Re: [libvirt] [PATCH 8/8] latency: Update cmdBlkStats to use new API
On Fri, Sep 02, 2011 at 07:23:38PM +0800, Osier Yang wrote:
于 2011年09月02日 18:21, Daniel Veillard 写道:
>On Wed, Aug 31, 2011 at 04:26:13PM +0800, Osier Yang wrote:
>>The modified function fallbacks to use virDomainBlockStats if
>>virDomainBlockStatsFlags is not supported by the hypervisor driver.
>>If the new API is supported, it will be invoked instead of the
>>old API.
> Again I would use a new command and keep the existing code
>as is,
>
>Daniel
>
What does "again" here mean? as I couldn't find another place which
introduce new code. :-)
Well I would not modify cmdDomblkstat and instead create a new command
and a new function for the new API
And per the new API has some same fields with old API, and seems
all of virsh commands try to fallback to old API if new API is introduced.
yes but the new API will provide informations in a different order
and is potentially more expensive, so I'm not sure I really want to use
the new API for the old command
Maybe something like "domblkfullstat"...
But if someone else disagrees with me I'm fine being in the minority :-)
Daniel
--
Daniel Veillard | libxml Gnome XML XSLT toolkit http://xmlsoft.org/
daniel(a)veillard.com | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library http://libvirt.org/