]> git.kernelconcepts.de Git - karo-tx-linux.git/commitdiff
powerpc/module: Fix stubs for BE
authorRusty Russell <rusty@rustcorp.com.au>
Mon, 12 May 2014 07:45:02 +0000 (17:15 +0930)
committerBenjamin Herrenschmidt <benh@kernel.crashing.org>
Tue, 20 May 2014 00:56:01 +0000 (10:56 +1000)
A simple patch which was supposed to swap r12 and r11 also
inexplicably changed the offset by two bytes.  This instruction
(to load r2) isn't used in LE, so it wasn't noticed.

Fixes: b1ce369e82 ("powerpc: modules: use r12 for stub jump address.)
Reported-by: Alistair Popple <alistair@popple.id.au>
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Tested-by: Alistair Popple <alistair@popple.id.au>
Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
arch/powerpc/kernel/module_64.c

index ef349d077129718eba1bb16eb65be1d96193a76f..077d2ce6c5a7c64b2a51404e7e33fbb5004541b9 100644 (file)
@@ -134,7 +134,7 @@ static u32 ppc64_stub_insns[] = {
        0xe98b0020,                     /* ld      r12,32(r11) */
 #if !defined(_CALL_ELF) || _CALL_ELF != 2
        /* Set up new r2 from function descriptor */
-       0xe84b0026,                     /* ld      r2,40(r11) */
+       0xe84b0028,                     /* ld      r2,40(r11) */
 #endif
        0x7d8903a6,                     /* mtctr   r12 */
        0x4e800420                      /* bctr */