> On 2 Apr 2024, at 12:40 AM, Alexander Wetzel <alexan...@wetzel-home.de> wrote:
>
> commit 27f58c04a8f4 ("scsi: sg: Avoid sg device teardown race")
> introduced an incorrect WARN_ON_ONCE() and missed a sequence where
> sg_device_destroy() was used after scsi_device_put().
>
> sg_device_destroy() is accessing the parent scsi_device request_queue which
> will already be set to NULL when the preceding call to scsi_device_put()
> removed the last reference to the parent scsi_device.
>
> Drop the incorrect WARN_ON_ONCE() - allowing more than one concurrent
> access to the sg device - and make sure sg_device_destroy() is not used
> after scsi_device_put() in the error handling.
>
> Link:
> https://lore.kernel.org/all/5375b275-d137-4d5f-be25-6af8acae4...@linux.ibm.com
> Fixes: 27f58c04a8f4 ("scsi: sg: Avoid sg device teardown race")
> Cc: sta...@vger.kernel.org
> Signed-off-by: Alexander Wetzel <alexan...@wetzel-home.de>
> ---
Thanks for the fix. I tested this patch and confirm it fixes the reported
problem.
Tested-by: Sachin Sant <sach...@linux.ibm.com>
— Sachin