[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 1/7] migration/savevm: respect qemu_fclose() error code in sa
From: |
Juan Quintela |
Subject: |
Re: [PATCH 1/7] migration/savevm: respect qemu_fclose() error code in save_snapshot() |
Date: |
Thu, 09 Jul 2020 12:26:44 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.3 (gnu/linux) |
"Denis V. Lunev" <den@openvz.org> wrote:
> qemu_fclose() could return error, f.e. if bdrv_co_flush() will return
> the error.
>
> This validation will become more important once we will start waiting of
> asynchronous IO operations, started from bdrv_write_vmstate(), which are
> coming soon.
>
> Signed-off-by: Denis V. Lunev <den@openvz.org>
> Reviewed-by: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
> Reviewed-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
> CC: Kevin Wolf <kwolf@redhat.com>
> CC: Max Reitz <mreitz@redhat.com>
> CC: Stefan Hajnoczi <stefanha@redhat.com>
> CC: Fam Zheng <fam@euphon.net>
> CC: Juan Quintela <quintela@redhat.com>
> CC: Denis Plotnikov <dplotnikov@virtuozzo.com>
Reviewed-by: Juan Quintela <quintela@redhat.com>
queued
> ---
> migration/savevm.c | 8 ++++++--
> 1 file changed, 6 insertions(+), 2 deletions(-)
>
> diff --git a/migration/savevm.c b/migration/savevm.c
> index b979ea6e7f..da3dead4e9 100644
> --- a/migration/savevm.c
> +++ b/migration/savevm.c
> @@ -2628,7 +2628,7 @@ int save_snapshot(const char *name, Error **errp)
> {
> BlockDriverState *bs, *bs1;
> QEMUSnapshotInfo sn1, *sn = &sn1, old_sn1, *old_sn = &old_sn1;
> - int ret = -1;
> + int ret = -1, ret2;
> QEMUFile *f;
> int saved_vm_running;
> uint64_t vm_state_size;
> @@ -2712,10 +2712,14 @@ int save_snapshot(const char *name, Error **errp)
> }
> ret = qemu_savevm_state(f, errp);
> vm_state_size = qemu_ftell(f);
> - qemu_fclose(f);
> + ret2 = qemu_fclose(f);
> if (ret < 0) {
> goto the_end;
> }
> + if (ret2 < 0) {
> + ret = ret2;
> + goto the_end;
> + }
>
> /* The bdrv_all_create_snapshot() call that follows acquires the
> AioContext
> * for itself. BDRV_POLL_WHILE() does not support nested locking because
- [PATCH v6 0/7] block: seriously improve savevm/loadvm performance, Denis V. Lunev, 2020/07/03
- [PATCH 6/7] block/io: improve savevm performance, Denis V. Lunev, 2020/07/03
- [PATCH 7/7] block/io: improve loadvm performance, Denis V. Lunev, 2020/07/03
- [PATCH 4/7] block/block-backend: remove always true check from blk_save_vmstate, Denis V. Lunev, 2020/07/03
- [PATCH 5/7] block, migration: add bdrv_finalize_vmstate helper, Denis V. Lunev, 2020/07/03
- [PATCH 3/7] block/aio_task: drop aio_task_pool_wait_one() helper, Denis V. Lunev, 2020/07/03
- [PATCH 1/7] migration/savevm: respect qemu_fclose() error code in save_snapshot(), Denis V. Lunev, 2020/07/03
- Re: [PATCH 1/7] migration/savevm: respect qemu_fclose() error code in save_snapshot(),
Juan Quintela <=
- [PATCH 2/7] block/aio_task: allow start/wait task from any coroutine, Denis V. Lunev, 2020/07/03
- Re: [PATCH v6 0/7] block: seriously improve savevm/loadvm performance, no-reply, 2020/07/03