netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* Re: [PATCH]: Remove backlog accounting from TBF, pass limit to the default inner bfifo qdisc only
       [not found] <3FE98BE9.2000708@trash.net>
@ 2003-12-25  3:22 ` David S. Miller
  0 siblings, 0 replies; only message in thread
From: David S. Miller @ 2003-12-25  3:22 UTC (permalink / raw)
  To: Patrick McHardy; +Cc: netdev

On Wed, 24 Dec 2003 13:51:53 +0100
Patrick McHardy <kaber@trash.net> wrote:

> when TBF was made classful, backlog accounting was kept inside
> TBF for reasons I don't understand. When used with SFQ as inner
> qdisc, TBF can not know the exact backlog once SFQ returned
> NET_CMIT_CN. An underestimate leads to backlog becoming negative
> when the queue drains, it stops accepting packets then.
 ...
> This patch removes backlog accounting from TBF, the limit
> parameter is just passed to the default inner bfifo, once it
> is replaced limit is not effective anymore.

Applied to both 2.4.x and 2.6.x, thanks Patrick.

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2003-12-25  3:22 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <3FE98BE9.2000708@trash.net>
2003-12-25  3:22 ` [PATCH]: Remove backlog accounting from TBF, pass limit to the default inner bfifo qdisc only David S. Miller

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