[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH 2/5] schemas: Add schemas for more CPU topology information in the caps XML



On 01/21/13 09:53, Martin Kletzander wrote:
On 01/19/2013 12:06 AM, Peter Krempa wrote:

[...]



IMHO this looks very good and it is understandable for management
applications.  I like that with this approach libvirt isn't obfuscating
modules as neither threads nor cores, but rather providing all the
information available, so management application can decide what it
wants (but doesn't have to).

However, I wasn't part of your previous discussion about this and I
don't want to skip other's decision on this by simply ACKing it without
others expressing their opinion, so small ACK from me.  I'd say it can
go in if nobody is particularly against it until the freeze.

Thanks for the review. Anybody else to review the design? (... also known as "Ping?")

PEter


Martin

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



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]