app/testpmd: fix incorrect flow arguments name
Checks
Commit Message
There should be no spaces in flow command arguments name.
This commit removes incorrect arguments name in queue based table
template.
Fixes: ecdc927b99f2 ("app/testpmd: add async flow create/destroy operations")
Cc: akozyrev@nvidia.com
Signed-off-by: Suanming Mou <suanmingm@nvidia.com>
---
app/test-pmd/cmdline_flow.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Comments
On 9/30/2022 5:09 PM, Suanming Mou wrote:
> There should be no spaces in flow command arguments name.
> This commit removes incorrect arguments name in queue based table
> template.
>
> Fixes: ecdc927b99f2 ("app/testpmd: add async flow create/destroy operations")
> Cc: akozyrev@nvidia.com
>
> Signed-off-by: Suanming Mou <suanmingm@nvidia.com>
> Acked-by: Aman Singh<aman.deep.singh@intel.com>
>
> ---
> app/test-pmd/cmdline_flow.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/app/test-pmd/cmdline_flow.c b/app/test-pmd/cmdline_flow.c
> index 148dbc082d..e59d64ffa9 100644
> --- a/app/test-pmd/cmdline_flow.c
> +++ b/app/test-pmd/cmdline_flow.c
> @@ -2968,7 +2968,7 @@ static const struct token token_list[] = {
> },
> /* Queue arguments. */
> [QUEUE_TEMPLATE_TABLE] = {
> - .name = "template table",
> + .name = "template_table",
> .help = "specify table id",
> .next = NEXT(NEXT_ENTRY(QUEUE_PATTERN_TEMPLATE),
> NEXT_ENTRY(COMMON_TABLE_ID)),
> -----Original Message-----
> From: Singh, Aman Deep <aman.deep.singh@intel.com>
> Sent: Thursday, 6 October 2022 10:59
>
>
>
> On 9/30/2022 5:09 PM, Suanming Mou wrote:
> > There should be no spaces in flow command arguments name.
> > This commit removes incorrect arguments name in queue based table
> > template.
> >
> > Fixes: ecdc927b99f2 ("app/testpmd: add async flow create/destroy
> operations")
> > Cc: akozyrev@nvidia.com
> >
> > Signed-off-by: Suanming Mou <suanmingm@nvidia.com>
> > Acked-by: Aman Singh<aman.deep.singh@intel.com>
> >
> > ---
Acked-by: Ori Kam <orika@nvidia.com>
Thanks,
Ori
On 10/11/22 09:55, Ori Kam wrote:
>
>
>> -----Original Message-----
>> From: Singh, Aman Deep <aman.deep.singh@intel.com>
>> Sent: Thursday, 6 October 2022 10:59
>>
>>
>>
>> On 9/30/2022 5:09 PM, Suanming Mou wrote:
>>> There should be no spaces in flow command arguments name.
>>> This commit removes incorrect arguments name in queue based table
>>> template.
>>>
>>> Fixes: ecdc927b99f2 ("app/testpmd: add async flow create/destroy
>> operations")
>>> Cc: akozyrev@nvidia.com
>>>
>>> Signed-off-by: Suanming Mou <suanmingm@nvidia.com>
>>> Acked-by: Aman Singh<aman.deep.singh@intel.com>
Acked-by: Aman Singh <aman.deep.singh@intel.com>
>>>
>>> ---
>
>
> Acked-by: Ori Kam <orika@nvidia.com>
> Thanks,
> Ori
Applied to dpdk-next-net/main, thanks.
@@ -2968,7 +2968,7 @@ static const struct token token_list[] = {
},
/* Queue arguments. */
[QUEUE_TEMPLATE_TABLE] = {
- .name = "template table",
+ .name = "template_table",
.help = "specify table id",
.next = NEXT(NEXT_ENTRY(QUEUE_PATTERN_TEMPLATE),
NEXT_ENTRY(COMMON_TABLE_ID)),