patch 'net/ngbe: fix MAC control frame forwarding' has been queued to stable release 24.11.3
Kevin Traynor
ktraynor at redhat.com
Fri Jul 18 21:31:19 CEST 2025
Hi,
FYI, your patch has been queued to stable release 24.11.3
Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet.
It will be pushed if I get no objections before 07/23/25. 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. This will indicate if there was any rebasing
needed to apply to the stable branch. If there were code changes for rebasing
(ie: not only metadata diffs), please double check that the rebase was
correctly done.
Queued patches are on a temporary branch at:
https://github.com/kevintraynor/dpdk-stable
This queued commit can be viewed at:
https://github.com/kevintraynor/dpdk-stable/commit/e58a17a64a7eede07d9ba4994a7feff0286878a0
Thanks.
Kevin
---
>From e58a17a64a7eede07d9ba4994a7feff0286878a0 Mon Sep 17 00:00:00 2001
From: Jiawen Wu <jiawenwu at trustnetic.com>
Date: Fri, 13 Jun 2025 16:41:54 +0800
Subject: [PATCH] net/ngbe: fix MAC control frame forwarding
[ upstream commit 31ae872822cdc89cd00926c3e162781ef927ca24 ]
Test failure on the case "test_pause_fwd_port_stop_start", which expect
MAC control frame forwarding setting still working after port stop/start.
Fix the bug to pass the test case.
Fixes: f40e9f0e2278 ("net/ngbe: support flow control")
Signed-off-by: Jiawen Wu <jiawenwu at trustnetic.com>
---
drivers/net/ngbe/base/ngbe_hw.c | 9 +++++++++
drivers/net/ngbe/base/ngbe_type.h | 1 +
drivers/net/ngbe/ngbe_ethdev.c | 1 +
3 files changed, 11 insertions(+)
diff --git a/drivers/net/ngbe/base/ngbe_hw.c b/drivers/net/ngbe/base/ngbe_hw.c
index 1caeececf0..0b7f3e8bae 100644
--- a/drivers/net/ngbe/base/ngbe_hw.c
+++ b/drivers/net/ngbe/base/ngbe_hw.c
@@ -865,4 +865,13 @@ s32 ngbe_setup_fc_em(struct ngbe_hw *hw)
}
+ /*
+ * Reconfig mac ctrl frame fwd rule to make sure it still
+ * working after port stop/start.
+ */
+ wr32m(hw, NGBE_MACRXFLT, NGBE_MACRXFLT_CTL_MASK,
+ (hw->fc.mac_ctrl_frame_fwd ?
+ NGBE_MACRXFLT_CTL_NOPS : NGBE_MACRXFLT_CTL_DROP));
+ ngbe_flush(hw);
+
err = hw->phy.set_pause_adv(hw, reg_cu);
diff --git a/drivers/net/ngbe/base/ngbe_type.h b/drivers/net/ngbe/base/ngbe_type.h
index a987bbe25b..6a3cada617 100644
--- a/drivers/net/ngbe/base/ngbe_type.h
+++ b/drivers/net/ngbe/base/ngbe_type.h
@@ -109,4 +109,5 @@ struct ngbe_fc_info {
u32 low_water; /* Flow Ctrl Low-water */
u16 pause_time; /* Flow Control Pause timer */
+ u8 mac_ctrl_frame_fwd; /* Forward MAC control frames */
bool send_xon; /* Flow control send XON */
bool strict_ieee; /* Strict IEEE mode */
diff --git a/drivers/net/ngbe/ngbe_ethdev.c b/drivers/net/ngbe/ngbe_ethdev.c
index 08e87471f6..a8f847de8d 100644
--- a/drivers/net/ngbe/ngbe_ethdev.c
+++ b/drivers/net/ngbe/ngbe_ethdev.c
@@ -2421,4 +2421,5 @@ ngbe_flow_ctrl_set(struct rte_eth_dev *dev, struct rte_eth_fc_conf *fc_conf)
hw->fc.send_xon = fc_conf->send_xon;
hw->fc.disable_fc_autoneg = !fc_conf->autoneg;
+ hw->fc.mac_ctrl_frame_fwd = fc_conf->mac_ctrl_frame_fwd;
err = hw->mac.fc_enable(hw);
--
2.50.0
---
Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- - 2025-07-18 20:29:16.103336375 +0100
+++ 0145-net-ngbe-fix-MAC-control-frame-forwarding.patch 2025-07-18 20:29:11.121907916 +0100
@@ -1 +1 @@
-From 31ae872822cdc89cd00926c3e162781ef927ca24 Mon Sep 17 00:00:00 2001
+From e58a17a64a7eede07d9ba4994a7feff0286878a0 Mon Sep 17 00:00:00 2001
@@ -5,0 +6,2 @@
+[ upstream commit 31ae872822cdc89cd00926c3e162781ef927ca24 ]
+
@@ -11 +12,0 @@
-Cc: stable at dpdk.org
@@ -21 +22 @@
-index 6688ae6a31..bf09f8a817 100644
+index 1caeececf0..0b7f3e8bae 100644
@@ -24 +25 @@
-@@ -866,4 +866,13 @@ s32 ngbe_setup_fc_em(struct ngbe_hw *hw)
+@@ -865,4 +865,13 @@ s32 ngbe_setup_fc_em(struct ngbe_hw *hw)
@@ -39 +40 @@
-index 7a3b52ffd4..fc571c7457 100644
+index a987bbe25b..6a3cada617 100644
@@ -42 +43 @@
-@@ -113,4 +113,5 @@ struct ngbe_fc_info {
+@@ -109,4 +109,5 @@ struct ngbe_fc_info {
More information about the stable
mailing list