[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <1528839361.3874.10.camel@linux.vnet.ibm.com>
Date: Tue, 12 Jun 2018 17:36:01 -0400
From: Mimi Zohar <zohar@...ux.vnet.ibm.com>
To: John Johansen <john.johansen@...onical.com>,
Paul Moore <paul@...l-moore.com>,
James Morris <jmorris@...ei.org>, Joe Perches <joe@...ches.com>
Cc: Casey Schaufler <casey@...aufler-ca.com>,
Dmitry Kasatkin <dmitry.kasatkin@...il.com>,
Stephen Smalley <sds@...ho.nsa.gov>,
Eric Paris <eparis@...isplace.org>,
Kentaro Takeda <takedakn@...data.co.jp>,
Tetsuo Handa <penguin-kernel@...ove.sakura.ne.jp>,
"Serge E. Hallyn" <serge@...lyn.com>,
linux-security-module@...r.kernel.org,
linux-kernel@...r.kernel.org, linux-integrity@...r.kernel.org,
selinux@...ho.nsa.gov
Subject: Re: [-next PATCH] security: use octal not symbolic permissions
On Tue, 2018-06-12 at 14:29 -0700, John Johansen wrote:
> On 06/12/2018 02:12 PM, Paul Moore wrote:
> > On Tue, Jun 12, 2018 at 4:32 PM, James Morris <jmorris@...ei.org> wrote:
> >> On Mon, 11 Jun 2018, Casey Schaufler wrote:
> >>
> >>> If you want to break this up by security module I would take
> >>> the Smack part as soon as James does the tree update. If James
> >>> wants to take the whole thing at once you can add my:
> >>>
> >>> Acked-by: Casey Schaufler <casey@...aufler-ca.com>
> >>>
> >>> for the Smack changes.
> >>
> >> It's probably simplest for me to take them as one patch.
> >
> > I would prefer if the SELinux changes were split into a separate
> > patch. I'm guessing John would probably want the same for the
> > AppArmor patches, but take his work for it, not mine.
>
> yes that would be preferred
Agreed
>
> >
> > Joe, in general I really appreciate the fixes you send, but these
> > patches that cross a lot of subsystem boundaries (this isn't the first
> > one that does this) causes unnecessary conflicts in -next and during
> > the merge window. Could you split your patches up from now on please?
> >
>
> yeah splitting patches at subsystem boundaries is highly recommended.
Agreed
Powered by blists - more mailing lists