[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] block migration: Allow compile time disable
From: |
Dr. David Alan Gilbert |
Subject: |
Re: [Qemu-devel] [PATCH] block migration: Allow compile time disable |
Date: |
Mon, 15 May 2017 13:13:01 +0100 |
User-agent: |
Mutt/1.8.2 (2017-04-18) |
* Eric Blake (address@hidden) wrote:
> On 05/03/2017 05:42 AM, Dr. David Alan Gilbert (git) wrote:
> > From: "Dr. David Alan Gilbert" <address@hidden>
> >
> > Many users now prefer to use drive_mirror over NBD as an
> > alternative to the older migrate -b option; drive_mirror is
> > more complex to setup but gives you more options (e.g. only
> > migrating some of the disks if some of them are shared).
> >
> > Allow the large chunk of block migration code to be compiled
> > out for those who don't use it.
> >
> > Based on a downstream-patch we've had for a while by Jeff Cody.
> >
> > Signed-off-by: Dr. David Alan Gilbert <address@hidden>
> > ---
> > @@ -1222,6 +1226,14 @@ void qmp_migrate(const char *uri, bool has_blk, bool
> > blk,
> > params.blk = has_blk && blk;
> > params.shared = has_inc && inc;
> >
> > +#ifndef CONFIG_LIVE_BLOCK_MIGRATION
> > + if (params.blk || params.shared) {
> > + error_setg(errp, "QEMU compiled without old-style block migration.
> > "
> > + "Use drive_mirror+NBD.");
>
> error_setg() should not be used with '.' (it should be a single phrase,
> here you are trying to stuff in two sentences). error_append_hint() can
> be used to supply the advice about using drive_mirror+NBD as the
> alternative.
>
> Otherwise this looks reasonable to me.
Done as:
error_setg(errp, "QEMU compiled without old-style block migration");
error_append_hint(errp, "Use drive_mirror+NBD.\n");
(All the places I can see seem to have . and \n in append_hint)
Dave
> --
> Eric Blake, Principal Software Engineer
> Red Hat, Inc. +1-919-301-3266
> Virtualization: qemu.org | libvirt.org
>
>
--
Dr. David Alan Gilbert / address@hidden / Manchester, UK