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

Reply via email to