From: Junlong Wang <wang.junlong1@zte.com.cn>
To: stephen@networkplumber.org
Cc: dev@dpdk.org, Junlong Wang <wang.junlong1@zte.com.cn>
Subject: [PATCH v2 3/5] net/zxdh: modify some VLAN related issues
Date: Tue, 17 Jun 2025 10:05:01 +0800 [thread overview]
Message-ID: <20250617020503.3554511-3-wang.junlong1@zte.com.cn> (raw)
In-Reply-To: <20250617020503.3554511-1-wang.junlong1@zte.com.cn>
[-- Attachment #1.1.1: Type: text/plain, Size: 8322 bytes --]
modify some vlan related issues,
which were discovered during the testing process.
Fixes: b4f996b7fcf6 ("net/zxdh: optimize VLAN filter and offload")
Signed-off-by: Junlong Wang <wang.junlong1@zte.com.cn>
---
drivers/net/zxdh/zxdh_ethdev.c | 9 ++++++++-
drivers/net/zxdh/zxdh_ethdev_ops.c | 4 ++--
drivers/net/zxdh/zxdh_msg.c | 28 ++++++++++++++++++++++++++--
drivers/net/zxdh/zxdh_tables.c | 28 ++++++++++++++++++++++------
drivers/net/zxdh/zxdh_tables.h | 7 ++++++-
5 files changed, 64 insertions(+), 12 deletions(-)
diff --git a/drivers/net/zxdh/zxdh_ethdev.c b/drivers/net/zxdh/zxdh_ethdev.c
index 74018821d1..d92df46d11 100644
--- a/drivers/net/zxdh/zxdh_ethdev.c
+++ b/drivers/net/zxdh/zxdh_ethdev.c
@@ -1907,6 +1907,7 @@ zxdh_np_init(struct rte_eth_dev *eth_dev)
static int
zxdh_tables_init(struct rte_eth_dev *dev)
{
+ struct zxdh_hw *hw = dev->data->dev_private;
int ret = 0;
ret = zxdh_port_attr_init(dev);
@@ -1927,12 +1928,18 @@ zxdh_tables_init(struct rte_eth_dev *dev)
return ret;
}
- ret = zxdh_vlan_filter_table_init(dev);
+ ret = zxdh_vlan_filter_table_init(hw, hw->vport.vport);
if (ret) {
PMD_DRV_LOG(ERR, "vlan filter table init failed");
return ret;
}
+ ret = zxdh_port_vlan_table_init(hw, hw->vport.vport);
+ if (ret) {
+ PMD_DRV_LOG(ERR, "port vlan table init failed");
+ return ret;
+ }
+
return ret;
}
diff --git a/drivers/net/zxdh/zxdh_ethdev_ops.c b/drivers/net/zxdh/zxdh_ethdev_ops.c
index c7a9dc7854..3eed388dac 100644
--- a/drivers/net/zxdh/zxdh_ethdev_ops.c
+++ b/drivers/net/zxdh/zxdh_ethdev_ops.c
@@ -553,7 +553,7 @@ zxdh_dev_mac_addr_add(struct rte_eth_dev *dev, struct rte_ether_addr *mac_addr,
zxdh_msg_reply_info,
ZXDH_ST_SZ_BYTES(msg_reply_info));
if (ret) {
- flag = ZXDH_GET(mac_reply_msg,
+ uint8_t flag = ZXDH_GET(mac_reply_msg,
mac_reply_msg_addr, mac_flag);
if (flag == ZXDH_EEXIST_MAC_FLAG) {
PMD_DRV_LOG(ERR, "pf mac add failed mac is in used");
@@ -895,7 +895,7 @@ zxdh_dev_vlan_offload_set(struct rte_eth_dev *dev, int mask)
return -EAGAIN;
}
} else {
- msg.data.vlan_filter_set_msg.enable = true;
+ msg.data.vlan_filter_set_msg.enable = false;
zxdh_msg_head_build(hw, ZXDH_VLAN_FILTER_SET, &msg);
ret = zxdh_vf_send_msg_to_pf(hw->eth_dev, &msg,
sizeof(struct zxdh_msg_info), NULL, 0);
diff --git a/drivers/net/zxdh/zxdh_msg.c b/drivers/net/zxdh/zxdh_msg.c
index 452c7ab2e3..e607ea1dab 100644
--- a/drivers/net/zxdh/zxdh_msg.c
+++ b/drivers/net/zxdh/zxdh_msg.c
@@ -1233,6 +1233,24 @@ zxdh_vf_promisc_uninit(struct zxdh_hw *hw, union zxdh_virport_num vport)
return ret;
}
+static int
+zxdh_vf_vlan_table_init(struct zxdh_hw *hw, uint16_t vport)
+{
+ int ret = 0;
+ ret = zxdh_vlan_filter_table_init(hw, vport);
+ if (ret) {
+ PMD_DRV_LOG(ERR, "vf vlan filter table init failed, code:%d", ret);
+ return -1;
+ }
+
+ ret = zxdh_port_vlan_table_init(hw, vport);
+ if (ret) {
+ PMD_DRV_LOG(ERR, "vf port vlan table init failed, code:%d", ret);
+ return -1;
+ }
+ return ret;
+}
+
static int
zxdh_vf_port_init(struct zxdh_hw *pf_hw, uint16_t vport, void *cfg_data,
void *res_info, uint16_t *res_len)
@@ -1252,9 +1270,8 @@ zxdh_vf_port_init(struct zxdh_hw *pf_hw, uint16_t vport, void *cfg_data,
port_attr.pf_vfid = pf_hw->vfid;
port_attr.hash_search_index = pf_hw->hash_search_index;
port_attr.port_base_qid = vf_init_msg->base_qid;
- uint16_t vfid = zxdh_vport_to_vfid(port);
- ret = zxdh_set_port_attr(pf_hw, vfid, &port_attr);
+ ret = zxdh_set_port_attr(pf_hw, vport, &port_attr);
if (ret) {
PMD_DRV_LOG(ERR, "set vport attr failed, code:%d", ret);
goto proc_end;
@@ -1265,6 +1282,13 @@ zxdh_vf_port_init(struct zxdh_hw *pf_hw, uint16_t vport, void *cfg_data,
PMD_DRV_LOG(ERR, "vf_promisc_table_init failed, code:%d", ret);
goto proc_end;
}
+
+ ret = zxdh_vf_vlan_table_init(pf_hw, vport);
+ if (ret) {
+ PMD_DRV_LOG(ERR, "vf vlan table init failed, code:%d", ret);
+ goto proc_end;
+ }
+
ZXDH_SET(msg_reply_body, res_info, flag, ZXDH_REPS_SUCC);
*res_len = sizeof(uint8_t);
diff --git a/drivers/net/zxdh/zxdh_tables.c b/drivers/net/zxdh/zxdh_tables.c
index 33f161c4cd..a213df4f23 100644
--- a/drivers/net/zxdh/zxdh_tables.c
+++ b/drivers/net/zxdh/zxdh_tables.c
@@ -699,10 +699,11 @@ zxdh_dev_multicast_table_set(struct zxdh_hw *hw, uint16_t vport, bool enable)
}
int
-zxdh_vlan_filter_table_init(struct rte_eth_dev *dev)
+zxdh_vlan_filter_table_init(struct zxdh_hw *hw, uint16_t vport)
{
- struct zxdh_hw *hw = dev->data->dev_private;
struct zxdh_dtb_shared_data *dtb_data = &hw->dev_sd->dtb_sd;
+ union zxdh_virport_num vport_num = (union zxdh_virport_num)vport;
+ uint16_t vfid = zxdh_vport_to_vfid(vport_num);
struct zxdh_vlan_filter_table vlan_table = {0};
int16_t ret = 0;
@@ -717,7 +718,7 @@ zxdh_vlan_filter_table_init(struct rte_eth_dev *dev)
} else {
vlan_table.vlans[0] = 0;
}
- uint32_t index = (vlan_group << 11) | hw->vport.vfid;
+ uint32_t index = (vlan_group << 11) | vfid;
ZXDH_DTB_ERAM_ENTRY_INFO_T entry_data = {
.index = index,
.p_data = (uint32_t *)&vlan_table
@@ -749,7 +750,7 @@ zxdh_vlan_filter_table_set(struct zxdh_hw *hw, uint16_t vport, uint16_t vlan_id,
memset(&vlan_table, 0, sizeof(struct zxdh_vlan_filter_table));
int table_num = vlan_id / ZXDH_VLAN_FILTER_VLANID_STEP;
uint32_t index = (table_num << 11) | vfid;
- uint16_t group = (vlan_id - table_num * ZXDH_VLAN_FILTER_VLANID_STEP) / 8 + 1;
+ uint16_t group = (vlan_id % ZXDH_VLAN_FILTER_VLANID_STEP) / 8 + 1;
uint8_t val = sizeof(struct zxdh_vlan_filter_table) / sizeof(uint32_t);
uint8_t vlan_tbl_index = group / val;
@@ -950,7 +951,7 @@ zxdh_vlan_relate_vport(struct rte_eth_dev *dev, uint16_t vport,
return 0;
}
-static int
+int
zxdh_set_port_vlan_attr(struct zxdh_hw *hw, uint16_t vport,
struct zxdh_port_vlan_table *port_vlan)
{
@@ -973,7 +974,7 @@ zxdh_set_port_vlan_attr(struct zxdh_hw *hw, uint16_t vport,
return ret;
}
-static int
+int
zxdh_get_port_vlan_attr(struct zxdh_hw *hw, uint16_t vport,
struct zxdh_port_vlan_table *port_vlan)
{
@@ -1045,3 +1046,18 @@ int zxdh_set_vlan_offload(struct zxdh_hw *hw, uint16_t vport, uint8_t type, uint
{
return set_vlan_config(hw, vport, type, enable);
}
+
+int zxdh_port_vlan_table_init(struct zxdh_hw *hw, uint16_t vport)
+{
+ struct zxdh_port_vlan_table port_vlan = {0};
+ int ret = 0;
+
+ if (!hw->is_pf)
+ return 0;
+
+ ret = zxdh_set_port_vlan_attr(hw, vport, &port_vlan);
+ if (ret)
+ PMD_DRV_LOG(ERR, "port vlan table init failed");
+
+ return ret;
+}
diff --git a/drivers/net/zxdh/zxdh_tables.h b/drivers/net/zxdh/zxdh_tables.h
index cb34e38be8..3280ff1f89 100644
--- a/drivers/net/zxdh/zxdh_tables.h
+++ b/drivers/net/zxdh/zxdh_tables.h
@@ -362,7 +362,7 @@ int zxdh_promisc_table_init(struct rte_eth_dev *dev);
int zxdh_promisc_table_uninit(struct rte_eth_dev *dev);
int zxdh_dev_unicast_table_set(struct zxdh_hw *hw, uint16_t vport, bool enable);
int zxdh_dev_multicast_table_set(struct zxdh_hw *hw, uint16_t vport, bool enable);
-int zxdh_vlan_filter_table_init(struct rte_eth_dev *dev);
+int zxdh_vlan_filter_table_init(struct zxdh_hw *hw, uint16_t vport);
int zxdh_vlan_filter_table_set(struct zxdh_hw *hw, uint16_t vport,
uint16_t vlan_id, uint8_t enable);
int zxdh_rss_table_set(struct zxdh_hw *hw, uint16_t vport, struct zxdh_rss_reta *rss_reta);
@@ -372,5 +372,10 @@ int zxdh_set_panel_attr(struct rte_eth_dev *dev, struct zxdh_panel_table *panel_
int zxdh_dev_broadcast_set(struct zxdh_hw *hw, uint16_t vport, bool enable);
int zxdh_set_vlan_filter(struct zxdh_hw *hw, uint16_t vport, uint8_t enable);
int zxdh_set_vlan_offload(struct zxdh_hw *hw, uint16_t vport, uint8_t type, uint8_t enable);
+int zxdh_set_port_vlan_attr(struct zxdh_hw *hw, uint16_t vport,
+ struct zxdh_port_vlan_table *port_vlan);
+int zxdh_get_port_vlan_attr(struct zxdh_hw *hw, uint16_t vport,
+ struct zxdh_port_vlan_table *port_vlan);
+int zxdh_port_vlan_table_init(struct zxdh_hw *hw, uint16_t vport);
#endif /* ZXDH_TABLES_H */
--
2.27.0
[-- Attachment #1.1.2: Type: text/html , Size: 17628 bytes --]
next prev parent reply other threads:[~2025-06-17 2:11 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-06-07 2:30 [PATCH v1 1/3] net/zxdh: modify some MAC config issues Junlong Wang
2025-06-07 2:30 ` [PATCH v1 2/3] net/zxdh: modify some VLAN related issues Junlong Wang
2025-06-16 16:20 ` Stephen Hemminger
2025-06-07 2:30 ` [PATCH v1 3/3] net/zxdh: modify RSS " Junlong Wang
2025-06-16 16:22 ` Stephen Hemminger
2025-06-12 0:14 ` [PATCH v1 1/3] net/zxdh: modify some MAC config issues Thomas Monjalon
2025-06-12 1:18 ` Junlong Wang
2025-06-16 16:16 ` Stephen Hemminger
2025-06-17 2:04 ` [PATCH v2 1/5] net/zxdh: fix meson.build issus in compilation Junlong Wang
2025-06-17 2:05 ` [PATCH v2 2/5] net/zxdh: modify some MAC config issues Junlong Wang
2025-06-17 2:05 ` Junlong Wang [this message]
2025-06-17 2:05 ` [PATCH v2 4/5] net/zxdh: modify RSS related issues Junlong Wang
2025-06-17 2:05 ` [PATCH v2 5/5] net/zxdh: add support VLAN TPID set ops Junlong Wang
2025-06-17 3:32 ` zhoumin
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=20250617020503.3554511-3-wang.junlong1@zte.com.cn \
--to=wang.junlong1@zte.com.cn \
--cc=dev@dpdk.org \
--cc=stephen@networkplumber.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).