[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 01/17] job: Fix missing locking due to mismer
From: |
John Snow |
Subject: |
Re: [Qemu-devel] [PATCH v2 01/17] job: Fix missing locking due to mismerge |
Date: |
Thu, 13 Sep 2018 13:38:47 -0400 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1 |
On 09/13/2018 08:52 AM, Kevin Wolf wrote:
> job_completed() had a problem with double locking that was recently
> fixed independently by two different commits:
>
> "job: Fix nested aio_poll() hanging in job_txn_apply"
> "jobs: add exit shim"
>
> One fix removed the first aio_context_acquire(), the other fix removed
> the other one. Now we have a bug again and the code is run without any
> locking.
>
> Add it back in one of the places.
>
> Signed-off-by: Kevin Wolf <address@hidden>
> ---
> job.c | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/job.c b/job.c
> index 82b46929bd..5c4e84f007 100644
> --- a/job.c
> +++ b/job.c
> @@ -847,7 +847,11 @@ static void job_completed(Job *job)
> static void job_exit(void *opaque)
> {
> Job *job = (Job *)opaque;
> + AioContext *ctx = job->aio_context;
> +
> + aio_context_acquire(ctx);
> job_completed(job);
> + aio_context_release(ctx);
> }
>
> /**
>
OK =]
Reviewed-by: John Snow <address@hidden>