patch 'net/txgbe: switch to FDIR when ntuple filter is full' has been queued to stable release 24.11.4

[Kevin Traynor <ktraynor@redhat.com>]

Hi,

FYI, your patch has been queued to stable release 24.11.4

Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet.
It will be pushed if I get no objections before 11/26/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/348509e3fe19edb1cfde5f36771775d916420e48

Thanks.

Kevin

---
From 348509e3fe19edb1cfde5f36771775d916420e48 Mon Sep 17 00:00:00 2001
From: Jiawen Wu <jiawenwu@trustnetic.com>
Date: Mon, 27 Oct 2025 11:15:39 +0800
Subject: [PATCH] net/txgbe: switch to FDIR when ntuple filter is full

[ upstream commit ccac5e093041f255f38a59ea472c6dee493ccc7c ]

Using ntuple filter has less performance loss on the hardware compared
to FDIR filter. So when the flow rule both match ntuple filter and FDIR
filter, ntuple filter will be created first. But there are only maximum
128 flow rules can be created on ntuple filter, it is far less than the
requirements of many users. So switch to use FDIR when ntuple filters
are full.

Fixes: 77a72b4d9dc0 ("net/txgbe: support ntuple filter add and delete")

Signed-off-by: Jiawen Wu <jiawenwu@trustnetic.com>
---
 drivers/net/txgbe/txgbe_ethdev.c | 5 ++++-
 drivers/net/txgbe/txgbe_ethdev.h | 1 +
 drivers/net/txgbe/txgbe_flow.c   | 8 ++++++++
 3 files changed, 13 insertions(+), 1 deletion(-)

diff --git a/drivers/net/txgbe/txgbe_ethdev.c b/drivers/net/txgbe/txgbe_ethdev.c
index 7df1841561..49ac1c1c82 100644
--- a/drivers/net/txgbe/txgbe_ethdev.c
+++ b/drivers/net/txgbe/txgbe_ethdev.c
@@ -854,4 +854,5 @@ static int txgbe_ntuple_filter_uninit(struct rte_eth_dev *eth_dev)
 	memset(filter_info->fivetuple_mask, 0,
 	       sizeof(uint32_t) * TXGBE_5TUPLE_ARRAY_SIZE);
+	filter_info->ntuple_is_full = false;
 
 	return 0;
@@ -4268,5 +4269,6 @@ txgbe_add_5tuple_filter(struct rte_eth_dev *dev,
 	}
 	if (i >= TXGBE_MAX_FTQF_FILTERS) {
-		PMD_DRV_LOG(ERR, "5tuple filters are full.");
+		PMD_DRV_LOG(INFO, "5tuple filters are full, switch to FDIR");
+		filter_info->ntuple_is_full = true;
 		return -ENOSYS;
 	}
