[libvirt] [PATCH] cpustat: fix regression when cpus are offline
Eric Blake
eblake at redhat.com
Fri Oct 26 16:02:57 UTC 2012
On 10/26/2012 07:59 AM, Peter Krempa wrote:
> On 10/25/12 22:57, Eric Blake wrote:
>> It turns out that the cpuacct results properly account for offline
>> cpus, and always returns results for every possible cpu, not just
>> the online ones. So there is no need to check the map of online
>> cpus in the first place, merely only a need to know the maximum
>> possible cpu. Meanwhile, virNodeGetCPUBitmap had a subtle change
>> from returning the maximum id to instead returning the width of
>> the bitmap (one larger than the maximum id), which made this code
>> encounter some off-by-one logic leading to bad error messages when
>> a cpu was offline:
>>
> virBitmapPtr map = NULL;
>> - virBitmapPtr map2 = NULL;
>
> When you apply this patch "map" is also not needed (never actualy used).
>
> Is there a way to replace
> "map = nodeGetCPUBitmap(domain->conn, &max_id);" to get max_id? That's
> the only thing that code is used to.
>
>> int rv = -1;
>> int i, id, max_id;
>> char *pos;
>
> Otherwise looks good. (ACK if there isn't any other option to get max_id
> than to call nodeGetCPUBitmap)
Hmm, good point. I'll spin a v2 that adds a new helper function in
nodeinfo.c for getting at just the number of host cpus.
--
Eric Blake eblake at redhat.com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 617 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20121026/28b3f8fb/attachment-0001.sig>
More information about the libvir-list
mailing list