Re: [PATCH 2/2] nohz: make nohz_full imply isolcpus

[Chris Metcalf]

On 04/05/2015 01:05 AM, Ingo Molnar wrote:
> * Chris Metcalf <cmetcalf@xxxxxxxxxx> wrote:
>
> > On 4/4/2015 10:10 AM, Rik van Riel wrote:
> > > > Rik, what's the change you're proposing that's similar to this one? Thanks!
> > > I don't have this particular one, and I like it.
> > >
> > > I know there are use cases where isolcpus= without
> > > nohz_full= makes sense, but I cannot think of the
> > > reverse.
> > >
> > > Acked-by: Rik van Riel<riel@xxxxxxxxxx>
> > Thanks, I'll push it via the tile tree unless someone would prefer otherwise.
> Yes, I'd prefer otherwise: please send the final, agreed upon patch to
> the timer tree.

Not having done this before, I assume I just ask you to take patches
from LKML, or to pull from my tree?

I've set up a "timers" branch in my kernel.org git repo in any case
(git://git.kernel.org/pub/scm/linux/kernel/git/cmetcalf/linux-tile.git)
that has the commits in it.

> > (The tick_nohz_full_set_cpus() and tick_nohz_full_clear_cpus()
> > routines are in earlier tile tree commits, the latter supporting a
> > change to the tile network driver.)
> This is absolutely not OK, please push this through the timer tree. We
> don't do generic timer changes through architecture trees.

Sure; makes sense.  I will post a v2 with two patches in the patchset
and see if anyone else wants to comment in the next couple of days,
and then ask you to pull, or take the patch series, at that point.

Thanks!

--
Chris Metcalf, EZChip Semiconductor
http://www.ezchip.com

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/