On Mon, Jan 30, 2012 at 16:13:37 -0700, Eric Blake wrote: > On 01/30/2012 09:01 AM, Jiri Denemark wrote: > > --- > > src/qemu/qemu_conf.h | 1 + > > src/qemu/qemu_driver.c | 83 ++++++++++++++++++++++++++++++++++++++++++ > > src/qemu/qemu_monitor.c | 40 ++++++++++++++++++++ > > src/qemu/qemu_monitor.h | 1 + > > src/qemu/qemu_monitor_json.c | 8 ++++ > > src/qemu/qemu_monitor_text.c | 15 ++++++++ > > 6 files changed, 148 insertions(+), 0 deletions(-) > > > ... > > +++ b/src/qemu/qemu_monitor_text.c > > @@ -839,6 +839,21 @@ int qemuMonitorTextGetBlockInfo(qemuMonitorPtr mon, > > VIR_DEBUG("error reading tray_open: %s", p); > > Do we still need to implement this in qemu_monitor_text, now that we > require JSON for any qemu version new enough to actually provide this > information in the first place? I guess we don't but v1 predates your patch and I tested the code worked so I just didn't remove it :-) Jirka -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list