diff options
| author | Daniel P. Berrangé | 2021-02-04 13:48:28 +0100 |
|---|---|---|
| committer | Dr. David Alan Gilbert | 2021-02-08 12:19:51 +0100 |
| commit | 3d3e9b1f669b60d9d3cb857edbfc3d54cbb9c0ef (patch) | |
| tree | 0c55e6564c1cea3901e2af1fd86eb9d7582b6182 /include | |
| parent | block: allow specifying name of block device for vmstate storage (diff) | |
| download | qemu-3d3e9b1f669b60d9d3cb857edbfc3d54cbb9c0ef.tar.gz qemu-3d3e9b1f669b60d9d3cb857edbfc3d54cbb9c0ef.tar.xz qemu-3d3e9b1f669b60d9d3cb857edbfc3d54cbb9c0ef.zip | |
block: rename and alter bdrv_all_find_snapshot semantics
Currently bdrv_all_find_snapshot() will return 0 if it finds
a snapshot, -1 if an error occurs, or if it fails to find a
snapshot. New callers to be added want to distinguish between
the error scenario and failing to find a snapshot.
Rename it to bdrv_all_has_snapshot and make it return -1 on
error, 0 if no snapshot is found and 1 if snapshot is found.
Reviewed-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
Message-Id: <20210204124834.774401-7-berrange@redhat.com>
Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Diffstat (limited to 'include')
| -rw-r--r-- | include/block/snapshot.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/include/block/snapshot.h b/include/block/snapshot.h index 8a6a37240d..940345692f 100644 --- a/include/block/snapshot.h +++ b/include/block/snapshot.h @@ -85,9 +85,9 @@ int bdrv_all_delete_snapshot(const char *name, int bdrv_all_goto_snapshot(const char *name, bool has_devices, strList *devices, Error **errp); -int bdrv_all_find_snapshot(const char *name, - bool has_devices, strList *devices, - Error **errp); +int bdrv_all_has_snapshot(const char *name, + bool has_devices, strList *devices, + Error **errp); int bdrv_all_create_snapshot(QEMUSnapshotInfo *sn, BlockDriverState *vm_state_bs, uint64_t vm_state_size, |