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

Re: [libvirt] [PATCH] cpumap: optimize for clients that don't need online count



On 11/01/2012 08:17 PM, Laine Stump wrote:
> On 11/01/2012 09:57 PM, Eric Blake wrote:
>> It turns out that calling virNodeGetCPUMap(conn, NULL, NULL, 0)
>> is both useful, and with Viktor's patches, common enough to
>> optimize.  Since this interface hasn't been released yet, we
>> can change the RPC call.
>>

> 
> After a small bit of explaining the context surrounding the change on
> IRC, I understand what's going on and the benefit, and I agree that it's
> either now or never.
> 
> ACK.

Thanks.  I edited the commit message to mention some of the context
(finding the max cpu is one sysfs file, but finding the bitmap of cpus
to determine which cpus are online involves more file reads, possibly
one per cpu depending on the age of the kernel, which can add up to a
lot of worthless syscalls if the caller passed NULL for both
parameters).  Now pushed.

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature


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