Re: [PATCH v8 2/4] app/testpmd: add skip cman support for testpmd

[Ferruh Yigit <ferruh.yigit@amd.com>]

On 2/14/2023 1:04 PM, Rakesh Kudurumalla wrote:
> added support for testpmd application to accept
> skip_cman action while configuring policy action
> 
> Signed-off-by: Rakesh Kudurumalla <rkudurumalla@marvell.com>
> ---
>  app/test-pmd/cmdline_flow.c | 9 +++++++++
>  1 file changed, 9 insertions(+)
> 
> diff --git a/app/test-pmd/cmdline_flow.c b/app/test-pmd/cmdline_flow.c
> index 88108498e0..fc003e0096 100644
> --- a/app/test-pmd/cmdline_flow.c
> +++ b/app/test-pmd/cmdline_flow.c
> @@ -472,6 +472,7 @@ enum index {
>  	ACTION_END,
>  	ACTION_VOID,
>  	ACTION_PASSTHRU,
> +	ACTION_SKIP_CMAN,
>  	ACTION_JUMP,
>  	ACTION_JUMP_GROUP,
>  	ACTION_MARK,
> @@ -1825,6 +1826,7 @@ static const enum index next_action[] = {
>  	ACTION_END,
>  	ACTION_VOID,
>  	ACTION_PASSTHRU,
> +	ACTION_SKIP_CMAN,
>  	ACTION_JUMP,
>  	ACTION_MARK,
>  	ACTION_FLAG,
> @@ -5159,6 +5161,13 @@ static const struct token token_list[] = {
>  		.next = NEXT(NEXT_ENTRY(ACTION_NEXT)),
>  		.call = parse_vc,
>  	},
> +	[ACTION_SKIP_CMAN] = {
> +		.name = "skip_cman",
> +		.help = "bypass cman on received packets",
> +		.priv = PRIV_ACTION(SKIP_CMAN, 0),
> +		.next = NEXT(NEXT_ENTRY(ACTION_NEXT)),
> +		.call = parse_vc,
> +	},
>  	[ACTION_JUMP] = {
>  		.name = "jump",
>  		.help = "redirect traffic to a given group",

Hi Ori,

any objection to this patch?