[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH V4 2/3] qerror: Add error telling that block dev
From: |
Luiz Capitulino |
Subject: |
Re: [Qemu-devel] [PATCH V4 2/3] qerror: Add error telling that block dev usage prevents migration |
Date: |
Mon, 23 Jul 2012 14:09:00 -0300 |
On Mon, 23 Jul 2012 16:22:59 +0200
address@hidden wrote:
> From: Benoît Canet <address@hidden>
>
> Signed-off-by: Benoit Canet <address@hidden>
> ---
> qerror.c | 4 ++++
> qerror.h | 3 +++
> 2 files changed, 7 insertions(+)
>
> diff --git a/qerror.c b/qerror.c
> index 92c4eff..d2e76ca 100644
> --- a/qerror.c
> +++ b/qerror.c
> @@ -283,6 +283,10 @@ static const QErrorStringTable qerror_table[] = {
> .desc = "Could not set password",
> },
> {
> + .error_fmt = QERR_BLOCK_DEV_IN_USE_MIGRATION_PREVENTED,
> + .desc = "Block device in use migration prevented",
> + },
Let's avoid an specific error. We have two options:
1. Use QERR_MIGRATION_NOT_SUPPORTED
2. Add QERR_MIGRATION_BLOCKED
> + {
> .error_fmt = QERR_TOO_MANY_FILES,
> .desc = "Too many open files",
> },
> diff --git a/qerror.h b/qerror.h
> index b4c8758..2ef40b5 100644
> --- a/qerror.h
> +++ b/qerror.h
> @@ -233,6 +233,9 @@ QError *qobject_to_qerror(const QObject *obj);
> #define QERR_SET_PASSWD_FAILED \
> "{ 'class': 'SetPasswdFailed', 'data': {} }"
>
> +#define QERR_BLOCK_DEV_IN_USE_MIGRATION_PREVENTED \
> + "{ 'class': 'BlockDevInUseMigrationPrevented', 'data': {} }"
> +
> #define QERR_TOO_MANY_FILES \
> "{ 'class': 'TooManyFiles', 'data': {} }"
>
- [Qemu-devel] [PATCH V4 0/3] Block migration if any of the block device is busy, benoit . canet, 2012/07/23
- [Qemu-devel] [PATCH V4 1/3] block: Add bdrv_are_busy(), benoit . canet, 2012/07/23
- [Qemu-devel] [PATCH V4 2/3] qerror: Add error telling that block dev usage prevents migration, benoit . canet, 2012/07/23
- Re: [Qemu-devel] [PATCH V4 2/3] qerror: Add error telling that block dev usage prevents migration,
Luiz Capitulino <=
- [Qemu-devel] [PATCH V4 3/3] migration: block migration when any of the block device is busy, benoit . canet, 2012/07/23
- Re: [Qemu-devel] [PATCH V4 0/3] Block migration if any of the block device is busy, Stefan Hajnoczi, 2012/07/23
- Re: [Qemu-devel] [PATCH V4 0/3] Block migration if any of the block device is busy, Kevin Wolf, 2012/07/24