diff options
| author |   Christophe Leroy <christophe.leroy@c-s.fr> | 2019-08-23 09:56:21 +0000 |
|---|---|---|
| committer |   Michael Ellerman <mpe@ellerman.id.au> | 2019-11-18 22:27:52 +1100 |
| commit | a2227a27774328507a5c2335a6dd600c079d1ff5 (patch) | |
| tree | 0dc5d7e43238c3fe65d26bcef49a37cd8b5f68e5 /arch/powerpc/mm/pgtable_32.c | |
| parent | powerpc/mm: Show if a bad page fault on data is read or write. (diff) | |
| download | linux-dev-a2227a27774328507a5c2335a6dd600c079d1ff5.tar.xz linux-dev-a2227a27774328507a5c2335a6dd600c079d1ff5.zip | |
powerpc/32: Don't populate page tables for block mapped pages except on the 8xx.
Commit d2f15e0979ee ("powerpc/32: always populate page tables for
Abatron BDI.") wrongly sets page tables for any PPC32 for using BDI,
and does't update them after init (remove RX on init section, set
text and rodata read-only)
Only the 8xx requires page tables to be populated for using the BDI.
They also need to be populated in order to see the mappings in
/sys/kernel/debug/kernel_page_tables
On BOOK3S_32, pages that are not mapped by page tables are mapped
by BATs. The BDI knows BATs and they can be viewed in
/sys/kernel/debug/powerpc/block_address_translation
Only set pagetables for RAM and IMMR on the 8xx and properly update
them at the end of init.
Signed-off-by: Christophe Leroy <christophe.leroy@c-s.fr>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Link: https://lore.kernel.org/r/c8610942203e0d93fcb02ad20c57edd3adb4c9d3.1566554029.git.christophe.leroy@c-s.fr
Diffstat (limited to '')
| -rw-r--r-- | arch/powerpc/mm/pgtable_32.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/arch/powerpc/mm/pgtable_32.c b/arch/powerpc/mm/pgtable_32.c index 8ec5dfb65b2e..73b84166d06a 100644 --- a/arch/powerpc/mm/pgtable_32.c +++ b/arch/powerpc/mm/pgtable_32.c @@ -117,10 +117,7 @@ void __init mapin_ram(void) if (base >= top) continue; base = mmu_mapin_ram(base, top); - if (IS_ENABLED(CONFIG_BDI_SWITCH)) - __mapin_ram_chunk(reg->base, top); - else - __mapin_ram_chunk(base, top); + __mapin_ram_chunk(base, top); } } |