scsi: qedi: Fix host removal with running sessions
authorMike Christie <michael.christie@oracle.com>
Wed, 9 Jun 2021 19:27:09 +0000 (14:27 -0500)
committerMartin K. Petersen <martin.petersen@oracle.com>
Thu, 10 Jun 2021 03:06:57 +0000 (23:06 -0400)
commitd1f2ce77638d681c1e34d597e432a3b28ce15c78
tree7e15b3c4c1f9c5bafeddf9046489b77f0593d256
parent2938bedd0efa0964db563999b9b3b866b2d1a090
scsi: qedi: Fix host removal with running sessions

qedi_clear_session_ctx() could race with the in-kernel or userspace driven
recovery/removal and we could access a NULL conn or do a double free.

We should be using iscsi_host_remove() to start the removal process from
the driver. It will start the in-kernel recovery and notify userspace that
the driver's scsi_hosts are being removed. iscsid will then drive the
session removal like is done when the logout command is run. When the
sessions are removed, iscsi_host_remove() will return so qedi can finish
knowing there are no running sessions and no new sessions will be allowed.

This also fixes an issue where we check for a NULL conn after already
accessing it introduced in commit 27e986289e73 ("scsi: iscsi: Drop suspend
calls from ep_disconnect") by just removing the function completely.

Link: https://lore.kernel.org/r/20210609192709.5094-1-michael.christie@oracle.com
Fixes: 27e986289e73 ("scsi: iscsi: Drop suspend calls from ep_disconnect")
Signed-off-by: Mike Christie <michael.christie@oracle.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/qedi/qedi_gbl.h
drivers/scsi/qedi/qedi_iscsi.c
drivers/scsi/qedi/qedi_main.c