patches for DPDK stable branches
 help / color / mirror / Atom feed
From: luca.boccassi@gmail.com
To: Chengwen Feng <fengchengwen@huawei.com>
Cc: Dengdui Huang <huangdengdui@huawei.com>, dpdk stable <stable@dpdk.org>
Subject: patch 'net/hns3: fix queue TC configuration on VF' has been queued to stable release 22.11.9
Date: Mon, 14 Jul 2025 16:11:08 +0100	[thread overview]
Message-ID: <20250714151121.219866-13-luca.boccassi@gmail.com> (raw)
In-Reply-To: <20250714151121.219866-1-luca.boccassi@gmail.com>

Hi,

FYI, your patch has been queued to stable release 22.11.9

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/16/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/bluca/dpdk-stable

This queued commit can be viewed at:
https://github.com/bluca/dpdk-stable/commit/3a3e27e23f1b0ba93b6653e1fa90a78ee37fac3c

Thanks.

Luca Boccassi

---
From 3a3e27e23f1b0ba93b6653e1fa90a78ee37fac3c Mon Sep 17 00:00:00 2001
From: Chengwen Feng <fengchengwen@huawei.com>
Date: Tue, 1 Jul 2025 17:09:59 +0800
Subject: [PATCH] net/hns3: fix queue TC configuration on VF

[ upstream commit a542f48bc0ec83c296ae01ad691479c17caf99b5 ]

The VF cannot configure the mapping of queue to TC by directly writing
the register. Instead, the mapping must be modified by using firmware
command.

Fixes: bba636698316 ("net/hns3: support Rx/Tx and related operations")

Signed-off-by: Chengwen Feng <fengchengwen@huawei.com>
Signed-off-by: Dengdui Huang <huangdengdui@huawei.com>
---
 drivers/net/hns3/hns3_cmd.h  |  8 ++++++++
 drivers/net/hns3/hns3_rxtx.c | 26 +++++++++++++++++++++-----
 2 files changed, 29 insertions(+), 5 deletions(-)

diff --git a/drivers/net/hns3/hns3_cmd.h b/drivers/net/hns3/hns3_cmd.h
index ae797a7692..936e6455f0 100644
--- a/drivers/net/hns3/hns3_cmd.h
+++ b/drivers/net/hns3/hns3_cmd.h
@@ -178,6 +178,7 @@ enum hns3_opcode_type {
 
 	/* TQP commands */
 	HNS3_OPC_QUERY_TX_STATUS        = 0x0B03,
+	HNS3_OPC_TQP_TX_QUEUE_TC        = 0x0B04,
 	HNS3_OPC_QUERY_RX_STATUS        = 0x0B13,
 	HNS3_OPC_CFG_COM_TQP_QUEUE      = 0x0B20,
 	HNS3_OPC_RESET_TQP_QUEUE        = 0x0B22,
@@ -942,6 +943,13 @@ struct hns3_reset_tqp_queue_cmd {
 	uint8_t rsv[19];
 };
 
+struct hns3vf_tx_ring_tc_cmd {
+	uint16_t tqp_id;
+	uint16_t rsv1;
+	uint8_t  tc_id;
+	uint8_t  rsv2[19];
+};
+
 #define HNS3_CFG_RESET_MAC_B		3
 #define HNS3_CFG_RESET_FUNC_B		7
 #define HNS3_CFG_RESET_RCB_B		1
diff --git a/drivers/net/hns3/hns3_rxtx.c b/drivers/net/hns3/hns3_rxtx.c
index 83837e5c9a..56060730b0 100644
--- a/drivers/net/hns3/hns3_rxtx.c
+++ b/drivers/net/hns3/hns3_rxtx.c
@@ -1179,12 +1179,14 @@ hns3_init_txq(struct hns3_tx_queue *txq)
 	hns3_init_tx_queue_hw(txq);
 }
 
