Re: [PATCH 4/9] x86, pkeys: override pkey when moving away from PROT_EXEC

From: Thomas Gleixner
Date: Fri Mar 23 2018 - 15:38:22 EST


On Fri, 23 Mar 2018, Dave Hansen wrote:

> On 03/23/2018 12:15 PM, Shakeel Butt wrote:
> >> We had a check for PROT_READ/WRITE, but it did not work
> >> for PROT_NONE. This entirely removes the PROT_* checks,
> >> which ensures that PROT_NONE now works.
> >>
> >> Reported-by: Shakeel Butt <shakeelb@xxxxxxxxxx>
> >> Signed-off-by: Dave Hansen <dave.hansen@xxxxxxxxxxxxxxx>
> > Should there be a 'Fixes' tag? Also should this patch go to stable?
>
> There could be, but I'm to lazy to dig up the original commit. Does it
> matter?
>
> And, yes, I think it probably makes sense for -stable. I'll add that if
> I resend this series.

The fixes tag makes sense in general even if the patch is not tagged for
stable. It gives you immediate context and I use it a lot to look why this
went unnoticed or what the context of that change was.

Thanks,

tglx