net/vhost: move to vhost library statistics API
[dpdk.git] / app / test-pmd / cmdline_mtr.c
index aa454af..57050ec 100644 (file)
@@ -37,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;
@@ -51,11 +53,18 @@ 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)
 {
@@ -92,13 +101,13 @@ parse_dscp_table_entries(char *str, enum rte_color **dscp_table)
        while (1) {
                if (strcmp(token, "G") == 0 ||
                        strcmp(token, "g") == 0)
-                       *dscp_table[i++] = RTE_COLOR_GREEN;
+                       (*dscp_table)[i++] = RTE_COLOR_GREEN;
                else if (strcmp(token, "Y") == 0 ||
                        strcmp(token, "y") == 0)
-                       *dscp_table[i++] = RTE_COLOR_YELLOW;
+                       (*dscp_table)[i++] = RTE_COLOR_YELLOW;
                else if (strcmp(token, "R") == 0 ||
                        strcmp(token, "r") == 0)
-                       *dscp_table[i++] = RTE_COLOR_RED;
+                       (*dscp_table)[i++] = RTE_COLOR_RED;
                else {
                        free(*dscp_table);
                        return -1;
@@ -192,19 +201,19 @@ struct cmd_show_port_meter_cap_result {
        uint16_t port_id;
 };
 
-cmdline_parse_token_string_t cmd_show_port_meter_cap_show =
+static cmdline_parse_token_string_t cmd_show_port_meter_cap_show =
        TOKEN_STRING_INITIALIZER(
                struct cmd_show_port_meter_cap_result, show, "show");
-cmdline_parse_token_string_t cmd_show_port_meter_cap_port =
+static cmdline_parse_token_string_t cmd_show_port_meter_cap_port =
        TOKEN_STRING_INITIALIZER(
                struct cmd_show_port_meter_cap_result, port, "port");
-cmdline_parse_token_string_t cmd_show_port_meter_cap_meter =
+static cmdline_parse_token_string_t cmd_show_port_meter_cap_meter =
        TOKEN_STRING_INITIALIZER(
                struct cmd_show_port_meter_cap_result, meter, "meter");
-cmdline_parse_token_string_t cmd_show_port_meter_cap_cap =
+static cmdline_parse_token_string_t cmd_show_port_meter_cap_cap =
        TOKEN_STRING_INITIALIZER(
                struct cmd_show_port_meter_cap_result, cap, "cap");
-cmdline_parse_token_num_t cmd_show_port_meter_cap_port_id =
+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);
 
@@ -299,46 +308,46 @@ struct cmd_add_port_meter_profile_srtcm_result {
        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, RTE_UINT16);
-cmdline_parse_token_num_t cmd_add_port_meter_profile_srtcm_profile_id =
+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, RTE_UINT32);
-cmdline_parse_token_num_t cmd_add_port_meter_profile_srtcm_cir =
+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, RTE_UINT64);
-cmdline_parse_token_num_t cmd_add_port_meter_profile_srtcm_cbs =
+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, RTE_UINT64);
-cmdline_parse_token_num_t cmd_add_port_meter_profile_srtcm_ebs =
+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, RTE_UINT64);
-cmdline_parse_token_num_t cmd_add_port_meter_profile_srtcm_packet_mode =
+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);
@@ -408,50 +417,50 @@ struct cmd_add_port_meter_profile_trtcm_result {
        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, RTE_UINT16);
-cmdline_parse_token_num_t cmd_add_port_meter_profile_trtcm_profile_id =
+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, RTE_UINT32);
-cmdline_parse_token_num_t cmd_add_port_meter_profile_trtcm_cir =
+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, RTE_UINT64);
-cmdline_parse_token_num_t cmd_add_port_meter_profile_trtcm_pir =
+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, RTE_UINT64);
-cmdline_parse_token_num_t cmd_add_port_meter_profile_trtcm_cbs =
+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, RTE_UINT64);
-cmdline_parse_token_num_t cmd_add_port_meter_profile_trtcm_pbs =
+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, RTE_UINT64);
-cmdline_parse_token_num_t cmd_add_port_meter_profile_trtcm_packet_mode =
+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);
@@ -523,52 +532,52 @@ struct cmd_add_port_meter_profile_trtcm_rfc4115_result {
        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, RTE_UINT16);
