[libvirt] [PATCH] qemu: Propagate QEMU errors during incoming migrations
Jiri Denemark
jdenemar at redhat.com
Mon Sep 8 11:36:43 UTC 2014
On Mon, Sep 08, 2014 at 13:21:52 +0200, Peter Krempa wrote:
> On 09/08/14 13:05, Jiri Denemark wrote:
> > When QEMU fails during incoming migration after we successfully started
> > it (i.e., during Perform or Finish phase), we report a rather unhelpful
> > message
> >
> > Unable to read from monitor: Connection reset by peer
> >
> > We already have a code that takes error messages from QEMU's error
> > output but we disable it once QEMU successfully starts. This patch
> > postpones this until the end of Finish phase during incoming migration
> > so that we can report a much better error message:
> >
> > internal error: early end of file from monitor: possible problem:
> > Unknown savevm section or instance '0000:00:05.0/virtio-balloon' 0
> > load of migration failed
> >
> > https://bugzilla.redhat.com/show_bug.cgi?id=1090093
> >
> > Signed-off-by: Jiri Denemark <jdenemar at redhat.com>
> > ---
> > src/qemu/qemu_migration.c | 2 ++
> > src/qemu/qemu_monitor.c | 7 ++++++-
> > src/qemu/qemu_process.c | 6 ++++--
> > 3 files changed, 12 insertions(+), 3 deletions(-)
>
> ACK, although you could state that this is done with migrations now too
> in the comment above the code that parses the monitor error message.
I did so and pushed the patch. Thanks.
Jirka
More information about the libvir-list
mailing list