From: Nick Piggin <nickpiggin@yahoo.com.au>
To: "Siddha, Suresh B" <suresh.b.siddha@intel.com>
Cc: mingo@elte.hu, akpm@osdl.org, linux-kernel@vger.kernel.org,
kenneth.w.chen@intel.com
Subject: Re: [patch] sched: improve pinned task handling again!
Date: Sat, 02 Apr 2005 13:11:20 +1000 [thread overview]
Message-ID: <424E0D58.1070700@yahoo.com.au> (raw)
In-Reply-To: <20050401185812.A5598@unix-os.sc.intel.com>
Siddha, Suresh B wrote:
>
> This time Ken Chen brought up this issue -- No it has nothing to do with
> industry db benchmark ;-)
>
> Even with the above mentioned Nick's patch in -mm, I see system livelock's
> if for example I have 7000 processes pinned onto one cpu (this is on the
> fastest 8-way system I have access to). I am sure there will be other
> systems where this problem can be encountered even with lesser pin count.
>
Thanks for testing these patches in -mm, by the way.
> We tried to fix this issue but as you know there is no good mechanism
> in fixing this issue with out letting the regular paths know about this.
>
> Our proposed solution is appended and we tried to minimize the affect on
> fast path. It builds up on Nick's patch and once this situation is detected,
> it will not do any more move_tasks as long as busiest cpu is always the
> same cpu and the queued processes on busiest_cpu, their
> cpu affinity remain same(found out by runqueue's "generation_num")
>
7000 running processes pinned into one CPU. I guess that isn't a
great deal :(
How important is this? Any application to real workloads? Even if
not, I agree it would be nice to improve this more. I don't know
if I really like this approach - I guess due to what it adds to
fastpaths.
Now presumably if the all_pinned logic is working properly in the
first place, and it is correctly causing balancing to back-off, you
could tweak that a bit to avoid livelocks? Perhaps the all_pinned
case should back off faster than the usual doubling of the interval,
and be allowed to exceed max_interval?
Any thoughts Ingo?
--
SUSE Labs, Novell Inc.
next prev parent reply other threads:[~2005-04-02 3:11 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-04-02 2:58 [patch] sched: improve pinned task handling again! Siddha, Suresh B
2005-04-02 3:11 ` Nick Piggin [this message]
2005-04-02 4:05 ` Siddha, Suresh B
2005-04-02 4:12 ` Nick Piggin
2005-04-04 1:46 ` Chen, Kenneth W
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=424E0D58.1070700@yahoo.com.au \
--to=nickpiggin@yahoo.com.au \
--cc=akpm@osdl.org \
--cc=kenneth.w.chen@intel.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@elte.hu \
--cc=suresh.b.siddha@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 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.