[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 05/11] migration: factor our snapshottability ch
From: |
Fam Zheng |
Subject: |
Re: [Qemu-devel] [PATCH 05/11] migration: factor our snapshottability check in load_vmstate |
Date: |
Tue, 17 Nov 2015 19:51:32 +0800 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Tue, 11/17 12:08, Denis V. Lunev wrote:
> We should check that all inserted and not read-only images support
> snapshotting. This could be made using already invented helper
> bdrv_all_can_snapshot().
>
> Signed-off-by: Denis V. Lunev <address@hidden>
> CC: Juan Quintela <address@hidden>
> CC: Stefan Hajnoczi <address@hidden>
> CC: Kevin Wolf <address@hidden>
> ---
> migration/savevm.c | 15 +++++++--------
> 1 file changed, 7 insertions(+), 8 deletions(-)
>
> diff --git a/migration/savevm.c b/migration/savevm.c
> index 254e51d..2ecc1b3 100644
> --- a/migration/savevm.c
> +++ b/migration/savevm.c
> @@ -2051,6 +2051,12 @@ int load_vmstate(const char *name)
> QEMUFile *f;
> int ret;
>
> + if (!bdrv_all_can_snapshot(&bs)) {
> + error_report("Device '%s' is writable but does not support
> snapshots.",
> + bdrv_get_device_name(bs));
> + return -ENOTSUP;
> + }
> +
> bs_vm_state = find_vmstate_bs();
> if (!bs_vm_state) {
> error_report("No block device supports snapshots");
> @@ -2071,15 +2077,8 @@ int load_vmstate(const char *name)
> writable and check if the requested snapshot is available too. */
> bs = NULL;
> while ((bs = bdrv_next(bs))) {
> -
> - if (!bdrv_is_inserted(bs) || bdrv_is_read_only(bs)) {
> - continue;
> - }
> -
> if (!bdrv_can_snapshot(bs)) {
> - error_report("Device '%s' is writable but does not support
> snapshots.",
> - bdrv_get_device_name(bs));
> - return -ENOTSUP;
> + continue;
> }
>
> ret = bdrv_snapshot_find(bs, &sn, name);
> --
> 2.5.0
>
>
Reviewed-by: Fam Zheng <address@hidden>
- [Qemu-devel] [PATCH for 2.9 v8 0/10] dataplane snapshot fixes, Denis V. Lunev, 2015/11/17
- [Qemu-devel] [PATCH 01/11] snapshot: create helper to test that block drivers supports snapshots, Denis V. Lunev, 2015/11/17
- [Qemu-devel] [PATCH 02/11] snapshot: return error code from bdrv_snapshot_delete_by_id_or_name, Denis V. Lunev, 2015/11/17
- [Qemu-devel] [PATCH 04/11] snapshot: create bdrv_all_goto_snapshot helper, Denis V. Lunev, 2015/11/17
- [Qemu-devel] [PATCH 03/11] snapshot: create bdrv_all_delete_snapshot helper, Denis V. Lunev, 2015/11/17
- [Qemu-devel] [PATCH 05/11] migration: factor our snapshottability check in load_vmstate, Denis V. Lunev, 2015/11/17
- [Qemu-devel] [PATCH 07/11] migration: drop find_vmstate_bs check in hmp_delvm, Denis V. Lunev, 2015/11/17
- [Qemu-devel] [PATCH 06/11] snapshot: create bdrv_all_find_snapshot helper, Denis V. Lunev, 2015/11/17
- [Qemu-devel] [PATCH 08/11] snapshot: create bdrv_all_create_snapshot helper, Denis V. Lunev, 2015/11/17
- [Qemu-devel] [PATCH 09/11] migration: reorder processing in hmp_savevm, Denis V. Lunev, 2015/11/17