Hi
> -----Original Message-----
> From: Min Hu (Connor) <humi...@huawei.com>
> Sent: Saturday, October 9, 2021 16:42
> To: dev@dpdk.org
> Cc: Yigit, Ferruh <ferruh.yi...@intel.com>; tho...@monjalon.net; Li, Xiaoyun
> <xiaoyun...@intel.com>
> Subject: [PATCH] app/testpmd: retain all original dev conf when config DCB
>
> From: Huisong Li <lihuis...@huawei.com>
>
> 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: sta...@dpdk.org
>
> Signed-off-by: Huisong Li <lihuis...@huawei.com>
> Signed-off-by: Min Hu (Connor) <humi...@huawei.com>
> ---
> 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 <xiaoyun...@intel.com>
