diff options
author | James Smart <jsmart2021@gmail.com> | 2021-04-12 03:31:13 +0200 |
---|---|---|
committer | Martin K. Petersen <martin.petersen@oracle.com> | 2021-04-13 07:39:13 +0200 |
commit | fffd18ec6579c2d9c72b212169259062fe747888 (patch) | |
tree | 81fe45957baea5980c55d433ce676dbc6d57b8ba /kernel/async.c | |
parent | scsi: lpfc: Fix rmmod crash due to bad ring pointers to abort_iotag (diff) | |
download | linux-fffd18ec6579c2d9c72b212169259062fe747888.tar.xz linux-fffd18ec6579c2d9c72b212169259062fe747888.zip |
scsi: lpfc: Fix crash when a REG_RPI mailbox fails triggering a LOGO response
Fix a crash caused by a double put on the node when the driver completed an
ACC for an unsolicted abort on the same node. The second put was executed
by lpfc_nlp_not_used() and is wrong because the completion routine executes
the nlp_put when the iocbq was released. Additionally, the driver is
issuing a LOGO then immediately calls lpfc_nlp_set_state to put the node
into NPR. This call does nothing.
Remove the lpfc_nlp_not_used call and additional set_state in the
completion routine. Remove the lpfc_nlp_set_state post issue_logo. Isn't
necessary.
Link: https://lore.kernel.org/r/20210412013127.2387-3-jsmart2021@gmail.com
Co-developed-by: Justin Tee <justin.tee@broadcom.com>
Signed-off-by: Justin Tee <justin.tee@broadcom.com>
Signed-off-by: James Smart <jsmart2021@gmail.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
Diffstat (limited to 'kernel/async.c')
0 files changed, 0 insertions, 0 deletions