Re: [PATCH 5/7] app/testpmd: add setting and querying of LLRS FEC mode

["Singh, Aman Deep" <aman.deep.singh@intel.com>]

On 4/8/2023 7:57 AM, Dongdong Liu wrote:
> From: Jie Hai <haijie1@huawei.com>
>
> This patch supports setting and querying of LLRS FEC mode.
>
> Signed-off-by: Jie Hai <haijie1@huawei.com>
> Signed-off-by: Dongdong Liu <liudongdong3@huawei.com>

Acked-by: Aman Singh <aman.deep.singh@intel.com>

> ---
>   app/test-pmd/cmdline.c                      | 5 ++++-
>   app/test-pmd/config.c                       | 4 ++++
>   doc/guides/testpmd_app_ug/testpmd_funcs.rst | 2 +-
>   3 files changed, 9 insertions(+), 2 deletions(-)
>
> diff --git a/app/test-pmd/cmdline.c b/app/test-pmd/cmdline.c
> index 7b20bef4e9..38fa0f507c 100644
> --- a/app/test-pmd/cmdline.c
> +++ b/app/test-pmd/cmdline.c
> @@ -11973,6 +11973,9 @@ cmd_show_fec_mode_parsed(void *parsed_result,
>   	case RTE_ETH_FEC_MODE_CAPA_MASK(RS):
>   		strlcpy(buf, "rs", sizeof(buf));
>   		break;
> +	case RTE_ETH_FEC_MODE_CAPA_MASK(LLRS):
> +		strlcpy(buf, "llrs", sizeof(buf));
> +		break;
>   	default:
>   		return;
>   	}
> @@ -12068,7 +12071,7 @@ cmd_set_port_fec_mode_parsed(
>   static cmdline_parse_inst_t cmd_set_fec_mode = {
>   	.f = cmd_set_port_fec_mode_parsed,
>   	.data = NULL,
> -	.help_str = "set port <port_id> fec_mode auto|off|rs|baser",
> +	.help_str = "set port <port_id> fec_mode auto|off|rs|baser|llrs",
>   	.tokens = {
>   		(void *)&cmd_set_port_fec_mode_set,
>   		(void *)&cmd_set_port_fec_mode_port,
> diff --git a/app/test-pmd/config.c b/app/test-pmd/config.c
> index 096c218c12..f306d678f9 100644
> --- a/app/test-pmd/config.c
> +++ b/app/test-pmd/config.c
> @@ -170,6 +170,10 @@ static const struct {
>   		.mode = RTE_ETH_FEC_RS,
>   		.name = "rs",
>   	},
> +	{
> +		.mode = RTE_ETH_FEC_LLRS,
> +		.name = "llrs",
> +	},
>   };
>   
>   static const struct {
> diff --git a/doc/guides/testpmd_app_ug/testpmd_funcs.rst b/doc/guides/testpmd_app_ug/testpmd_funcs.rst
> index 8f23847859..fa1cea3ed6 100644
> --- a/doc/guides/testpmd_app_ug/testpmd_funcs.rst
> +++ b/doc/guides/testpmd_app_ug/testpmd_funcs.rst
> @@ -1800,7 +1800,7 @@ Set fec mode
>   
>   Set fec mode for a specific port::
>   
> -  testpmd> set port (port_id) fec_mode auto|off|rs|baser
> +  testpmd> set port (port_id) fec_mode auto|off|rs|baser|llrs
>   
>   Config Sample actions list
>   ~~~~~~~~~~~~~~~~~~~~~~~~~~