-static void
+static int
 hns3_init_tx_ring_tc(struct hns3_adapter *hns)
 {
+	struct hns3_cmd_desc desc;
+	struct hns3vf_tx_ring_tc_cmd *req = (struct hns3vf_tx_ring_tc_cmd *)desc.data;
 	struct hns3_hw *hw = &hns->hw;
 	struct hns3_tx_queue *txq;
-	int i, num;
+	int i, num, ret;
 
 	for (i = 0; i < HNS3_MAX_TC_NUM; i++) {
 		struct hns3_tc_queue_info *tc_queue = &hw->tc_queue[i];
@@ -1199,9 +1201,24 @@ hns3_init_tx_ring_tc(struct hns3_adapter *hns)
 			if (txq == NULL)
 				continue;
 
-			hns3_write_dev(txq, HNS3_RING_TX_TC_REG, tc_queue->tc);
+			if (!hns->is_vf) {
+				hns3_write_dev(txq, HNS3_RING_TX_TC_REG, tc_queue->tc);
+				continue;
+			}
+
+			hns3_cmd_setup_basic_desc(&desc, HNS3_OPC_TQP_TX_QUEUE_TC, false);
+			req->tqp_id = rte_cpu_to_le_16(num);
+			req->tc_id  = tc_queue->tc;
+			ret = hns3_cmd_send(hw, &desc, 1);
+			if (ret != 0) {
+				hns3_err(hw, "config Tx queue (%u)'s TC failed! ret = %d.",
+					 num, ret);
+				return ret;
+			}
 		}
 	}
+
+	return 0;
 }
 
 static int
@@ -1277,9 +1294,8 @@ hns3_init_tx_queues(struct hns3_adapter *hns)
 		txq = (struct hns3_tx_queue *)hw->fkq_data.tx_queues[i];
 		hns3_init_txq(txq);
 	}
-	hns3_init_tx_ring_tc(hns);
 
