[libvirt] [PATCH V3] Expose resource control capabilites on cache bank

[Eli Qiao]

> >  
> > </bank>
> > </cache>
> >  
> >  
> > @Daniel,  
> >  
> > the enum values are same with `type`
> >  
> > unified: 0
> > instruction: 1
> > data: 2
> >  
> > but scope should be both(0)/code(1)/data(2), so the attribute name will be
>  
> 'both' and 'unified' mean the same thing.
>  
> 'instruction' and 'code' mean the same thing to.
>  
> So we should use the same terminology for both attributes. IOW, I
> suggest we use 'both', 'code', and 'data' everywhere.
>  

Hi Daniel/Martin

I update V4 patch of “extend resctrl capabilites” [1]

I would like to request for comment, this is something different for your suggestion
I added the reason in P.S section of the comment message.

Looking for your comments.


[1] https://www.redhat.com/archives/libvir-list/2017-April/msg00516.html

Thanks.

Eli.
  
>  
>  
> Regards,
> Daniel
> --  
> |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :|
> |: http://libvirt.org -o- http://virt-manager.org :|
> |: http://entangle-photo.org -o- http://search.cpan.org/~danberr/ :|
>  
>  


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20170411/c62babcd/attachment-0001.htm>