public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Ingo Molnar <mingo@elte.hu>
To: Tong Li <tong.n.li@intel.com>
Cc: linux-kernel@vger.kernel.org, Chris Snook <csnook@redhat.com>
Subject: Re: [RFC] scheduler: improve SMP fairness in CFS
Date: Wed, 25 Jul 2007 21:24:42 +0200	[thread overview]
Message-ID: <20070725192442.GC4463@elte.hu> (raw)
In-Reply-To: <Pine.LNX.4.64.0707251014520.14515@tongli.jf.intel.com>


* Tong Li <tong.n.li@intel.com> wrote:

> Thanks for the patch. It doesn't work well on my 8-way box. Here's the 
> output at two different times. It's also changing all the time.

you need to measure it over longer periods of time. Its not worth 
balancing for such a thing in any high-frequency manner. (we'd trash the 
cache constantly migrating tasks back and forth.)

> [time 2]
>  3721 tongli    20   0  7864  108   16 R   91  0.0   1:44.67 loop
>  3720 tongli    20   0  7864  108   16 R   89  0.0   1:40.55 loop
>  3722 tongli    20   0  7864  108   16 R   86  0.0   1:41.40 loop
>  3723 tongli    20   0  7864  108   16 R   82  0.0   1:38.94 loop
>  3715 tongli    20   0  7864  108   16 R   82  0.0   1:43.09 loop
>  3717 tongli    20   0  7864  108   16 R   79  0.0   1:46.38 loop
>  3718 tongli    20   0  7864  108   16 R   79  0.0   1:40.72 loop
>  3714 tongli    20   0  7864  108   16 R   77  0.0   1:40.17 loop
>  3719 tongli    20   0  7864  108   16 R   73  0.0   1:35.41 loop
>  3716 tongli    20   0  7864  108   16 R   61  0.0   1:38.66 loop

the runtime ranges from 1:38.66 to 1:46.38 - that's a +-3% spread which 
is pretty acceptable for ~90 seconds of runtime. Note that the 
percentages you see above were likely done over a shorter period that's 
why you see them range from 61% to 91%.

> I'm running 2.6.23-rc1 with your patch on a two-socket quad-core 
> system. Top refresh rate is the default 3s.

the 3s is the problem: change that to 60s! We no way want to 
over-migrate for SMP fairness, the change i did gives us reasonable 
long-term SMP fairness without the need for high-rate rebalancing.

	Ingo

  reply	other threads:[~2007-07-25 19:25 UTC|newest]

Thread overview: 45+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-07-23 18:38 [RFC] scheduler: improve SMP fairness in CFS Tong Li
2007-07-23 20:00 ` Andi Kleen
2007-07-23 21:10   ` Li, Tong N
2007-07-23 21:25     ` Chris Friesen
2007-07-24  9:43       ` Andi Kleen
2007-07-23 23:40 ` Chris Snook
2007-07-24  8:07   ` Chris Snook
2007-07-24 17:11     ` Li, Tong N
2007-07-24 17:07   ` Tong Li
2007-07-24 18:08     ` Chris Snook
2007-07-24 19:47       ` Chris Friesen
2007-07-24 20:39         ` Chris Snook
2007-07-24 20:58           ` Li, Tong N
2007-07-24 21:09             ` Chris Snook
2007-07-24 21:23               ` Chris Friesen
2007-07-24 21:45                 ` Chris Snook
2007-07-24 23:33                   ` Chris Friesen
2007-07-24 21:06           ` Bill Huey
2007-07-24 21:22             ` Chris Snook
2007-07-24 23:14               ` Bill Huey
2007-07-24 21:12           ` Chris Friesen
2007-07-25 11:01 ` Ingo Molnar
2007-07-25 12:03   ` Ingo Molnar
2007-07-25 17:23     ` Tong Li
2007-07-25 19:24       ` Ingo Molnar [this message]
2007-07-25 20:38         ` Chris Friesen
2007-07-25 20:55           ` Chris Snook
2007-07-25 21:15             ` Li, Tong N
2007-07-25 22:24               ` Chris Snook
2007-07-26 19:00         ` Tong Li
2007-07-26 21:31           ` Ingo Molnar
2007-07-26 22:00             ` Li, Tong N
2007-07-27  1:34               ` Tong Li
2007-07-27 17:16                 ` Chris Snook
2007-07-27 19:03                   ` Tong Li
2007-07-27 22:20                     ` Bill Huey
2007-07-27 23:36                     ` Chris Snook
2007-07-28  0:54                       ` Bill Huey
2007-07-28  2:59                         ` Chris Snook
2007-07-28 19:38                           ` Tong Li
2007-07-29  2:40                             ` Chris Snook
2007-07-28 19:23                       ` Tong Li
2007-07-29  3:01                         ` Chris Snook
2007-07-25 18:20     ` Li, Tong N
2007-07-25 19:18       ` Ingo Molnar

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=20070725192442.GC4463@elte.hu \
    --to=mingo@elte.hu \
    --cc=csnook@redhat.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=tong.n.li@intel.com \
    /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