-	return 0;
+	return hns3_init_tx_ring_tc(hns);
 }
 
 /*
-- 
2.47.2

---
  Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- -	2025-07-14 16:09:41.195711707 +0100
+++ 0013-net-hns3-fix-queue-TC-configuration-on-VF.patch	2025-07-14 16:09:40.708448899 +0100
@@ -1 +1 @@
-From a542f48bc0ec83c296ae01ad691479c17caf99b5 Mon Sep 17 00:00:00 2001
+From 3a3e27e23f1b0ba93b6653e1fa90a78ee37fac3c Mon Sep 17 00:00:00 2001
@@ -5,0 +6,2 @@
+[ upstream commit a542f48bc0ec83c296ae01ad691479c17caf99b5 ]
+
@@ -11 +12,0 @@
-Cc: stable@dpdk.org
@@ -21 +22 @@
-index 4d707c13b2..e21a2b652f 100644
+index ae797a7692..936e6455f0 100644
@@ -32 +33 @@
-@@ -970,6 +971,13 @@ struct hns3_reset_tqp_queue_cmd {
+@@ -942,6 +943,13 @@ struct hns3_reset_tqp_queue_cmd {
@@ -47 +48 @@
-index f9fde3948a..785e06db9c 100644
+index 83837e5c9a..56060730b0 100644
@@ -50 +51 @@
-@@ -1180,12 +1180,14 @@ hns3_init_txq(struct hns3_tx_queue *txq)
+@@ -1179,12 +1179,14 @@ hns3_init_txq(struct hns3_tx_queue *txq)
@@ -67 +68 @@
-@@ -1200,9 +1202,24 @@ hns3_init_tx_ring_tc(struct hns3_adapter *hns)
+@@ -1199,9 +1201,24 @@ hns3_init_tx_ring_tc(struct hns3_adapter *hns)
@@ -93 +94 @@
-@@ -1278,9 +1295,8 @@ hns3_init_tx_queues(struct hns3_adapter *hns)
+@@ -1277,9 +1294,8 @@ hns3_init_tx_queues(struct hns3_adapter *hns)

  parent reply	other threads:[~2025-07-14 15:12 UTC|newest]

Thread overview: 121+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-06-12 21:06 patch 'net/i40e/base: remove unused variables' " luca.boccassi
2025-06-12 21:06 ` patch 'ci: bump tested distributions in GHA' " luca.boccassi
2025-06-12 21:06 ` patch 'net/fm10k/base: fix compilation warnings' " luca.boccassi
2025-06-12 21:06 ` patch 'net/ixgbe/base: correct definition of endianness macro' " luca.boccassi
2025-06-12 21:06 ` patch 'net/ixgbe/base: fix compilation warnings' " luca.boccassi
2025-06-12 21:06 ` patch 'net/i40e/base: fix unused value " luca.boccassi
2025-06-12 21:06 ` patch 'net/i40e/base: fix compiler " luca.boccassi
2025-06-12 21:06 ` patch 'acl: fix build with GCC 15 on aarch64' " luca.boccassi
2025-06-12 21:06 ` patch 'eal/linux: improve ASLR check' " luca.boccassi
2025-06-12 21:06 ` patch 'net/e1000: fix xstats name' " luca.boccassi
2025-06-12 21:06 ` patch 'net/e1000: fix EEPROM dump' " luca.boccassi
2025-06-12 21:06 ` patch 'net/ixgbe: fix port mask default value in filter' " luca.boccassi
2025-06-12 21:06 ` patch 'net/e1000: fix igb Tx queue offloads capability' " luca.boccassi
2025-06-12 21:06 ` patch 'vhost/crypto: fix cipher data length' " luca.boccassi
2025-06-12 21:06 ` patch 'crypto/virtio: fix cipher data source " luca.boccassi
2025-06-12 21:06 ` patch 'app/crypto-perf: fix AAD offset alignment' " luca.boccassi
2025-06-12 21:06 ` patch 'crypto/qat: fix out-of-place header bytes in AEAD raw API' " luca.boccassi
2025-06-12 21:06 ` patch 'crypto/qat: fix out-of-place chain/cipher/auth headers' " luca.boccassi
2025-06-12 21:06 ` patch 'net/mlx5: fix header modify action on group 0' " luca.boccassi
2025-06-12 21:06 ` patch 'net/mlx5: validate GTP PSC QFI width' " luca.boccassi
2025-06-12 21:06 ` patch 'net/mlx5: fix counter service cleanup on init failure' " luca.boccassi
2025-06-12 21:06 ` patch 'net/mlx5: remove unsupported flow meter action in HWS' " luca.boccassi
2025-06-12 21:06 ` patch 'net/mlx5: fix maximal queue size query' " luca.boccassi
2025-06-12 21:06 ` patch 'net/mlx5: align PF and VF/SF MAC address handling' " luca.boccassi
2025-06-12 21:06 ` patch 'app/testpmd: fix RSS hash key update' " luca.boccassi
2025-06-12 21:06 ` patch 'net/af_xdp: fix use after free in zero-copy Tx' " luca.boccassi
2025-06-12 21:06 ` patch 'net/hns3: fix integer overflow in interrupt unmap' " luca.boccassi
2025-06-12 21:06 ` patch 'net/hns3: fix memory leak on failure' " luca.boccassi
2025-06-12 21:06 ` patch 'net/hns3: fix extra wait for link up' " luca.boccassi
2025-06-12 21:06 ` patch 'net/hns3: fix memory leak for indirect flow action' " luca.boccassi
2025-06-12 21:06 ` patch 'net/hns3: fix interrupt rollback' " luca.boccassi
2025-06-12 21:06 ` patch 'net/hns3: fix divide by zero' " luca.boccassi
2025-06-12 21:06 ` patch 'net/hns3: fix resources release on reset' " luca.boccassi
2025-06-12 21:06 ` patch 'net/qede: fix use after free' " luca.boccassi
2025-06-12 21:06 ` patch 'bus/fslmc: " luca.boccassi
2025-06-12 21:06 ` patch 'net/null: fix packet copy' " luca.boccassi
2025-06-12 21:06 ` patch 'bus/vmbus: align ring buffer data to page boundary' " luca.boccassi
2025-06-12 21:06 ` patch 'bus/vmbus: use Hyper-V page size' " luca.boccassi
2025-06-12 21:06 ` patch 'net/netvsc: " luca.boccassi
2025-06-12 21:06 ` patch 'net/netvsc: add stats counters from VF' " luca.boccassi
2025-06-12 21:06 ` patch 'app/testpmd: relax number of TCs in DCB command' " luca.boccassi
2025-06-12 21:06 ` patch 'net/mana: check vendor ID when probing RDMA device' " luca.boccassi
2025-06-12 21:07 ` patch 'net/hns3: fix CRC data segment' " luca.boccassi
2025-06-12 21:07 ` patch 'net/tap: fix qdisc add failure handling' " luca.boccassi
2025-06-12 21:07 ` patch 'net/mlx5: fix VLAN stripping on hairpin queue' " luca.boccassi
2025-06-12 21:07 ` patch 'mem: fix lockup on address space shortage' " luca.boccassi
2025-06-12 21:07 ` patch 'test/malloc: improve resiliency' " luca.boccassi
2025-06-12 21:07 ` patch 'trace: fix overflow in per-lcore trace buffer' " luca.boccassi
2025-06-12 21:07 ` patch 'common/cnxk: fix E-tag pattern parsing' " luca.boccassi
2025-06-12 21:07 ` patch 'common/cnxk: fix CQ tail drop' " luca.boccassi
2025-06-12 21:07 ` patch 'net/cnxk: fix descriptor count update on reconfig' " luca.boccassi
2025-06-12 21:07 ` patch 'ethdev: fix error struct in flow configure' " luca.boccassi
2025-06-12 21:07 ` patch 'net/ice/base: fix integer overflow' " luca.boccassi
2025-06-12 21:07 ` patch 'net/ice/base: fix typo in device ID description' " luca.boccassi
2025-06-12 21:07 ` patch 'common/dpaax: fix PDCP key command race condition' " luca.boccassi
2025-06-12 21:07 ` patch 'common/dpaax: fix PDCP AES only 12-bit SN' " luca.boccassi
2025-06-12 21:07 ` patch 'crypto/virtio: add request check on request side' " luca.boccassi
2025-06-12 21:07 ` patch 'crypto/virtio: fix driver cleanup' " luca.boccassi
2025-06-12 21:07 ` patch 'ethdev: keep promiscuous/allmulti value before disabling' " luca.boccassi
2025-06-12 21:07 ` patch 'eal: fix return value of lcore role' " luca.boccassi
2025-06-12 21:07 ` patch 'eal: warn if no lcore is available' " luca.boccassi
2025-06-12 21:07 ` patch 'bus: cleanup device lists' " luca.boccassi
2025-06-12 21:07 ` patch 'eal/linux: unregister alarm callback before free' " luca.boccassi
2025-06-12 21:07 ` patch 'eal/freebsd: " luca.boccassi
2025-06-12 21:07 ` patch 'bus/pci/bsd: fix device existence check' " luca.boccassi
2025-06-17 18:38   ` Jake Freeland
2025-06-12 21:07 ` patch 'power/intel_uncore: fix crash closing uninitialized driver' " luca.boccassi
2025-06-12 21:07 ` patch 'pcapng: fix null dereference in close' " luca.boccassi
2025-06-12 21:07 ` patch 'net/mlx5: avoid setting kernel MTU if not needed' " luca.boccassi
2025-06-12 21:07 ` patch 'net/mlx5: fix hypervisor detection in VLAN workaround' " luca.boccassi
2025-06-12 21:07 ` patch 'net/hns3: check requirement for hardware GRO' " luca.boccassi
2025-06-12 21:07 ` patch 'net/hns3: allow Tx vector when fast free not enabled' " luca.boccassi
2025-06-12 21:07 ` patch 'net/hns3: allow Rx vector mode with VLAN filter' " luca.boccassi
2025-06-30 12:25   ` patch 'common/mlx5: fix extraction of auxiliary device name' " luca.boccassi
2025-06-30 12:25     ` patch 'net/mlx5: fix crash on age query with indirect conntrack' " luca.boccassi
2025-07-02 15:32       ` Khadem Ullah
2025-07-03 11:57         ` Khadem Ullah
2025-06-30 12:25     ` patch 'net/mlx5: fix WQE size calculation for Tx queue' " luca.boccassi
2025-06-30 12:25     ` patch 'net/ixgbe: fix indentation' " luca.boccassi
2025-06-30 12:25     ` patch 'net/ice: fix querying RSS hash for DCF' " luca.boccassi
2025-06-30 12:25     ` patch 'net/iavf: fix VLAN strip setting after enabling filter' " luca.boccassi
2025-06-30 12:25     ` patch 'common/cnxk: fix null pointer checks' " luca.boccassi
2025-06-30 12:25     ` patch 'vhost: search virtqueues driver data in read-only area' " luca.boccassi
2025-06-30 12:25     ` patch 'net/virtio: fix check of threshold for Tx freeing' " luca.boccassi
2025-07-04 12:01       ` Maxime Coquelin
2025-07-04 13:13         ` Luca Boccassi
2025-06-30 12:25     ` patch 'eal/unix: fix log message for madvise failure' " luca.boccassi
2025-06-30 12:25     ` patch 'common/cnxk: fix qsize in CPT iq enable' " luca.boccassi
2025-06-30 12:25     ` patch 'crypto/qat: fix out-of-place chain/cipher/auth headers' " luca.boccassi
2025-06-30 12:25     ` patch 'examples/ipsec-secgw: fix crash with IPv6' " luca.boccassi
2025-06-30 12:25     ` patch 'examples/ipsec-secgw: fix crash in event vector mode' " luca.boccassi
2025-06-30 12:25     ` patch 'test/crypto: fix auth and cipher case IV length' " luca.boccassi
2025-06-30 12:25     ` patch 'test/crypto: set to null after freeing operation' " luca.boccassi
2025-06-30 12:25     ` patch 'crypto/openssl: include private exponent in RSA session' " luca.boccassi
2025-06-30 12:25     ` patch 'test/crypto: fix RSA decrypt validation' " luca.boccassi
2025-06-30 12:25     ` patch 'event/dlb2: fix dequeue with CQ depth <= 16' " luca.boccassi
2025-06-30 12:25     ` patch 'event/dlb2: fix validaton of LDB port COS ID arguments' " luca.boccassi
2025-06-30 12:25     ` patch 'event/dlb2: fix num single link ports for DLB2.5' " luca.boccassi
2025-06-30 12:25     ` patch 'event/dlb2: fix default credits based on HW version' " luca.boccassi
2025-06-30 12:25     ` patch 'app/eventdev: fix number of releases sent during cleanup' " luca.boccassi
2025-07-14 15:10       ` patch 'net/txgbe: fix ntuple filter parsing' " luca.boccassi
2025-07-14 15:10         ` patch 'net/txgbe: fix raw pattern match for FDIR rule' " luca.boccassi
2025-07-14 15:10         ` patch 'net/txgbe: fix packet type for FDIR filter' " luca.boccassi
2025-07-14 15:10         ` patch 'net/txgbe: fix to create FDIR filter for SCTP packet' " luca.boccassi
2025-07-14 15:11         ` patch 'net/txgbe: fix FDIR perfect mode for IPv6' " luca.boccassi
2025-07-14 15:11         ` patch 'net/txgbe: fix reserved extra FDIR headroom' " luca.boccassi
2025-07-14 15:11         ` patch 'net/txgbe: fix MAC control frame forwarding' " luca.boccassi
2025-07-14 15:11         ` patch 'net/ngbe: " luca.boccassi
2025-07-14 15:11         ` patch 'net/txgbe: fix device statistics' " luca.boccassi
2025-07-14 15:11         ` patch 'net/ngbe: " luca.boccassi
2025-07-14 15:11         ` patch 'net/txgbe: restrict VLAN strip configuration on VF' " luca.boccassi
2025-07-14 15:11         ` patch 'net/txgbe: add LRO flag in mbuf when enabled' " luca.boccassi
2025-07-14 15:11         ` luca.boccassi [this message]
2025-07-14 15:11         ` patch 'net/bonding: avoid RSS RETA update in flow isolation mode' " luca.boccassi
2025-07-14 15:11         ` patch 'net/iavf: fix VLAN strip disabling for ADQ v2 capability' " luca.boccassi
2025-07-14 15:11         ` patch 'net/i40e: fix RSS on plain IPv4' " luca.boccassi
2025-07-14 15:11         ` patch 'bus/auxiliary: fix crash in cleanup' " luca.boccassi
2025-07-14 15:11         ` patch 'net/mlx5: fix masked indirect age action validation' " luca.boccassi
2025-07-14 15:11         ` patch 'examples/multi_process: fix ports cleanup on exit' " luca.boccassi
2025-07-14 15:11         ` patch 'examples/ntb: check more heap allocations' " luca.boccassi
2025-07-14 15:11         ` patch 'examples/ipsec-secgw: fix number of queue pairs' " luca.boccassi

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20250714151121.219866-13-luca.boccassi@gmail.com \
    --to=luca.boccassi@gmail.com \
    --cc=fengchengwen@huawei.com \
    --cc=huangdengdui@huawei.com \
    --cc=stable@dpdk.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).