Re: [PATCH 1/2] scsi: alua: Revert "Move a scsi_device_put() call out of alua_check_vpd()"

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




> On 18-Nov-2022, at 12:06 AM, Bart Van Assche <bvanassche@xxxxxxx> wrote:
> 
> There is a bug in commit 0b25e17e9018 ("scsi: alua: Move a
> scsi_device_put() call out of alua_check_vpd()"): that patch may cause
> alua_rtpg_queue() callers to call scsi_device_put() even if that function
> should not be called. Revert that commit to prepare for a different
> solution.
> 
> Cc: Hannes Reinecke <hare@xxxxxxx>
> Cc: Martin Wilck <mwilck@xxxxxxxx>
> Cc: Sachin Sant <sachinp@xxxxxxxxxxxxx>
> Cc: Benjamin Block <bblock@xxxxxxxxxxxxx>
> Reported-by: Sachin Sant <sachinp@xxxxxxxxxxxxx>
> Reported-by: Benjamin Block <bblock@xxxxxxxxxxxxx>
> Signed-off-by: Bart Van Assche <bvanassche@xxxxxxx>
> ---
> drivers/scsi/device_handler/scsi_dh_alua.c | 23 ++++++++--------------
> 1 file changed, 8 insertions(+), 15 deletions(-)

Thanks for the patch. 
Tested it on top of next-20221117. The reported warning is not seen.

Tested-by: Sachin Sant <sachinp@xxxxxxxxxxxxx <mailto:sachinp@xxxxxxxxxxxxx>>

- Sachin



[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]

  Powered by Linux