-cmdline_parse_token_num_t cmd_add_port_meter_profile_trtcm_rfc4115_profile_id =
+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, RTE_UINT32);
-cmdline_parse_token_num_t cmd_add_port_meter_profile_trtcm_rfc4115_cir =
+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, RTE_UINT64);
-cmdline_parse_token_num_t cmd_add_port_meter_profile_trtcm_rfc4115_eir =
+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, RTE_UINT64);
-cmdline_parse_token_num_t cmd_add_port_meter_profile_trtcm_rfc4115_cbs =
+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, RTE_UINT64);
-cmdline_parse_token_num_t cmd_add_port_meter_profile_trtcm_rfc4115_ebs =
+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, RTE_UINT64);
-cmdline_parse_token_num_t
+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,
@@ -637,26 +646,26 @@ 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, RTE_UINT16);
-cmdline_parse_token_num_t cmd_del_port_meter_profile_profile_id =
+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, RTE_UINT32);
@@ -705,6 +714,7 @@ struct cmd_create_port_meter_result {
        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;
@@ -714,43 +724,37 @@ struct cmd_create_port_meter_result {
        cmdline_multi_string_t meter_input_color;
 };
 
-cmdline_parse_token_string_t cmd_create_port_meter_create =
+static cmdline_parse_token_string_t cmd_create_port_meter_create =
        TOKEN_STRING_INITIALIZER(
                struct cmd_create_port_meter_result, create, "create");
-cmdline_parse_token_string_t cmd_create_port_meter_port =
+static cmdline_parse_token_string_t cmd_create_port_meter_port =
        TOKEN_STRING_INITIALIZER(
                struct cmd_create_port_meter_result, port, "port");
-cmdline_parse_token_string_t cmd_create_port_meter_meter =
+static cmdline_parse_token_string_t cmd_create_port_meter_meter =
        TOKEN_STRING_INITIALIZER(
                struct cmd_create_port_meter_result, meter, "meter");
-cmdline_parse_token_num_t cmd_create_port_meter_port_id =
+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);
-cmdline_parse_token_num_t cmd_create_port_meter_mtr_id =
+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);
-cmdline_parse_token_num_t cmd_create_port_meter_profile_id =
+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);
-cmdline_parse_token_string_t cmd_create_port_meter_meter_enable =
+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");
-cmdline_parse_token_string_t cmd_create_port_meter_g_action =
-       TOKEN_STRING_INITIALIZER(struct cmd_create_port_meter_result,
-               g_action, "R#Y#G#D#r#y#g#d");
-cmdline_parse_token_string_t cmd_create_port_meter_y_action =
-       TOKEN_STRING_INITIALIZER(struct cmd_create_port_meter_result,
-               y_action, "R#Y#G#D#r#y#g#d");
-cmdline_parse_token_string_t cmd_create_port_meter_r_action =
-       TOKEN_STRING_INITIALIZER(struct cmd_create_port_meter_result,
-               r_action, "R#Y#G#D#r#y#g#d");
-cmdline_parse_token_num_t cmd_create_port_meter_statistics_mask =
+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);
-cmdline_parse_token_num_t cmd_create_port_meter_shared =
+static cmdline_parse_token_num_t cmd_create_port_meter_shared =
        TOKEN_NUM_INITIALIZER(struct cmd_create_port_meter_result,
                shared, RTE_UINT32);
-cmdline_parse_token_string_t cmd_create_port_meter_input_color =
+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);
 
@@ -775,11 +779,12 @@ static void cmd_create_port_meter_parsed(void *parsed_result,
        /* Meter params */
        memset(&params, 0, sizeof(struct rte_mtr_params));
        params.meter_profile_id = res->profile_id;
-
+       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) {
-               printf(" Meter input color params string parse error\n");
+               fprintf(stderr,
+                       " Meter input color params string parse error\n");
                return;
        }
 
@@ -790,7 +795,6 @@ static void cmd_create_port_meter_parsed(void *parsed_result,
                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);
@@ -804,9 +808,8 @@ static void cmd_create_port_meter_parsed(void *parsed_result,
 cmdline_parse_inst_t cmd_create_port_meter = {
        .f = cmd_create_port_meter_parsed,
        .data = NULL,
-       .help_str = "create port meter <port_id> <mtr_id> <profile_id> <meter_enable>(yes|no) "
-               "<g_action>(R|Y|G|D) <y_action>(R|Y|G|D) <r_action>(R|Y|G|D) "
-               "<stats_mask> <shared> <use_pre_meter_color> "
+       .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_create_port_meter_create,
@@ -815,10 +818,8 @@ cmdline_parse_inst_t cmd_create_port_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_g_action,
-               (void *)&cmd_create_port_meter_y_action,
-               (void *)&cmd_create_port_meter_r_action,
                (void *)&cmd_create_port_meter_statistics_mask,
                (void *)&cmd_create_port_meter_shared,
                (void *)&cmd_create_port_meter_input_color,
@@ -835,19 +836,19 @@ struct cmd_enable_port_meter_result {
        uint32_t mtr_id;
 };
 
-cmdline_parse_token_string_t cmd_enable_port_meter_enable =
+static cmdline_parse_token_string_t cmd_enable_port_meter_enable =
        TOKEN_STRING_INITIALIZER(
                struct cmd_enable_port_meter_result, enable, "enable");
-cmdline_parse_token_string_t cmd_enable_port_meter_port =
+static cmdline_parse_token_string_t cmd_enable_port_meter_port =
        TOKEN_STRING_INITIALIZER(
                struct cmd_enable_port_meter_result, port, "port");
-cmdline_parse_token_string_t cmd_enable_port_meter_meter =
+static cmdline_parse_token_string_t cmd_enable_port_meter_meter =
        TOKEN_STRING_INITIALIZER(
                struct cmd_enable_port_meter_result, meter, "meter");
-cmdline_parse_token_num_t cmd_enable_port_meter_port_id =
+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);
-cmdline_parse_token_num_t cmd_enable_port_meter_mtr_id =
+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);
 
