[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20171027190446.miszsgfu5ijwjz4n@gmail.com>
Date: Fri, 27 Oct 2017 21:04:46 +0200
From: Ingo Molnar <mingo@...nel.org>
To: Frederic Weisbecker <frederic@...nel.org>
Cc: Peter Zijlstra <peterz@...radead.org>, cmetcalf@...lanox.com,
linux-kernel@...r.kernel.org, tglx@...utronix.de,
torvalds@...ux-foundation.org, hpa@...or.com, riel@...hat.com,
cl@...ux.com, efault@....de, kernellwp@...il.com,
paulmck@...ux.vnet.ibm.com, lcapitulino@...hat.com,
linux-tip-commits@...r.kernel.org
Subject: Re: [tip:sched/core] sched/isolation: Document the isolcpus= flags
* Frederic Weisbecker <frederic@...nel.org> wrote:
> 2017-10-27 20:21 UTC+02:00, Ingo Molnar <mingo@...nel.org>:
> >
> > * Frederic Weisbecker <frederic@...nel.org> wrote:
> >
> >> 2017-10-27 19:06 UTC+02:00, Ingo Molnar <mingo@...nel.org>:
> >> >
> >> > * Frederic Weisbecker <frederic@...nel.org> wrote:
> >> >
> >> >> 2017-10-27 15:58 UTC+02:00, Peter Zijlstra <peterz@...radead.org>:
> >> >> > On Fri, Oct 27, 2017 at 05:06:25AM -0700, tip-bot for Frederic
> >> >> > Weisbecker
> >> >> > wrote:
> >> >> >> + isolcpus= [KNL,SMP] Isolate a given set of CPUs from disturbance.
> >> >> >> + Format: [flag-list,]<cpu-list>
> >> >> >> +
> >> >> >> + Specify one or more CPUs to isolate from disturbances
> >> >> >> + specified in the flag list (default: domain):
> >> >> >> +
> >> >> >> + nohz
> >> >> >> + Disable the tick when a single task runs.
> >> >> >> + domain
> >> >> >> + Isolate from the general SMP balancing and scheduling
> >> >> >> + algorithms. This option is the preferred way to isolate
> >> >> >> + CPUs from tasks.
> >> >> >
> >> >> > I _strongly_ object to this statement, isolcpus is _not_ the
> >> >> > preferred
> >> >> > way, cpusets are.
> >> >> >
> >> >> > And yes, while cpusets suffers some problems, we _should_ really fix
> >> >> > those and not promote this piece of shit isolcpus crap.
> >> >>
> >> >> I definitely agree with that so your position is a relief :-) This
> >> >> patch only indented the existing parameter documentation so fixing its
> >> >> content was beyond its scope. I'll send a patch to correct the text.
> >> >
> >> > Since it was the last commit in tip:sched/core that was pushed out just
> >> > hours ago
> >> > I zapped that last commit, please send an updated patch which we can
> >> > apply
> >> > and get
> >> > a clean series.
> >> >
> >> > Thanks,
> >>
> >> Note the issue was there before that patch. But nevermind I'll resend
> >> an updated version of the patch.
> >
> > Yeah - so we get a single 'improve documentation' commit.
>
> Ah ok I see. No problem I'll resend.
Thanks!
Ingo
Powered by blists - more mailing lists