Commit c21f5a9e authored by Christophe Leroy's avatar Christophe Leroy Committed by Michael Ellerman

powerpc/32s: fix booting with CONFIG_PPC_EARLY_DEBUG_BOOTX

When booting through OF, setup_disp_bat() does nothing because
disp_BAT are not set. By change, it used to work because BOOTX
buffer is mapped 1:1 at address 0x81000000 by the bootloader, and
btext_setup_display() sets virt addr same as phys addr.

But since commit 215b8237 ("powerpc/32s: set up an early static
hash table for KASAN."), a temporary page table overrides the
bootloader mapping.

This 0x81000000 is also problematic with the newly implemented
Kernel Userspace Access Protection (KUAP) because it is within user
address space.

This patch fixes those issues by properly setting disp_BAT through
a call to btext_prepare_BAT(), allowing setup_disp_bat() to
properly setup BAT3 for early bootx screen buffer access.
Reported-by: default avatarMathieu Malaterre <malat@debian.org>
Fixes: 215b8237 ("powerpc/32s: set up an early static hash table for KASAN.")
Signed-off-by: default avatarChristophe Leroy <christophe.leroy@c-s.fr>
Tested-by: default avatarMathieu Malaterre <malat@debian.org>
Signed-off-by: default avatarMichael Ellerman <mpe@ellerman.id.au>
parent a00196a2
...@@ -13,7 +13,11 @@ extern void btext_update_display(unsigned long phys, int width, int height, ...@@ -13,7 +13,11 @@ extern void btext_update_display(unsigned long phys, int width, int height,
int depth, int pitch); int depth, int pitch);
extern void btext_setup_display(int width, int height, int depth, int pitch, extern void btext_setup_display(int width, int height, int depth, int pitch,
unsigned long address); unsigned long address);
#ifdef CONFIG_PPC32
extern void btext_prepare_BAT(void); extern void btext_prepare_BAT(void);
#else
static inline void btext_prepare_BAT(void) { }
#endif
extern void btext_map(void); extern void btext_map(void);
extern void btext_unmap(void); extern void btext_unmap(void);
......
...@@ -2340,6 +2340,7 @@ static void __init prom_check_displays(void) ...@@ -2340,6 +2340,7 @@ static void __init prom_check_displays(void)
prom_printf("W=%d H=%d LB=%d addr=0x%x\n", prom_printf("W=%d H=%d LB=%d addr=0x%x\n",
width, height, pitch, addr); width, height, pitch, addr);
btext_setup_display(width, height, 8, pitch, addr); btext_setup_display(width, height, 8, pitch, addr);
btext_prepare_BAT();
} }
#endif /* CONFIG_PPC_EARLY_DEBUG_BOOTX */ #endif /* CONFIG_PPC_EARLY_DEBUG_BOOTX */
} }
......
...@@ -27,7 +27,7 @@ fi ...@@ -27,7 +27,7 @@ fi
WHITELIST="add_reloc_offset __bss_start __bss_stop copy_and_flush WHITELIST="add_reloc_offset __bss_start __bss_stop copy_and_flush
_end enter_prom $MEM_FUNCS reloc_offset __secondary_hold _end enter_prom $MEM_FUNCS reloc_offset __secondary_hold
__secondary_hold_acknowledge __secondary_hold_spinloop __start __secondary_hold_acknowledge __secondary_hold_spinloop __start
logo_linux_clut224 logo_linux_clut224 btext_prepare_BAT
reloc_got2 kernstart_addr memstart_addr linux_banner _stext reloc_got2 kernstart_addr memstart_addr linux_banner _stext
__prom_init_toc_start __prom_init_toc_end btext_setup_display TOC." __prom_init_toc_start __prom_init_toc_end btext_setup_display TOC."
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment