Thread (12 messages) flat view 12 messages, 5 authors, 2017-08-09

Re: [PATCH 3/3] powerpc/mm: Mark __init memory no-execute when STRICT_KERNEL_RWX=y

From: Michael Ellerman <mpe@ellerman.id.au>
Date: 2017-08-09 13:13:43

Christophe LEROY [off-list ref] writes:
...
At least it is correct for the ones that use regular pages, and kernel 
can also be started with nobats or noltlbs at command line, in which 
case it is usefull to have the page tables correct.
Yep OK.
quoted
So yes we *should* always mark it no-execute but in practice we can't
because it's not page aligned.
On 32 bit it seems to always be aligned to the normal page size, so no 
problem.
quoted
But if that's different on (some?) 32-bit then we could introduce a new
CONFIG symbol that is enabled in the right cases.
For the time being, I have added an "|| CONFIG_PPC32 " on the ifdef, is 
that OK ?
See https://patchwork.ozlabs.org/patch/796625/
Yeah looks fine.

cheers
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help