Re: [PATCH v3 09/16] jobs: remove aiocontext locks since the functions are under BQL

2022-01-26 Thread Emanuele Giuseppe Esposito
On 19/01/2022 12:09, Paolo Bonzini wrote: >> @@ -3707,15 +3707,11 @@ BlockJobInfoList *qmp_query_block_jobs(Error >> **errp) >>     for (job = block_job_next(NULL); job; job = >> block_job_next(job)) { >>   BlockJobInfo *value; >> -    AioContext *aio_context; >>     if

Re: [PATCH v3 09/16] jobs: remove aiocontext locks since the functions are under BQL

2022-01-19 Thread Paolo Bonzini
On 1/5/22 15:02, Emanuele Giuseppe Esposito wrote: In preparation to the job_lock/unlock patch, remove these aiocontext locks. The main reason these two locks are removed here is because they are inside a loop iterating on the jobs list. Once the job_lock is added, it will have to protect the

[PATCH v3 09/16] jobs: remove aiocontext locks since the functions are under BQL

2022-01-05 Thread Emanuele Giuseppe Esposito
In preparation to the job_lock/unlock patch, remove these aiocontext locks. The main reason these two locks are removed here is because they are inside a loop iterating on the jobs list. Once the job_lock is added, it will have to protect the whole loop, wrapping also the aiocontext