From: Martin Devera <devik@cdi.cz>
To: jdb@comx.dk
Cc: "David S. Miller" <davem@davemloft.net>,
"netdev@vger.kernel.org" <netdev@vger.kernel.org>
Subject: Re: [PATCH 0/2] HTB scheduler HTB_HYSTERESIS modifications
Date: Wed, 04 Jun 2008 15:05:39 +0200 [thread overview]
Message-ID: <48469323.8000506@cdi.cz> (raw)
In-Reply-To: <1212501803.13164.27.camel@localhost.localdomain>
Jesper Dangaard Brouer wrote:
> This patch set modifies the HTB schedulers hysteresis mode. The HTB
> hysteresis is a hack to reduce the CPU load, it introduces significant
> jitter, enough to disturbe VoIP quality on e.g. ADSL.
>
> The first patch changes HTB_HYSTERESIS to default off.
>
> The second patch Change HTB_HYSTERESIS to a runtime parameter
> htb_hysteresis, adjustable via
> sys/module/sch_htb/parameters/htb_hysteresis.
>
> For details about my measurements see masters thesis
> (http://www.adsl-optimizer.dk/thesis/), chapter 7, section 7.3.1,
> page 69-70.
>
Hello Jesper,
I just took time to read your thesis. Nice!
I'm thinking about removing HYSTERESIS code altogether, but
(and it holds for setting HYSTERESIS=off as default) I'd like
to measure cpu impact again.
IMHO we need to assure that there will be no surprise when some big
router is upgraded to new kernel and will not catch on speed
(unlikely, but....).
Also, I'm a bit busy for a few weeks, would you be able/interested
to do such testing ?
thanks, Martin
next prev parent reply other threads:[~2008-06-04 13:47 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-06-03 14:03 [PATCH 0/2] HTB scheduler HTB_HYSTERESIS modifications Jesper Dangaard Brouer
2008-06-03 14:05 ` [PATCH 1/2] HTB scheduler, change default hysteresis mode to off Jesper Dangaard Brouer
2008-06-03 14:06 ` [PATCH 2/2] Change HTB_HYSTERESIS to a runtime parameter htb_hysteresis Jesper Dangaard Brouer
2008-06-04 13:05 ` Martin Devera [this message]
2008-06-04 14:28 ` [PATCH 0/2] HTB scheduler HTB_HYSTERESIS modifications Jesper Dangaard Brouer
2008-06-04 15:31 ` Martin Devera
2008-06-12 10:01 ` Jesper Dangaard Brouer
2008-06-14 9:18 ` Martin Devera
2008-06-16 23:40 ` David Miller
2008-06-17 1:00 ` Denys Fedoryshchenko
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=48469323.8000506@cdi.cz \
--to=devik@cdi.cz \
--cc=davem@davemloft.net \
--cc=jdb@comx.dk \
--cc=netdev@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).