Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1480198

Improper TTBCR for arm 32bit kernel decompression

Show all headers | View raw

Hello,

While trying to boot arm-32 bit kernel, I came across a problem where 
TTBCR is in improper state. If the bootloader uses the long descriptor 
format and jumps to kernel decompressor code, TTBCR may not be in the 
right state. So, as soon as the MMU is enabled, execution can not 
proceed further.

Before enabling the MMU, it is required to clear the TTBCR.PD0 field to 
use TTBR0 for translation table walks. Also, TTBCR.N should be reset to 
'0' to indicate the correct base address width. The 'commit 
dbece45894d3a ("ARM: 7501/1: decompressor: reset ttbcr for VMSA ARMv7 
cores")' does the reset of TTBCR.N, but doesn't consider all the bits 
for the size of TTBCR.N.

when i tried the below change where i explicitly clear TTBCR.PD0 and use 
correct mask for TTBCR.N, I see proper memory after MMU is enabled and 
decompression succeeds.

Request your comments on the change below. If it looks good, I can 
submit a patch for inclusion.

---------------------8<----------------------------------
diff --git a/arch/arm/boot/compressed/head.S 
b/arch/arm/boot/compressed/head.S
index af11c2f..5769f1f 100644
--- a/arch/arm/boot/compressed/head.S
+++ b/arch/arm/boot/compressed/head.S
@@ -779,7 +779,8 @@ __armv7_mmu_cache_on:
                 orrne   r0, r0, #1              @ MMU enabled
                 movne   r1, #0xfffffffd         @ domain 0 = client
                 bic     r6, r6, #1 << 31        @ 32-bit translation system
-               bic     r6, r6, #3 << 0         @ use only ttbr0
+               bic     r6, r6, #7 << 0         @ width of base address 
field
+               bic     r6, r6, #1 << 4         @ use only ttbr0
                 mcrne   p15, 0, r3, c2, c0, 0   @ load page table pointer
                 mcrne   p15, 0, r1, c3, c0, 0   @ load domain access 
control
                 mcrne   p15, 0, r6, c2, c0, 2   @ load ttb control
---------------------8<----------------------------------

Thanks,
-- Srinivas R

-- 
Qualcomm India Private Limited, on behalf of Qualcomm Innovation Center, 
Inc., is a member of Code Aurora Forum, a Linux Foundation Collaborative 
Project.

Back to linux.kernel | Previous | Next — Next in thread | Find similar | Unroll thread

Thread

Improper TTBCR for arm 32bit kernel decompression Srinivas Ramana <sramana@codeaurora.org> - 2016-09-09 18:40 +0200
  Re: Improper TTBCR for arm 32bit kernel decompression Nicolas Pitre <nicolas.pitre@linaro.org> - 2016-09-09 19:40 +0200
    Re: Improper TTBCR for arm 32bit kernel decompression Srinivas Ramana <sramana@codeaurora.org> - 2016-09-10 08:00 +0200
    Re: Improper TTBCR for arm 32bit kernel decompression Srinivas Ramana <sramana@codeaurora.org> - 2016-09-10 10:20 +0200
      [PATCH] ARM: decompressor: reset ttbcr fields to use TTBR0 on ARMv7 Srinivas Ramana <sramana@codeaurora.org> - 2016-09-12 09:00 +0200
        Re: [PATCH] ARM: decompressor: reset ttbcr fields to use TTBR0 on  ARMv7 Russell King - ARM Linux <linux@armlinux.org.uk> - 2016-09-12 12:30 +0200
        Re: [PATCH] ARM: decompressor: reset ttbcr fields to use TTBR0 on  ARMv7 Robin Murphy <robin.murphy@arm.com> - 2016-09-12 20:00 +0200
          Re: [PATCH] ARM: decompressor: reset ttbcr fields to use TTBR0 on  ARMv7 Srinivas Ramana <sramana@codeaurora.org> - 2016-09-13 17:00 +0200

csiph-web