LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
* dbench 15% regression with 2.6.28-rc1
@ 2008-10-28  8:41 Zhang, Yanmin
  2008-10-31  9:14 ` Zhang, Yanmin
  0 siblings, 1 reply; 3+ messages in thread
From: Zhang, Yanmin @ 2008-10-28  8:41 UTC (permalink / raw)
  To: Peter Zijlstra; +Cc: LKML

Comparing with 2.6.27, dbench result has regression with 2.6.28-rc1 on 2 machines.
1) 8-core stoakley: 15%
2) 8 core+mutl-thread new-model x86-64: 12%

Bisect located below patch.

695698500912c4479ddf4723e492de3970ff8530 is first bad commit
commit 695698500912c4479ddf4723e492de3970ff8530
Author: Peter Zijlstra <a.p.zijlstra@chello.nl>
Date:   Tue Sep 23 14:54:23 2008 +0200

    sched: rework wakeup preemption
    
    Rework the wakeup preemption to work on real runtime instead of
    the virtual runtime. This greatly simplifies the code.
    
    Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
    Signed-off-by: Ingo Molnar <mingo@elte.hu>


I reverted the patch against 2.6.28-rc2 and the regression mostly disappears
on 8-core stoakley and 8-core+multiThread x86-64 machines.


On other 2 machines, I see improvement instead of regression.
1) 16-core tigerton: improvement 48%
2) 8-core+hyperThreading tulsa: 10%.
I just checked it by reverting above patch to see if the patch improves it. At least
it isn't on tigerton. I'm doing a new bisect on tigerton to see what patch improves
dbench result.


I start online cpu number of dbench processes.

-yanmin



^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: dbench 15% regression with 2.6.28-rc1
  2008-10-28  8:41 dbench 15% regression with 2.6.28-rc1 Zhang, Yanmin
@ 2008-10-31  9:14 ` Zhang, Yanmin
  2008-10-31  9:23   ` Peter Zijlstra
  0 siblings, 1 reply; 3+ messages in thread
From: Zhang, Yanmin @ 2008-10-31  9:14 UTC (permalink / raw)
  To: Peter Zijlstra; +Cc: LKML


On Tue, 2008-10-28 at 16:41 +0800, Zhang, Yanmin wrote:
> Comparing with 2.6.27, dbench result has regression with 2.6.28-rc1 on 2 machines.
> 1) 8-core stoakley: 15%
> 2) 8 core+mutl-thread new-model x86-64: 12%
> 
> Bisect located below patch.
> 
> 695698500912c4479ddf4723e492de3970ff8530 is first bad commit
> commit 695698500912c4479ddf4723e492de3970ff8530
> Author: Peter Zijlstra <a.p.zijlstra@chello.nl>
> Date:   Tue Sep 23 14:54:23 2008 +0200
> 
>     sched: rework wakeup preemption
>     
>     Rework the wakeup preemption to work on real runtime instead of
>     the virtual runtime. This greatly simplifies the code.
>     
>     Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
>     Signed-off-by: Ingo Molnar <mingo@elte.hu>
> 
> 
> I reverted the patch against 2.6.28-rc2 and the regression mostly disappears
> on 8-core stoakley and 8-core+multiThread x86-64 machines.
> 
> 
> On other 2 machines, I see improvement instead of regression.
> 1) 16-core tigerton: improvement 48%
> 2) 8-core+hyperThreading tulsa: 10%.
> I just checked it by reverting above patch to see if the patch improves it. At least
> it isn't on tigerton. I'm doing a new bisect on tigerton to see what patch improves
> dbench result.
The improvement on tigerton isn't caused by the patch. It seems it is caused by
other scheduler patches.

Well, comparing with 2.6.27, the result of sysbench+mysql (oltp readonly) with 2.6.28-rc2
has about 10% improvement, especially with high thread number. I located that's casued
by the rework_wakeup_preemption patch.

So the patch improves oltp result, but downgrades dbench result.

-yanmin



^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: dbench 15% regression with 2.6.28-rc1
  2008-10-31  9:14 ` Zhang, Yanmin
@ 2008-10-31  9:23   ` Peter Zijlstra
  0 siblings, 0 replies; 3+ messages in thread
From: Peter Zijlstra @ 2008-10-31  9:23 UTC (permalink / raw)
  To: Zhang, Yanmin; +Cc: LKML

On Fri, 2008-10-31 at 17:14 +0800, Zhang, Yanmin wrote:
> On Tue, 2008-10-28 at 16:41 +0800, Zhang, Yanmin wrote:
> > Comparing with 2.6.27, dbench result has regression with 2.6.28-rc1 on 2 machines.
> > 1) 8-core stoakley: 15%
> > 2) 8 core+mutl-thread new-model x86-64: 12%
> > 
> > Bisect located below patch.
> > 
> > 695698500912c4479ddf4723e492de3970ff8530 is first bad commit
> > commit 695698500912c4479ddf4723e492de3970ff8530
> > Author: Peter Zijlstra <a.p.zijlstra@chello.nl>
> > Date:   Tue Sep 23 14:54:23 2008 +0200
> > 
> >     sched: rework wakeup preemption
> >     
> >     Rework the wakeup preemption to work on real runtime instead of
> >     the virtual runtime. This greatly simplifies the code.
> >     
> >     Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
> >     Signed-off-by: Ingo Molnar <mingo@elte.hu>
> > 
> > 
> > I reverted the patch against 2.6.28-rc2 and the regression mostly disappears
> > on 8-core stoakley and 8-core+multiThread x86-64 machines.
> > 
> > 
> > On other 2 machines, I see improvement instead of regression.
> > 1) 16-core tigerton: improvement 48%
> > 2) 8-core+hyperThreading tulsa: 10%.
> > I just checked it by reverting above patch to see if the patch improves it. At least
> > it isn't on tigerton. I'm doing a new bisect on tigerton to see what patch improves
> > dbench result.
> The improvement on tigerton isn't caused by the patch. It seems it is caused by
> other scheduler patches.
> 
> Well, comparing with 2.6.27, the result of sysbench+mysql (oltp readonly) with 2.6.28-rc2
> has about 10% improvement, especially with high thread number. I located that's casued
> by the rework_wakeup_preemption patch.
> 
> So the patch improves oltp result, but downgrades dbench result.

The thing is, a later patch undoes this one. I'm just not sure when its
landed in -linus.

This vruntime -> real-time wakeup preemption was a test to see if it
would work because the code is so much simpler, but the sad truth is
that it doesn't work all that well, so we went back already.

Its just that the patch going back to vruntime based wakeup didn't make
it in -rc1 (and possibly -rc2).




^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2008-10-31  9:23 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-10-28  8:41 dbench 15% regression with 2.6.28-rc1 Zhang, Yanmin
2008-10-31  9:14 ` Zhang, Yanmin
2008-10-31  9:23   ` Peter Zijlstra

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