From: "Siddha, Suresh B" <suresh.b.siddha@intel.com>
To: Paul Jackson <pj@sgi.com>
Cc: "Siddha, Suresh B" <suresh.b.siddha@intel.com>,
dino@in.ibm.com, menage@google.com, Simon.Derr@bull.net,
linux-kernel@vger.kernel.org, mbligh@google.com,
rohitseth@google.com, dipankar@in.ibm.com,
nickpiggin@yahoo.com.au
Subject: Re: [RFC] Cpuset: explicit dynamic sched domain control flags
Date: Wed, 18 Oct 2006 10:50:35 -0700 [thread overview]
Message-ID: <20061018105035.B26521@unix-os.sc.intel.com> (raw)
In-Reply-To: <20061018000512.1d13aabd.pj@sgi.com>; from pj@sgi.com on Wed, Oct 18, 2006 at 12:05:12AM -0700
On Wed, Oct 18, 2006 at 12:05:12AM -0700, Paul Jackson wrote:
> Suresh wrote:
> > > What happens then is that the job manager marks the cpuset of this
> > > newly activated job as being a sched_domain.
> >
> > With your patch, that will fail because there is already a cpuset defining
> > a sched domain and which overlaps with the one that is becoming active.
>
> No, this does not fail. The job manager also turned off the other cpusets
> sched_domain flag, because that job went inactive.
>
> The job manager does not run active jobs on overlapping CPUs. It marks
> the cpusets of only the active jobs as sched domains, and turns off the
> sched_domain flag on the cpusets of the inactive jobs.
So why can't the job manager remove the cpus from inactive cpusets and add
it to the cpuset that job manager is interested in.
Not sure what additional functionality your are adding here.
> > In this case, shouldn't we remove cpus0-3 from "cs1" cpus_allowed?
>
> No, you missed my point. The point of my example wasn't to show how to
> do this right. It was to show that even the existing cpu_exclusive
> flag lets one create sched domains configurations that might not be
> what one wanted.
I agree with your point. Lets take this out of discussion for now.
thanks,
suresh
next prev parent reply other threads:[~2006-10-18 18:11 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-10-16 23:03 [RFC] Cpuset: explicit dynamic sched domain control flags Paul Jackson
2006-10-17 18:43 ` Siddha, Suresh B
2006-10-17 19:18 ` Paul Jackson
2006-10-18 2:01 ` Siddha, Suresh B
2006-10-18 7:05 ` Paul Jackson
2006-10-18 17:50 ` Siddha, Suresh B [this message]
2006-10-19 6:30 ` Paul Jackson
2006-10-19 6:39 ` Nick Piggin
2006-10-19 7:03 ` Paul Jackson
2006-10-19 8:09 ` Nick Piggin
2006-10-19 8:15 ` Paul Jackson
2006-10-19 8:18 ` Paul Jackson
2006-10-18 17:49 ` Dinakar Guniguntala
2006-10-19 6:00 ` Paul Jackson
2006-10-19 6:28 ` Paul Jackson
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=20061018105035.B26521@unix-os.sc.intel.com \
--to=suresh.b.siddha@intel.com \
--cc=Simon.Derr@bull.net \
--cc=dino@in.ibm.com \
--cc=dipankar@in.ibm.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mbligh@google.com \
--cc=menage@google.com \
--cc=nickpiggin@yahoo.com.au \
--cc=pj@sgi.com \
--cc=rohitseth@google.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