This is a note to let you know that I've just added the patch titled
bnxt_en: Fix MSIX request logic for RDMA driver.
to the 5.4-stable tree which can be found at:
http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary
The filename of the patch is:
bnxt_en-fix-msix-request-logic-for-rdma-driver.patch
and it can be found in the queue-5.4 subdirectory.
If you, or anyone else, feels it should not be added to the stable tree,
please let <stable@xxxxxxxxxxxxxxx> know about it.
>From foo@baz Wed 01 Jan 2020 10:35:31 PM CET
From: Michael Chan <michael.chan@xxxxxxxxxxxx>
Date: Tue, 10 Dec 2019 02:49:07 -0500
Subject: bnxt_en: Fix MSIX request logic for RDMA driver.
From: Michael Chan <michael.chan@xxxxxxxxxxxx>
[ Upstream commit 0c722ec0a289c7f6b53f89bad1cfb7c4db3f7a62 ]
The logic needs to check both bp->total_irqs and the reserved IRQs in
hw_resc->resv_irqs if applicable and see if both are enough to cover
the L2 and RDMA requested vectors. The current code is only checking
bp->total_irqs and can fail in some code paths, such as the TX timeout
code path with the RDMA driver requesting vectors after recovery. In
this code path, we have not reserved enough MSIX resources for the
RDMA driver yet.
Fixes: 75720e6323a1 ("bnxt_en: Keep track of reserved IRQs.")
Signed-off-by: Michael Chan <michael.chan@xxxxxxxxxxxx>
Signed-off-by: David S. Miller <davem@xxxxxxxxxxxxx>
Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
---
drivers/net/ethernet/broadcom/bnxt/bnxt_ulp.c | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)
--- a/drivers/net/ethernet/broadcom/bnxt/bnxt_ulp.c
+++ b/drivers/net/ethernet/broadcom/bnxt/bnxt_ulp.c
@@ -113,8 +113,10 @@ static int bnxt_req_msix_vecs(struct bnx
{
struct net_device *dev = edev->net;
struct bnxt *bp = netdev_priv(dev);
+ struct bnxt_hw_resc *hw_resc;
int max_idx, max_cp_rings;
int avail_msix, idx;
+ int total_vecs;
int rc = 0;
ASSERT_RTNL();
@@ -142,7 +144,10 @@ static int bnxt_req_msix_vecs(struct bnx
}
edev->ulp_tbl[ulp_id].msix_base = idx;
edev->ulp_tbl[ulp_id].msix_requested = avail_msix;
- if (bp->total_irqs < (idx + avail_msix)) {
+ hw_resc = &bp->hw_resc;
+ total_vecs = idx + avail_msix;
+ if (bp->total_irqs < total_vecs ||
+ (BNXT_NEW_RM(bp) && hw_resc->resv_irqs < total_vecs)) {
if (netif_running(dev)) {
bnxt_close_nic(bp, true, false);
rc = bnxt_open_nic(bp, true, false);
@@ -156,7 +161,6 @@ static int bnxt_req_msix_vecs(struct bnx
}
if (BNXT_NEW_RM(bp)) {
- struct bnxt_hw_resc *hw_resc = &bp->hw_resc;
int resv_msix;
resv_msix = hw_resc->resv_irqs - bp->cp_nr_rings;
Patches currently in stable-queue which might be from michael.chan@xxxxxxxxxxxx are
queue-5.4/bnxt_en-fix-bp-fw_health-allocation-and-free-logic.patch
queue-5.4/bnxt_en-fix-the-logic-that-creates-the-health-reporters.patch
queue-5.4/bnxt_en-return-error-if-fw-returns-more-data-than-dump-length.patch
queue-5.4/bnxt_en-fix-msix-request-logic-for-rdma-driver.patch
queue-5.4/bnxt_en-free-context-memory-in-the-open-path-if-firmware-has-been-reset.patch
queue-5.4/bnxt-apply-computed-clamp-value-for-coalece-parameter.patch
queue-5.4/bnxt_en-remove-unnecessary-null-checks-for-fw_health.patch
queue-5.4/bnxt_en-add-missing-devlink-health-reporters-for-vfs.patch
