[PATCH 0/3] net/hns3: bugfix on hns3

[PATCH 0/3] net/hns3: bugfix on hns3

[Jie Hai]

The patchset fixes some codes.

Dengdui Huang (1):
  net/hns3: fix simple Tx path incorrect free the mbuf

Jie Hai (2):
  net/hns3: remove pvid info dump for VF
  net/hns3: rename RAS module

 drivers/net/hns3/hns3_dump.c | 4 ++++
 drivers/net/hns3/hns3_intr.c | 4 ++--
 drivers/net/hns3/hns3_intr.h | 2 +-
 drivers/net/hns3/hns3_rxtx.c | 2 +-
 4 files changed, 8 insertions(+), 4 deletions(-)

-- 
2.22.0

[PATCH 1/3] net/hns3: fix simple Tx path incorrect free the mbuf

[Jie Hai]

From: Dengdui Huang <huangdengdui@huawei.com>

When RTE_ETH_TX_OFFLOAD_MBUF_FAST_FREE offload is not set,
use rte_pktmbuf_free_seg() to free the mbuf.

Fixes: 7ef933908f04 ("net/hns3: add simple Tx path")
Cc: stable@dpdk.org

Signed-off-by: Dengdui Huang <huangdengdui@huawei.com>
Signed-off-by: Jie Hai <haijie1@huawei.com>
---
 drivers/net/hns3/hns3_rxtx.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/hns3/hns3_rxtx.c b/drivers/net/hns3/hns3_rxtx.c
index 03bbbc435fac..09e39cb673cb 100644
--- a/drivers/net/hns3/hns3_rxtx.c
+++ b/drivers/net/hns3/hns3_rxtx.c
@@ -4006,7 +4006,7 @@ hns3_tx_free_buffer_simple(struct hns3_tx_queue *txq)
 		for (i = 0; i < txq->tx_rs_thresh; i++)
 			rte_prefetch0((tx_entry + i)->mbuf);
 		for (i = 0; i < txq->tx_rs_thresh; i++, tx_entry++) {
-			rte_mempool_put(tx_entry->mbuf->pool, tx_entry->mbuf);
+			rte_pktmbuf_free_seg(tx_entry->mbuf);
 			tx_entry->mbuf = NULL;
 		}
 
-- 
2.22.0

[PATCH 2/3] net/hns3: remove pvid info dump for VF

[Jie Hai]

Since the pvid status obtained from kernel varies on different
platform, and the pvid of VF can be accessed by 'ip link show'
command, so remove it in case of misunderstanding.

Fixes: 871e5a4f881b ("net/hns3: dump VLAN configuration info")
Cc: stable@dpdk.org

Signed-off-by: Jie Hai <haijie1@huawei.com>
---
 drivers/net/hns3/hns3_dump.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/drivers/net/hns3/hns3_dump.c b/drivers/net/hns3/hns3_dump.c
index 738dcb0c42fc..f21d32e6a2b5 100644
--- a/drivers/net/hns3/hns3_dump.c
+++ b/drivers/net/hns3/hns3_dump.c
@@ -693,6 +693,10 @@ hns3_get_vlan_tx_offload_cfg(FILE *file, struct hns3_hw *hw)
 static void
 hns3_get_port_pvid_info(FILE *file, struct hns3_hw *hw)
 {
+	struct hns3_adapter *hns = HNS3_DEV_HW_TO_ADAPTER(hw);
+	if (hns->is_vf)
+		return;
+
 	fprintf(file, "  - pvid status: %s\n",
 		hw->port_base_vlan_cfg.state ? "On" : "Off");
 }
-- 
2.22.0

[PATCH 3/3] net/hns3: rename RAS module

[Jie Hai]

Rename ROH_MAC module as HIMAC to avoid misunderstandings.

Fixes: 1c1eb759e9d7 ("net/hns3: support RAS process in Kunpeng 930")
Cc: stable@dpdk.org

Signed-off-by: Jie Hai <haijie1@huawei.com>
---
 drivers/net/hns3/hns3_intr.c | 4 ++--
 drivers/net/hns3/hns3_intr.h | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/net/hns3/hns3_intr.c b/drivers/net/hns3/hns3_intr.c
