[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH v2 07/16] backup: Do initial aio context move of
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-block] [PATCH v2 07/16] backup: Do initial aio context move of target via BB interface |
Date: |
Thu, 11 May 2017 16:00:01 -0400 |
User-agent: |
Mutt/1.8.0 (2017-02-23) |
On Wed, Apr 19, 2017 at 05:43:47PM +0800, Fam Zheng wrote:
> The old aio context check is hacky because when it was added we didn't
> have the permission system to enforce a general rule. It only checks if
> the target BDS has a BB, which is in fact insufficient because there may
> be other BBs in the graph that cannot handle the aio context change.
>
> Do this through blk_set_aio_context interface, in backup_job_create()
> which is a central place for both drive-backup and blockdev-backup, to
> take care of the compatibility check.
>
> Also the bdrv_set_aio_context in do_drive_backup could have been
> conditional, to save a recursion when possible.
>
> Signed-off-by: Fam Zheng <address@hidden>
> ---
> block/backup.c | 9 +++++++++
> blockdev.c | 14 --------------
> 2 files changed, 9 insertions(+), 14 deletions(-)
Reviewed-by: Stefan Hajnoczi <address@hidden>
signature.asc
Description: PGP signature
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-block] [PATCH v2 07/16] backup: Do initial aio context move of target via BB interface,
Stefan Hajnoczi <=