@@ -896,19 +897,19 @@ struct cmd_disable_port_meter_result {
        uint32_t mtr_id;
 };
 
-cmdline_parse_token_string_t cmd_disable_port_meter_disable =
+static cmdline_parse_token_string_t cmd_disable_port_meter_disable =
        TOKEN_STRING_INITIALIZER(
                struct cmd_disable_port_meter_result, disable, "disable");
-cmdline_parse_token_string_t cmd_disable_port_meter_port =
+static cmdline_parse_token_string_t cmd_disable_port_meter_port =
        TOKEN_STRING_INITIALIZER(
                struct cmd_disable_port_meter_result, port, "port");
-cmdline_parse_token_string_t cmd_disable_port_meter_meter =
+static cmdline_parse_token_string_t cmd_disable_port_meter_meter =
        TOKEN_STRING_INITIALIZER(
                struct cmd_disable_port_meter_result, meter, "meter");
-cmdline_parse_token_num_t cmd_disable_port_meter_port_id =
+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);
-cmdline_parse_token_num_t cmd_disable_port_meter_mtr_id =
+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);
 
@@ -948,6 +949,71 @@ cmdline_parse_inst_t cmd_disable_port_meter = {
        },
 };
 
+/* *** 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,
+       },
+};
+
 /* *** Delete Port Meter Object *** */
 struct cmd_del_port_meter_result {
        cmdline_fixed_string_t del;
@@ -957,19 +1023,19 @@ 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, RTE_UINT16);
-cmdline_parse_token_num_t cmd_del_port_meter_mtr_id =
+static cmdline_parse_token_num_t cmd_del_port_meter_mtr_id =
        TOKEN_NUM_INITIALIZER(
                struct cmd_del_port_meter_result, mtr_id, RTE_UINT32);
 
@@ -1020,27 +1086,27 @@ 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,
                RTE_UINT16);
-cmdline_parse_token_num_t cmd_set_port_meter_profile_mtr_id =
+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,
                RTE_UINT32);
-cmdline_parse_token_num_t cmd_set_port_meter_profile_profile_id =
+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,
                RTE_UINT32);
@@ -1094,20 +1160,20 @@ struct cmd_set_port_meter_dscp_table_result {
        cmdline_multi_string_t token_string;
 };
 
-cmdline_parse_token_string_t cmd_set_port_meter_dscp_table_set =
+static cmdline_parse_token_string_t cmd_set_port_meter_dscp_table_set =
        TOKEN_STRING_INITIALIZER(
                struct cmd_set_port_meter_dscp_table_result, set, "set");
-cmdline_parse_token_string_t cmd_set_port_meter_dscp_table_port =
+static cmdline_parse_token_string_t cmd_set_port_meter_dscp_table_port =
        TOKEN_STRING_INITIALIZER(
                struct cmd_set_port_meter_dscp_table_result, port, "port");
-cmdline_parse_token_string_t cmd_set_port_meter_dscp_table_meter =
+static cmdline_parse_token_string_t cmd_set_port_meter_dscp_table_meter =
        TOKEN_STRING_INITIALIZER(
                struct cmd_set_port_meter_dscp_table_result, meter, "meter");
-cmdline_parse_token_string_t cmd_set_port_meter_dscp_table_dscp_table =
+static cmdline_parse_token_string_t cmd_set_port_meter_dscp_table_dscp_table =
        TOKEN_STRING_INITIALIZER(
                struct cmd_set_port_meter_dscp_table_result,
                dscp_table, "dscp table");
-cmdline_parse_token_string_t cmd_set_port_meter_dscp_table_token_string =
+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);
 
@@ -1126,7 +1192,7 @@ static void cmd_set_port_meter_dscp_table_parsed(void *parsed_result,
        /* Parse string */
        ret = parse_multi_token_string(t_str, &port_id, &mtr_id, &dscp_table);
        if (ret) {
-               printf(" Multi token string parse error\n");
+               fprintf(stderr, " Multi token string parse error\n");
                return;
        }
 
@@ -1170,30 +1236,30 @@ 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,
                RTE_UINT16);
-cmdline_parse_token_num_t cmd_set_port_meter_stats_mask_mtr_id =
+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,
                RTE_UINT32);
-cmdline_parse_token_num_t cmd_set_port_meter_stats_mask_stats_mask =
+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,
                RTE_UINT64);
@@ -1247,25 +1313,25 @@ struct cmd_show_port_meter_stats_result {
        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 =
+static cmdline_parse_token_num_t cmd_show_port_meter_stats_port_id =
        TOKEN_NUM_INITIALIZER(
                struct cmd_show_port_meter_stats_result, port_id, RTE_UINT16);
-cmdline_parse_token_num_t cmd_show_port_meter_stats_mtr_id =
+static cmdline_parse_token_num_t cmd_show_port_meter_stats_mtr_id =
        TOKEN_NUM_INITIALIZER(
                struct cmd_show_port_meter_stats_result, mtr_id, RTE_UINT32);
-cmdline_parse_token_string_t cmd_show_port_meter_stats_clear =
+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");