[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 00/11] Preparation for block job mutex
From: |
Jeff Cody |
Subject: |
Re: [Qemu-devel] [PATCH v2 00/11] Preparation for block job mutex |
Date: |
Mon, 22 May 2017 21:57:47 -0400 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
On Mon, May 08, 2017 at 04:12:59PM +0200, Paolo Bonzini wrote:
> These are a bunch of cleanups and patches extracted from the AioContext
> lock removal series. As a general theme, the patches reorganize
> blockjob.c to follow the blockjob.h/blockjob_int.h separation more
> closely. For this reason, a lot of the patches are just moving functions
> around.
>
> The blockjob.h/blockjob_int.h split later will correspond to different
> locking rules, but the patches are independent from this change, and
> can be applied/reviewed separately.
>
> There is no code change from v1, but all patches now have Reviewed-by
> from at least one of John and Stefan.
>
> Thanks,
>
> Paolo
>
>
> Paolo Bonzini (11):
> blockjob: remove unnecessary check
> blockjob: remove iostatus_reset callback
> blockjob: introduce block_job_early_fail
> blockjob: introduce block_job_pause/resume_all
> blockjob: separate monitor and blockjob APIs
> blockjob: move iostatus reset inside block_job_user_resume
> blockjob: introduce block_job_cancel_async, check iostatus invariants
> blockjob: group BlockJob transaction functions together
> blockjob: strengthen a bit test-blockjob-txn
> blockjob: reorganize block_job_completed_txn_abort
> blockjob: use deferred_to_main_loop to indicate the coroutine has
> ended
>
> block/backup.c | 2 +-
> block/commit.c | 2 +-
> block/io.c | 19 +-
> block/mirror.c | 2 +-
> blockdev.c | 1 -
> blockjob.c | 900
> +++++++++++++++++++++++--------------------
> include/block/blockjob.h | 16 -
> include/block/blockjob_int.h | 27 +-
> tests/test-blockjob-txn.c | 7 +-
> tests/test-blockjob.c | 10 +-
> 10 files changed, 522 insertions(+), 464 deletions(-)
>
> --
> 2.12.2
>
Thanks,
Applied to my block branch:
git://github.com/codyprime/qemu-kvm-jtc block
-Jeff
- [Qemu-devel] [PATCH 08/11] blockjob: group BlockJob transaction functions together, (continued)
- [Qemu-devel] [PATCH 08/11] blockjob: group BlockJob transaction functions together, Paolo Bonzini, 2017/05/08
- [Qemu-devel] [PATCH 05/11] blockjob: separate monitor and blockjob APIs, Paolo Bonzini, 2017/05/08
- [Qemu-devel] [PATCH 07/11] blockjob: introduce block_job_cancel_async, check iostatus invariants, Paolo Bonzini, 2017/05/08
- [Qemu-devel] [PATCH 10/11] blockjob: reorganize block_job_completed_txn_abort, Paolo Bonzini, 2017/05/08
- [Qemu-devel] [PATCH 11/11] blockjob: use deferred_to_main_loop to indicate the coroutine has ended, Paolo Bonzini, 2017/05/08
- [Qemu-devel] [PATCH 09/11] blockjob: strengthen a bit test-blockjob-txn, Paolo Bonzini, 2017/05/08
- Re: [Qemu-devel] [PATCH v2 00/11] Preparation for block job mutex, Paolo Bonzini, 2017/05/21
- Re: [Qemu-devel] [PATCH v2 00/11] Preparation for block job mutex,
Jeff Cody <=