All of lore.kernel.org
 help / color / mirror / Atom feed
From: Frans Pop <elendil@planet.nl>
To: Ingo Molnar <mingo@elte.hu>
Cc: linux-kernel@vger.kernel.org,
	Peter Zijlstra <a.p.zijlstra@chello.nl>,
	Jens Axboe <jens.axboe@oracle.com>
Subject: Re: [long] Another BFS versus CFS shakedown
Date: Fri, 11 Sep 2009 09:04:49 +0200	[thread overview]
Message-ID: <200909110904.50487.elendil@planet.nl> (raw)
In-Reply-To: <20090911062032.GB27833@elte.hu>

On Friday 11 September 2009, Ingo Molnar wrote:
> Note, the one you used was a still buggy version of latt.c producing
> bogus latency numbers - you will need the fix to it attached below.

Yes, I'm aware of that and have already copied Jens' latest version.

> Furthermore, the following tune might be needed on mainline to make
> it produce consistently good max numbers (not just good averages):
>
>    echo 0 > /proc/sys/kernel/sched_wakeup_granularity_ns

Ack. I've seen the patches to change some defaults floating by.
Hmmm. I think the proposed new default for my system is 2ms with 2 CPUs?

I will not test against TIP at this time, but I plan to do the following:
- repeat my tests now using vanilla 2.6.31 for both BFS and CFS
  This will provide a baseline to verify improvements.
- do two additional runs with CFS with some modified tunables
- do one more run probably when .32-rc2 is out
  I'd expect that to have the scheduler fixes, while the worst post-merge
  issues should be resolved.

I also have a couple of ideas for getting additional data. I'll post my 
results as follow-ups.

I'm very impressed with the responses to the issues that have been raised, 
but I think we do owe Con a huge thank you for setting off that process.

I also think there is a lot to be said for having a very straightforward 
alternative scheduler available for baseline comparisons. It's much 
easier to come out and say "something's broken" if you know some latency 
issue is not due to buggy hardware or applications or orange bunnies with 
a cosmic ray gun. I'll not go into the question whether such a scheduler 
should be in mainline or not.

Cheers,
FJP

  reply	other threads:[~2009-09-11  7:04 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-09-08 23:42 [long] Another BFS versus CFS shakedown Frans Pop
2009-09-09  0:01 ` Nikos Chantziaras
2009-09-09  0:44 ` Frans Pop
2009-09-11  6:20 ` Ingo Molnar
2009-09-11  7:04   ` Frans Pop [this message]
2009-09-11  7:17     ` Jens Axboe
2009-09-11  7:53       ` Ingo Molnar
2009-09-11  7:58         ` Jens Axboe

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=200909110904.50487.elendil@planet.nl \
    --to=elendil@planet.nl \
    --cc=a.p.zijlstra@chello.nl \
    --cc=jens.axboe@oracle.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@elte.hu \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.