Messages in this thread | | | Date | Tue, 17 Nov 2015 10:56:10 +0100 | Subject | Re: [PATCHv2] arm64: Fix R/O permissions in mark_rodata_ro | From | Ard Biesheuvel <> |
| |
On 16 November 2015 at 23:50, Laura Abbott <labbott@redhat.com> wrote: > On 11/14/2015 11:38 PM, Ard Biesheuvel wrote: >> >> On 12 November 2015 at 21:21, Laura Abbott <labbott@fedoraproject.org> >> wrote: >>> >>> The permissions in mark_rodata_ro trigger a build error >>> with STRICT_MM_TYPECHECKS. Fix this by introducing >>> PAGE_KERNEL_ROX for the same reasons as PAGE_KERNEL_RO. >> >> >> Forgive the bikeshedding but perhaps PAGE_KERNEL_EXEC_RO is more >> consistent? >> > > powerpc was using PAGE_KERNEL_ROX so I went with that. Blame powerpc ;) >
OK, in that case, let's keep PAGE_KERNEL_ROX
>>> >>> From Ard: >>> >>> "PAGE_KERNEL_EXEC has PTE_WRITE set as well, making the range >>> writeable under the ARMv8.1 DBM feature, that manages the >>> dirty bit in hardware (writing to a page with the PTE_RDONLY >>> and PTE_WRITE bits both set will clear the PTE_RDONLY bit in that case)" >>> >>> Signed-off-by: Laura Abbott <labbott@fedoraproject.org> >> >> >> Acked-by: Ard Biesheuvel <ard.biesheuvel@linaro.org> >> > > Thanks, > Laura >
| |