diff options
| author |   Justin Tee <justin.tee@broadcom.com> | 2023-01-09 15:33:11 -0800 |
|---|---|---|
| committer |   Martin K. Petersen <martin.petersen@oracle.com> | 2023-01-12 00:03:14 -0500 |
| commit | ecdf4ddf4eb7a9135abdb358e98a8e6c1e8effe6 (patch) | |
| tree | b4ce80aa7094f3d6bacdbe7baf9d77ad6d23f631 /lib/parser.c | |
| parent | scsi: lpfc: Remove redundant clean up code in disable_vport() (diff) | |
| download | linux-rng-ecdf4ddf4eb7a9135abdb358e98a8e6c1e8effe6.tar.xz linux-rng-ecdf4ddf4eb7a9135abdb358e98a8e6c1e8effe6.zip | |
scsi: lpfc: Remove duplicate ndlp kref decrement in lpfc_cleanup_rpis()
With faulty cables in PT2PT topology, an unintentional ndlp double kref
decrement can occur.
If a FLOGI request is outstanding before the link goes down, the missing
FLOGI_ACC causes an F_Port ndlp to remain in the UNUSED state. During link
down, lpfc_cleanup_rpis() is called and decrements an ndlp kref.
Additionally, when the driver later decides to abort the FLOGI, the FLOGI
completion handler decrements the ndlp kref a second time.
Remove duplicate clean up logic in lpfc_cleanup_rpis() because the updated
FLOGI completion handler already handles the ndlp kref decrement.
Signed-off-by: Justin Tee <justin.tee@broadcom.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
Diffstat (limited to 'lib/parser.c')
0 files changed, 0 insertions, 0 deletions