[libvirt] [PATCHv6 0/6] Add virNodeGetCPUStats() API

[Eric Blake]

On 05/29/2011 07:54 PM, Minoru Usui wrote:
>>> So I wonder if we should add a parameter for the CPU num
>>> to the API:
>>>
>>>  int                     virNodeGetCPUStats (virConnectPtr conn,
>>>                                              int cpuNum,
>>>                                              virCPUStatsPtr params,
>>>                                              int *nparams,
>>>                                              unsigned int flags);
>>>
>>> And have a constant for getting the host data as a whole:
>>>
>>>    #define  VIR_NODE_CPU_STATS_ALL_CPUS -1

>>  That way the API and virsh command are ready for more precise use
>> and we won't need to make a second set of APIs.
>>
>>   thanks !
> 
> OK.
> I'll add cpu/cell argments for more generalization.

At this point, I'd feel more comfortable delaying this patch series (as
well the virNodeGetMemoryStats series) until after the 0.9.2 release.
Feel free to post v7 for review before them, but I think we've had
enough churn on the interface, and we've already hit the freeze with
release candidate 1, that I don't feel right pushing this upstream this
late in a release cycle, even though the principle idea has been ACK'd.

-- 
Eric Blake   eblake at redhat.com    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 619 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20110531/37d05238/attachment-0001.sig>