@@ -4296,4 +4298,5 @@ txgbe_remove_5tuple_filter(struct rte_eth_dev *dev,
 	TAILQ_REMOVE(&filter_info->fivetuple_list, filter, entries);
 	rte_free(filter);
+	filter_info->ntuple_is_full = false;
 
 	wr32(hw, TXGBE_5TFDADDR(index), 0);
diff --git a/drivers/net/txgbe/txgbe_ethdev.h b/drivers/net/txgbe/txgbe_ethdev.h
index 288d9a43da..33d72e68d5 100644
--- a/drivers/net/txgbe/txgbe_ethdev.h
+++ b/drivers/net/txgbe/txgbe_ethdev.h
@@ -244,4 +244,5 @@ struct txgbe_filter_info {
 	uint32_t fivetuple_mask[TXGBE_5TUPLE_ARRAY_SIZE];
 	struct txgbe_5tuple_filter_list fivetuple_list;
+	bool ntuple_is_full;
 	/* store the SYN filter info */
 	uint32_t syn_info;
diff --git a/drivers/net/txgbe/txgbe_flow.c b/drivers/net/txgbe/txgbe_flow.c
index cb8004e48c..c87bb4dd63 100644
--- a/drivers/net/txgbe/txgbe_flow.c
+++ b/drivers/net/txgbe/txgbe_flow.c
@@ -581,6 +581,10 @@ txgbe_parse_ntuple_filter(struct rte_eth_dev *dev,
 			  struct rte_flow_error *error)
 {
+	struct txgbe_filter_info *filter_info = TXGBE_DEV_FILTER(dev);
 	int ret;
 
+	if (filter_info->ntuple_is_full)
+		return -ENOSYS;
+
 	ret = cons_parse_ntuple_filter(attr, pattern, actions, filter, error);
 
@@ -3201,4 +3205,5 @@ txgbe_flow_create(struct rte_eth_dev *dev,
 	struct txgbe_rss_conf_ele *rss_filter_ptr;
 	struct txgbe_flow_mem *txgbe_flow_mem_ptr;
+	struct txgbe_filter_info *filter_info = TXGBE_DEV_FILTER(dev);
 	uint8_t first_mask = FALSE;
 
@@ -3246,8 +3251,11 @@ txgbe_flow_create(struct rte_eth_dev *dev,
 			flow->filter_type = RTE_ETH_FILTER_NTUPLE;
 			return flow;
+		} else if (filter_info->ntuple_is_full) {
+			goto next;
 		}
 		goto out;
 	}
 
+next:
 	memset(&ethertype_filter, 0, sizeof(struct rte_eth_ethertype_filter));
 	ret = txgbe_parse_ethertype_filter(dev, attr, pattern,
-- 
2.51.0

---
  Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- -	2025-11-21 11:05:10.721727017 +0000
+++ 0036-net-txgbe-switch-to-FDIR-when-ntuple-filter-is-full.patch	2025-11-21 11:05:09.431201033 +0000
@@ -1 +1 @@
-From ccac5e093041f255f38a59ea472c6dee493ccc7c Mon Sep 17 00:00:00 2001
+From 348509e3fe19edb1cfde5f36771775d916420e48 Mon Sep 17 00:00:00 2001
@@ -5,0 +6,2 @@
+[ upstream commit ccac5e093041f255f38a59ea472c6dee493ccc7c ]
+
@@ -14 +15,0 @@
-Cc: stable@dpdk.org
@@ -24 +25 @@
-index f650c5b7a4..21f0711762 100644
+index 7df1841561..49ac1c1c82 100644
@@ -27 +28 @@
-@@ -894,4 +894,5 @@ int txgbe_ntuple_filter_uninit(struct rte_eth_dev *eth_dev)
+@@ -854,4 +854,5 @@ static int txgbe_ntuple_filter_uninit(struct rte_eth_dev *eth_dev)
@@ -33 +34 @@
-@@ -4496,5 +4497,6 @@ txgbe_add_5tuple_filter(struct rte_eth_dev *dev,
+@@ -4268,5 +4269,6 @@ txgbe_add_5tuple_filter(struct rte_eth_dev *dev,
@@ -41 +42 @@
-@@ -4527,4 +4529,5 @@ txgbe_remove_5tuple_filter(struct rte_eth_dev *dev,
+@@ -4296,4 +4298,5 @@ txgbe_remove_5tuple_filter(struct rte_eth_dev *dev,
@@ -46 +47 @@
- 	if (!txgbe_is_pf(TXGBE_DEV_HW(dev))) {
+ 	wr32(hw, TXGBE_5TFDADDR(index), 0);
@@ -48 +49 @@
-index 053aa1645f..1e7cc5ea80 100644
+index 288d9a43da..33d72e68d5 100644
@@ -51 +52 @@
-@@ -246,4 +246,5 @@ struct txgbe_filter_info {
+@@ -244,4 +244,5 @@ struct txgbe_filter_info {
@@ -58 +59 @@
-index d3113b6fc8..cd05ceffed 100644
+index cb8004e48c..c87bb4dd63 100644
@@ -78 +79 @@
-@@ -3246,4 +3251,6 @@ txgbe_flow_create(struct rte_eth_dev *dev,
+@@ -3246,8 +3251,11 @@ txgbe_flow_create(struct rte_eth_dev *dev,
@@ -85 +85,0 @@
-@@ -3255,4 +3262,5 @@ txgbe_flow_create(struct rte_eth_dev *dev,