net/i40e: move testpmd commands
[dpdk.git] / app / test-pmd / cmdline_mtr.c
index d8d806d..57050ec 100644 (file)
@@ -1,34 +1,5 @@
-/*-
- *   BSD LICENSE
- *
- *   Copyright(c) 2017 Intel Corporation. All rights reserved.
- *   All rights reserved.
- *
- *   Redistribution and use in source and binary forms, with or without
- *   modification, are permitted provided that the following conditions
- *   are met:
- *
- *     * Redistributions of source code must retain the above copyright
- *       notice, this list of conditions and the following disclaimer.
- *     * Redistributions in binary form must reproduce the above copyright
- *       notice, this list of conditions and the following disclaimer in
- *       the documentation and/or other materials provided with the
- *       distribution.
- *     * Neither the name of Intel Corporation nor the names of its
- *       contributors may be used to endorse or promote products derived
- *       from this software without specific prior written permission.
- *
- *   THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
- *   "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
- *   LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
- *   A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
- *   OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
- *   SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
- *   LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
- *   DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
- *   THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
- *   (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
- *   OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+/* SPDX-License-Identifier: BSD-3-Clause
+ * Copyright(c) 2017 Intel Corporation
  */
 
 #include <cmdline_parse.h>
@@ -42,6 +13,9 @@
 #include "testpmd.h"
 #include "cmdline_mtr.h"
 
+#define PARSE_DELIMITER                                " \f\n\r\t\v"
+#define MAX_DSCP_TABLE_ENTRIES         64
+
 /** Display Meter Error Message */
 static void
 print_err_msg(struct rte_mtr_error *error)
@@ -63,6 +37,8 @@ print_err_msg(struct rte_mtr_error *error)
                [RTE_MTR_ERROR_TYPE_STATS] = "stats",
                [RTE_MTR_ERROR_TYPE_SHARED]
                        = "shared meter",
+               [RTE_MTR_ERROR_TYPE_METER_POLICY_ID] = "meter policy id",
+               [RTE_MTR_ERROR_TYPE_METER_POLICY] = "meter policy null",
        };
 
        const char *errstr;
@@ -77,29 +53,246 @@ print_err_msg(struct rte_mtr_error *error)
        if (error->cause)
                snprintf(buf, sizeof(buf), "cause: %p, ", error->cause);
 
-       printf("%s: %s%s (error %d)\n", errstr, error->cause ? buf : "",
+       fprintf(stderr, "%s: %s%s (error %d)\n",
+               errstr, error->cause ? buf : "",
                error->message ? error->message : "(no stated reason)",
                error->type);
 }
 
+void
+print_mtr_err_msg(struct rte_mtr_error *error)
+{
+       print_err_msg(error);
+}
+
+static int
+parse_uint(uint64_t *value, const char *str)
+{
+       char *next = NULL;
+       uint64_t n;
+
+       errno = 0;
+       /* Parse number string */
+       n = strtol(str, &next, 10);
+       if (errno != 0 || str == next || *next != '\0')
+               return -1;
+
+       *value = n;
+
+       return 0;
+}
+
+static int
+parse_dscp_table_entries(char *str, enum rte_color **dscp_table)
+{
+       char *token;
+       int i = 0;
+
+       token = strtok_r(str, PARSE_DELIMITER, &str);
+       if (token == NULL)
+               return 0;
+
+       /* Allocate memory for dscp table */
+       *dscp_table = (enum rte_color *)malloc(MAX_DSCP_TABLE_ENTRIES *
+               sizeof(enum rte_color));
+       if (*dscp_table == NULL)
+               return -1;
+
+       while (1) {
+               if (strcmp(token, "G") == 0 ||
+                       strcmp(token, "g") == 0)
+                       (*dscp_table)[i++] = RTE_COLOR_GREEN;
+               else if (strcmp(token, "Y") == 0 ||
+                       strcmp(token, "y") == 0)
+                       (*dscp_table)[i++] = RTE_COLOR_YELLOW;
+               else if (strcmp(token, "R") == 0 ||
+                       strcmp(token, "r") == 0)
+                       (*dscp_table)[i++] = RTE_COLOR_RED;
+               else {
+                       free(*dscp_table);
+                       return -1;
+               }
+               if (i == MAX_DSCP_TABLE_ENTRIES)
+                       break;
+
+               token = strtok_r(str, PARSE_DELIMITER, &str);
+               if (token == NULL) {
+                       free(*dscp_table);
+                       return -1;
+               }
+       }
+       return 0;
+}
+
+static int
+parse_meter_color_str(char *c_str, uint32_t *use_prev_meter_color,
+       enum rte_color **dscp_table)
+{
+       char *token;
+       uint64_t previous_mtr_color = 0;
+       int ret;
+
+       /* First token: use previous meter color */
+       token = strtok_r(c_str, PARSE_DELIMITER, &c_str);
+       if (token ==  NULL)
+               return -1;
+
+       ret = parse_uint(&previous_mtr_color, token);
+       if (ret != 0)
+               return -1;
+
+       /* Check if previous meter color to be used */
+       if (previous_mtr_color) {
+               *use_prev_meter_color = previous_mtr_color;
+               return 0;
+       }
+
+       /* Parse dscp table entries */
+       ret = parse_dscp_table_entries(c_str, dscp_table);
+       if (ret != 0)
+               return -1;
+
+       return 0;
+}
+
 static int
-string_to_policer_action(char *s)
+parse_multi_token_string(char *t_str, uint16_t *port_id,
+       uint32_t *mtr_id, enum rte_color **dscp_table)
 {
-       if (strcmp(s, "G") == 0)
-               return MTR_POLICER_ACTION_COLOR_GREEN;
+       char *token;
+       uint64_t val;
+       int ret;
+
+       /* First token: port id */
+       token = strtok_r(t_str, PARSE_DELIMITER, &t_str);
+       if (token ==  NULL)
+               return -1;
+
+       ret = parse_uint(&val, token);
+       if (ret != 0 || val > UINT16_MAX)
+               return -1;
+
+       *port_id = val;
+
+       /* Second token: meter id */
+       token = strtok_r(t_str, PARSE_DELIMITER, &t_str);
+       if (token == NULL)
+               return 0;
+
+       ret = parse_uint(&val, token);
+       if (ret != 0 || val > UINT32_MAX)
+               return -1;
+
+       *mtr_id = val;
 
-       if (strcmp(s, "Y") == 0)
-               return MTR_POLICER_ACTION_COLOR_YELLOW;
+       ret = parse_dscp_table_entries(t_str, dscp_table);
+       if (ret != 0)
+               return -1;
 
-       if (strcmp(s, "R") == 0)
-               return MTR_POLICER_ACTION_COLOR_RED;
+       return 0;
+}
+
+/* *** Show Port Meter Capabilities *** */
+struct cmd_show_port_meter_cap_result {
+       cmdline_fixed_string_t show;
+       cmdline_fixed_string_t port;
+       cmdline_fixed_string_t meter;
+       cmdline_fixed_string_t cap;
+       uint16_t port_id;
+};
+
+static cmdline_parse_token_string_t cmd_show_port_meter_cap_show =
+       TOKEN_STRING_INITIALIZER(
+               struct cmd_show_port_meter_cap_result, show, "show");
+static cmdline_parse_token_string_t cmd_show_port_meter_cap_port =
+       TOKEN_STRING_INITIALIZER(
+               struct cmd_show_port_meter_cap_result, port, "port");
+static cmdline_parse_token_string_t cmd_show_port_meter_cap_meter =
+       TOKEN_STRING_INITIALIZER(
+               struct cmd_show_port_meter_cap_result, meter, "meter");
+static cmdline_parse_token_string_t cmd_show_port_meter_cap_cap =
+       TOKEN_STRING_INITIALIZER(
+               struct cmd_show_port_meter_cap_result, cap, "cap");
+static cmdline_parse_token_num_t cmd_show_port_meter_cap_port_id =
+       TOKEN_NUM_INITIALIZER(
+               struct cmd_show_port_meter_cap_result, port_id, RTE_UINT16);
+
+static void cmd_show_port_meter_cap_parsed(void *parsed_result,
+       __rte_unused struct cmdline *cl,
+       __rte_unused void *data)
+{
+       struct cmd_show_port_meter_cap_result *res = parsed_result;
+       struct rte_mtr_capabilities cap;
+       struct rte_mtr_error error;
+       uint16_t port_id = res->port_id;
+       int ret;
+
+       if (port_id_is_invalid(port_id, ENABLED_WARN))
+               return;
 
-       if (strcmp(s, "D") == 0)
-               return MTR_POLICER_ACTION_DROP;
+       memset(&cap, 0, sizeof(struct rte_mtr_capabilities));
+       ret = rte_mtr_capabilities_get(port_id, &cap, &error);
+       if (ret) {
+               print_err_msg(&error);
+               return;
+       }
 
-       return -1;
+       printf("\n****   Port Meter Object Capabilities ****\n\n");
+       printf("cap.n_max %" PRIu32 "\n", cap.n_max);
+       printf("cap.n_shared_max %" PRIu32 "\n", cap.n_shared_max);
+       printf("cap.identical %" PRId32 "\n", cap.identical);
+       printf("cap.shared_identical %" PRId32 "\n",
+               cap.shared_identical);
+       printf("cap.shared_n_flows_per_mtr_max %" PRIu32 "\n",
+               cap.shared_n_flows_per_mtr_max);
+       printf("cap.chaining_n_mtrs_per_flow_max %" PRIu32 "\n",
+               cap.chaining_n_mtrs_per_flow_max);
+       printf("cap.chaining_use_prev_mtr_color_supported %" PRId32 "\n",
+               cap.chaining_use_prev_mtr_color_supported);
+       printf("cap.chaining_use_prev_mtr_color_enforced %" PRId32 "\n",
+               cap.chaining_use_prev_mtr_color_enforced);
+       printf("cap.meter_srtcm_rfc2697_n_max %" PRIu32 "\n",
+               cap.meter_srtcm_rfc2697_n_max);
+       printf("cap.meter_trtcm_rfc2698_n_max %" PRIu32 "\n",
+               cap.meter_trtcm_rfc2698_n_max);
+       printf("cap.meter_trtcm_rfc4115_n_max %" PRIu32 "\n",
+               cap.meter_trtcm_rfc4115_n_max);
+       printf("cap.meter_rate_max %" PRIu64 "\n", cap.meter_rate_max);
+       printf("cap.color_aware_srtcm_rfc2697_supported %" PRId32 "\n",
+               cap.color_aware_srtcm_rfc2697_supported);
+       printf("cap.color_aware_trtcm_rfc2698_supported %" PRId32 "\n",
+               cap.color_aware_trtcm_rfc2698_supported);
+       printf("cap.color_aware_trtcm_rfc4115_supported %" PRId32 "\n",
+               cap.color_aware_trtcm_rfc4115_supported);
+       printf("cap.srtcm_rfc2697_byte_mode_supported %" PRId32 "\n",
+               cap.srtcm_rfc2697_byte_mode_supported);
+       printf("cap.srtcm_rfc2697_packet_mode_supported %" PRId32 "\n",
+               cap.srtcm_rfc2697_packet_mode_supported);
+       printf("cap.trtcm_rfc2698_byte_mode_supported %" PRId32 "\n",
+               cap.trtcm_rfc2698_byte_mode_supported);
+       printf("cap.trtcm_rfc2698_packet_mode_supported %" PRId32 "\n",
+               cap.trtcm_rfc2698_packet_mode_supported);
+       printf("cap.trtcm_rfc4115_byte_mode_supported %" PRId32 "\n",
+               cap.trtcm_rfc4115_byte_mode_supported);
+       printf("cap.trtcm_rfc4115_packet_mode_supported %" PRId32 "\n",
+               cap.trtcm_rfc4115_packet_mode_supported);
+       printf("cap.stats_mask %" PRIx64 "\n", cap.stats_mask);
 }
 
