scsi: sg: Allow waiting for commands to complete on removed device
authorTony Battersby <tonyb@cybernetics.com>
Mon, 11 Jul 2022 14:51:32 +0000 (10:51 -0400)
committerMartin K. Petersen <martin.petersen@oracle.com>
Tue, 19 Jul 2022 02:33:04 +0000 (22:33 -0400)
commit3455607fd7be10b449f5135c00dc306b85dc0d21
tree123d7c365c6918c70eea96a6f6156b8a9f8563cd
parent6c20cc4885c5c11065a83c82dd8ce2074fe5c774
scsi: sg: Allow waiting for commands to complete on removed device

When a SCSI device is removed while in active use, currently sg will
immediately return -ENODEV on any attempt to wait for active commands that
were sent before the removal.  This is problematic for commands that use
SG_FLAG_DIRECT_IO since the data buffer may still be in use by the kernel
when userspace frees or reuses it after getting ENODEV, leading to
corrupted userspace memory (in the case of READ-type commands) or corrupted
data being sent to the device (in the case of WRITE-type commands).  This
has been seen in practice when logging out of a iscsi_tcp session, where
the iSCSI driver may still be processing commands after the device has been
marked for removal.

Change the policy to allow userspace to wait for active sg commands even
when the device is being removed.  Return -ENODEV only when there are no
more responses to read.

Link: https://lore.kernel.org/r/5ebea46f-fe83-2d0b-233d-d0dcb362dd0a@cybernetics.com
Cc: <stable@vger.kernel.org>
Acked-by: Douglas Gilbert <dgilbert@interlog.com>
Signed-off-by: Tony Battersby <tonyb@cybernetics.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/sg.c