Re: [PATCH V2] cpuset: Add knob to make allowed masks hotplug invariant on legacy hierarchy

From: Serge E. Hallyn
Date: Wed Apr 15 2015 - 12:15:48 EST


On Wed, Apr 15, 2015 at 11:48:02AM -0400, Tejun Heo wrote:
> On Wed, Apr 15, 2015 at 10:30:35AM -0500, Serge E. Hallyn wrote:
> > On Wed, Apr 15, 2015 at 05:19:26PM +0200, Peter Zijlstra wrote:
> > > On Wed, Apr 15, 2015 at 10:03:02AM -0500, Serge E. Hallyn wrote:
> > >
> > > > Peter, is the question "why can't we just use the unified hierarchy for
> > > > cpusets"?
> > >
> > > No, the question is why can't you use the unified hierarchy cpuset
> > > semantics -- without the actual unified hierarchy stuff.
> >
> > Ok - that I don't know the answer to. Does sound like it would
> > simplify things, reduce # of sets of semantics.
>
> What'd be the reason for not using the unified hierarchy tho? The

The reason would be because it breaks "legacy" software. So that
would only matter if Preeti needs to run such software.

A particularly pernicious example would be needing to run nested
containers, with an older (stable release) container wanting to run
containers inside itself. The host may have all-new software
capable of working with unifiied hierarchy, but the stable release
container will not. (Honestly I don't yet know what we're going to
do about that, but it's going to be great fun for the next few months)

> adaptable required isn't that big and the patchset already proposes
> significant amount of behavior change. Sure it'd be a bit more hassle
> but does that really justify introducing yet another operation mode?
> This is why unified hierarchy is being added in the first place. One
> can argue "but *I* just need this specific part changed" but allowing
> combinations of all those little variations is gonna lead us to a
> hellish place.

Agreed, these things quickly get out of hand. This patch looks
straightforward enough in itself, though and the question is, is this
a special enough case for an exception.

-serge
--
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/