From: Jesper Dangaard Brouer <jdb@comx.dk>
To: Martin Devera <devik@cdi.cz>
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 16:28:33 +0200 [thread overview]
Message-ID: <1212589713.13164.75.camel@localhost.localdomain> (raw)
In-Reply-To: <48469323.8000506@cdi.cz>
On Wed, 2008-06-04 at 15:05 +0200, Martin Devera wrote:
> 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.
> I just took time to read your thesis. Nice!
Thanks
> I'm thinking about removing HYSTERESIS code altogether, but
> (and it holds for setting HYSTERESIS=off as default)
I prefer a turn knob, where the default is off.
> 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....).
Agreed, lets measure the impact.
With current testing (on 2.6.25.4) I could not see any changes with CPU
usage, but thats probably caused by the fast CPU on the test system
(Dual Xeon). BUT an interesting observation was that the number of
interrupts increased significantly, from approx 1300 to 2000 per sec.
The other observation was that I got significantly less jitter.
> Also, I'm a bit busy for a few weeks, would you be able/interested
> to do such testing ?
Okay, lets wait until I get this kernel into our real-life production
system. Then I can report on the real-life impact on CPU performance.
--
Med venlig hilsen / Best regards
Jesper Brouer
ComX Networks A/S
Linux Network developer
Cand. Scient Datalog / MSc.
Author of http://adsl-optimizer.dk
LinkedIn: http://www.linkedin.com/in/brouer
next prev parent reply other threads:[~2008-06-04 14:28 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 ` [PATCH 0/2] HTB scheduler HTB_HYSTERESIS modifications Martin Devera
2008-06-04 14:28 ` Jesper Dangaard Brouer [this message]
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=1212589713.13164.75.camel@localhost.localdomain \
--to=jdb@comx.dk \
--cc=davem@davemloft.net \
--cc=devik@cdi.cz \
--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).