Re: [PATCH] app/testpmd: fix set Tx offload command

2023-11-17 Thread Ferruh Yigit
On 11/17/2023 1:31 AM, lihuisong (C) wrote: >> On 2023/11/17 1:21, Ferruh Yigit wrote: >>> In command to set Tx offload: >>> "port config tx_offload on|off", >>> >>> there is a defect in "on|off" comparison, so command does opposite of >>> what is intended. Fixed comparison. >>> >>> Fixes: 6280f

Re: [PATCH] app/testpmd: fix set Tx offload command

2023-11-16 Thread lihuisong (C)
Acked-by: Huisong Li 在 2023/11/17 9:05, fengchengwen 写道: Reviewed-by: Chengwen Feng On 2023/11/17 1:21, Ferruh Yigit wrote: In command to set Tx offload: "port config tx_offload on|off", there is a defect in "on|off" comparison, so command does opposite of what is intended. Fixed comparis

Re: [PATCH] app/testpmd: fix set Tx offload command

2023-11-16 Thread fengchengwen
Reviewed-by: Chengwen Feng On 2023/11/17 1:21, Ferruh Yigit wrote: > In command to set Tx offload: > "port config tx_offload on|off", > > there is a defect in "on|off" comparison, so command does opposite of > what is intended. Fixed comparison. > > Fixes: 6280fe565b44 ("app/testpmd: allow of

[PATCH] app/testpmd: fix set Tx offload command

2023-11-16 Thread Ferruh Yigit
In command to set Tx offload: "port config tx_offload on|off", there is a defect in "on|off" comparison, so command does opposite of what is intended. Fixed comparison. Fixes: 6280fe565b44 ("app/testpmd: allow offload config for all ports") Signed-off-by: Ferruh Yigit --- Cc: huangdeng...@hua