Re: [Qemu-devel] Re: Machine-readable or parseable qemu output

["Daniel P. Berrange" <berrange@redhat.com>]

On Thu, Jan 15, 2009 at 02:27:05PM -0600, Anthony Liguori wrote:
> Daniel P. Berrange wrote:
> >On Wed, Jan 14, 2009 at 04:40:05PM +0530, Amit Shah wrote:
> >  
> >>Hello,
> >>
> >>Continuing from the thread at [1], building on Daniel's suggestions, I
> >>have jot down a few points as to how a libqemumonitor API could be
> >>developed.
> >>
> >>To recap, there has to be an interface to the qemu monitor in
> >>a way that wouldn't break even if the monitor output changes. This API
> >>will remain the same, so that consumers (libvirt, etc) can safely
> >>integrate with the monitor.
> >>
> >>Please see Dan's email at [1] to get the background details.
> >>
> >>[1] http://www.mail-archive.com/qemu-devel@nongnu.org/msg14615.html
> >>
> >>Here are some initial thoughts. Comments?
> >>    
> >
> >I'd like to add one more requirement
> >
> > - Works with existing QEMU monitor for releases >= 0.8.0
> >  
> 
> I think this is a bad requirement.
> 
> >This is because libvirt currently supports all QEMU >= 0.8.0,
> 
> libvirt has no bearing on upstream QEMU support :-)
> 
> > so if
> >we're to be able to make use of this library we can't restrict it
> >to just new releases.
> 
> But you can conditionally use the new library instead of your custom 
> parsing code for newer QEMU versions.

That is true, but if there are a number of apps around which want to
support multiple versions of QEMU, it is beneficial to centralize
this conditional logic in libqemumonitor.so instead, of making each
app implement the compat logic for the existing monitor format. I'm
not against adding a new machine friendly monitor format, I'd just
prefer it if one library API could provide impl for both old and new
format, obviously preferring to use the new format where available.

Daniel
-- 
|: Red Hat, Engineering, London   -o-   http://people.redhat.com/berrange/ :|
|: http://libvirt.org  -o-  http://virt-manager.org  -o-  http://ovirt.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: GnuPG: 7D3B9505  -o-  F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|