|
From: | Vladimir Sementsov-Ogievskiy |
Subject: | Re: [PATCH v11 16/21] blockjob: protect iostatus field in BlockJob struct |
Date: | Wed, 14 Sep 2022 16:52:16 +0300 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.8.1 |
On 8/26/22 16:20, Emanuele Giuseppe Esposito wrote:
iostatus is the only field (together with .job) that needs protection using the job mutex. It is set in the main loop (GLOBAL_STATE functions) but read in I/O code (block_job_error_action). In order to protect it, change block_job_iostatus_set_err to block_job_iostatus_set_err_locked(), always called under job lock. Signed-off-by: Emanuele Giuseppe Esposito<eesposit@redhat.com> Reviewed-by: Kevin Wolf<kwolf@redhat.com>
Reviewed-by: Vladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru> -- Best regards, Vladimir
[Prev in Thread] | Current Thread | [Next in Thread] |