[libvirt] [PATCH v2 2/3] qemu: Implement virDomainGetDiskErrors

Jiri Denemark jdenemar at redhat.com
Tue Jan 31 19:34:19 UTC 2012


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




More information about the libvir-list mailing list