DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] Old oversubscription related checkin
@ 2016-05-04 21:14 Sridhar.V.Iyer
  2016-05-05 12:49 ` Dumitrescu, Cristian
  0 siblings, 1 reply; 2+ messages in thread
From: Sridhar.V.Iyer @ 2016-05-04 21:14 UTC (permalink / raw)
  To: dev, Dumitrescu, Cristian; +Cc: Ananda

Hi Cristian,

 I stumbled into an old from 2013 (http://dpdk.org/browse/dpdk/patch/lib/librte_sched/rte_sched.c?id=835c5409a7bac3055b82bebee65d8ada7f20d332 <http://dpdk.org/browse/dpdk/patch/lib/librte_sched/rte_sched.c?id=835c5409a7bac3055b82bebee65d8ada7f20d332>)

I couldn’t find any context for the patch from the mail archives. Could you please let me know why oversubscription was removed from all the other traffic classes and just given to tc3?
Were there huge performance penalties? Would there be any issues if I add this patch back again locally (enable/disabled via config).

Regards,

Sridhar V Iyer









 <http://www.versa-networks.com/> <http://www.versa-networks.com/>

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [dpdk-dev] Old oversubscription related checkin
  2016-05-04 21:14 [dpdk-dev] Old oversubscription related checkin Sridhar.V.Iyer
@ 2016-05-05 12:49 ` Dumitrescu, Cristian
  0 siblings, 0 replies; 2+ messages in thread
From: Dumitrescu, Cristian @ 2016-05-05 12:49 UTC (permalink / raw)
  To: Sridhar.V.Iyer, dev; +Cc: Ananda

Hi Sridhar,

I think this patch is simply the implementation of the oversubscription feature, as introduced by DPDK release 1.4.1. Most likely, this patch is the way Thomas added this Intel release to dpdk.org.

The oversubscription feature was never implemented for all 4 traffic classes. The patch you are pointing to simply introduced this feature for TC3 only. Before this patch, there might have been just a placeholder in the code for the oversubscription feature that was to be developed later on.

Again, the reason for introducing this feature for TC3 only is that the higher priority traffic classes TC0 .. TC2 are typically fully provisioned, as the amount of TC0 .. TC2 traffic is much less than TC3 (Best Effort) traffic, which is usually hugely overprovisioned. This feature can potentially be extended to TC0 .. TC2 as well, as the way this feature basically works is deciding the quanta to be applied for all pipes in the subport during the next scheduling decision based on the amount of unused credits found at the end of the current scheduling period.

Regards,
Cristian

From: Sridhar.V.Iyer [mailto:sridhariyer@versa-networks.com]
Sent: Wednesday, May 4, 2016 10:15 PM
To: dev@dpdk.org; Dumitrescu, Cristian <cristian.dumitrescu@intel.com>
Cc: Ananda <ananda@versa-networks.com>
Subject: Old oversubscription related checkin

Hi Cristian,

 I stumbled into an old from 2013 (http://dpdk.org/browse/dpdk/patch/lib/librte_sched/rte_sched.c?id=835c5409a7bac3055b82bebee65d8ada7f20d332)

I couldn’t find any context for the patch from the mail archives. Could you please let me know why oversubscription was removed from all the other traffic classes and just given to tc3?
Were there huge performance penalties? Would there be any issues if I add this patch back again locally (enable/disabled via config).

Regards,

Sridhar V Iyer

<http://www.versa-networks.com/>






<http://www.versa-networks.com/>


<http://www.versa-networks.com/>


<http://www.versa-networks.com/>


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2016-05-05 12:49 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-05-04 21:14 [dpdk-dev] Old oversubscription related checkin Sridhar.V.Iyer
2016-05-05 12:49 ` Dumitrescu, Cristian

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).