[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 09/11] migration: add missed aio_context_acquire
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH 09/11] migration: add missed aio_context_acquire for state writing/reading |
Date: |
Fri, 6 Nov 2015 15:37:04 +0000 |
User-agent: |
Mutt/1.5.23 (2015-06-09) |
On Wed, Nov 04, 2015 at 08:19:40PM +0300, Denis V. Lunev wrote:
> aio_context should be locked in the similar way as was done in QMP
> snapshot creation in the other case there are a lot of possible
> troubles if native AIO mode is enabled for disk.
>
> qemu_fopen_bdrv and bdrv_fclose are used in real snapshot operations only
> along with block drivers. This change should influence only HMP snapshot
> operations.
>
> AioContext lock is reqursive. Thus nested locking should not be a problem.
hmp_savevm() and load_vmstate() look up the BlockDriverState and perform
other operations on it. So the natural place to call acquire/release is
in hmp_savevm() and load_vmstate(). They need that anyway since they
also perform other operations.
qemu_fopen_bdrv() and bdrv_fclose() don't need to worry about
AioContext.
signature.asc
Description: PGP signature
- Re: [Qemu-devel] [PATCH 01/11] snapshot: create helper to test that block drivers supports snapshots, (continued)
- [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
- [Qemu-devel] [PATCH 09/11] migration: add missed aio_context_acquire for state writing/reading, Denis V. Lunev, 2015/11/04
- Re: [Qemu-devel] [PATCH 09/11] migration: add missed aio_context_acquire for state writing/reading,
Stefan Hajnoczi <=
- [Qemu-devel] [PATCH 06/11] migration: drop find_vmstate_bs check in hmp_delvm, Denis V. Lunev, 2015/11/04
- [Qemu-devel] [PATCH 11/11] monitor: add missed aio_context_acquire into vm_completion call, Denis V. Lunev, 2015/11/04
- [Qemu-devel] [PATCH 10/11] snapshot: create bdrv_all_create_snapshot helper, Denis V. Lunev, 2015/11/04
- Re: [Qemu-devel] [PATCH 2.5 v5 0/11] dataplane snapshot fixes, Stefan Hajnoczi, 2015/11/06