riscv: Fix set_direct_map_default_noflush() to reset _PAGE_EXEC
authorAlexandre Ghiti <alexghiti@rivosinc.com>
Wed, 13 Dec 2023 13:40:27 +0000 (14:40 +0100)
committerPalmer Dabbelt <palmer@rivosinc.com>
Tue, 9 Jan 2024 18:59:08 +0000 (10:59 -0800)
When resetting the linear mapping permissions, we must make sure that we
clear the X bit so that do not end up with WX mappings (since we set
PAGE_KERNEL).

Fixes: 395a21ff859c ("riscv: add ARCH_HAS_SET_DIRECT_MAP support")
Signed-off-by: Alexandre Ghiti <alexghiti@rivosinc.com>
Link: https://lore.kernel.org/r/20231213134027.155327-3-alexghiti@rivosinc.com
Signed-off-by: Palmer Dabbelt <palmer@rivosinc.com>
arch/riscv/mm/pageattr.c

index 96cbda683936bda1c5aac11d3d3057dd206bc6e6..01398fee5cf8281b5c6df39d3f6adb113a7436f0 100644 (file)
@@ -383,7 +383,7 @@ int set_direct_map_invalid_noflush(struct page *page)
 int set_direct_map_default_noflush(struct page *page)
 {
        return __set_memory((unsigned long)page_address(page), 1,
-                           PAGE_KERNEL, __pgprot(0));
+                           PAGE_KERNEL, __pgprot(_PAGE_EXEC));
 }
 
 #ifdef CONFIG_DEBUG_PAGEALLOC