On Fri, Jul 20, 2012 at 8:32 PM, <benoit.ca...@gmail.com> wrote: > From: Benoît Canet <ben...@irqsave.net> > > Signed-off-by: Benoit Canet <ben...@irqsave.net> > --- > migration.c | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/migration.c b/migration.c > index 8db1b43..dfce680 100644 > --- a/migration.c > +++ b/migration.c > @@ -425,6 +425,11 @@ void qmp_migrate(const char *uri, bool has_blk, bool blk, > return; > } > > + if (bdrv_have_block_jobs()) { > + error_set(errp, QERR_STREAMING_BLOCKS_MIGRATION); > + return; > + }
I think bdrv_have_block_jobs() is too specific and would use bdrv_in_use(bs) here to give basically an EBUSY-type error. Stefan