RE: [PATCH] sched: fix for tc_ov_enable flag position in subport structure.

2023-02-07 Thread Ajmera, Megha
> > > > 10/01/2023 12:27, David Marchand: > > > On Mon, Jan 9, 2023 at 3:59 PM Megha Ajmera > > wrote: > > > > > > > > Current position of "tv_ov_enable" variable in > > > > > > tc_ov_enabled* > > > > > > > > > > rte_sched_subport structure makes the "memory" variable unused. > > > > > > I did not

RE: [PATCH] sched: fix for tc_ov_enable flag position in subport structure.

2023-02-06 Thread Dumitrescu, Cristian
Re: [PATCH] sched: fix for tc_ov_enable flag position in subport > structure. > > 10/01/2023 12:27, David Marchand: > > On Mon, Jan 9, 2023 at 3:59 PM Megha Ajmera > wrote: > > > > > > Current position of "tv_ov_enable" variable in > > > > tc_ov_

Re: [PATCH] sched: fix for tc_ov_enable flag position in subport structure.

2023-02-05 Thread Thomas Monjalon
10/01/2023 12:27, David Marchand: > On Mon, Jan 9, 2023 at 3:59 PM Megha Ajmera wrote: > > > > Current position of "tv_ov_enable" variable in > > tc_ov_enabled* > > > > rte_sched_subport structure makes the "memory" variable unused. > > I did not enter the beast... but my understanding is that

Re: [PATCH] sched: fix for tc_ov_enable flag position in subport structure.

2023-01-10 Thread David Marchand
On Mon, Jan 9, 2023 at 3:59 PM Megha Ajmera wrote: > > Current position of "tv_ov_enable" variable in tc_ov_enabled* > rte_sched_subport structure makes the "memory" variable unused. I did not enter the beast... but my understanding is that some object internal to rte_sched_subport currently s

RE: [PATCH] sched: fix for tc_ov_enable flag position in subport structure.

2023-01-10 Thread Dumitrescu, Cristian
> -Original Message- > From: Ajmera, Megha > Sent: Monday, January 9, 2023 2:55 PM > To: dev@dpdk.org; Singh, Jasvinder ; Dumitrescu, > Cristian > Cc: sta...@dpdk.org; marcinx.danilew...@intel.com > Subject: [PATCH] sched: fix for tc_ov_enable flag position in

[PATCH] sched: fix for tc_ov_enable flag position in subport structure.

2023-01-09 Thread Megha Ajmera
Current position of "tv_ov_enable" variable in rte_sched_subport structure makes the "memory" variable unused. Fixes: f5e60154ade ("sched: enable traffic class oversubscription conditionally") CC: marcinx.danilew...@intel.com Signed-off-by: Megha Ajmera --- lib/sched/rte_sched.c | 5 +++-- 1 fi