[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [Qemu-devel] [PATCH v5 06/16] block/mirror: conservativ
From: |
Max Reitz |
Subject: |
Re: [Qemu-block] [Qemu-devel] [PATCH v5 06/16] block/mirror: conservative mirror_exit refactor |
Date: |
Sat, 8 Sep 2018 01:16:56 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1 |
On 2018-09-06 22:31, John Snow wrote:
>
>
> On 09/06/2018 12:57 PM, Jeff Cody wrote:
>> On Thu, Sep 06, 2018 at 09:02:15AM -0400, John Snow wrote:
>>> For purposes of minimum code movement, refactor the mirror_exit
>>> callback to use the post-finalization callbacks in a trivial way.
>>>
>>> Signed-off-by: John Snow <address@hidden>
>>> ---
>>> block/mirror.c | 39 ++++++++++++++++++++++++++++-----------
>>> 1 file changed, 28 insertions(+), 11 deletions(-)
>>>
>>> diff --git a/block/mirror.c b/block/mirror.c
>>> index bd3e908710..a92b4702c5 100644
>>> --- a/block/mirror.c
>>> +++ b/block/mirror.c
>>> @@ -79,6 +79,7 @@ typedef struct MirrorBlockJob {
>>> int max_iov;
>>> bool initial_zeroing_ongoing;
>>> int in_active_write_counter;
>>> + bool prepared;
>>> } MirrorBlockJob;
>>>
>>> typedef struct MirrorBDSOpaque {
>>> @@ -607,7 +608,7 @@ static void mirror_wait_for_all_io(MirrorBlockJob *s)
>>> }
>>> }
>>>
>>> -static void mirror_exit(Job *job)
>
> /**
> * mirror_exit_common: handle both abort() and prepare() cases.
> * for .prepare, returns 0 on success and -errno on failure.
> * for .abort cases, denoted by abort = true, MUST return 0.
> */
Any case:
Reviewed-by: Max Reitz <address@hidden>
signature.asc
Description: OpenPGP digital signature
- [Qemu-block] [PATCH v5 00/16] jobs: Job Exit Refactoring Pt 2, John Snow, 2018/09/06
- [Qemu-block] [PATCH v5 03/16] block/stream: add block job creation flags, John Snow, 2018/09/06
- [Qemu-block] [PATCH v5 05/16] block/mirror: don't install backing chain on abort, John Snow, 2018/09/06
- [Qemu-block] [PATCH v5 02/16] block/mirror: add block job creation flags, John Snow, 2018/09/06
- [Qemu-block] [PATCH v5 10/16] tests/test-blockjob-txn: move .exit to .clean, John Snow, 2018/09/06
- [Qemu-block] [PATCH v5 14/16] qapi/block-stream: expose new job properties, John Snow, 2018/09/06
- [Qemu-block] [PATCH v5 04/16] block/commit: refactor commit to use job callbacks, John Snow, 2018/09/06