+cmdline_parse_inst_t cmd_show_port_meter_cap = {
+       .f = cmd_show_port_meter_cap_parsed,
+       .data = NULL,
+       .help_str = "show port meter cap <port_id>",
+       .tokens = {
+               (void *)&cmd_show_port_meter_cap_show,
+               (void *)&cmd_show_port_meter_cap_port,
+               (void *)&cmd_show_port_meter_cap_meter,
+               (void *)&cmd_show_port_meter_cap_cap,
+               (void *)&cmd_show_port_meter_cap_port_id,
+               NULL,
+       },
+};
+
 /* *** Add Port Meter Profile srtcm_rfc2697 *** */
 struct cmd_add_port_meter_profile_srtcm_result {
        cmdline_fixed_string_t add;
@@ -112,52 +305,56 @@ struct cmd_add_port_meter_profile_srtcm_result {
        uint64_t cir;
        uint64_t cbs;
        uint64_t ebs;
-       uint8_t color_aware;
+       int packet_mode;
 };
 
-cmdline_parse_token_string_t cmd_add_port_meter_profile_srtcm_add =
+static cmdline_parse_token_string_t cmd_add_port_meter_profile_srtcm_add =
        TOKEN_STRING_INITIALIZER(
                struct cmd_add_port_meter_profile_srtcm_result, add, "add");
-cmdline_parse_token_string_t cmd_add_port_meter_profile_srtcm_port =
+static cmdline_parse_token_string_t cmd_add_port_meter_profile_srtcm_port =
        TOKEN_STRING_INITIALIZER(
                struct cmd_add_port_meter_profile_srtcm_result,
                        port, "port");
-cmdline_parse_token_string_t cmd_add_port_meter_profile_srtcm_meter =
+static cmdline_parse_token_string_t cmd_add_port_meter_profile_srtcm_meter =
        TOKEN_STRING_INITIALIZER(
                struct cmd_add_port_meter_profile_srtcm_result,
                        meter, "meter");
-cmdline_parse_token_string_t cmd_add_port_meter_profile_srtcm_profile =
+static cmdline_parse_token_string_t cmd_add_port_meter_profile_srtcm_profile =
        TOKEN_STRING_INITIALIZER(
                struct cmd_add_port_meter_profile_srtcm_result,
                        profile, "profile");
-cmdline_parse_token_string_t cmd_add_port_meter_profile_srtcm_srtcm_rfc2697 =
+static cmdline_parse_token_string_t cmd_add_port_meter_profile_srtcm_srtcm_rfc2697 =
        TOKEN_STRING_INITIALIZER(
                struct cmd_add_port_meter_profile_srtcm_result,
                        srtcm_rfc2697, "srtcm_rfc2697");
-cmdline_parse_token_num_t cmd_add_port_meter_profile_srtcm_port_id =
+static cmdline_parse_token_num_t cmd_add_port_meter_profile_srtcm_port_id =
        TOKEN_NUM_INITIALIZER(
                struct cmd_add_port_meter_profile_srtcm_result,
-                       port_id, UINT16);
-cmdline_parse_token_num_t cmd_add_port_meter_profile_srtcm_profile_id =
+                       port_id, RTE_UINT16);
+static cmdline_parse_token_num_t cmd_add_port_meter_profile_srtcm_profile_id =
        TOKEN_NUM_INITIALIZER(
                struct cmd_add_port_meter_profile_srtcm_result,
-                       profile_id, UINT32);
-cmdline_parse_token_num_t cmd_add_port_meter_profile_srtcm_cir =
+                       profile_id, RTE_UINT32);
+static cmdline_parse_token_num_t cmd_add_port_meter_profile_srtcm_cir =
        TOKEN_NUM_INITIALIZER(
                struct cmd_add_port_meter_profile_srtcm_result,
-                       cir, UINT64);
-cmdline_parse_token_num_t cmd_add_port_meter_profile_srtcm_cbs =
+                       cir, RTE_UINT64);
+static cmdline_parse_token_num_t cmd_add_port_meter_profile_srtcm_cbs =
        TOKEN_NUM_INITIALIZER(
                struct cmd_add_port_meter_profile_srtcm_result,
-                       cbs, UINT64);
-cmdline_parse_token_num_t cmd_add_port_meter_profile_srtcm_ebs =
+                       cbs, RTE_UINT64);
+static cmdline_parse_token_num_t cmd_add_port_meter_profile_srtcm_ebs =
        TOKEN_NUM_INITIALIZER(
                struct cmd_add_port_meter_profile_srtcm_result,
-                       ebs, UINT64);
+                       ebs, RTE_UINT64);
+static cmdline_parse_token_num_t cmd_add_port_meter_profile_srtcm_packet_mode =
+       TOKEN_NUM_INITIALIZER(
+               struct cmd_add_port_meter_profile_srtcm_result,
+                       packet_mode, RTE_UINT32);
 
 static void cmd_add_port_meter_profile_srtcm_parsed(void *parsed_result,
-       __attribute__((unused)) struct cmdline *cl,
-       __attribute__((unused)) void *data)
+       __rte_unused struct cmdline *cl,
+       __rte_unused void *data)
 {
        struct cmd_add_port_meter_profile_srtcm_result *res = parsed_result;
        struct rte_mtr_meter_profile mp;
@@ -171,10 +368,11 @@ static void cmd_add_port_meter_profile_srtcm_parsed(void *parsed_result,
 
        /* Private shaper profile params */
        memset(&mp, 0, sizeof(struct rte_mtr_meter_profile));
-       mp.alg = 0;
+       mp.alg = RTE_MTR_SRTCM_RFC2697;
        mp.srtcm_rfc2697.cir = res->cir;
        mp.srtcm_rfc2697.cbs = res->cbs;
        mp.srtcm_rfc2697.ebs = res->ebs;
+       mp.packet_mode = res->packet_mode;
 
        ret = rte_mtr_meter_profile_add(port_id, profile_id, &mp, &error);
        if (ret != 0) {
@@ -186,18 +384,19 @@ static void cmd_add_port_meter_profile_srtcm_parsed(void *parsed_result,
 cmdline_parse_inst_t cmd_add_port_meter_profile_srtcm = {
        .f = cmd_add_port_meter_profile_srtcm_parsed,
        .data = NULL,
-       .help_str = "Add port meter profile srtcm (rfc2697)",
+       .help_str = "add port meter profile srtcm_rfc2697 <port_id> <profile_id> <cir> <cbs> <ebs> <packet_mode>",
        .tokens = {
                (void *)&cmd_add_port_meter_profile_srtcm_add,
                (void *)&cmd_add_port_meter_profile_srtcm_port,
                (void *)&cmd_add_port_meter_profile_srtcm_meter,
                (void *)&cmd_add_port_meter_profile_srtcm_profile,
+               (void *)&cmd_add_port_meter_profile_srtcm_srtcm_rfc2697,
                (void *)&cmd_add_port_meter_profile_srtcm_port_id,
                (void *)&cmd_add_port_meter_profile_srtcm_profile_id,
-               (void *)&cmd_add_port_meter_profile_srtcm_srtcm_rfc2697,
                (void *)&cmd_add_port_meter_profile_srtcm_cir,
                (void *)&cmd_add_port_meter_profile_srtcm_cbs,
                (void *)&cmd_add_port_meter_profile_srtcm_ebs,
+               (void *)&cmd_add_port_meter_profile_srtcm_packet_mode,
                NULL,
        },
 };
@@ -215,55 +414,60 @@ struct cmd_add_port_meter_profile_trtcm_result {
        uint64_t pir;
        uint64_t cbs;
        uint64_t pbs;
+       int packet_mode;
 };
 
-cmdline_parse_token_string_t cmd_add_port_meter_profile_trtcm_add =
+static cmdline_parse_token_string_t cmd_add_port_meter_profile_trtcm_add =
        TOKEN_STRING_INITIALIZER(
                struct cmd_add_port_meter_profile_trtcm_result, add, "add");
-cmdline_parse_token_string_t cmd_add_port_meter_profile_trtcm_port =
+static cmdline_parse_token_string_t cmd_add_port_meter_profile_trtcm_port =
        TOKEN_STRING_INITIALIZER(
                struct cmd_add_port_meter_profile_trtcm_result,
                        port, "port");
-cmdline_parse_token_string_t cmd_add_port_meter_profile_trtcm_meter =
+static cmdline_parse_token_string_t cmd_add_port_meter_profile_trtcm_meter =
        TOKEN_STRING_INITIALIZER(
                struct cmd_add_port_meter_profile_trtcm_result,
                        meter, "meter");
-cmdline_parse_token_string_t cmd_add_port_meter_profile_trtcm_profile =
+static cmdline_parse_token_string_t cmd_add_port_meter_profile_trtcm_profile =
        TOKEN_STRING_INITIALIZER(
                struct cmd_add_port_meter_profile_trtcm_result,
                        profile, "profile");
-cmdline_parse_token_string_t cmd_add_port_meter_profile_trtcm_trtcm_rfc2698 =
+static cmdline_parse_token_string_t cmd_add_port_meter_profile_trtcm_trtcm_rfc2698 =
        TOKEN_STRING_INITIALIZER(
                struct cmd_add_port_meter_profile_trtcm_result,
                        trtcm_rfc2698, "trtcm_rfc2698");
-cmdline_parse_token_num_t cmd_add_port_meter_profile_trtcm_port_id =
+static cmdline_parse_token_num_t cmd_add_port_meter_profile_trtcm_port_id =
        TOKEN_NUM_INITIALIZER(
                struct cmd_add_port_meter_profile_trtcm_result,
-                       port_id, UINT16);
-cmdline_parse_token_num_t cmd_add_port_meter_profile_trtcm_profile_id =
+                       port_id, RTE_UINT16);
+static cmdline_parse_token_num_t cmd_add_port_meter_profile_trtcm_profile_id =
        TOKEN_NUM_INITIALIZER(
                struct cmd_add_port_meter_profile_trtcm_result,
-                       profile_id, UINT32);
-cmdline_parse_token_num_t cmd_add_port_meter_profile_trtcm_cir =
+                       profile_id, RTE_UINT32);
+static cmdline_parse_token_num_t cmd_add_port_meter_profile_trtcm_cir =
        TOKEN_NUM_INITIALIZER(
                struct cmd_add_port_meter_profile_trtcm_result,
-                       cir, UINT64);
-cmdline_parse_token_num_t cmd_add_port_meter_profile_trtcm_pir =
+                       cir, RTE_UINT64);
+static cmdline_parse_token_num_t cmd_add_port_meter_profile_trtcm_pir =
        TOKEN_NUM_INITIALIZER(
                struct cmd_add_port_meter_profile_trtcm_result,
-                       pir, UINT64);
-cmdline_parse_token_num_t cmd_add_port_meter_profile_trtcm_cbs =
+                       pir, RTE_UINT64);
+static cmdline_parse_token_num_t cmd_add_port_meter_profile_trtcm_cbs =
        TOKEN_NUM_INITIALIZER(
                struct cmd_add_port_meter_profile_trtcm_result,
-                       cbs, UINT64);
-cmdline_parse_token_num_t cmd_add_port_meter_profile_trtcm_pbs =
+                       cbs, RTE_UINT64);
+static cmdline_parse_token_num_t cmd_add_port_meter_profile_trtcm_pbs =
        TOKEN_NUM_INITIALIZER(
                struct cmd_add_port_meter_profile_trtcm_result,
-                       pbs, UINT64);
+                       pbs, RTE_UINT64);
+static cmdline_parse_token_num_t cmd_add_port_meter_profile_trtcm_packet_mode =
+       TOKEN_NUM_INITIALIZER(
+               struct cmd_add_port_meter_profile_trtcm_result,
+                       packet_mode, RTE_UINT32);
 
 static void cmd_add_port_meter_profile_trtcm_parsed(void *parsed_result,
-       __attribute__((unused)) struct cmdline *cl,
-       __attribute__((unused)) void *data)
+       __rte_unused struct cmdline *cl,
+       __rte_unused void *data)
 {
        struct cmd_add_port_meter_profile_trtcm_result *res = parsed_result;
        struct rte_mtr_meter_profile mp;
@@ -277,11 +481,12 @@ static void cmd_add_port_meter_profile_trtcm_parsed(void *parsed_result,
 
        /* Private shaper profile params */
        memset(&mp, 0, sizeof(struct rte_mtr_meter_profile));
-       mp.alg = 0;
+       mp.alg = RTE_MTR_TRTCM_RFC2698;
        mp.trtcm_rfc2698.cir = res->cir;
        mp.trtcm_rfc2698.pir = res->pir;
        mp.trtcm_rfc2698.cbs = res->cbs;
        mp.trtcm_rfc2698.pbs = res->pbs;
+       mp.packet_mode = res->packet_mode;
 
        ret = rte_mtr_meter_profile_add(port_id, profile_id, &mp, &error);
        if (ret != 0) {
@@ -293,19 +498,20 @@ static void cmd_add_port_meter_profile_trtcm_parsed(void *parsed_result,
 cmdline_parse_inst_t cmd_add_port_meter_profile_trtcm = {
        .f = cmd_add_port_meter_profile_trtcm_parsed,
        .data = NULL,
-       .help_str = "Add port meter profile trtcm (rfc2698)",
+       .help_str = "add port meter profile trtcm_rfc2698 <port_id> <profile_id> <cir> <pir> <cbs> <pbs> <packet_mode>",
        .tokens = {
                (void *)&cmd_add_port_meter_profile_trtcm_add,
                (void *)&cmd_add_port_meter_profile_trtcm_port,
                (void *)&cmd_add_port_meter_profile_trtcm_meter,
                (void *)&cmd_add_port_meter_profile_trtcm_profile,
+               (void *)&cmd_add_port_meter_profile_trtcm_trtcm_rfc2698,
                (void *)&cmd_add_port_meter_profile_trtcm_port_id,
                (void *)&cmd_add_port_meter_profile_trtcm_profile_id,
-               (void *)&cmd_add_port_meter_profile_trtcm_trtcm_rfc2698,
                (void *)&cmd_add_port_meter_profile_trtcm_cir,
                (void *)&cmd_add_port_meter_profile_trtcm_pir,
                (void *)&cmd_add_port_meter_profile_trtcm_cbs,
                (void *)&cmd_add_port_meter_profile_trtcm_pbs,
+               (void *)&cmd_add_port_meter_profile_trtcm_packet_mode,
                NULL,
        },
 };
@@ -323,58 +529,64 @@ struct cmd_add_port_meter_profile_trtcm_rfc4115_result {
        uint64_t eir;
        uint64_t cbs;
        uint64_t ebs;
+       int packet_mode;
 };
 
-cmdline_parse_token_string_t cmd_add_port_meter_profile_trtcm_rfc4115_add =
+static cmdline_parse_token_string_t cmd_add_port_meter_profile_trtcm_rfc4115_add =
        TOKEN_STRING_INITIALIZER(
                struct cmd_add_port_meter_profile_trtcm_rfc4115_result, add,
                "add");
-cmdline_parse_token_string_t cmd_add_port_meter_profile_trtcm_rfc4115_port =
+static cmdline_parse_token_string_t cmd_add_port_meter_profile_trtcm_rfc4115_port =
        TOKEN_STRING_INITIALIZER(
                struct cmd_add_port_meter_profile_trtcm_rfc4115_result,
                        port, "port");
-cmdline_parse_token_string_t cmd_add_port_meter_profile_trtcm_rfc4115_meter =
+static cmdline_parse_token_string_t cmd_add_port_meter_profile_trtcm_rfc4115_meter =
        TOKEN_STRING_INITIALIZER(
                struct cmd_add_port_meter_profile_trtcm_rfc4115_result,
                        meter, "meter");
-cmdline_parse_token_string_t cmd_add_port_meter_profile_trtcm_rfc4115_profile =
+static cmdline_parse_token_string_t cmd_add_port_meter_profile_trtcm_rfc4115_profile =
        TOKEN_STRING_INITIALIZER(
                struct cmd_add_port_meter_profile_trtcm_rfc4115_result,
                        profile, "profile");
-cmdline_parse_token_string_t
+static cmdline_parse_token_string_t
        cmd_add_port_meter_profile_trtcm_rfc4115_trtcm_rfc4115 =
        TOKEN_STRING_INITIALIZER(
                struct cmd_add_port_meter_profile_trtcm_rfc4115_result,
                        trtcm_rfc4115, "trtcm_rfc4115");
-cmdline_parse_token_num_t cmd_add_port_meter_profile_trtcm_rfc4115_port_id =
+static cmdline_parse_token_num_t cmd_add_port_meter_profile_trtcm_rfc4115_port_id =
        TOKEN_NUM_INITIALIZER(
                struct cmd_add_port_meter_profile_trtcm_rfc4115_result,
-                       port_id, UINT16);
-cmdline_parse_token_num_t cmd_add_port_meter_profile_trtcm_rfc4115_profile_id =
+                       port_id, RTE_UINT16);
+static cmdline_parse_token_num_t cmd_add_port_meter_profile_trtcm_rfc4115_profile_id =
        TOKEN_NUM_INITIALIZER(
                struct cmd_add_port_meter_profile_trtcm_rfc4115_result,
-                       profile_id, UINT32);
-cmdline_parse_token_num_t cmd_add_port_meter_profile_trtcm_rfc4115_cir =
+                       profile_id, RTE_UINT32);
+static cmdline_parse_token_num_t cmd_add_port_meter_profile_trtcm_rfc4115_cir =
        TOKEN_NUM_INITIALIZER(
                struct cmd_add_port_meter_profile_trtcm_rfc4115_result,
-                       cir, UINT64);
-cmdline_parse_token_num_t cmd_add_port_meter_profile_trtcm_rfc4115_eir =
+                       cir, RTE_UINT64);
+static cmdline_parse_token_num_t cmd_add_port_meter_profile_trtcm_rfc4115_eir =
        TOKEN_NUM_INITIALIZER(
                struct cmd_add_port_meter_profile_trtcm_rfc4115_result,
-                       eir, UINT64);
-cmdline_parse_token_num_t cmd_add_port_meter_profile_trtcm_rfc4115_cbs =
+                       eir, RTE_UINT64);
+static cmdline_parse_token_num_t cmd_add_port_meter_profile_trtcm_rfc4115_cbs =
        TOKEN_NUM_INITIALIZER(
                struct cmd_add_port_meter_profile_trtcm_rfc4115_result,
-                       cbs, UINT64);
-cmdline_parse_token_num_t cmd_add_port_meter_profile_trtcm_rfc4115_ebs =
+                       cbs, RTE_UINT64);
+static cmdline_parse_token_num_t cmd_add_port_meter_profile_trtcm_rfc4115_ebs =
        TOKEN_NUM_INITIALIZER(
                struct cmd_add_port_meter_profile_trtcm_rfc4115_result,
-                       ebs, UINT64);
+                       ebs, RTE_UINT64);
+static cmdline_parse_token_num_t
+       cmd_add_port_meter_profile_trtcm_rfc4115_packet_mode =
+       TOKEN_NUM_INITIALIZER(
+               struct cmd_add_port_meter_profile_trtcm_rfc4115_result,
+                       packet_mode, RTE_UINT32);
 
 static void cmd_add_port_meter_profile_trtcm_rfc4115_parsed(
        void *parsed_result,
-       __attribute__((unused)) struct cmdline *cl,
-       __attribute__((unused)) void *data)
+       __rte_unused struct cmdline *cl,
+       __rte_unused void *data)
 {
        struct cmd_add_port_meter_profile_trtcm_rfc4115_result *res =
                parsed_result;
@@ -389,11 +601,12 @@ static void cmd_add_port_meter_profile_trtcm_rfc4115_parsed(
 
        /* Private shaper profile params */
        memset(&mp, 0, sizeof(struct rte_mtr_meter_profile));
-       mp.alg = 0;
+       mp.alg = RTE_MTR_TRTCM_RFC4115;
        mp.trtcm_rfc4115.cir = res->cir;
        mp.trtcm_rfc4115.eir = res->eir;
        mp.trtcm_rfc4115.cbs = res->cbs;
        mp.trtcm_rfc4115.ebs = res->ebs;
+       mp.packet_mode = res->packet_mode;
 
        ret = rte_mtr_meter_profile_add(port_id, profile_id, &mp, &error);
        if (ret != 0) {
@@ -405,19 +618,20 @@ static void cmd_add_port_meter_profile_trtcm_rfc4115_parsed(
 cmdline_parse_inst_t cmd_add_port_meter_profile_trtcm_rfc4115 = {
        .f = cmd_add_port_meter_profile_trtcm_rfc4115_parsed,
        .data = NULL,
-       .help_str = "Add port meter profile trtcm (rfc4115)",
+       .help_str = "add port meter profile trtcm_rfc4115 <port_id> <profile_id> <cir> <eir> <cbs> <ebs> <packet_mode>",
        .tokens = {
                (void *)&cmd_add_port_meter_profile_trtcm_rfc4115_add,
                (void *)&cmd_add_port_meter_profile_trtcm_rfc4115_port,
                (void *)&cmd_add_port_meter_profile_trtcm_rfc4115_meter,
                (void *)&cmd_add_port_meter_profile_trtcm_rfc4115_profile,
+               (void *)&cmd_add_port_meter_profile_trtcm_rfc4115_trtcm_rfc4115,
                (void *)&cmd_add_port_meter_profile_trtcm_rfc4115_port_id,
                (void *)&cmd_add_port_meter_profile_trtcm_rfc4115_profile_id,
-               (void *)&cmd_add_port_meter_profile_trtcm_rfc4115_trtcm_rfc4115,
                (void *)&cmd_add_port_meter_profile_trtcm_rfc4115_cir,
                (void *)&cmd_add_port_meter_profile_trtcm_rfc4115_eir,
                (void *)&cmd_add_port_meter_profile_trtcm_rfc4115_cbs,
                (void *)&cmd_add_port_meter_profile_trtcm_rfc4115_ebs,
+               (void *)&cmd_add_port_meter_profile_trtcm_rfc4115_packet_mode,
                NULL,
        },
 };
@@ -432,33 +646,33 @@ struct cmd_del_port_meter_profile_result {
        uint32_t profile_id;
 };
 
-cmdline_parse_token_string_t cmd_del_port_meter_profile_del =
+static cmdline_parse_token_string_t cmd_del_port_meter_profile_del =
        TOKEN_STRING_INITIALIZER(
                struct cmd_del_port_meter_profile_result, del, "del");
-cmdline_parse_token_string_t cmd_del_port_meter_profile_port =
+static cmdline_parse_token_string_t cmd_del_port_meter_profile_port =
        TOKEN_STRING_INITIALIZER(
                struct cmd_del_port_meter_profile_result,
                        port, "port");
-cmdline_parse_token_string_t cmd_del_port_meter_profile_meter =
+static cmdline_parse_token_string_t cmd_del_port_meter_profile_meter =
        TOKEN_STRING_INITIALIZER(
                struct cmd_del_port_meter_profile_result,
                        meter, "meter");
-cmdline_parse_token_string_t cmd_del_port_meter_profile_profile =
+static cmdline_parse_token_string_t cmd_del_port_meter_profile_profile =
        TOKEN_STRING_INITIALIZER(
                struct cmd_del_port_meter_profile_result,
                        profile, "profile");
-cmdline_parse_token_num_t cmd_del_port_meter_profile_port_id =
+static cmdline_parse_token_num_t cmd_del_port_meter_profile_port_id =
        TOKEN_NUM_INITIALIZER(
                struct cmd_del_port_meter_profile_result,
-                       port_id, UINT16);
-cmdline_parse_token_num_t cmd_del_port_meter_profile_profile_id =
+                       port_id, RTE_UINT16);
+static cmdline_parse_token_num_t cmd_del_port_meter_profile_profile_id =
        TOKEN_NUM_INITIALIZER(
                struct cmd_del_port_meter_profile_result,
-                       profile_id, UINT32);
+                       profile_id, RTE_UINT32);
 
 static void cmd_del_port_meter_profile_parsed(void *parsed_result,
-       __attribute__((unused)) struct cmdline *cl,
-       __attribute__((unused)) void *data)
+       __rte_unused struct cmdline *cl,
+       __rte_unused void *data)
 {
        struct cmd_del_port_meter_profile_result *res = parsed_result;
        struct rte_mtr_error error;
@@ -480,7 +694,7 @@ static void cmd_del_port_meter_profile_parsed(void *parsed_result,
 cmdline_parse_inst_t cmd_del_port_meter_profile = {
        .f = cmd_del_port_meter_profile_parsed,
        .data = NULL,
-       .help_str = "Delete port meter profile",
+       .help_str = "del port meter profile <port_id> <profile_id>",
        .tokens = {
                (void *)&cmd_del_port_meter_profile_del,
                (void *)&cmd_del_port_meter_profile_port,
@@ -493,65 +707,70 @@ cmdline_parse_inst_t cmd_del_port_meter_profile = {
 };
 
 /* *** Create Port Meter Object *** */
-struct cmd_set_port_meter_result {
-       cmdline_fixed_string_t set;
+struct cmd_create_port_meter_result {
+       cmdline_fixed_string_t create;
        cmdline_fixed_string_t port;
        cmdline_fixed_string_t meter;
        uint16_t port_id;
        uint32_t mtr_id;
        uint32_t profile_id;
+       uint32_t policy_id;
+       cmdline_fixed_string_t meter_enable;
        cmdline_fixed_string_t g_action;
        cmdline_fixed_string_t y_action;
        cmdline_fixed_string_t r_action;
        uint64_t statistics_mask;
        uint32_t shared;
+       cmdline_multi_string_t meter_input_color;
 };
 
-cmdline_parse_token_string_t cmd_set_port_meter_set =
-       TOKEN_STRING_INITIALIZER(
-               struct cmd_set_port_meter_result, set, "set");
-cmdline_parse_token_string_t cmd_set_port_meter_port =
-       TOKEN_STRING_INITIALIZER(
-               struct cmd_set_port_meter_result, port, "port");
-cmdline_parse_token_string_t cmd_set_port_meter_meter =
-       TOKEN_STRING_INITIALIZER(
-               struct cmd_set_port_meter_result, meter, "meter");
-cmdline_parse_token_num_t cmd_set_port_meter_port_id =
-       TOKEN_NUM_INITIALIZER(
-               struct cmd_set_port_meter_result, port_id, UINT16);
-cmdline_parse_token_num_t cmd_set_port_meter_mtr_id =
-       TOKEN_NUM_INITIALIZER(
-               struct cmd_set_port_meter_result, mtr_id, UINT32);
-cmdline_parse_token_num_t cmd_set_port_meter_profile_id =
-       TOKEN_NUM_INITIALIZER(
-               struct cmd_set_port_meter_result, profile_id, UINT32);
-cmdline_parse_token_string_t cmd_set_port_meter_g_action =
-       TOKEN_STRING_INITIALIZER(struct cmd_set_port_meter_result,
-               g_action, "R#Y#G#D");
-cmdline_parse_token_string_t cmd_set_port_meter_y_action =
-       TOKEN_STRING_INITIALIZER(struct cmd_set_port_meter_result,
-               y_action, "R#Y#G#D");
-cmdline_parse_token_string_t cmd_set_port_meter_r_action =
-       TOKEN_STRING_INITIALIZER(struct cmd_set_port_meter_result,
-               r_action, "R#Y#G#D");
-cmdline_parse_token_num_t cmd_set_port_meter_statistics_mask =
-       TOKEN_NUM_INITIALIZER(struct cmd_set_port_meter_result,
-               statistics_mask, UINT64);
-cmdline_parse_token_num_t cmd_set_port_meter_shared =
-       TOKEN_NUM_INITIALIZER(struct cmd_set_port_meter_result,
-               shared, UINT32);
-
-static void cmd_set_port_meter_parsed(void *parsed_result,
-       __attribute__((unused)) struct cmdline *cl,
-       __attribute__((unused)) void *data)
+static cmdline_parse_token_string_t cmd_create_port_meter_create =
+       TOKEN_STRING_INITIALIZER(
+               struct cmd_create_port_meter_result, create, "create");
+static cmdline_parse_token_string_t cmd_create_port_meter_port =
+       TOKEN_STRING_INITIALIZER(
+               struct cmd_create_port_meter_result, port, "port");
+static cmdline_parse_token_string_t cmd_create_port_meter_meter =
+       TOKEN_STRING_INITIALIZER(
+               struct cmd_create_port_meter_result, meter, "meter");
+static cmdline_parse_token_num_t cmd_create_port_meter_port_id =
+       TOKEN_NUM_INITIALIZER(
+               struct cmd_create_port_meter_result, port_id, RTE_UINT16);
+static cmdline_parse_token_num_t cmd_create_port_meter_mtr_id =
+       TOKEN_NUM_INITIALIZER(
+               struct cmd_create_port_meter_result, mtr_id, RTE_UINT32);
+static cmdline_parse_token_num_t cmd_create_port_meter_profile_id =
+       TOKEN_NUM_INITIALIZER(
+               struct cmd_create_port_meter_result, profile_id, RTE_UINT32);
+static cmdline_parse_token_num_t cmd_create_port_meter_policy_id =
+       TOKEN_NUM_INITIALIZER(
+               struct cmd_create_port_meter_result, policy_id, RTE_UINT32);
+static cmdline_parse_token_string_t cmd_create_port_meter_meter_enable =
+       TOKEN_STRING_INITIALIZER(struct cmd_create_port_meter_result,
+               meter_enable, "yes#no");
+static cmdline_parse_token_num_t cmd_create_port_meter_statistics_mask =
+       TOKEN_NUM_INITIALIZER(struct cmd_create_port_meter_result,
+               statistics_mask, RTE_UINT64);
+static cmdline_parse_token_num_t cmd_create_port_meter_shared =
+       TOKEN_NUM_INITIALIZER(struct cmd_create_port_meter_result,
+               shared, RTE_UINT32);
+static cmdline_parse_token_string_t cmd_create_port_meter_input_color =
+       TOKEN_STRING_INITIALIZER(struct cmd_create_port_meter_result,
+               meter_input_color, TOKEN_STRING_MULTI);
+
+static void cmd_create_port_meter_parsed(void *parsed_result,
+       __rte_unused struct cmdline *cl,
+       __rte_unused void *data)
 {
-       struct cmd_set_port_meter_result *res = parsed_result;
+       struct cmd_create_port_meter_result *res = parsed_result;
        struct rte_mtr_error error;
        struct rte_mtr_params params;
        uint32_t mtr_id = res->mtr_id;
        uint32_t shared = res->shared;
+       uint32_t use_prev_meter_color = 0;
        uint16_t port_id = res->port_id;
-
+       enum rte_color *dscp_table = NULL;
+       char *c_str = res->meter_input_color;
        int ret;
 
        if (port_id_is_invalid(port_id, ENABLED_WARN))
@@ -560,40 +779,237 @@ static void cmd_set_port_meter_parsed(void *parsed_result,
        /* Meter params */
        memset(&params, 0, sizeof(struct rte_mtr_params));
        params.meter_profile_id = res->profile_id;
-       params.use_prev_mtr_color = 1;
-       params.dscp_table = NULL;
-       params.meter_enable = 1;
-       params.action[RTE_MTR_GREEN] =
-               string_to_policer_action(res->g_action);
-       params.action[RTE_MTR_YELLOW] =
-               string_to_policer_action(res->y_action);
-       params.action[RTE_MTR_RED] =
-               string_to_policer_action(res->r_action);
+       params.meter_policy_id = res->policy_id;
+       /* Parse meter input color string params */
+       ret = parse_meter_color_str(c_str, &use_prev_meter_color, &dscp_table);
+       if (ret) {
+               fprintf(stderr,
+                       " Meter input color params string parse error\n");
+               return;
+       }
+
+       params.use_prev_mtr_color = use_prev_meter_color;
+       params.dscp_table = dscp_table;
+
+       if (strcmp(res->meter_enable, "yes") == 0)
+               params.meter_enable = 1;
+       else
+               params.meter_enable = 0;
        params.stats_mask = res->statistics_mask;
 
        ret = rte_mtr_create(port_id, mtr_id, &params, shared, &error);
        if (ret != 0) {
+               free(dscp_table);
                print_err_msg(&error);
                return;
        }
 }
 
-cmdline_parse_inst_t cmd_set_port_meter = {
-       .f = cmd_set_port_meter_parsed,
+cmdline_parse_inst_t cmd_create_port_meter = {
+       .f = cmd_create_port_meter_parsed,
        .data = NULL,
-       .help_str = "Set port meter",
+       .help_str = "create port meter <port_id> <mtr_id> <profile_id> <policy_id> "
+               "<meter_enable>(yes|no) <stats_mask> <shared> <use_pre_meter_color> "
+               "[<dscp_tbl_entry0> <dscp_tbl_entry1> ...<dscp_tbl_entry63>]",
        .tokens = {
-               (void *)&cmd_set_port_meter_set,
-               (void *)&cmd_set_port_meter_port,
-               (void *)&cmd_set_port_meter_meter,
-               (void *)&cmd_set_port_meter_port_id,
-               (void *)&cmd_set_port_meter_mtr_id,
-               (void *)&cmd_set_port_meter_profile_id,
-               (void *)&cmd_set_port_meter_g_action,
-               (void *)&cmd_set_port_meter_y_action,
-               (void *)&cmd_set_port_meter_r_action,
-               (void *)&cmd_set_port_meter_statistics_mask,
-               (void *)&cmd_set_port_meter_shared,
+               (void *)&cmd_create_port_meter_create,
+               (void *)&cmd_create_port_meter_port,
+               (void *)&cmd_create_port_meter_meter,
+               (void *)&cmd_create_port_meter_port_id,
+               (void *)&cmd_create_port_meter_mtr_id,
+               (void *)&cmd_create_port_meter_profile_id,
+               (void *)&cmd_create_port_meter_policy_id,
+               (void *)&cmd_create_port_meter_meter_enable,
+               (void *)&cmd_create_port_meter_statistics_mask,
+               (void *)&cmd_create_port_meter_shared,
+               (void *)&cmd_create_port_meter_input_color,
+               NULL,
+       },
+};
+
+/* *** Enable Meter of MTR Object  *** */
+struct cmd_enable_port_meter_result {
+       cmdline_fixed_string_t enable;
+       cmdline_fixed_string_t port;
+       cmdline_fixed_string_t meter;
+       uint16_t port_id;
+       uint32_t mtr_id;
+};
+
+static cmdline_parse_token_string_t cmd_enable_port_meter_enable =
+       TOKEN_STRING_INITIALIZER(
+               struct cmd_enable_port_meter_result, enable, "enable");
+static cmdline_parse_token_string_t cmd_enable_port_meter_port =
+       TOKEN_STRING_INITIALIZER(
+               struct cmd_enable_port_meter_result, port, "port");
+static cmdline_parse_token_string_t cmd_enable_port_meter_meter =
+       TOKEN_STRING_INITIALIZER(
+               struct cmd_enable_port_meter_result, meter, "meter");
+static cmdline_parse_token_num_t cmd_enable_port_meter_port_id =
+       TOKEN_NUM_INITIALIZER(
+               struct cmd_enable_port_meter_result, port_id, RTE_UINT16);
+static cmdline_parse_token_num_t cmd_enable_port_meter_mtr_id =
+       TOKEN_NUM_INITIALIZER(
+               struct cmd_enable_port_meter_result, mtr_id, RTE_UINT32);
+
+static void cmd_enable_port_meter_parsed(void *parsed_result,
+       __rte_unused struct cmdline *cl,
+       __rte_unused void *data)
+{
+       struct cmd_enable_port_meter_result *res = parsed_result;
+       struct rte_mtr_error error;
+       uint32_t mtr_id = res->mtr_id;
+       uint16_t port_id = res->port_id;
+
+       int ret;
+
+       if (port_id_is_invalid(port_id, ENABLED_WARN))
+               return;
+
+       /* Enable Meter */
+       ret = rte_mtr_meter_enable(port_id, mtr_id, &error);
+       if (ret != 0) {
+               print_err_msg(&error);
+               return;
+       }
+}
+
+cmdline_parse_inst_t cmd_enable_port_meter = {
+       .f = cmd_enable_port_meter_parsed,
+       .data = NULL,
+       .help_str = "enable port meter <port_id> <mtr_id>",
+       .tokens = {
+               (void *)&cmd_enable_port_meter_enable,
+               (void *)&cmd_enable_port_meter_port,
+               (void *)&cmd_enable_port_meter_meter,
+               (void *)&cmd_enable_port_meter_port_id,
+               (void *)&cmd_enable_port_meter_mtr_id,
+               NULL,
+       },
+};
+
+/* *** Disable Meter of MTR Object  *** */
+struct cmd_disable_port_meter_result {
+       cmdline_fixed_string_t disable;
+       cmdline_fixed_string_t port;
+       cmdline_fixed_string_t meter;
+       uint16_t port_id;
+       uint32_t mtr_id;
+};
+
+static cmdline_parse_token_string_t cmd_disable_port_meter_disable =
+       TOKEN_STRING_INITIALIZER(
+               struct cmd_disable_port_meter_result, disable, "disable");
+static cmdline_parse_token_string_t cmd_disable_port_meter_port =
+       TOKEN_STRING_INITIALIZER(
+               struct cmd_disable_port_meter_result, port, "port");
+static cmdline_parse_token_string_t cmd_disable_port_meter_meter =
+       TOKEN_STRING_INITIALIZER(
+               struct cmd_disable_port_meter_result, meter, "meter");
+static cmdline_parse_token_num_t cmd_disable_port_meter_port_id =
+       TOKEN_NUM_INITIALIZER(
+               struct cmd_disable_port_meter_result, port_id, RTE_UINT16);
+static cmdline_parse_token_num_t cmd_disable_port_meter_mtr_id =
+       TOKEN_NUM_INITIALIZER(
+               struct cmd_disable_port_meter_result, mtr_id, RTE_UINT32);
+
+static void cmd_disable_port_meter_parsed(void *parsed_result,
+       __rte_unused struct cmdline *cl,
+       __rte_unused void *data)
+{
+       struct cmd_disable_port_meter_result *res = parsed_result;
+       struct rte_mtr_error error;
+       uint32_t mtr_id = res->mtr_id;
+       uint16_t port_id = res->port_id;
+
+       int ret;
+
+       if (port_id_is_invalid(port_id, ENABLED_WARN))
+               return;
+
+       /* Disable Meter */
+       ret = rte_mtr_meter_disable(port_id, mtr_id, &error);
+       if (ret != 0) {
+               print_err_msg(&error);
+               return;
+       }
+}
+
+cmdline_parse_inst_t cmd_disable_port_meter = {
+       .f = cmd_disable_port_meter_parsed,
+       .data = NULL,
+       .help_str = "disable port meter <port_id> <mtr_id>",
+       .tokens = {
+               (void *)&cmd_disable_port_meter_disable,
+               (void *)&cmd_disable_port_meter_port,
+               (void *)&cmd_disable_port_meter_meter,
+               (void *)&cmd_disable_port_meter_port_id,
+               (void *)&cmd_disable_port_meter_mtr_id,
+               NULL,
+       },
+};
+
+/* *** Delete Port Meter Policy Object *** */
+struct cmd_del_port_meter_policy_result {
+       cmdline_fixed_string_t del;
+       cmdline_fixed_string_t port;
+       cmdline_fixed_string_t meter;
+       cmdline_fixed_string_t policy;
+       uint16_t port_id;
+       uint32_t policy_id;
+};
+
+static cmdline_parse_token_string_t cmd_del_port_meter_policy_del =
+       TOKEN_STRING_INITIALIZER(
+               struct cmd_del_port_meter_policy_result, del, "del");
+static cmdline_parse_token_string_t cmd_del_port_meter_policy_port =
+       TOKEN_STRING_INITIALIZER(
+               struct cmd_del_port_meter_policy_result, port, "port");
+static cmdline_parse_token_string_t cmd_del_port_meter_policy_meter =
+       TOKEN_STRING_INITIALIZER(
+               struct cmd_del_port_meter_policy_result, meter, "meter");
+static cmdline_parse_token_string_t cmd_del_port_meter_policy_policy =
+       TOKEN_STRING_INITIALIZER(
+               struct cmd_del_port_meter_policy_result, policy, "policy");
+static cmdline_parse_token_num_t cmd_del_port_meter_policy_port_id =
+       TOKEN_NUM_INITIALIZER(
+               struct cmd_del_port_meter_policy_result, port_id, RTE_UINT16);
+static cmdline_parse_token_num_t cmd_del_port_meter_policy_policy_id =
+       TOKEN_NUM_INITIALIZER(
+               struct cmd_del_port_meter_policy_result, policy_id, RTE_UINT32);
+
+static void cmd_del_port_meter_policy_parsed(void *parsed_result,
+       __rte_unused struct cmdline *cl,
+       __rte_unused void *data)
+{
+       struct cmd_del_port_meter_policy_result *res = parsed_result;
+       struct rte_mtr_error error;
+       uint32_t policy_id = res->policy_id;
+       uint16_t port_id = res->port_id;
+       int ret;
+
+       if (port_id_is_invalid(port_id, ENABLED_WARN))
+               return;
+
+       /* Delete Meter Policy*/
+       ret = rte_mtr_meter_policy_delete(port_id, policy_id, &error);
+       if (ret != 0) {
+               print_err_msg(&error);
+               return;
+       }
+}
+
+cmdline_parse_inst_t cmd_del_port_meter_policy = {
+       .f = cmd_del_port_meter_policy_parsed,
+       .data = NULL,
+       .help_str = "Delete port meter policy",
+       .tokens = {
+               (void *)&cmd_del_port_meter_policy_del,
+               (void *)&cmd_del_port_meter_policy_port,
+               (void *)&cmd_del_port_meter_policy_meter,
+               (void *)&cmd_del_port_meter_policy_policy,
+               (void *)&cmd_del_port_meter_policy_port_id,
+               (void *)&cmd_del_port_meter_policy_policy_id,
                NULL,
        },
 };
@@ -607,25 +1023,25 @@ struct cmd_del_port_meter_result {
        uint32_t mtr_id;
 };
 
-cmdline_parse_token_string_t cmd_del_port_meter_del =
+static cmdline_parse_token_string_t cmd_del_port_meter_del =
        TOKEN_STRING_INITIALIZER(
                struct cmd_del_port_meter_result, del, "del");
-cmdline_parse_token_string_t cmd_del_port_meter_port =
+static cmdline_parse_token_string_t cmd_del_port_meter_port =
        TOKEN_STRING_INITIALIZER(
                struct cmd_del_port_meter_result, port, "port");
-cmdline_parse_token_string_t cmd_del_port_meter_meter =
+static cmdline_parse_token_string_t cmd_del_port_meter_meter =
        TOKEN_STRING_INITIALIZER(
                struct cmd_del_port_meter_result, meter, "meter");
-cmdline_parse_token_num_t cmd_del_port_meter_port_id =
+static cmdline_parse_token_num_t cmd_del_port_meter_port_id =
        TOKEN_NUM_INITIALIZER(
-               struct cmd_del_port_meter_result, port_id, UINT16);
-cmdline_parse_token_num_t cmd_del_port_meter_mtr_id =
+               struct cmd_del_port_meter_result, port_id, RTE_UINT16);
+static cmdline_parse_token_num_t cmd_del_port_meter_mtr_id =
        TOKEN_NUM_INITIALIZER(
-               struct cmd_del_port_meter_result, mtr_id, UINT32);
+               struct cmd_del_port_meter_result, mtr_id, RTE_UINT32);
 
 static void cmd_del_port_meter_parsed(void *parsed_result,
-       __attribute__((unused)) struct cmdline *cl,
-       __attribute__((unused)) void *data)
+       __rte_unused struct cmdline *cl,
+       __rte_unused void *data)
 {
        struct cmd_del_port_meter_result *res = parsed_result;
        struct rte_mtr_error error;
@@ -648,7 +1064,7 @@ static void cmd_del_port_meter_parsed(void *parsed_result,
 cmdline_parse_inst_t cmd_del_port_meter = {
        .f = cmd_del_port_meter_parsed,
        .data = NULL,
-       .help_str = "Delete port meter",
+       .help_str = "del port meter <port_id> <mtr_id>",
        .tokens = {
                (void *)&cmd_del_port_meter_del,
                (void *)&cmd_del_port_meter_port,
@@ -670,31 +1086,34 @@ struct cmd_set_port_meter_profile_result {
        uint32_t profile_id;
 };
 
-cmdline_parse_token_string_t cmd_set_port_meter_profile_set =
+static cmdline_parse_token_string_t cmd_set_port_meter_profile_set =
        TOKEN_STRING_INITIALIZER(
                struct cmd_set_port_meter_profile_result, set, "set");
-cmdline_parse_token_string_t cmd_set_port_meter_profile_port =
+static cmdline_parse_token_string_t cmd_set_port_meter_profile_port =
        TOKEN_STRING_INITIALIZER(
                struct cmd_set_port_meter_profile_result, port, "port");
-cmdline_parse_token_string_t cmd_set_port_meter_profile_meter =
+static cmdline_parse_token_string_t cmd_set_port_meter_profile_meter =
        TOKEN_STRING_INITIALIZER(
                struct cmd_set_port_meter_profile_result, meter, "meter");
-cmdline_parse_token_string_t cmd_set_port_meter_profile_profile =
+static cmdline_parse_token_string_t cmd_set_port_meter_profile_profile =
        TOKEN_STRING_INITIALIZER(
                struct cmd_set_port_meter_profile_result, profile, "profile");
-cmdline_parse_token_num_t cmd_set_port_meter_profile_port_id =
+static cmdline_parse_token_num_t cmd_set_port_meter_profile_port_id =
        TOKEN_NUM_INITIALIZER(
-               struct cmd_set_port_meter_profile_result, port_id, UINT16);
-cmdline_parse_token_num_t cmd_set_port_meter_profile_mtr_id =
+               struct cmd_set_port_meter_profile_result, port_id,
+               RTE_UINT16);
+static cmdline_parse_token_num_t cmd_set_port_meter_profile_mtr_id =
        TOKEN_NUM_INITIALIZER(
-               struct cmd_set_port_meter_profile_result, mtr_id, UINT32);
-cmdline_parse_token_num_t cmd_set_port_meter_profile_profile_id =
+               struct cmd_set_port_meter_profile_result, mtr_id,
+               RTE_UINT32);
+static cmdline_parse_token_num_t cmd_set_port_meter_profile_profile_id =
        TOKEN_NUM_INITIALIZER(
-               struct cmd_set_port_meter_profile_result, profile_id, UINT32);
+               struct cmd_set_port_meter_profile_result, profile_id,
+               RTE_UINT32);
 
 static void cmd_set_port_meter_profile_parsed(void *parsed_result,
-       __attribute__((unused)) struct cmdline *cl,
-       __attribute__((unused)) void *data)
+       __rte_unused struct cmdline *cl,
+       __rte_unused void *data)
 {
        struct cmd_set_port_meter_profile_result *res = parsed_result;
        struct rte_mtr_error error;
@@ -719,7 +1138,7 @@ static void cmd_set_port_meter_profile_parsed(void *parsed_result,
 cmdline_parse_inst_t cmd_set_port_meter_profile = {
        .f = cmd_set_port_meter_profile_parsed,
        .data = NULL,
-       .help_str = "Set port meter profile",
+       .help_str = "set port meter profile <port_id> <mtr_id> <profile_id>",
        .tokens = {
                (void *)&cmd_set_port_meter_profile_set,
                (void *)&cmd_set_port_meter_profile_port,
@@ -732,111 +1151,75 @@ cmdline_parse_inst_t cmd_set_port_meter_profile = {
        },
 };
 
-/* *** Set Port Meter Policer Action *** */
-struct cmd_set_port_meter_policer_action_result {
+/* *** Set Port Meter DSCP Table *** */
+struct cmd_set_port_meter_dscp_table_result {
        cmdline_fixed_string_t set;
        cmdline_fixed_string_t port;
        cmdline_fixed_string_t meter;
-       cmdline_fixed_string_t policer;
-       cmdline_fixed_string_t action;
-       uint16_t port_id;
-       uint32_t mtr_id;
-       cmdline_fixed_string_t color;
-       cmdline_fixed_string_t policer_action;
+       cmdline_fixed_string_t dscp_table;
+       cmdline_multi_string_t token_string;
 };
 
-cmdline_parse_token_string_t cmd_set_port_meter_policer_action_set =
+static cmdline_parse_token_string_t cmd_set_port_meter_dscp_table_set =
        TOKEN_STRING_INITIALIZER(
-               struct cmd_set_port_meter_policer_action_result, set, "set");
-cmdline_parse_token_string_t cmd_set_port_meter_policer_action_port =
+               struct cmd_set_port_meter_dscp_table_result, set, "set");
+static cmdline_parse_token_string_t cmd_set_port_meter_dscp_table_port =
        TOKEN_STRING_INITIALIZER(
-               struct cmd_set_port_meter_policer_action_result, port, "port");
-cmdline_parse_token_string_t cmd_set_port_meter_policer_action_meter =
+               struct cmd_set_port_meter_dscp_table_result, port, "port");
+static cmdline_parse_token_string_t cmd_set_port_meter_dscp_table_meter =
        TOKEN_STRING_INITIALIZER(
-               struct cmd_set_port_meter_policer_action_result, meter,
-               "meter");
-cmdline_parse_token_string_t cmd_set_port_meter_policer_action_policer =
+               struct cmd_set_port_meter_dscp_table_result, meter, "meter");
+static cmdline_parse_token_string_t cmd_set_port_meter_dscp_table_dscp_table =
        TOKEN_STRING_INITIALIZER(
-               struct cmd_set_port_meter_policer_action_result, policer,
-               "policer");
-cmdline_parse_token_string_t cmd_set_port_meter_policer_action_action =
-       TOKEN_STRING_INITIALIZER(
-               struct cmd_set_port_meter_policer_action_result, action,
-               "action");
-cmdline_parse_token_num_t cmd_set_port_meter_policer_action_port_id =
-       TOKEN_NUM_INITIALIZER(
-               struct cmd_set_port_meter_policer_action_result, port_id,
-               UINT16);
-cmdline_parse_token_num_t cmd_set_port_meter_policer_action_mtr_id =
-       TOKEN_NUM_INITIALIZER(
-               struct cmd_set_port_meter_policer_action_result, mtr_id,
-               UINT32);
-cmdline_parse_token_string_t cmd_set_port_meter_policer_action_color =
-       TOKEN_STRING_INITIALIZER(
-               struct cmd_set_port_meter_policer_action_result, color,
-               "G#Y#R");
-cmdline_parse_token_string_t cmd_set_port_meter_policer_action_policer_action =
-       TOKEN_STRING_INITIALIZER(
-               struct cmd_set_port_meter_policer_action_result,
-               policer_action, "G#Y#R#D");
-
-static void cmd_set_port_meter_policer_action_parsed(void *parsed_result,
-       __attribute__((unused)) struct cmdline *cl,
-       __attribute__((unused)) void *data)
+               struct cmd_set_port_meter_dscp_table_result,
+               dscp_table, "dscp table");
+static cmdline_parse_token_string_t cmd_set_port_meter_dscp_table_token_string =
+       TOKEN_STRING_INITIALIZER(struct cmd_set_port_meter_dscp_table_result,
+               token_string, TOKEN_STRING_MULTI);
+
+static void cmd_set_port_meter_dscp_table_parsed(void *parsed_result,
+       __rte_unused struct cmdline *cl,
+       __rte_unused void *data)
 {
-       struct cmd_set_port_meter_policer_action_result *res = parsed_result;
-       enum rte_mtr_color color;
-       enum rte_mtr_policer_action action[RTE_MTR_COLORS];
+       struct cmd_set_port_meter_dscp_table_result *res = parsed_result;
        struct rte_mtr_error error;
-       uint32_t mtr_id = res->mtr_id;
-       uint16_t port_id = res->port_id;
-       char *c = res->color;
-       char *a = res->policer_action;
+       enum rte_color *dscp_table = NULL;
+       char *t_str = res->token_string;
+       uint32_t mtr_id = 0;
+       uint16_t port_id;
        int ret;
 
-       if (port_id_is_invalid(port_id, ENABLED_WARN))
+       /* Parse string */
+       ret = parse_multi_token_string(t_str, &port_id, &mtr_id, &dscp_table);
+       if (ret) {
+               fprintf(stderr, " Multi token string parse error\n");
                return;
+       }
 
-       /* Color */
-       if (strcmp(c, "G") == 0)
-               color = RTE_MTR_GREEN;
-       else if (strcmp(c, "Y") == 0)
-               color = RTE_MTR_YELLOW;
-       else
-               color = RTE_MTR_RED;
-
-       /* Action */
-       if (strcmp(a, "G") == 0)
-               action[color] = MTR_POLICER_ACTION_COLOR_GREEN;
-       else if (strcmp(a, "Y") == 0)
-               action[color] = MTR_POLICER_ACTION_COLOR_YELLOW;
-       else if (strcmp(a, "R") == 0)
-               action[color] = MTR_POLICER_ACTION_COLOR_RED;
-       else
-               action[color] = MTR_POLICER_ACTION_DROP;
+       if (port_id_is_invalid(port_id, ENABLED_WARN))
+               goto free_table;
 
-       ret = rte_mtr_policer_actions_update(port_id, mtr_id,
-               1 << color, action, &error);
-       if (ret != 0) {
+       /* Update Meter DSCP Table*/
+       ret = rte_mtr_meter_dscp_table_update(port_id, mtr_id,
+               dscp_table, &error);
+       if (ret != 0)
                print_err_msg(&error);
-               return;
-       }
+
+free_table:
+       free(dscp_table);
 }
 
-cmdline_parse_inst_t cmd_set_port_meter_policer_action = {
-       .f = cmd_set_port_meter_policer_action_parsed,
+cmdline_parse_inst_t cmd_set_port_meter_dscp_table = {
+       .f = cmd_set_port_meter_dscp_table_parsed,
        .data = NULL,
-       .help_str = "Set port meter policer action",
+       .help_str = "set port meter dscp table <port_id> <mtr_id> "
+               "[<dscp_tbl_entry0> <dscp_tbl_entry1> ... <dscp_tbl_entry63>]",
        .tokens = {
-               (void *)&cmd_set_port_meter_policer_action_set,
-               (void *)&cmd_set_port_meter_policer_action_port,
-               (void *)&cmd_set_port_meter_policer_action_meter,
-               (void *)&cmd_set_port_meter_policer_action_policer,
-               (void *)&cmd_set_port_meter_policer_action_action,
-               (void *)&cmd_set_port_meter_policer_action_port_id,
-               (void *)&cmd_set_port_meter_policer_action_mtr_id,
-               (void *)&cmd_set_port_meter_policer_action_color,
-               (void *)&cmd_set_port_meter_policer_action_policer_action,
+               (void *)&cmd_set_port_meter_dscp_table_set,
+               (void *)&cmd_set_port_meter_dscp_table_port,
+               (void *)&cmd_set_port_meter_dscp_table_meter,
+               (void *)&cmd_set_port_meter_dscp_table_dscp_table,
+               (void *)&cmd_set_port_meter_dscp_table_token_string,
                NULL,
        },
 };
@@ -853,35 +1236,37 @@ struct cmd_set_port_meter_stats_mask_result {
        uint64_t stats_mask;
 };
 
-cmdline_parse_token_string_t cmd_set_port_meter_stats_mask_set =
+static cmdline_parse_token_string_t cmd_set_port_meter_stats_mask_set =
        TOKEN_STRING_INITIALIZER(
                struct cmd_set_port_meter_stats_mask_result, set, "set");
-cmdline_parse_token_string_t cmd_set_port_meter_stats_mask_port =
+static cmdline_parse_token_string_t cmd_set_port_meter_stats_mask_port =
        TOKEN_STRING_INITIALIZER(
                struct cmd_set_port_meter_stats_mask_result, port, "port");
-cmdline_parse_token_string_t cmd_set_port_meter_stats_mask_meter =
+static cmdline_parse_token_string_t cmd_set_port_meter_stats_mask_meter =
        TOKEN_STRING_INITIALIZER(
                struct cmd_set_port_meter_stats_mask_result, meter, "meter");
-cmdline_parse_token_string_t cmd_set_port_meter_stats_mask_stats =
+static cmdline_parse_token_string_t cmd_set_port_meter_stats_mask_stats =
        TOKEN_STRING_INITIALIZER(
                struct cmd_set_port_meter_stats_mask_result, stats, "stats");
-cmdline_parse_token_string_t cmd_set_port_meter_stats_mask_mask =
+static cmdline_parse_token_string_t cmd_set_port_meter_stats_mask_mask =
        TOKEN_STRING_INITIALIZER(
                struct cmd_set_port_meter_stats_mask_result, mask, "mask");
-cmdline_parse_token_num_t cmd_set_port_meter_stats_mask_port_id =
+static cmdline_parse_token_num_t cmd_set_port_meter_stats_mask_port_id =
        TOKEN_NUM_INITIALIZER(
-               struct cmd_set_port_meter_stats_mask_result, port_id, UINT16);
-cmdline_parse_token_num_t cmd_set_port_meter_stats_mask_mtr_id =
+               struct cmd_set_port_meter_stats_mask_result, port_id,
+               RTE_UINT16);
+static cmdline_parse_token_num_t cmd_set_port_meter_stats_mask_mtr_id =
        TOKEN_NUM_INITIALIZER(
-               struct cmd_set_port_meter_stats_mask_result, mtr_id, UINT32);
-cmdline_parse_token_num_t cmd_set_port_meter_stats_mask_stats_mask =
+               struct cmd_set_port_meter_stats_mask_result, mtr_id,
+               RTE_UINT32);
+static cmdline_parse_token_num_t cmd_set_port_meter_stats_mask_stats_mask =
        TOKEN_NUM_INITIALIZER(
                struct cmd_set_port_meter_stats_mask_result, stats_mask,
-               UINT64);
+               RTE_UINT64);
 
 static void cmd_set_port_meter_stats_mask_parsed(void *parsed_result,
-       __attribute__((unused)) struct cmdline *cl,
-       __attribute__((unused)) void *data)
+       __rte_unused struct cmdline *cl,
+       __rte_unused void *data)
 {
        struct cmd_set_port_meter_stats_mask_result *res = parsed_result;
        struct rte_mtr_error error;
@@ -903,7 +1288,7 @@ static void cmd_set_port_meter_stats_mask_parsed(void *parsed_result,
 cmdline_parse_inst_t cmd_set_port_meter_stats_mask = {
        .f = cmd_set_port_meter_stats_mask_parsed,
        .data = NULL,
-       .help_str = "Set port meter stats mask",
+       .help_str = "set port meter stats mask <port_id> <mtr_id> <stats_mask>",
        .tokens = {
                (void *)&cmd_set_port_meter_stats_mask_set,
                (void *)&cmd_set_port_meter_stats_mask_port,
@@ -925,47 +1310,50 @@ struct cmd_show_port_meter_stats_result {
        cmdline_fixed_string_t stats;
        uint16_t port_id;
        uint32_t mtr_id;
-       uint32_t clear;
+       cmdline_fixed_string_t clear;
 };
 
-cmdline_parse_token_string_t cmd_show_port_meter_stats_show =
+static cmdline_parse_token_string_t cmd_show_port_meter_stats_show =
        TOKEN_STRING_INITIALIZER(
                struct cmd_show_port_meter_stats_result, show, "show");
-cmdline_parse_token_string_t cmd_show_port_meter_stats_port =
+static cmdline_parse_token_string_t cmd_show_port_meter_stats_port =
        TOKEN_STRING_INITIALIZER(
                struct cmd_show_port_meter_stats_result, port, "port");
-cmdline_parse_token_string_t cmd_show_port_meter_stats_meter =
+static cmdline_parse_token_string_t cmd_show_port_meter_stats_meter =
        TOKEN_STRING_INITIALIZER(
                struct cmd_show_port_meter_stats_result, meter, "meter");
-cmdline_parse_token_string_t cmd_show_port_meter_stats_stats =
+static cmdline_parse_token_string_t cmd_show_port_meter_stats_stats =
        TOKEN_STRING_INITIALIZER(
                struct cmd_show_port_meter_stats_result, stats, "stats");
-cmdline_parse_token_num_t cmd_show_port_meter_stats_port_id =
-       TOKEN_NUM_INITIALIZER(
-               struct cmd_show_port_meter_stats_result, port_id, UINT16);
-cmdline_parse_token_num_t cmd_show_port_meter_stats_mtr_id =
+static cmdline_parse_token_num_t cmd_show_port_meter_stats_port_id =
        TOKEN_NUM_INITIALIZER(
-               struct cmd_show_port_meter_stats_result, mtr_id, UINT32);
-cmdline_parse_token_num_t cmd_show_port_meter_stats_clear =
+               struct cmd_show_port_meter_stats_result, port_id, RTE_UINT16);
+static cmdline_parse_token_num_t cmd_show_port_meter_stats_mtr_id =
        TOKEN_NUM_INITIALIZER(
-               struct cmd_show_port_meter_stats_result, clear, UINT32);
+               struct cmd_show_port_meter_stats_result, mtr_id, RTE_UINT32);
+static cmdline_parse_token_string_t cmd_show_port_meter_stats_clear =
+       TOKEN_STRING_INITIALIZER(
+               struct cmd_show_port_meter_stats_result, clear, "yes#no");
 
 static void cmd_show_port_meter_stats_parsed(void *parsed_result,
-       __attribute__((unused)) struct cmdline *cl,
-       __attribute__((unused)) void *data)
+       __rte_unused struct cmdline *cl,
+       __rte_unused void *data)
 {
        struct cmd_show_port_meter_stats_result *res = parsed_result;
        struct rte_mtr_stats stats;
        uint64_t stats_mask = 0;
        struct rte_mtr_error error;
        uint32_t mtr_id = res->mtr_id;
-       uint32_t clear = res->clear;
+       uint32_t clear = 0;
        uint16_t port_id = res->port_id;
        int ret;
 
        if (port_id_is_invalid(port_id, ENABLED_WARN))
                return;
 
+       if (strcmp(res->clear, "yes") == 0)
+               clear = 1;
+
        memset(&stats, 0, sizeof(struct rte_mtr_stats));
        ret = rte_mtr_stats_read(port_id, mtr_id, &stats,
                &stats_mask, clear, &error);
@@ -977,22 +1365,22 @@ static void cmd_show_port_meter_stats_parsed(void *parsed_result,
        /* Display stats */
        if (stats_mask & RTE_MTR_STATS_N_PKTS_GREEN)
                printf("\tPkts G: %" PRIu64 "\n",
-                       stats.n_pkts[RTE_MTR_GREEN]);
+                       stats.n_pkts[RTE_COLOR_GREEN]);
        if (stats_mask & RTE_MTR_STATS_N_BYTES_GREEN)
                printf("\tBytes G: %" PRIu64 "\n",
-                       stats.n_bytes[RTE_MTR_GREEN]);
+                       stats.n_bytes[RTE_COLOR_GREEN]);
        if (stats_mask & RTE_MTR_STATS_N_PKTS_YELLOW)
                printf("\tPkts Y: %" PRIu64 "\n",
-                       stats.n_pkts[RTE_MTR_YELLOW]);
+                       stats.n_pkts[RTE_COLOR_YELLOW]);
        if (stats_mask & RTE_MTR_STATS_N_BYTES_YELLOW)
                printf("\tBytes Y: %" PRIu64 "\n",
-                       stats.n_bytes[RTE_MTR_YELLOW]);
+                       stats.n_bytes[RTE_COLOR_YELLOW]);
        if (stats_mask & RTE_MTR_STATS_N_PKTS_RED)
                printf("\tPkts R: %" PRIu64 "\n",
-                       stats.n_pkts[RTE_MTR_RED]);
+                       stats.n_pkts[RTE_COLOR_RED]);
        if (stats_mask & RTE_MTR_STATS_N_BYTES_RED)
-               printf("\tBytes Y: %" PRIu64 "\n",
-                       stats.n_bytes[RTE_MTR_RED]);
+               printf("\tBytes R: %" PRIu64 "\n",
+                       stats.n_bytes[RTE_COLOR_RED]);
        if (stats_mask & RTE_MTR_STATS_N_PKTS_DROPPED)
                printf("\tPkts DROPPED: %" PRIu64 "\n",
                        stats.n_pkts_dropped);
@@ -1004,7 +1392,7 @@ static void cmd_show_port_meter_stats_parsed(void *parsed_result,
 cmdline_parse_inst_t cmd_show_port_meter_stats = {
        .f = cmd_show_port_meter_stats_parsed,
        .data = NULL,
-       .help_str = "Show port meter stats",
+       .help_str = "show port meter stats <port_id> <mtr_id> <clear>(yes|no)",
        .tokens = {
                (void *)&cmd_show_port_meter_stats_show,
                (void *)&cmd_show_port_meter_stats_port,