> -----Original Message-----
> From: Singh, Jasvinder
> Sent: Wednesday, July 31, 2019 12:33 PM
> To: dev@dpdk.org
> Cc: Dumitrescu, Cristian <cristian.dumitre...@intel.com>
> Subject: [PATCH] examples/ip_pipeline: remove tc ov flag from pipe profile
> cli
> 
> Since sched library always check the non-zero value of subport tc
> ov weight. Therefore, it is necessary to set right value of tc ov
> weight even if RTE_SCHED_SUBPORT_TC_OV is disabled.
> 
> Signed-off-by: Jasvinder Singh <jasvinder.si...@intel.com>
> ---
>  examples/ip_pipeline/cli.c | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/examples/ip_pipeline/cli.c b/examples/ip_pipeline/cli.c
> index f4c2be8b8..c6cf4204e 100644
> --- a/examples/ip_pipeline/cli.c
> +++ b/examples/ip_pipeline/cli.c
> @@ -469,12 +469,10 @@ cmd_tmgr_pipe_profile(char **tokens,
>               return;
>       }
> 
> -#ifdef RTE_SCHED_SUBPORT_TC_OV
>       if (parser_read_uint8(&p.tc_ov_weight, tokens[19]) != 0) {
>               snprintf(out, out_size, MSG_ARG_INVALID,
> "tc_ov_weight");
>               return;
>       }
> -#endif
> 
>       for (i = 0; i < RTE_SCHED_BE_QUEUES_PER_PIPE; i++)
>               if (parser_read_uint8(&p.wrr_weights[i], tokens[20 + i]) != 0)
> {
> --
> 2.21.0

Acked-by: Cristian Dumitrescu <cristian.dumitre...@intel.com>

We should check that this macro does not show up anymore in any of our apps.

Reply via email to