Date: Fri, 11 Sep 2009 09:53:31 +0200
From: Ingo Molnar <mingo@elte.hu>
To: Jens Axboe <jens.axboe@oracle.com>
Cc: Frans Pop <elendil@planet.nl>, linux-kernel@vger.kernel.org,
       Peter Zijlstra <a.p.zijlstra@chello.nl>
Subject: Re: [long] Another BFS versus CFS shakedown
Message-ID: <20090911075331.GA14489@elte.hu>
References: <200909090142.41738.elendil@planet.nl> <20090911062032.GB27833@elte.hu> <200909110904.50487.elendil@planet.nl> <20090911071710.GB18599@kernel.dk>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20090911071710.GB18599@kernel.dk>
User-Agent: Mutt/1.5.18 (2008-05-17)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1790
Lines: 46


* Jens Axboe <jens.axboe@oracle.com> wrote:

> On Fri, Sep 11 2009, Frans Pop wrote:
> > 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.
> 
> BTW, I put it in a git repo, it quickly gets really confusing with so
> many version going around. So that can be accessed here:
> 
> git://git.kernel.dk/latt.git
> 
> and as with my other repos, snapshots are automatically generated every
> hour when new commits have been made. To get the very latest latt and
> not have to use git, download:
> 
> http://brick.kernel.dk/snaps/latt-git-latest.tar.gz

Btw., your earlier latt reports should be discarded as invalid due 
to that bug.

With the fixed latt.c version the mainline latencies (both 
worst-case and average) were reported to be better after the poll() 
bug got fixed, so in that area, for this kind of measurement, 
mainline seems to be working well.

[ What happened is that the poll() bug was creating false latencies
  in the mainline scheduler tests. (BFS avoided measuring that bug
  incidentally, by its agressive balancer moved the wakee tasks away
  from the buggy busy-looping poll() looping parent task. Two
  instances of latt.c would possibly have shown similar latencies.) ]

I see you added new 'work generator' changes to latt.c now, will 
check/validate that version of latt.c too.

Thanks,

	Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/