Re: [PATCH V3] Expose resource control capabilites on cache bank

[Eli Qiao <qiaoliyong@xxxxxxxxx>]

</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/ :|

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list