Thread (38 messages) 38 messages, 8 authors, 2014-10-02

Re: [net-next PATCH] dql: add a burst attribute

From: Jesper Dangaard Brouer <hidden>
Date: 2014-09-30 21:47:02

On Tue, 30 Sep 2014 07:55:36 -0700
Eric Dumazet [off-list ref] wrote:
On Tue, 2014-09-30 at 16:31 +0200, Florian Westphal wrote:
quoted
Eric Dumazet [off-list ref] wrote:
quoted
If you feel not comfortable with "burst", rename it to whatever you
think is best.
But please, do not hard code magic 7 in your code.
I had hoped that this 'magic' value could be removed
completely, only using bql data for bulking decisions.
But it is apparently not the case, since you guys decided to had it set
to 8, then to 7 later, based on experiments.
The "magic" limit is only a conservative save guard.  We do plan to
remove this completely.  That is the reason for not exporting this, as
we want free-hands to remove this completely.

Guess, I'll remove it completely now, so we can move on.


I would like some off/on switch, exported to userspace, for disabling
this bulking (then we don't need this conservative magic number).  How
would that be done best?

-- 
Best regards,
  Jesper Dangaard Brouer
  MSc.CS, Sr. Network Kernel Developer at Red Hat
  Author of http://www.iptv-analyzer.org
  LinkedIn: http://www.linkedin.com/in/brouer
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help