Reviewed-by: Chengwen Feng <fengcheng...@huawei.com>
On 2023/11/17 1:21, Ferruh Yigit wrote:
> In command to set Tx offload:
> "port config <port_id> tx_offload <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 <ferruh.yi...@amd.com>
> ---
> Cc: huangdeng...@huawei.com
> Cc: lihuis...@huawei.com
> Cc: Song Jiale <songx.ji...@intel.com>
> Cc: Zhimin Huang <zhiminx.hu...@intel.com>
> ---
> app/test-pmd/cmdline.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/app/test-pmd/cmdline.c b/app/test-pmd/cmdline.c
> index c040de7a7991..9369d3b4c526 100644
> --- a/app/test-pmd/cmdline.c
> +++ b/app/test-pmd/cmdline.c
> @@ -11737,7 +11737,7 @@ cmd_config_per_port_tx_offload_parsed(void
> *parsed_result,
> struct cmd_config_per_port_tx_offload_result *res = parsed_result;
> bool on;
>
> - on = strcmp(res->on_off, "on");
> + on = strcmp(res->on_off, "on") == 0;
> config_port_tx_offload(res->port_id, res->offload, on);
> }
>
>
