[dpdk-stable] patch 'net/bnxt: fix registration of VF async event completion ring' has been queued to LTS release 17.11.5

Yongseok Koh yskoh at mellanox.com
Fri Nov 30 00:10:45 CET 2018


Hi,

FYI, your patch has been queued to LTS release 17.11.5

Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet.
It will be pushed if I get no objections before 12/01/18. So please
shout if anyone has objections.

Also note that after the patch there's a diff of the upstream commit vs the patch applied
to the branch. If the code is different (ie: not only metadata diffs), due for example to
a change in context or macro names, please double check it.

Thanks.

Yongseok

---
>From db8802d754ce932c1aef911d991b49390d34c4cb Mon Sep 17 00:00:00 2001
From: Yongping Zhang <yongping.zhang at broadcom.com>
Date: Fri, 28 Sep 2018 19:00:01 -0700
Subject: [PATCH] net/bnxt: fix registration of VF async event completion ring

[ upstream commit 5f267bae3ee6da6ee158276055255b6ef5287caf ]

While registering vf's event completion ring to FW, use the wrong
macro, FW doesn't set up the event completion ring successfully,
VF can't receive any async event.

Fixes: 4535cad39515 ("net/bnxt: handle VF/PF initialization appropriately")

Signed-off-by: Yongping Zhang <yongping.zhang at broadcom.com>
Signed-off-by: Ajit Khaparde <ajit.khaparde at broadcom.com>
---
 drivers/net/bnxt/bnxt_hwrm.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/bnxt/bnxt_hwrm.c b/drivers/net/bnxt/bnxt_hwrm.c
index db3222f4e..3cadfc341 100644
--- a/drivers/net/bnxt/bnxt_hwrm.c
+++ b/drivers/net/bnxt/bnxt_hwrm.c
@@ -2744,7 +2744,7 @@ int bnxt_hwrm_vf_func_cfg_def_cp(struct bnxt *bp)
 	HWRM_PREP(req, FUNC_VF_CFG);
 
 	req.enables = rte_cpu_to_le_32(
-			HWRM_FUNC_CFG_INPUT_ENABLES_ASYNC_EVENT_CR);
+			HWRM_FUNC_VF_CFG_INPUT_ENABLES_ASYNC_EVENT_CR);
 	req.async_event_cr = rte_cpu_to_le_16(
 			bp->def_cp_ring->cp_ring_struct->fw_ring_id);
 	rc = bnxt_hwrm_send_message(bp, &req, sizeof(req));
-- 
2.11.0

---
  Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- -	2018-11-29 15:01:47.480975275 -0800
+++ 0051-net-bnxt-fix-registration-of-VF-async-event-completi.patch	2018-11-29 15:01:45.113959000 -0800
@@ -1,14 +1,15 @@
-From 5f267bae3ee6da6ee158276055255b6ef5287caf Mon Sep 17 00:00:00 2001
+From db8802d754ce932c1aef911d991b49390d34c4cb Mon Sep 17 00:00:00 2001
 From: Yongping Zhang <yongping.zhang at broadcom.com>
 Date: Fri, 28 Sep 2018 19:00:01 -0700
 Subject: [PATCH] net/bnxt: fix registration of VF async event completion ring
 
+[ upstream commit 5f267bae3ee6da6ee158276055255b6ef5287caf ]
+
 While registering vf's event completion ring to FW, use the wrong
 macro, FW doesn't set up the event completion ring successfully,
 VF can't receive any async event.
 
 Fixes: 4535cad39515 ("net/bnxt: handle VF/PF initialization appropriately")
-Cc: stable at dpdk.org
 
 Signed-off-by: Yongping Zhang <yongping.zhang at broadcom.com>
 Signed-off-by: Ajit Khaparde <ajit.khaparde at broadcom.com>
@@ -17,18 +18,18 @@
  1 file changed, 1 insertion(+), 1 deletion(-)
 
 diff --git a/drivers/net/bnxt/bnxt_hwrm.c b/drivers/net/bnxt/bnxt_hwrm.c
-index 62da254b9..51fe3a4c2 100644
+index db3222f4e..3cadfc341 100644
 --- a/drivers/net/bnxt/bnxt_hwrm.c
 +++ b/drivers/net/bnxt/bnxt_hwrm.c
-@@ -3001,7 +3001,7 @@ int bnxt_hwrm_vf_func_cfg_def_cp(struct bnxt *bp)
- 	HWRM_PREP(req, FUNC_VF_CFG, BNXT_USE_CHIMP_MB);
+@@ -2744,7 +2744,7 @@ int bnxt_hwrm_vf_func_cfg_def_cp(struct bnxt *bp)
+ 	HWRM_PREP(req, FUNC_VF_CFG);
  
  	req.enables = rte_cpu_to_le_32(
 -			HWRM_FUNC_CFG_INPUT_ENABLES_ASYNC_EVENT_CR);
 +			HWRM_FUNC_VF_CFG_INPUT_ENABLES_ASYNC_EVENT_CR);
  	req.async_event_cr = rte_cpu_to_le_16(
  			bp->def_cp_ring->cp_ring_struct->fw_ring_id);
- 	rc = bnxt_hwrm_send_message(bp, &req, sizeof(req), BNXT_USE_CHIMP_MB);
+ 	rc = bnxt_hwrm_send_message(bp, &req, sizeof(req));
 -- 
 2.11.0
 


More information about the stable mailing list