[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 for-2.11 1/4] blockjob: do not allow coroutin
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH v2 for-2.11 1/4] blockjob: do not allow coroutine double entry or entry-after-completion |
Date: |
Tue, 21 Nov 2017 14:12:32 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0 |
On 21/11/2017 11:49, Stefan Hajnoczi wrote:
> On Mon, Nov 20, 2017 at 09:23:23PM -0500, Jeff Cody wrote:
>> @@ -291,10 +291,10 @@ void block_job_start(BlockJob *job)
>> {
>> assert(job && !block_job_started(job) && job->paused &&
>> job->driver && job->driver->start);
>> - job->co = qemu_coroutine_create(block_job_co_entry, job);
>> job->pause_count--;
>> job->busy = true;
>> job->paused = false;
>> + job->co = qemu_coroutine_create(block_job_co_entry, job);
>> bdrv_coroutine_enter(blk_bs(job->blk), job->co);
>> }
>
> Please see discussion on v1 about this hunk.
>
> The rest looks good.
I'm okay with this hunk, but I would appreciate that the commit message
said why it's okay to delay block job cancellation after
block_job_sleep_ns returns.
Thanks,
Paolo
signature.asc
Description: OpenPGP digital signature