Re: [-next PATCH] security: use octal not symbolic permissions

From: John Johansen
Date: Tue Jun 12 2018 - 17:29:52 EST


On 06/12/2018 02:12 PM, Paul Moore wrote:
> On Tue, Jun 12, 2018 at 4:32 PM, James Morris <jmorris@xxxxxxxxx> 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@xxxxxxxxxxxxxxxx>
>>>
>>> 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

>
> 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.