index 2de2b86b0297..db5c84061ca3 100644
--- a/drivers/net/hns3/hns3_intr.c
+++ b/drivers/net/hns3/hns3_intr.c
@@ -1432,8 +1432,8 @@ static const struct hns3_hw_mod_name hns3_hw_module_name[] = {
 		.module_name = MODULE_MASTER,
 		.msg = "MODULE_MASTER"
 	}, {
-		.module_name = MODULE_ROH_MAC,
-		.msg = "MODULE_ROH_MAC"
+		.module_name = MODULE_HIMAC,
+		.msg = "MODULE_HIMAC"
 	}
 };
 
diff --git a/drivers/net/hns3/hns3_intr.h b/drivers/net/hns3/hns3_intr.h
index 1edb07de361b..e88b00c9c986 100644
--- a/drivers/net/hns3/hns3_intr.h
+++ b/drivers/net/hns3/hns3_intr.h
@@ -104,7 +104,7 @@ enum hns3_mod_name_list {
 	MODULE_RCB_TX,
 	MODULE_TXDMA,
 	MODULE_MASTER,
-	MODULE_ROH_MAC,
+	MODULE_HIMAC,
 };
 
 enum hns3_err_type_list {
-- 
2.22.0

Re: [PATCH 0/3] net/hns3: bugfix on hns3

[Stephen Hemminger]

On Mon, 30 Dec 2024 14:54:02 +0800
Jie Hai <haijie1@huawei.com> wrote:

> From: Jie Hai <haijie1@huawei.com>
> To: <dev@dpdk.org>, <thomas@monjalon.net>, <ferruh.yigit@amd.com>,  <david.marchand@redhat.com>, <andrew.rybchenko@oktetlabs.ru>
> CC: <lihuisong@huawei.com>, <fengchengwen@huawei.com>, <haijie1@huawei.com>,  <huangdengdui@huawei.com>
> Subject: [PATCH 0/3] net/hns3: bugfix on hns3
> Date: Mon, 30 Dec 2024 14:54:02 +0800
> X-Mailer: git-send-email 2.22.0
> 
> The patchset fixes some codes.
> 
> Dengdui Huang (1):
>   net/hns3: fix simple Tx path incorrect free the mbuf
> 
> Jie Hai (2):
>   net/hns3: remove pvid info dump for VF
>   net/hns3: rename RAS module
> 
>  drivers/net/hns3/hns3_dump.c | 4 ++++
>  drivers/net/hns3/hns3_intr.c | 4 ++--
>  drivers/net/hns3/hns3_intr.h | 2 +-
>  drivers/net/hns3/hns3_rxtx.c | 2 +-
>  4 files changed, 8 insertions(+), 4 deletions(-)


While reviewing these, I noticed that hns3 driver is open coding its own version
of rte_pktmbuf_alloc(). That was a mistake, if a future changes to rte_mbuf.c change
the allocation logic, the driver will break.

Re: [PATCH 1/3] net/hns3: fix simple Tx path incorrect free the mbuf

[Stephen Hemminger]

On Mon, 30 Dec 2024 14:54:03 +0800
Jie Hai <haijie1@huawei.com> wrote:

> From: Jie Hai <haijie1@huawei.com>
> To: <dev@dpdk.org>, <thomas@monjalon.net>, <ferruh.yigit@amd.com>,  <david.marchand@redhat.com>, <andrew.rybchenko@oktetlabs.ru>, Chengwen Feng  <fengchengwen@huawei.com>, "Wei Hu (Xavier)" <xavier.huwei@huawei.com>,  Huisong Li <lihuisong@huawei.com>
> CC: <haijie1@huawei.com>, <huangdengdui@huawei.com>
> Subject: [PATCH 1/3] net/hns3: fix simple Tx path incorrect free the mbuf
> Date: Mon, 30 Dec 2024 14:54:03 +0800
> X-Mailer: git-send-email 2.22.0
> 
> From: Dengdui Huang <huangdengdui@huawei.com>
> 
> When RTE_ETH_TX_OFFLOAD_MBUF_FAST_FREE offload is not set,
> use rte_pktmbuf_free_seg() to free the mbuf.
> 
> Fixes: 7ef933908f04 ("net/hns3: add simple Tx path")
> Cc: stable@dpdk.org
> 
> Signed-off-by: Dengdui Huang <huangdengdui@huawei.com>
> Signed-off-by: Jie Hai <haijie1@huawei.com>

What about the fast free case which is using rte_mempool_put_bulk when
it should use rte_pktmbuf_free_bulk instead?