[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 02/11] snapshot: return error code from bdrv_sna
From: |
Greg Kurz |
Subject: |
Re: [Qemu-devel] [PATCH 02/11] snapshot: return error code from bdrv_snapshot_delete_by_id_or_name |
Date: |
Thu, 5 Nov 2015 08:55:58 +0100 |
On Wed, 4 Nov 2015 20:19:33 +0300
"Denis V. Lunev" <address@hidden> wrote:
> this will make code better in the next patch
>
This changelog is not very useful. IMHO this calls for squashing this patch
into the next one: then we clearly see how the return value is used and why
the code is better.
> Signed-off-by: Denis V. Lunev <address@hidden>
> CC: Juan Quintela <address@hidden>
> CC: Stefan Hajnoczi <address@hidden>
> CC: Kevin Wolf <address@hidden>
> ---
> block/snapshot.c | 7 ++++---
> include/block/snapshot.h | 6 +++---
> 2 files changed, 7 insertions(+), 6 deletions(-)
>
> diff --git a/block/snapshot.c b/block/snapshot.c
> index 2ef4545..d729c76 100644
> --- a/block/snapshot.c
> +++ b/block/snapshot.c
> @@ -253,9 +253,9 @@ int bdrv_snapshot_delete(BlockDriverState *bs,
> return -ENOTSUP;
> }
>
> -void bdrv_snapshot_delete_by_id_or_name(BlockDriverState *bs,
> - const char *id_or_name,
> - Error **errp)
> +int bdrv_snapshot_delete_by_id_or_name(BlockDriverState *bs,
> + const char *id_or_name,
> + Error **errp)
> {
> int ret;
> Error *local_err = NULL;
> @@ -270,6 +270,7 @@ void bdrv_snapshot_delete_by_id_or_name(BlockDriverState
> *bs,
> if (ret < 0) {
> error_propagate(errp, local_err);
> }
> + return ret;
> }
>
> int bdrv_snapshot_list(BlockDriverState *bs,
> diff --git a/include/block/snapshot.h b/include/block/snapshot.h
> index 0a3cf0d..76cf6a3 100644
> --- a/include/block/snapshot.h
> +++ b/include/block/snapshot.h
> @@ -63,9 +63,9 @@ int bdrv_snapshot_delete(BlockDriverState *bs,
> const char *snapshot_id,
> const char *name,
> Error **errp);
> -void bdrv_snapshot_delete_by_id_or_name(BlockDriverState *bs,
> - const char *id_or_name,
> - Error **errp);
> +int bdrv_snapshot_delete_by_id_or_name(BlockDriverState *bs,
> + const char *id_or_name,
> + Error **errp);
> int bdrv_snapshot_list(BlockDriverState *bs,
> QEMUSnapshotInfo **psn_info);
> int bdrv_snapshot_load_tmp(BlockDriverState *bs,
- [Qemu-devel] [PATCH 2.5 v5 0/11] dataplane snapshot fixes, Denis V. Lunev, 2015/11/04
- [Qemu-devel] [PATCH 03/11] snapshot: create bdrv_all_delete_snapshot helper, Denis V. Lunev, 2015/11/04
- [Qemu-devel] [PATCH 02/11] snapshot: return error code from bdrv_snapshot_delete_by_id_or_name, Denis V. Lunev, 2015/11/04
- Re: [Qemu-devel] [PATCH 02/11] snapshot: return error code from bdrv_snapshot_delete_by_id_or_name,
Greg Kurz <=
- [Qemu-devel] [PATCH 01/11] snapshot: create helper to test that block drivers supports snapshots, Denis V. Lunev, 2015/11/04
- [Qemu-devel] [PATCH 04/11] snapshot: create bdrv_all_goto_snapshot helper, Denis V. Lunev, 2015/11/04
- [Qemu-devel] [PATCH 07/11] migration: reorder processing in hmp_savevm, Denis V. Lunev, 2015/11/04
- [Qemu-devel] [PATCH 05/11] snapshot: create bdrv_all_find_snapshot helper, Denis V. Lunev, 2015/11/04
- [Qemu-devel] [PATCH 08/11] migration: implement bdrv_all_find_vmstate_bs and bdrv_unlock helpers, Denis V. Lunev, 2015/11/04