linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Max Krasnyansky <maxk@qualcomm.com>
To: Ingo Molnar <mingo@elte.hu>
Cc: Nish Aravamudan <nish.aravamudan@gmail.com>,
	Peter Zijlstra <peterz@infradead.org>,
	Gregory Haskins <ghaskins@novell.com>,
	Dimitri Sivanich <sivanich@sgi.com>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: Using cpusets for configuration/isolation [Was Re: RT sched: cpupri_vec lock contention with def_root_domain and no load balance]
Date: Wed, 19 Nov 2008 12:01:47 -0800	[thread overview]
Message-ID: <492470AB.5060306@qualcomm.com> (raw)
In-Reply-To: <20081119174448.GB31560@elte.hu>

Ingo Molnar wrote:
> * Max Krasnyansky <maxk@qualcomm.com> wrote:
> 
>> Ingo Molnar wrote:
>>> * Max Krasnyansky <maxk@qualcomm.com> wrote:
>>>
>>>> What you described is almost exactly what I did in my original 
>>>> cpu isolation patch, which did get NAKed :). Basically I used 
>>>> global cpu_isolated_map and exposed 'isolated' bit, etc.
>>> Please extend cpusets according to the plan outlined by PeterZ a 
>>> few months ago - that's the right place to do partitioning.
>> Already did. It's all in mainline. The part you quoted was just 
>> pointing out that the original approach was not correct.
> 
> Yeah, we have bits of it (i merged them, and i still remember them ;-) 
> - but we still dont have the "system set" concept suggested by Peter 
> though. We could go further and make it really easy to partition all 
> scheduling and irq aspects of the system via cpusets.

Actually we (or maybe just me) gave up on those for now. We went back an forth
on the 'system set' and what it supposed to mean. Both Paul J. and Paul M.
were against the concept and especially backward compatibility with existing
user-space tools that use cpusets. Plus it's really really easy to setup the
'system' set from user-space and I just ended up writing 'syspart' thing that
I mentioned before.

Similar thing happened to "managing irqs via cpusets" idea. Peter and I wanted
to represent them ask tasks and Paul J. was very vocally :) opposed to it.
What we settled on was that we will manage irqs via proc for now. I added a
notion of 'default' irq affinity (ie /proc/irq/default_smp_affinity) which is
already in mainline.

We can probably revisit irq management and 'system' cpuset again. At this
point I'm swamped with other stuff though.

Max

      reply	other threads:[~2008-11-19 20:01 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-11-07 19:23 Using cpusets for configuration/isolation [Was Re: RT sched: cpupri_vec lock contention with def_root_domain and no load balance] Nish Aravamudan
2008-11-19  1:59 ` Max Krasnyansky
2008-11-19  2:11   ` Nish Aravamudan
2008-11-19  5:14     ` Max Krasnyansky
2008-11-19 12:30       ` Gregory Haskins
2008-11-19 16:28         ` Max Krasnyansky
2008-11-19 22:11       ` Nish Aravamudan
2008-11-19 12:51   ` Ingo Molnar
2008-11-19 16:31     ` Max Krasnyansky
2008-11-19 17:44       ` Ingo Molnar
2008-11-19 20:01         ` Max Krasnyansky [this message]

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=492470AB.5060306@qualcomm.com \
    --to=maxk@qualcomm.com \
    --cc=ghaskins@novell.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@elte.hu \
    --cc=nish.aravamudan@gmail.com \
    --cc=peterz@infradead.org \
    --cc=sivanich@sgi.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;
as well as URLs for NNTP newsgroup(s).