[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 0/6] block: Fixes regarding dataplane and man
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [PATCH v2 0/6] block: Fixes regarding dataplane and management operations |
Date: |
Fri, 7 Apr 2017 14:45:43 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
Am 07.04.2017 um 08:54 hat Fam Zheng geschrieben:
> v2: - Drop patch 4 in v1. A second thought made me feel neither it nor Kevin's
> suggestion to move the BH process to bdrv_drain_recurse/BDRV_POLL_WHILE
> is a complete fix. So leave it for a separate patch.
> - Add rev-by to patches 1, 3, 4.
> - Split from patch 1 in v1 and add patch 2, for the new assertions.
> [Kevin]
> - Rewrite patch 5. Fix block job's co when a BDS is moved to a different
> aio context. [Kevin]
> - Add patch 6.
>
> Crashes are reported on dataplane devices when doing snapshot and commit under
> guest I/O.
>
> With this series, Ed's test case '176' now passes (reran 10+ times):
>
> https://github.com/skyportsystems/qemu-1/commits/eswierk-iotests-2.9
>
> The biggest fix for this is patch 5, which fixes a race condition between main
> thread and iothread.
Thanks, applied patch 1-3 for now. I feel the other patches still need
closer review and potentially discussion.
Kevin
- Re: [Qemu-devel] [PATCH v2 5/6] coroutine: Explicitly specify AioContext when entering coroutine, (continued)
[Qemu-devel] [PATCH v2 6/6] tests/block-job-txn: Don't start block job before adding to txn, Fam Zheng, 2017/04/07
Re: [Qemu-devel] [PATCH v2 0/6] block: Fixes regarding dataplane and management operations,
Kevin Wolf <=