Skip to content

Commit 7e66cbc

Browse files
Konstantin KhlebnikovRussell King
authored andcommitted
ARM: 8132/1: LPAE: drop wrong carry flag correction after adding TTBR1_OFFSET
ARM: LPAE: drop wrong carry flag correction after adding TTBR1_OFFSET In commit 7fb00c2 ("ARM: 8114/1: LPAE: load upper bits of early TTBR0/TTBR1") part which fixes carrying in adding TTBR1_OFFSET to TTRR1 was wrong: addls ttbr1, ttbr1, #TTBR1_OFFSET adcls tmp, tmp, #0 addls doesn't update flags, adcls adds carry from cmp above: cmp ttbr1, tmp @ PHYS_OFFSET > PAGE_OFFSET? Condition 'ls' means carry flag is clear or zero flag is set, thus only one case is affected: when PHYS_OFFSET == PAGE_OFFSET. It seems safer to remove this fixup. Bug is here for ages and nobody complained. Let's fix it separately. Reported-and-Tested-by: Jassi Brar <[email protected]> Signed-off-by: Konstantin Khlebnikov <[email protected]> Signed-off-by: Russell King <[email protected]>
1 parent eba1c71 commit 7e66cbc

File tree

1 file changed

+0
-1
lines changed

1 file changed

+0
-1
lines changed

arch/arm/mm/proc-v7-3level.S

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -146,7 +146,6 @@ ENDPROC(cpu_v7_set_pte_ext)
146146
mov \tmp, \ttbr1, lsr #(32 - ARCH_PGD_SHIFT) @ upper bits
147147
mov \ttbr1, \ttbr1, lsl #ARCH_PGD_SHIFT @ lower bits
148148
addls \ttbr1, \ttbr1, #TTBR1_OFFSET
149-
adcls \tmp, \tmp, #0
150149
mcrr p15, 1, \ttbr1, \tmp, c2 @ load TTBR1
151150
mov \tmp, \ttbr0, lsr #(32 - ARCH_PGD_SHIFT) @ upper bits
152151
mov \ttbr0, \ttbr0, lsl #ARCH_PGD_SHIFT @ lower bits

0 commit comments

Comments
 (0)