[dpdk-dev] [PATCH] app/test-pmd: add and identify shaper profile parameters
Xu, Rosen
rosen.xu at intel.com
Mon Oct 22 10:49:03 CEST 2018
> -----Original Message-----
> From: Singh, Jasvinder
> Sent: Tuesday, September 25, 2018 19:56
> To: Xu, Rosen <rosen.xu at intel.com>; dev at dpdk.org
> Cc: Dumitrescu, Cristian <cristian.dumitrescu at intel.com>; Lu, Wenzhuo
> <wenzhuo.lu at intel.com>; Yigit, Ferruh <ferruh.yigit at intel.com>
> Subject: RE: [PATCH] app/test-pmd: add and identify shaper profile
> parameters
>
>
>
> > -----Original Message-----
> > From: Xu, Rosen
> > Sent: Monday, August 13, 2018 11:35 AM
> > To: dev at dpdk.org
> > Cc: Dumitrescu, Cristian <cristian.dumitrescu at intel.com>; Lu, Wenzhuo
> > <wenzhuo.lu at intel.com>; Singh, Jasvinder <jasvinder.singh at intel.com>;
> > Xu, Rosen <rosen.xu at intel.com>; Yigit, Ferruh <ferruh.yigit at intel.com>
> > Subject: [PATCH] app/test-pmd: add and identify shaper profile
> > parameters
> >
> > As struct rte_tm_shaper_params defined, the command line of test-pmd
> > should include committed and peak parameters, but right now the
> > command line doesn't identify whether it's committed or peak
> > parameter. This patch identifies and adds the clarify definition
> >
> > Signed-off-by: Rosen Xu <rosen.xu at intel.com>
> > Fixes: bddc2f40b594 ("app/testpmd: add commands for shaper and wred
> > profiles")
> > Cc: jasvinder.singh at intel.com
> > ---
> > app/test-pmd/cmdline_tm.c | 34 ++++++++++++++++++++++++----------
> > 1 file changed, 24 insertions(+), 10 deletions(-)
> >
> > diff --git a/app/test-pmd/cmdline_tm.c b/app/test-pmd/cmdline_tm.c
> > index
> > 631f179..8ee2785 100644
> > --- a/app/test-pmd/cmdline_tm.c
> > +++ b/app/test-pmd/cmdline_tm.c
> > @@ -771,8 +771,10 @@ struct
> cmd_add_port_tm_node_shaper_profile_result
> > {
> > cmdline_fixed_string_t profile;
> > uint16_t port_id;
> > uint32_t shaper_id;
> > - uint64_t tb_rate;
> > - uint64_t tb_size;
> > + uint64_t cmit_tb_rate;
> > + uint64_t cmit_tb_size;
> > + uint64_t peak_tb_rate;
> > + uint64_t peak_tb_size;
> > uint32_t pktlen_adjust;
> > };
> >
> > @@ -807,14 +809,22 @@ struct
> > cmd_add_port_tm_node_shaper_profile_result {
> > TOKEN_NUM_INITIALIZER(
> > struct cmd_add_port_tm_node_shaper_profile_result,
> > shaper_id, UINT32);
> > -cmdline_parse_token_num_t
> cmd_add_port_tm_node_shaper_profile_tb_rate
> > =
> > +cmdline_parse_token_num_t
> > +cmd_add_port_tm_node_shaper_profile_cmit_tb_rate =
> > TOKEN_NUM_INITIALIZER(
> > struct cmd_add_port_tm_node_shaper_profile_result,
> > - tb_rate, UINT64);
> > -cmdline_parse_token_num_t
> cmd_add_port_tm_node_shaper_profile_tb_size
> > =
> > + cmit_tb_rate, UINT64);
> > +cmdline_parse_token_num_t
> > +cmd_add_port_tm_node_shaper_profile_cmit_tb_size =
> > TOKEN_NUM_INITIALIZER(
> > struct cmd_add_port_tm_node_shaper_profile_result,
> > - tb_size, UINT64);
> > + cmit_tb_size, UINT64);
> > +cmdline_parse_token_num_t
> > cmd_add_port_tm_node_shaper_profile_peak_tb_rate =
> > + TOKEN_NUM_INITIALIZER(
> > + struct cmd_add_port_tm_node_shaper_profile_result,
> > + peak_tb_rate, UINT64);
> > +cmdline_parse_token_num_t
> > cmd_add_port_tm_node_shaper_profile_peak_tb_size =
> > + TOKEN_NUM_INITIALIZER(
> > + struct cmd_add_port_tm_node_shaper_profile_result,
> > + peak_tb_size, UINT64);
> > cmdline_parse_token_num_t
> > cmd_add_port_tm_node_shaper_profile_pktlen_adjust =
> > TOKEN_NUM_INITIALIZER(
> > struct cmd_add_port_tm_node_shaper_profile_result,
> > @@ -838,8 +848,10 @@ static void
> > cmd_add_port_tm_node_shaper_profile_parsed(void *parsed_result,
> > /* Private shaper profile params */
> > memset(&sp, 0, sizeof(struct rte_tm_shaper_params));
> > memset(&error, 0, sizeof(struct rte_tm_error));
> > - sp.peak.rate = res->tb_rate;
> > - sp.peak.size = res->tb_size;
> > + sp.committed.rate = res->cmit_tb_rate;
> > + sp.committed.size = res->cmit_tb_size;
> > + sp.peak.rate = res->peak_tb_rate;
> > + sp.peak.size = res->peak_tb_size;
> > sp.pkt_length_adjust = pkt_len_adjust;
> >
> > ret = rte_tm_shaper_profile_add(port_id, shaper_id, &sp, &error);
> @@
> > -862,8 +874,10 @@ static void
> > cmd_add_port_tm_node_shaper_profile_parsed(void *parsed_result,
> > (void *)&cmd_add_port_tm_node_shaper_profile_profile,
> > (void *)&cmd_add_port_tm_node_shaper_profile_port_id,
> > (void *)&cmd_add_port_tm_node_shaper_profile_shaper_id,
> > - (void *)&cmd_add_port_tm_node_shaper_profile_tb_rate,
> > - (void *)&cmd_add_port_tm_node_shaper_profile_tb_size,
> > + (void
> > *)&cmd_add_port_tm_node_shaper_profile_cmit_tb_rate,
> > + (void
> > *)&cmd_add_port_tm_node_shaper_profile_cmit_tb_size,
> > + (void
> > *)&cmd_add_port_tm_node_shaper_profile_peak_tb_rate,
> > + (void
> > *)&cmd_add_port_tm_node_shaper_profile_peak_tb_size,
> > (void
> > *)&cmd_add_port_tm_node_shaper_profile_pktlen_adjust,
> > NULL,
> > },
> > --
>
> Can you also update the help command section (test-pmd/cmdline.c) as well
> as the documentation (doc/guides/testpmd_app_ug) for these changes.
> Thanks!
It's done, thanks.
More information about the dev
mailing list