From: "Singh, Jasvinder" <jasvinder.singh@intel.com>
To: "Wu, Jingjing" <jingjing.wu@intel.com>, "dev@dpdk.org" <dev@dpdk.org>
Cc: "Dumitrescu, Cristian" <cristian.dumitrescu@intel.com>,
"Pei, Yulong" <yulong.pei@intel.com>
Subject: Re: [dpdk-dev] [PATCH v9 3/3] app/test-pmd: add CLI for TM nodes and hierarchy commit
Date: Mon, 16 Oct 2017 18:18:02 +0000 [thread overview]
Message-ID: <54CBAA185211B4429112C315DA58FF6D3326F551@IRSMSX103.ger.corp.intel.com> (raw)
In-Reply-To: <9BB6961774997848B5B42BEC655768F810E96EBB@SHSMSX103.ccr.corp.intel.com>
<snip>
> > diff --git a/app/test-pmd/cmdline_tm.c b/app/test-pmd/cmdline_tm.c
> > index 38048ae..002209c 100644
> > --- a/app/test-pmd/cmdline_tm.c
> > +++ b/app/test-pmd/cmdline_tm.c
> > @@ -42,6 +42,17 @@
> > #include "testpmd.h"
> > #include "cmdline_tm.h"
> >
> > +#define PARSE_DELIMITER " \f\n\r\t\v"
> > +#define MAX_NUM_SHARED_SHAPERS 256
> > +
> > +#define skip_white_spaces(pos) \
> > +({ \
> > + __typeof__(pos) _p = (pos); \
> > + for ( ; isspace(*_p); _p++) \
> > + ; \
> > + _p; \
> > +})
> > +
> > /** Display TM Error Message */
> > static void
> > print_err_msg(struct rte_tm_error *error) @@ -118,6 +129,100 @@
> > print_err_msg(struct rte_tm_error *error)
> > error->type);
> > }
> >
> > +static int
> > +read_uint64(uint64_t *value, const char *p) {
> > + char *next;
> > + uint64_t val;
> > +
> > + p = skip_white_spaces(p);
> > + if (!isdigit(*p))
> > + return -EINVAL;
> > +
> > + val = strtoul(p, &next, 10);
> > + if (p == next)
> > + return -EINVAL;
> > +
> > + p = next;
> > + switch (*p) {
> > + case 'T':
> > + val *= 1024ULL;
> > + /* fall through */
> > + case 'G':
> > + val *= 1024ULL;
> > + /* fall through */
> > + case 'M':
> > + val *= 1024ULL;
> > + /* fall through */
> > + case 'k':
> > + case 'K':
> > + val *= 1024ULL;
> > + p++;
> > + break;
> > + }
> > +
> > + p = skip_white_spaces(p);
> > + if (*p != '\0')
> > + return -EINVAL;
> > +
> > + *value = val;
> > + return 0;
> > +}
> > +
> > +static int
> > +read_uint32(uint32_t *value, const char *p) {
> > + uint64_t val = 0;
> > + int ret = read_uint64(&val, p);
> > +
> > + if (ret < 0)
> > + return ret;
> > +
> > + if (val > UINT32_MAX)
> > + return -ERANGE;
> > +
> > + *value = val;
> > + return 0;
> > +}
> > +
> > +static int
> > +parse_multi_ss_id_str(char *s_str, uint32_t *n_ssp, uint32_t
> > +shaper_id[]) {
> > + uint32_t n_shared_shapers = 0, i = 0;
> > + char *token;
> > +
> > + /* First token: num of shared shapers */
> > + token = strtok_r(s_str, PARSE_DELIMITER, &s_str);
> > + if (token == NULL)
> > + return -1;
> > +
> > + if (read_uint32(&n_shared_shapers, token))
> > + return -1;
> > +
> > + /* Check: num of shared shaper */
> > + if (n_shared_shapers >= MAX_NUM_SHARED_SHAPERS) {
> > + printf(" Number of shared shapers exceed the max
> (error)\n");
> > + return -1;
> > + }
> > +
> > + /* Parse shared shaper ids */
> > + while (1) {
> > + token = strtok_r(s_str, PARSE_DELIMITER, &s_str);
> > + if ((token != NULL && n_shared_shapers == 0) ||
> > + (token == NULL && i < n_shared_shapers))
> > + return -1;
> > +
> > + if (token == NULL)
> > + break;
> > +
> > + if (read_uint32(&shaper_id[i], token))
> > + return -1;
> > + i++;
> > + }
> > + *n_ssp = n_shared_shapers;
> > +
> > + return 0;
> > +}
>
> Thanks for the change, can those be done by rte_strsplit?
On CLI, user can inadvertently press tab, etc., instead of space character while specifying the cli arguments. The above approach is robust to such incidents by allowing range of characters to be used as delim. On the other hand, rte_strsplit() allows only one character to be used as delim and restricts the user to specify only that character on CLI.
> > /* *** Port TM Capability *** */
> > struct cmd_show_port_tm_cap_result {
> > cmdline_fixed_string_t show;
> > @@ -1410,3 +1515,549 @@ cmdline_parse_inst_t
> > cmd_set_port_tm_node_shaper_profile = {
> > NULL,
> > },
> > };
> > +
> > +/* *** Add Port TM nonleaf node *** */ struct
> > +cmd_add_port_tm_nonleaf_node_result {
> > + cmdline_fixed_string_t add;
> > + cmdline_fixed_string_t port;
> > + cmdline_fixed_string_t tm;
> > + cmdline_fixed_string_t nonleaf;
> > + cmdline_fixed_string_t node;
> > + uint16_t port_id;
> > + uint32_t node_id;
> > + int32_t parent_node_id;
> > + uint32_t priority;
> > + uint32_t weight;
> > + uint32_t level_id;
> > + uint32_t shaper_profile_id;
> > + uint32_t n_sp_priorities;
> > + uint64_t stats_mask;
> > + cmdline_multi_string_t multi_shared_shaper_id; };
> > +
> > +cmdline_parse_token_string_t cmd_add_port_tm_nonleaf_node_add =
> > + TOKEN_STRING_INITIALIZER(
> > + struct cmd_add_port_tm_nonleaf_node_result, add, "add");
> > +cmdline_parse_token_string_t cmd_add_port_tm_nonleaf_node_port =
> > + TOKEN_STRING_INITIALIZER(
> > + struct cmd_add_port_tm_nonleaf_node_result, port, "port");
> > +cmdline_parse_token_string_t cmd_add_port_tm_nonleaf_node_tm =
> > + TOKEN_STRING_INITIALIZER(
> > + struct cmd_add_port_tm_nonleaf_node_result, tm, "tm");
> > +cmdline_parse_token_string_t cmd_add_port_tm_nonleaf_node_nonleaf
> =
> > + TOKEN_STRING_INITIALIZER(
> > + struct cmd_add_port_tm_nonleaf_node_result, nonleaf,
> > "nonleaf");
> > +cmdline_parse_token_string_t cmd_add_port_tm_nonleaf_node_node =
> > + TOKEN_STRING_INITIALIZER(
> > + struct cmd_add_port_tm_nonleaf_node_result, node,
> "node");
> > +cmdline_parse_token_num_t cmd_add_port_tm_nonleaf_node_port_id =
> > + TOKEN_NUM_INITIALIZER(
> > + struct cmd_add_port_tm_nonleaf_node_result,
> > + port_id, UINT16);
> > +cmdline_parse_token_num_t cmd_add_port_tm_nonleaf_node_node_id
> =
> > + TOKEN_NUM_INITIALIZER(struct
> > cmd_add_port_tm_nonleaf_node_result,
> > + node_id, UINT32);
> > +cmdline_parse_token_num_t
> > cmd_add_port_tm_nonleaf_node_parent_node_id =
> > + TOKEN_NUM_INITIALIZER(struct
> > cmd_add_port_tm_nonleaf_node_result,
> > + parent_node_id, INT32);
> > +cmdline_parse_token_num_t cmd_add_port_tm_nonleaf_node_priority =
> > + TOKEN_NUM_INITIALIZER(struct
> > cmd_add_port_tm_nonleaf_node_result,
> > + priority, UINT32);
> > +cmdline_parse_token_num_t cmd_add_port_tm_nonleaf_node_weight =
> > + TOKEN_NUM_INITIALIZER(struct
> > cmd_add_port_tm_nonleaf_node_result,
> > + weight, UINT32);
> > +cmdline_parse_token_num_t cmd_add_port_tm_nonleaf_node_level_id
> =
> > + TOKEN_NUM_INITIALIZER(struct
> > cmd_add_port_tm_nonleaf_node_result,
> > + level_id, UINT32);
> > +cmdline_parse_token_num_t
> > cmd_add_port_tm_nonleaf_node_shaper_profile_id =
> > + TOKEN_NUM_INITIALIZER(struct
> > cmd_add_port_tm_nonleaf_node_result,
> > + shaper_profile_id, UINT32);
> > +cmdline_parse_token_num_t
> > cmd_add_port_tm_nonleaf_node_n_sp_priorities =
> > + TOKEN_NUM_INITIALIZER(struct
> > cmd_add_port_tm_nonleaf_node_result,
> > + n_sp_priorities, UINT32);
> > +cmdline_parse_token_num_t
> cmd_add_port_tm_nonleaf_node_stats_mask =
> > + TOKEN_NUM_INITIALIZER(struct
> > cmd_add_port_tm_nonleaf_node_result,
> > + stats_mask, UINT64);
> > +cmdline_parse_token_string_t
> > + cmd_add_port_tm_nonleaf_node_multi_shared_shaper_id =
> > + TOKEN_STRING_INITIALIZER(struct
> > cmd_add_port_tm_nonleaf_node_result,
> > + multi_shared_shaper_id, TOKEN_STRING_MULTI);
> > +
> > +static void cmd_add_port_tm_nonleaf_node_parsed(void *parsed_result,
> > + __attribute__((unused)) struct cmdline *cl,
> > + __attribute__((unused)) void *data)
> > +{
> > + struct cmd_add_port_tm_nonleaf_node_result *res = parsed_result;
> > + struct rte_tm_error error;
> > + struct rte_tm_node_params np;
> > + uint32_t *shared_shaper_id;
> > + uint32_t parent_node_id, n_shared_shapers = 0;
> > + char *s_str = res->multi_shared_shaper_id;
> > + portid_t port_id = res->port_id;
> > + int ret;
> > +
> > + if (port_id_is_invalid(port_id, ENABLED_WARN))
> > + return;
> > +
> > + /* Port status */
> > + if (port_is_started(port_id)) {
> > + printf(" Port %u not stopped (error)\n", port_id);
> > + return;
> > + }
> > +
> > + memset(&np, 0, sizeof(struct rte_tm_node_params));
> > +
> > + /* Node parameters */
> > + if (res->parent_node_id < 0)
> > + parent_node_id = UINT32_MAX;
> > + else
> > + parent_node_id = res->parent_node_id;
> > +
> > + shared_shaper_id = (uint32_t
> *)malloc(MAX_NUM_SHARED_SHAPERS
> > *
> > + sizeof(uint32_t));
> > + /* Parse multi shared shaper id string */
> > + ret = parse_multi_ss_id_str(s_str, &n_shared_shapers,
> > shared_shaper_id);
> > + if (ret) {
> > + printf(" Shared shapers params string parse error\n");
> > + return;
>
> You forgot to free shared_shaper_id.
>
> > + }
> > +
> > + np.shaper_profile_id = res->shaper_profile_id;
> > + np.n_shared_shapers = n_shared_shapers;
> > + if (np.n_shared_shapers)
> > + np.shared_shaper_id = &shared_shaper_id[0];
> > + else
> > + np.shared_shaper_id = NULL;
> > +
> > + np.nonleaf.n_sp_priorities = res->n_sp_priorities;
> > + np.stats_mask = res->stats_mask;
> > + np.nonleaf.wfq_weight_mode = NULL;
> > +
> > + ret = rte_tm_node_add(port_id, res->node_id, parent_node_id,
> > + res->priority, res->weight, res->level_id,
> > + &np, &error);
> > + if (ret != 0) {
> > + print_err_msg(&error);
> > + return;
> > + }
>
> You forgot to free shared_shaper_id.
>
> > +}
> > +
> > +cmdline_parse_inst_t cmd_add_port_tm_nonleaf_node = {
> > + .f = cmd_add_port_tm_nonleaf_node_parsed,
> > + .data = NULL,
> > + .help_str = "Add port tm nonleaf node",
> > + .tokens = {
> > + (void *)&cmd_add_port_tm_nonleaf_node_add,
> > + (void *)&cmd_add_port_tm_nonleaf_node_port,
> > + (void *)&cmd_add_port_tm_nonleaf_node_tm,
> > + (void *)&cmd_add_port_tm_nonleaf_node_nonleaf,
> > + (void *)&cmd_add_port_tm_nonleaf_node_node,
> > + (void *)&cmd_add_port_tm_nonleaf_node_port_id,
> > + (void *)&cmd_add_port_tm_nonleaf_node_node_id,
> > + (void *)&cmd_add_port_tm_nonleaf_node_parent_node_id,
> > + (void *)&cmd_add_port_tm_nonleaf_node_priority,
> > + (void *)&cmd_add_port_tm_nonleaf_node_weight,
> > + (void *)&cmd_add_port_tm_nonleaf_node_level_id,
> > + (void
> *)&cmd_add_port_tm_nonleaf_node_shaper_profile_id,
> > + (void *)&cmd_add_port_tm_nonleaf_node_n_sp_priorities,
> > + (void *)&cmd_add_port_tm_nonleaf_node_stats_mask,
> > + (void
> > *)&cmd_add_port_tm_nonleaf_node_multi_shared_shaper_id,
> > + NULL,
> > + },
> > +};
> > +
> > +/* *** Add Port TM leaf node *** */
> > +struct cmd_add_port_tm_leaf_node_result {
> > + cmdline_fixed_string_t add;
> > + cmdline_fixed_string_t port;
> > + cmdline_fixed_string_t tm;
> > + cmdline_fixed_string_t leaf;
> > + cmdline_fixed_string_t node;
> > + uint16_t port_id;
> > + uint32_t node_id;
> > + int32_t parent_node_id;
> > + uint32_t priority;
> > + uint32_t weight;
> > + uint32_t level_id;
> > + uint32_t shaper_profile_id;
> > + uint32_t cman_mode;
> > + uint32_t wred_profile_id;
> > + uint64_t stats_mask;
> > + cmdline_multi_string_t multi_shared_shaper_id; };
> > +
> > +cmdline_parse_token_string_t cmd_add_port_tm_leaf_node_add =
> > + TOKEN_STRING_INITIALIZER(
> > + struct cmd_add_port_tm_leaf_node_result, add, "add");
> > +cmdline_parse_token_string_t cmd_add_port_tm_leaf_node_port =
> > + TOKEN_STRING_INITIALIZER(
> > + struct cmd_add_port_tm_leaf_node_result, port, "port");
> > +cmdline_parse_token_string_t cmd_add_port_tm_leaf_node_tm =
> > + TOKEN_STRING_INITIALIZER(
> > + struct cmd_add_port_tm_leaf_node_result, tm, "tm");
> > +cmdline_parse_token_string_t cmd_add_port_tm_leaf_node_nonleaf =
> > + TOKEN_STRING_INITIALIZER(
> > + struct cmd_add_port_tm_leaf_node_result, leaf, "leaf");
> > +cmdline_parse_token_string_t cmd_add_port_tm_leaf_node_node =
> > + TOKEN_STRING_INITIALIZER(
> > + struct cmd_add_port_tm_leaf_node_result, node, "node");
> > +cmdline_parse_token_num_t cmd_add_port_tm_leaf_node_port_id =
> > + TOKEN_NUM_INITIALIZER(struct
> cmd_add_port_tm_leaf_node_result,
> > + port_id, UINT16);
> > +cmdline_parse_token_num_t cmd_add_port_tm_leaf_node_node_id =
> > + TOKEN_NUM_INITIALIZER(struct
> cmd_add_port_tm_leaf_node_result,
> > + node_id, UINT32);
> > +cmdline_parse_token_num_t
> cmd_add_port_tm_leaf_node_parent_node_id =
> > + TOKEN_NUM_INITIALIZER(struct
> cmd_add_port_tm_leaf_node_result,
> > + parent_node_id, INT32);
> > +cmdline_parse_token_num_t cmd_add_port_tm_leaf_node_priority =
> > + TOKEN_NUM_INITIALIZER(struct
> cmd_add_port_tm_leaf_node_result,
> > + priority, UINT32);
> > +cmdline_parse_token_num_t cmd_add_port_tm_leaf_node_weight =
> > + TOKEN_NUM_INITIALIZER(struct
> cmd_add_port_tm_leaf_node_result,
> > + weight, UINT32);
> > +cmdline_parse_token_num_t cmd_add_port_tm_leaf_node_level_id =
> > + TOKEN_NUM_INITIALIZER(struct
> cmd_add_port_tm_leaf_node_result,
> > + level_id, UINT32);
> > +cmdline_parse_token_num_t
> cmd_add_port_tm_leaf_node_shaper_profile_id
> > =
> > + TOKEN_NUM_INITIALIZER(struct
> cmd_add_port_tm_leaf_node_result,
> > + shaper_profile_id, UINT32);
> > +cmdline_parse_token_num_t cmd_add_port_tm_leaf_node_cman_mode
> =
> > + TOKEN_NUM_INITIALIZER(struct
> cmd_add_port_tm_leaf_node_result,
> > + cman_mode, UINT32);
> > +cmdline_parse_token_num_t
> cmd_add_port_tm_leaf_node_wred_profile_id
> > =
> > + TOKEN_NUM_INITIALIZER(struct
> cmd_add_port_tm_leaf_node_result,
> > + wred_profile_id, UINT32);
> > +cmdline_parse_token_num_t cmd_add_port_tm_leaf_node_stats_mask =
> > + TOKEN_NUM_INITIALIZER(struct
> cmd_add_port_tm_leaf_node_result,
> > + stats_mask, UINT64);
> > +cmdline_parse_token_string_t
> > + cmd_add_port_tm_leaf_node_multi_shared_shaper_id =
> > + TOKEN_STRING_INITIALIZER(struct
> > cmd_add_port_tm_leaf_node_result,
> > + multi_shared_shaper_id, TOKEN_STRING_MULTI);
> > +
> > +static void cmd_add_port_tm_leaf_node_parsed(void *parsed_result,
> > + __attribute__((unused)) struct cmdline *cl,
> > + __attribute__((unused)) void *data)
> > +{
> > + struct cmd_add_port_tm_leaf_node_result *res = parsed_result;
> > + struct rte_tm_error error;
> > + struct rte_tm_node_params np;
> > + uint32_t *shared_shaper_id;
> > + uint32_t parent_node_id, n_shared_shapers = 0;
> > + portid_t port_id = res->port_id;
> > + char *s_str = res->multi_shared_shaper_id;
> > + int ret;
> > +
> > + if (port_id_is_invalid(port_id, ENABLED_WARN))
> > + return;
> > +
> > + /* Port status */
> > + if (port_is_started(port_id)) {
> > + printf(" Port %u not stopped (error)\n", port_id);
> > + return;
> > + }
> > +
> > + memset(&np, 0, sizeof(struct rte_tm_node_params));
> > +
> > + /* Node parameters */
> > + if (res->parent_node_id < 0)
> > + parent_node_id = UINT32_MAX;
> > + else
> > + parent_node_id = res->parent_node_id;
> > +
> > + shared_shaper_id = (uint32_t
> *)malloc(MAX_NUM_SHARED_SHAPERS
> > *
> > + sizeof(uint32_t));
> > + /* Parse multi shared shaper id string */
> > + ret = parse_multi_ss_id_str(s_str, &n_shared_shapers,
> > shared_shaper_id);
> > + if (ret) {
> > + printf(" Shared shapers params string parse error\n");
> The same, free the memory you malloced.
Oops, will fix this in next version. Thanks.
next prev parent reply other threads:[~2017-10-16 18:18 UTC|newest]
Thread overview: 49+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-08-22 17:02 [dpdk-dev] [PATCH 1/2] app/testpmd: add traffic management forwarding mode Jasvinder Singh
2017-08-22 17:02 ` [dpdk-dev] [PATCH 2/2] app/testpmd: add CLI for tm mode Jasvinder Singh
2017-09-14 11:53 ` [dpdk-dev] [PATCH v2 1/2] app/testpmd: add traffic management forwarding mode Jasvinder Singh
2017-09-14 11:53 ` [dpdk-dev] [PATCH v2 2/2] app/testpmd: add CLI for tm mode Jasvinder Singh
2017-09-18 14:03 ` De Lara Guarch, Pablo
2017-09-19 15:06 ` Singh, Jasvinder
2017-09-18 13:54 ` [dpdk-dev] [PATCH v2 1/2] app/testpmd: add traffic management forwarding mode De Lara Guarch, Pablo
2017-09-19 15:04 ` Singh, Jasvinder
2017-09-20 9:56 ` [dpdk-dev] [PATCH v3 1/5] " Jasvinder Singh
2017-09-20 9:56 ` [dpdk-dev] [PATCH v3 2/5] app/test-pmd: add CLI for TM capability and stats Jasvinder Singh
2017-09-20 9:56 ` [dpdk-dev] [PATCH v3 3/5] app/test-pmd: add CLI for shaper and wred profiles Jasvinder Singh
2017-09-20 9:56 ` [dpdk-dev] [PATCH v3 4/5] app/test-pmd: add CLI for TM nodes and hierarchy commit Jasvinder Singh
2017-09-20 9:56 ` [dpdk-dev] [PATCH v3 5/5] app/test-pmd: add CLI for TM packet classification Jasvinder Singh
2017-09-22 16:35 ` [dpdk-dev] [PATCH v4 1/5] app/testpmd: add traffic management forwarding mode Jasvinder Singh
2017-09-22 16:35 ` [dpdk-dev] [PATCH v4 2/5] app/test-pmd: add CLI for TM capability and stats Jasvinder Singh
2017-09-29 12:38 ` [dpdk-dev] [PATCH v5 1/3] " Jasvinder Singh
2017-09-29 12:38 ` [dpdk-dev] [PATCH v5 2/3] app/test-pmd: add CLI for shaper and wred profiles Jasvinder Singh
2017-09-29 12:38 ` [dpdk-dev] [PATCH v5 3/3] app/test-pmd: add CLI for TM nodes and hierarchy commit Jasvinder Singh
2017-10-09 19:07 ` [dpdk-dev] [PATCH v6 1/3] app/test-pmd: add CLI for TM capability and stats Jasvinder Singh
2017-10-09 19:07 ` [dpdk-dev] [PATCH v6 2/3] app/test-pmd: add CLI for shaper and wred profiles Jasvinder Singh
2017-10-09 19:07 ` [dpdk-dev] [PATCH v6 3/3] app/test-pmd: add CLI for TM nodes and hierarchy commit Jasvinder Singh
2017-10-11 8:40 ` Pei, Yulong
2017-10-11 8:51 ` Singh, Jasvinder
2017-10-11 9:26 ` [dpdk-dev] [PATCH v7 1/3] app/test-pmd: add CLI for TM capability and stats Jasvinder Singh
2017-10-11 9:26 ` [dpdk-dev] [PATCH v7 2/3] app/test-pmd: add CLI for shaper and wred profiles Jasvinder Singh
2017-10-13 2:33 ` Wu, Jingjing
2017-10-13 8:58 ` Singh, Jasvinder
2017-10-11 9:26 ` [dpdk-dev] [PATCH v7 3/3] app/test-pmd: add CLI for TM nodes and hierarchy commit Jasvinder Singh
2017-10-13 2:58 ` Wu, Jingjing
2017-10-13 10:50 ` Singh, Jasvinder
2017-10-12 20:55 ` [dpdk-dev] [PATCH v7 1/3] app/test-pmd: add CLI for TM capability and stats Thomas Monjalon
2017-10-13 2:34 ` Wu, Jingjing
2017-10-13 6:22 ` Pei, Yulong
2017-10-13 16:59 ` [dpdk-dev] [PATCH v8 " Jasvinder Singh
2017-10-13 16:59 ` [dpdk-dev] [PATCH v8 2/3] app/test-pmd: add CLI for shaper and wred profiles Jasvinder Singh
2017-10-13 17:00 ` [dpdk-dev] [PATCH v8 3/3] app/test-pmd: add CLI for TM nodes and hierarchy commit Jasvinder Singh
2017-10-14 10:20 ` [dpdk-dev] [PATCH v9 1/3] app/test-pmd: add CLI for TM capability and stats Jasvinder Singh
2017-10-14 10:20 ` [dpdk-dev] [PATCH v9 2/3] app/test-pmd: add CLI for shaper and wred profiles Jasvinder Singh
2017-10-14 10:20 ` [dpdk-dev] [PATCH v9 3/3] app/test-pmd: add CLI for TM nodes and hierarchy commit Jasvinder Singh
2017-10-16 10:18 ` Wu, Jingjing
2017-10-16 18:18 ` Singh, Jasvinder [this message]
2017-10-16 18:55 ` [dpdk-dev] [PATCH v10 1/3] app/test-pmd: add CLI for TM capability and stats Jasvinder Singh
2017-10-16 18:55 ` [dpdk-dev] [PATCH v10 2/3] app/test-pmd: add CLI for shaper and wred profiles Jasvinder Singh
2017-10-16 18:55 ` [dpdk-dev] [PATCH v10 3/3] app/test-pmd: add CLI for TM nodes and hierarchy commit Jasvinder Singh
2017-10-17 8:44 ` [dpdk-dev] [PATCH v10 1/3] app/test-pmd: add CLI for TM capability and stats Wu, Jingjing
2017-10-20 14:08 ` Dumitrescu, Cristian
2017-09-22 16:35 ` [dpdk-dev] [PATCH v4 3/5] app/test-pmd: add CLI for shaper and wred profiles Jasvinder Singh
2017-09-22 16:35 ` [dpdk-dev] [PATCH v4 4/5] app/test-pmd: add CLI for TM nodes and hierarchy commit Jasvinder Singh
2017-09-22 16:35 ` [dpdk-dev] [PATCH v4 5/5] app/test-pmd: add CLI for TM packet classification Jasvinder Singh
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=54CBAA185211B4429112C315DA58FF6D3326F551@IRSMSX103.ger.corp.intel.com \
--to=jasvinder.singh@intel.com \
--cc=cristian.dumitrescu@intel.com \
--cc=dev@dpdk.org \
--cc=jingjing.wu@intel.com \
--cc=yulong.pei@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).