examples/pipeline: fix help command
authorYogesh Jangra <yogesh.jangra@intel.com>
Thu, 22 Oct 2020 11:01:52 +0000 (07:01 -0400)
committerDavid Marchand <david.marchand@redhat.com>
Fri, 23 Oct 2020 11:40:32 +0000 (13:40 +0200)
This patch has the changes to list all supported commands and fix
supported commands help details.

Fixes: 5074e1d55107 ("examples/pipeline: add configuration commands")

Signed-off-by: Yogesh Jangra <yogesh.jangra@intel.com>
Acked-by: Cristian Dumitrescu <cristian.dumitrescu@intel.com>
examples/pipeline/cli.c

index 5800cc9..8ac6b3f 100644 (file)
@@ -480,7 +480,7 @@ cmd_pipeline_create(char **tokens,
 static const char cmd_pipeline_port_in_help[] =
 "pipeline <pipeline_name> port in <port_id>\n"
 "   link <link_name> rxq <queue_id> bsz <burst_size>\n"
-"   source <mempool_name> <fie_name>\n";
+"   | source <mempool_name> <file_name>\n";
 
 static void
 cmd_pipeline_port_in(char **tokens,
@@ -1163,7 +1163,18 @@ cmd_help(char **tokens,
 
        if (n_tokens == 0) {
                snprintf(out, out_size,
-                       "Type 'help <command>' for command details.\n\n");
+                       "Type 'help <command>' for command details.\n\n"
+                       "List of commands:\n"
+                       "\tmempool\n"
+                       "\tlink\n"
+                       "\tpipeline create\n"
+                       "\tpipeline port in\n"
+                       "\tpipeline port out\n"
+                       "\tpipeline build\n"
+                       "\tpipeline table update\n"
+                       "\tpipeline stats\n"
+                       "\tthread pipeline enable\n"
+                       "\tthread pipeline disable\n\n");
                return;
        }
 
@@ -1178,20 +1189,20 @@ cmd_help(char **tokens,
        }
 
        if ((strcmp(tokens[0], "pipeline") == 0) &&
-               ((n_tokens == 1) && (strcmp(tokens[2], "create")) == 0)) {
+               (n_tokens == 2) && (strcmp(tokens[1], "create") == 0)) {
                snprintf(out, out_size, "\n%s\n", cmd_pipeline_create_help);
                return;
        }
 
        if ((strcmp(tokens[0], "pipeline") == 0) &&
-               (strcmp(tokens[1], "port") == 0)) {
-               if ((n_tokens == 3) && (strcmp(tokens[2], "in")) == 0) {
+               (n_tokens == 3) && (strcmp(tokens[1], "port") == 0)) {
+               if (strcmp(tokens[2], "in") == 0) {
                        snprintf(out, out_size, "\n%s\n",
                                cmd_pipeline_port_in_help);
                        return;
                }
 
-               if ((n_tokens == 3) && (strcmp(tokens[2], "out")) == 0) {
+               if (strcmp(tokens[2], "out") == 0) {
                        snprintf(out, out_size, "\n%s\n",
                                cmd_pipeline_port_out_help);
                        return;
@@ -1199,20 +1210,22 @@ cmd_help(char **tokens,
        }
 
        if ((strcmp(tokens[0], "pipeline") == 0) &&
-               ((n_tokens >= 2) && (strcmp(tokens[2], "build")) == 0)) {
+               (n_tokens == 2) && (strcmp(tokens[1], "build") == 0)) {
                snprintf(out, out_size, "\n%s\n", cmd_pipeline_build_help);
                return;
        }
 
        if ((strcmp(tokens[0], "pipeline") == 0) &&
-               ((n_tokens >= 2) && (strcmp(tokens[2], "table")) == 0)) {
+               (n_tokens == 3) &&
+               (strcmp(tokens[1], "table") == 0) &&
+               (strcmp(tokens[2], "update") == 0)) {
                snprintf(out, out_size, "\n%s\n",
                        cmd_pipeline_table_update_help);
                return;
        }
 
        if ((strcmp(tokens[0], "pipeline") == 0) &&
-               ((n_tokens >= 2) && (strcmp(tokens[2], "stats")) == 0)) {
+               (n_tokens == 2) && (strcmp(tokens[1], "stats") == 0)) {
                snprintf(out, out_size, "\n%s\n", cmd_pipeline_stats_help);
                return;
        }