testpmd provides commands to test tunnel offload rte_flow
capabilities. A command that creates a tunnel had a syntax bug that
allowed to spesify the 'create' command part several times.

Current patch fixed that fault. Correct syntax for tunnel cretion is:
testpmd> flow tunnel create <port> type <tunnel type>

Signed-off-by: Gregory Etelson <getel...@nvidia.com>
Acked-by: Ori Kam <or...@nvidia.com>
---
 app/test-pmd/cmdline_flow.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/app/test-pmd/cmdline_flow.c b/app/test-pmd/cmdline_flow.c
index 457f74078d..bf00ba0acb 100644
--- a/app/test-pmd/cmdline_flow.c
+++ b/app/test-pmd/cmdline_flow.c
@@ -812,8 +812,7 @@ static const enum index next_vc_attr[] = {
        ZERO,
 };
 
-static const enum index tunnel_create_attr[] = {
-       TUNNEL_CREATE,
+static const enum index tunnel_create_type_attr[] = {
        TUNNEL_CREATE_TYPE,
        END,
        ZERO,
@@ -2009,14 +2008,15 @@ static const struct token token_list[] = {
        [TUNNEL_CREATE] = {
                .name = "create",
                .help = "create new tunnel object",
-               .next = NEXT(tunnel_create_attr, NEXT_ENTRY(PORT_ID)),
+               .next = NEXT(NEXT_ENTRY(TUNNEL_CREATE_TYPE),
+                            NEXT_ENTRY(PORT_ID)),
                .args = ARGS(ARGS_ENTRY(struct buffer, port)),
                .call = parse_tunnel,
        },
        [TUNNEL_CREATE_TYPE] = {
                .name = "type",
                .help = "create new tunnel",
-               .next = NEXT(tunnel_create_attr, NEXT_ENTRY(FILE_PATH)),
+               .next = NEXT(tunnel_create_type_attr, NEXT_ENTRY(FILE_PATH)),
                .args = ARGS(ARGS_ENTRY(struct tunnel_ops, type)),
                .call = parse_tunnel,
        },
-- 
2.29.2

Reply via email to