> -----Original Message-----
> From: Danilewicz, MarcinX <marcinx.danilew...@intel.com>
> Sent: Tuesday, May 24, 2022 2:44 PM
> To: dev@dpdk.org; Singh, Jasvinder <jasvinder.si...@intel.com>;
> Dumitrescu, Cristian <cristian.dumitre...@intel.com>
> Cc: Ajmera, Megha <megha.ajm...@intel.com>
> Subject: [PATCH v4] sched: enable traffic class oversubscription conditionally
>
> Added new API to enable or disable TC over subscription for best
> effort traffic class at subport level.
> Added changes after review and increased throughput.
>
> By default TC OV is disabled.
> History:
> - v1 - TC OV disabled by default
> - v2 - throughput improvements
> - v3, v4 - changes from comments
>
> Signed-off-by: Marcin Danilewicz <marcinx.danilew...@intel.com>
> ---
> lib/sched/rte_sched.c | 189 +++++++++++++++++++++++++++++++++++------
Marcin,
I don't see any of my comments on the previous V3 version addressed. You
mention in the change log that you addressed comments, but I see that all my
comments were silently disregarded. Jasvinder also noted the same for his
comments in a previous version. Please address the comments and do not keep
sending the same code over and over.
This change was supposed to be straightforward, but for some reason the
progress is extremely slow on your side. I think at this point we are at risk
of missing the RC1 deadline for this feature.
Regards,
Cristian
