On 04/02/2012 06:23 PM, Wen Congyang wrote: >> >> Why are we bothering with a text implementation? We know that the >> feature is only present if you have qemu 1.1 or later (assuming that >> qemu-devel did apply your series adding the monitor command), and >> therefore we know that we have QMP, so we should only implement this for >> JSON and not bother with the text monitor. >> > > In most cases, the libvirt is built with json. But if the libvirt is built > without json, we will use text monitor. If libvirt is built without json, it will die a horrible death on any qemu newer than 0.15, thanks to commit 6e769eba. You can safely assume that a qemu 1.1 feature implies that libvirt was compiled with json support. The _only_ situation that requires the text monitor is when building libvirt for RHEL 5 where qemu is still stuck at 0.10 and where JSON libraries do not exist. I suspect no one will be backporting the new dump-guest-monitor command to RHEL 5. -- Eric Blake eblake@xxxxxxxxxx +1-919-301-3266 Libvirt virtualization library http://libvirt.org
Attachment:
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list