在 2021/10/11 14:49, Li, Xiaoyun 写道:
Hi
-----Original Message-----
From: Min Hu (Connor) <[email protected]>
Sent: Saturday, October 9, 2021 16:42
To: [email protected]
Cc: Yigit, Ferruh <[email protected]>; [email protected]; Li, Xiaoyun
<[email protected]>
Subject: [PATCH] app/testpmd: retain all original dev conf when config DCB
From: Huisong Li <[email protected]>
When configuring DCB, testpmd retains the rx_mode/tx_mode configuration in
rte_port->dev_conf. But some configurations, such as the link_speed, were not
saved if it were set before configuring DCB.
"if they were" or "if it was"
Fixes: 1a572499beb6 ("app/testpmd: setup DCB forwarding based on traffic
class")
Cc: [email protected]
Signed-off-by: Huisong Li <[email protected]>
Signed-off-by: Min Hu (Connor) <[email protected]>
---
app/test-pmd/testpmd.c | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/app/test-pmd/testpmd.c b/app/test-pmd/testpmd.c index
eec4f1b81d..12a0db8796 100644
--- a/app/test-pmd/testpmd.c
+++ b/app/test-pmd/testpmd.c
@@ -3729,10 +3729,8 @@ init_port_dcb_config(portid_t pid,
}
rte_port = &ports[pid];
- memset(&port_conf, 0, sizeof(struct rte_eth_conf));
-
- port_conf.rxmode = rte_port->dev_conf.rxmode;
- port_conf.txmode = rte_port->dev_conf.txmode;
+ /* retain the original device configuration. */
+ memcpy(&port_conf, &rte_port->dev_conf, sizeof(struct rte_eth_conf));
/*set configuration of DCB in vt mode and DCB in non-vt mode*/
retval = get_eth_dcb_conf(pid, &port_conf, dcb_mode, num_tcs,
pfc_en);
--
2.33.0
Except the small issue in commit log,
Acked-by: Xiaoyun Li <[email protected]>
Thanks Xiaoyun,
V2 has been sent.
.
