From: "Kevin D. Kissell" <kevink@paralogos.com>
To: Ralf Baechle <ralf@linux-mips.org>
Cc: Julia Lawall <julia@diku.dk>,
linux-mips@linux-mips.org, linux-kernel@vger.kernel.org,
kernel-janitors@vger.kernel.org
Subject: Affinity Automation (was Re: [PATCH 3/17] arch/mips/kernel: Add missing
Date: Wed, 26 May 2010 17:27:53 +0000 [thread overview]
Message-ID: <4BFD5A19.5030602@paralogos.com> (raw)
In-Reply-To: <20100526163227.GA17043@linux-mips.org>
Ralf Baechle wrote:
> Your patch appears correct - and mipsmt_sys_sched_setaffinity() even
> more broken than you thought. It duplicates some code from kernel/sched.c
> and has gotten out of sync.
Yeah, that was inevitable. Since the distribution of the previous
message seems to cover concerned developers outside the MIPS community,
let me make one final(?) plea to actually do this right.
The MIPS SMTC support for managing a single FPU context on a processor
with multiple integer TC contexts involves having the system make
automous, real time, decisions about scheduling affinity. It may be a
first, but it can't possibly be the only case, especially as we've
started seeing more and more mainstream multi-core, multi-thread
designs. System and chip resources are going to be "closer" to one
processor or another. The current Linux paradigm is that it's the
responsibility of programs, or users, to know what the optimal placement
of processes should be for a given system platform, and while it's
absolutely appropriate to provide that level of control for the cases
where the user really does know best, it's mildly insane to make that
the only way that thread placement can be optimized. It's really the
OS's job to match demand to resources.
My contention years ago was, and remains, that it would be a bad idea to
burden the main scheduler loop with checks for two different levels of
affinity, system-automatic and user-specified. It would add
non-trivially to the cache footprint and execution overhead of thread
dispatch, and there's no logical need for it. So the model I proposed,
and implemented in the cloned affinity system calls for SMTC, was that a
*single* affinity mask continue to be used by the scheduler, but that
the per-thread data structures carry two: The one requested explicitly
by the user, and the one actually used by the scheduler. The idea is
that normally those two are the same, but that the system - in the MIPS
SMTC case, the FPU emulator - can overlay its constraints with the
user's constraints to come up with an intersection-of-sets constraint
that satisfies both (there was a clause which prevents system affinity
heuristics from restricting the affinity mask to a null set of CPUs,
though of course the user can do that if he really wants to).
There's nothing particularly MIPS-specific about the problem or the
solution. Most of the mechanisms should really be in
platform-independent code, so we don't get the drift of cloned components.
I don't recall who owned the scheduler at the time, but whoever it was
was too busy dealing with mainstream processor problems to even engage
in a dialogue about this. Is it time to raise the question again?
Regards,
Kevin K.
next prev parent reply other threads:[~2010-05-26 17:27 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-05-26 15:54 [PATCH 3/17] arch/mips/kernel: Add missing read_unlock Julia Lawall
2010-05-26 16:32 ` Ralf Baechle
2010-05-26 17:27 ` Kevin D. Kissell [this message]
2010-05-26 17:38 ` Affinity Automation (was Re: [PATCH 3/17] arch/mips/kernel: Add Ralf Baechle
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=4BFD5A19.5030602@paralogos.com \
--to=kevink@paralogos.com \
--cc=julia@diku.dk \
--cc=kernel-janitors@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mips@linux-mips.org \
--cc=ralf@linux-mips.org \
/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;
as well as URLs for NNTP newsgroup(s).