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

[Anthony Liguori <anthony@codemonkey.ws>]

Avi Kivity wrote:
> Daniel P. Berrange wrote:
>>> 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.
>>   
>
> Perhaps libvirt can donate its parsing code for use as a fallback in 
> libqemumonitor.so.

Or split it out into an another library.  I really don't think it's a 
good idea to maintain it within QEMU because it's against an interface 
that was never supported in QEMU and doesn't exist in the existing QEMU 
code base.

Regards,

Anthony Liguori