[libvirt] [PATCH 8/8] latency: Update cmdBlkStats to use new API

Daniel Veillard veillard at redhat.com
Fri Sep 2 11:34:08 UTC 2011


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 at veillard.com  | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library  http://libvirt.org/




More information about the libvir-list mailing list