Re: [PATCH v7 06/18] jobs: protect jobs with job_lock/unlock

[Vladimir Sementsov-Ogievskiy]

On 6/16/22 16:18, Emanuele Giuseppe Esposito wrote:
>       }
> @@ -1939,7 +1943,9 @@ static void blockdev_backup_abort(BlkActionState *common)
>           aio_context = bdrv_get_aio_context(state->bs);
>           aio_context_acquire(aio_context);
>   
> -        job_cancel_sync(&state->job->job, true);
> +        WITH_JOB_LOCK_GUARD() {
> +            job_cancel_sync(&state->job->job, true);
> +        }
>   

Definitely this patch will be simplified, if we add job_cancel_sync_locked and 
make job_cancel_sync a wrapper on it, like other functions in 05 patch. And may 
be some other functions too.

--
Best regards,
Vladimir