diff options
author | Catalin Marinas <catalin.marinas@arm.com> | 2009-10-12 12:31:20 -0400 |
---|---|---|
committer | Russell King <rmk+kernel@arm.linux.org.uk> | 2009-10-14 05:33:05 -0400 |
commit | a771fe6e4e3e58f2056823ef9c30a554ec48f453 (patch) | |
tree | 06f60c5b96bd4b7c1690f86b0e44d74f6e3ffcc9 /arch/arm/kernel | |
parent | 80f506918fdaaca6b574ba931536a58ce015c7be (diff) |
ARM: 5757/1: Thumb-2: Correct "mov.w pc, lr" instruction which is unpredictable
The 32-bit wide variant of "mov pc, reg" in Thumb-2 is unpredictable
causing improper handling of the undefined instructions not caught by
the kernel. This patch adds a movw_pc macro for such situations
(currently only used in call_fpe).
Signed-off-by: Catalin Marinas <catalin.marinas@arm.com>
Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
Diffstat (limited to 'arch/arm/kernel')
-rw-r--r-- | arch/arm/kernel/entry-armv.S | 28 | ||||
-rw-r--r-- | arch/arm/kernel/entry-header.S | 15 |
2 files changed, 29 insertions, 14 deletions
diff --git a/arch/arm/kernel/entry-armv.S b/arch/arm/kernel/entry-armv.S index 322410be573c..0022b4d57f8b 100644 --- a/arch/arm/kernel/entry-armv.S +++ b/arch/arm/kernel/entry-armv.S | |||
@@ -608,33 +608,33 @@ call_fpe: | |||
608 | THUMB( add pc, r8 ) | 608 | THUMB( add pc, r8 ) |
609 | nop | 609 | nop |
610 | 610 | ||
611 | W(mov) pc, lr @ CP#0 | 611 | movw_pc lr @ CP#0 |
612 | W(b) do_fpe @ CP#1 (FPE) | 612 | W(b) do_fpe @ CP#1 (FPE) |
613 | W(b) do_fpe @ CP#2 (FPE) | 613 | W(b) do_fpe @ CP#2 (FPE) |
614 | W(mov) pc, lr @ CP#3 | 614 | movw_pc lr @ CP#3 |
615 | #ifdef CONFIG_CRUNCH | 615 | #ifdef CONFIG_CRUNCH |
616 | b crunch_task_enable @ CP#4 (MaverickCrunch) | 616 | b crunch_task_enable @ CP#4 (MaverickCrunch) |
617 | b crunch_task_enable @ CP#5 (MaverickCrunch) | 617 | b crunch_task_enable @ CP#5 (MaverickCrunch) |
618 | b crunch_task_enable @ CP#6 (MaverickCrunch) | 618 | b crunch_task_enable @ CP#6 (MaverickCrunch) |
619 | #else | 619 | #else |
620 | W(mov) pc, lr @ CP#4 | 620 | movw_pc lr @ CP#4 |
621 | W(mov) pc, lr @ CP#5 | 621 | movw_pc lr @ CP#5 |
622 | W(mov) pc, lr @ CP#6 | 622 | movw_pc lr @ CP#6 |
623 | #endif | 623 | #endif |
624 | W(mov) pc, lr @ CP#7 | 624 | movw_pc lr @ CP#7 |
625 | W(mov) pc, lr @ CP#8 | 625 | movw_pc lr @ CP#8 |
626 | W(mov) pc, lr @ CP#9 | 626 | movw_pc lr @ CP#9 |
627 | #ifdef CONFIG_VFP | 627 | #ifdef CONFIG_VFP |
628 | W(b) do_vfp @ CP#10 (VFP) | 628 | W(b) do_vfp @ CP#10 (VFP) |
629 | W(b) do_vfp @ CP#11 (VFP) | 629 | W(b) do_vfp @ CP#11 (VFP) |
630 | #else | 630 | #else |
631 | W(mov) pc, lr @ CP#10 (VFP) | 631 | movw_pc lr @ CP#10 (VFP) |
632 | W(mov) pc, lr @ CP#11 (VFP) | 632 | movw_pc lr @ CP#11 (VFP) |
633 | #endif | 633 | #endif |
634 | W(mov) pc, lr @ CP#12 | 634 | movw_pc lr @ CP#12 |
635 | W(mov) pc, lr @ CP#13 | 635 | movw_pc lr @ CP#13 |
636 | W(mov) pc, lr @ CP#14 (Debug) | 636 | movw_pc lr @ CP#14 (Debug) |
637 | W(mov) pc, lr @ CP#15 (Control) | 637 | movw_pc lr @ CP#15 (Control) |
638 | 638 | ||
639 | #ifdef CONFIG_NEON | 639 | #ifdef CONFIG_NEON |
640 | .align 6 | 640 | .align 6 |
diff --git a/arch/arm/kernel/entry-header.S b/arch/arm/kernel/entry-header.S index ac34c0d9384b..7e9ed1eea40a 100644 --- a/arch/arm/kernel/entry-header.S +++ b/arch/arm/kernel/entry-header.S | |||
@@ -110,6 +110,13 @@ | |||
110 | mov \rd, sp, lsr #13 | 110 | mov \rd, sp, lsr #13 |
111 | mov \rd, \rd, lsl #13 | 111 | mov \rd, \rd, lsl #13 |
112 | .endm | 112 | .endm |
113 | |||
114 | @ | ||
115 | @ 32-bit wide "mov pc, reg" | ||
116 | @ | ||
117 | .macro movw_pc, reg | ||
118 | mov pc, \reg | ||
119 | .endm | ||
113 | #else /* CONFIG_THUMB2_KERNEL */ | 120 | #else /* CONFIG_THUMB2_KERNEL */ |
114 | .macro svc_exit, rpsr | 121 | .macro svc_exit, rpsr |
115 | clrex @ clear the exclusive monitor | 122 | clrex @ clear the exclusive monitor |
@@ -146,6 +153,14 @@ | |||
146 | lsr \rd, \rd, #13 | 153 | lsr \rd, \rd, #13 |
147 | mov \rd, \rd, lsl #13 | 154 | mov \rd, \rd, lsl #13 |
148 | .endm | 155 | .endm |
156 | |||
157 | @ | ||
158 | @ 32-bit wide "mov pc, reg" | ||
159 | @ | ||
160 | .macro movw_pc, reg | ||
161 | mov pc, \reg | ||
162 | nop | ||
163 | .endm | ||
149 | #endif /* !CONFIG_THUMB2_KERNEL */ | 164 | #endif /* !CONFIG_THUMB2_KERNEL */ |
150 | 165 | ||
151 | /* | 166 | /* |