|
Daniel Wagner |
8dcfc3 |
From: James Smart <jsmart2021@gmail.com>
|
|
Daniel Wagner |
8dcfc3 |
Date: Fri, 3 Dec 2021 16:26:38 -0800
|
|
Daniel Wagner |
8dcfc3 |
Subject: scsi: lpfc: Fix lpfc_force_rscn ndlp kref imbalance
|
|
Daniel Wagner |
8dcfc3 |
Patch-mainline: Queued in subsystem maintainer repository
|
|
Daniel Wagner |
8dcfc3 |
Git-repo: git://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi.git
|
|
Daniel Wagner |
8dcfc3 |
Git-commit: 7576d48c64f36f6fea9df2882f710a474fa35f40
|
|
Daniel Wagner |
8dcfc3 |
References: bsc1192145
|
|
Daniel Wagner |
8dcfc3 |
|
|
Daniel Wagner |
8dcfc3 |
Issuing lpfc_force_rscn twice results in an ndlp kref use-after-free call
|
|
Daniel Wagner |
8dcfc3 |
trace.
|
|
Daniel Wagner |
8dcfc3 |
|
|
Daniel Wagner |
8dcfc3 |
A prior patch reworked the get/put handling by ensuring nlp_get was done
|
|
Daniel Wagner |
8dcfc3 |
before WQE submission and a put was done in the completion path.
|
|
Daniel Wagner |
8dcfc3 |
Unfortunately, the issue_els_rscn path had a piece of legacy code that did
|
|
Daniel Wagner |
8dcfc3 |
a nlp_put, causing an imbalance on the ref counts.
|
|
Daniel Wagner |
8dcfc3 |
|
|
Daniel Wagner |
8dcfc3 |
Fixed by removing the unnecessary legacy code snippet.
|
|
Daniel Wagner |
8dcfc3 |
|
|
Daniel Wagner |
8dcfc3 |
Link: https://lore.kernel.org/r/20211204002644.116455-4-jsmart2021@gmail.com
|
|
Daniel Wagner |
8dcfc3 |
Fixes: 4430f7fd09ec ("scsi: lpfc: Rework locations of ndlp reference taking")
|
|
Daniel Wagner |
8dcfc3 |
Cc: <stable@vger.kernel.org> # v5.11+
|
|
Daniel Wagner |
8dcfc3 |
Co-developed-by: Justin Tee <justin.tee@broadcom.com>
|
|
Daniel Wagner |
8dcfc3 |
Signed-off-by: Justin Tee <justin.tee@broadcom.com>
|
|
Daniel Wagner |
8dcfc3 |
Signed-off-by: James Smart <jsmart2021@gmail.com>
|
|
Daniel Wagner |
8dcfc3 |
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
|
|
Daniel Wagner |
8dcfc3 |
Acked-by: Daniel Wagner <dwagner@suse.de>
|
|
Daniel Wagner |
8dcfc3 |
---
|
|
Daniel Wagner |
8dcfc3 |
drivers/scsi/lpfc/lpfc_els.c | 5 -----
|
|
Daniel Wagner |
8dcfc3 |
1 file changed, 5 deletions(-)
|
|
Daniel Wagner |
8dcfc3 |
|
|
Daniel Wagner |
8dcfc3 |
--- a/drivers/scsi/lpfc/lpfc_els.c
|
|
Daniel Wagner |
8dcfc3 |
+++ b/drivers/scsi/lpfc/lpfc_els.c
|
|
Daniel Wagner |
8dcfc3 |
@@ -3490,11 +3490,6 @@ lpfc_issue_els_rscn(struct lpfc_vport *v
|
|
Daniel Wagner |
8dcfc3 |
return 1;
|
|
Daniel Wagner |
8dcfc3 |
}
|
|
Daniel Wagner |
8dcfc3 |
|
|
Daniel Wagner |
8dcfc3 |
- /* This will cause the callback-function lpfc_cmpl_els_cmd to
|
|
Daniel Wagner |
8dcfc3 |
- * trigger the release of node.
|
|
Daniel Wagner |
8dcfc3 |
- */
|
|
Daniel Wagner |
8dcfc3 |
- if (!(vport->fc_flag & FC_PT2PT))
|
|
Daniel Wagner |
8dcfc3 |
- lpfc_nlp_put(ndlp);
|
|
Daniel Wagner |
8dcfc3 |
return 0;
|
|
Daniel Wagner |
8dcfc3 |
}
|
|
Daniel Wagner |
8dcfc3 |
|