diff options
| author | Nicholas Piggin <npiggin@gmail.com> | 2019-07-24 21:24:40 +1000 |
|---|---|---|
| committer | Oliver O'Halloran <oohall@gmail.com> | 2019-07-26 15:31:41 +1000 |
| commit | 4cf05f35c9158341606e170c4a4c7a0835d7b41b (patch) | |
| tree | 9e017b91afd08b0acf20206ba8bed73785a53311 /asm | |
| parent | 6017ba6a5a7eb7dface68d29ab7372320615ad47 (diff) | |
| download | skiboot-4cf05f35c9158341606e170c4a4c7a0835d7b41b.zip skiboot-4cf05f35c9158341606e170c4a4c7a0835d7b41b.tar.gz skiboot-4cf05f35c9158341606e170c4a4c7a0835d7b41b.tar.bz2 | |
asm/head: set skiboot TOC in interrupt handler entry
Some code runs with r2 not set to the skiboot TOC, for example the
secureboot CVC (see call_rom_entry). If a system reset or machine
check is taken at this time, the skiboot interrupt handler will
crash badly rather than report it.
So set the skiboot TOC into r2 in the interrupt entry code. r2 is
already saved and restored, so in the case of recoverable exceptions,
this will restore the correct r2 when returning to such code.
This issue was found by Stewart, and this patch is based on his
initial fix, with some modification.
Cc: Stewart Smith <stewart@linux.ibm.com>
Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
Signed-off-by: Oliver O'Halloran <oohall@gmail.com>
Diffstat (limited to 'asm')
| -rw-r--r-- | asm/head.S | 8 |
1 files changed, 5 insertions, 3 deletions
@@ -226,10 +226,12 @@ _exception: stw %r4,STACK_XER(%r1) std %r5,STACK_CTR(%r1) std %r6,STACK_LR(%r1) - mr %r3,%r1 LOAD_IMM64(%r4, SKIBOOT_BASE) - LOAD_IMM32(%r5, exception_entry_foo - __head) - add %r4,%r4,%r5 + LOAD_IMM32(%r5,__toc_start - __head) + LOAD_IMM32(%r6, exception_entry_foo - __head) + add %r2,%r4,%r5 + mr %r3,%r1 + add %r4,%r4,%r6 mtctr %r4 bctr exception_entry_foo: |