aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGabor Juhos <juhosg@openwrt.org>2016-01-17 15:29:03 +0000
committerGabor Juhos <juhosg@openwrt.org>2016-01-17 15:29:03 +0000
commit55b0e261a41bb6b27c2df8669c3708e8c1ab70a4 (patch)
tree8ba0dc22bd9e6ffe29a37a5a511ba525c43dee79
parentc3b01ead447d89f106492f5a97bea6560548b8fd (diff)
downloadupstream-55b0e261a41bb6b27c2df8669c3708e8c1ab70a4.tar.gz
upstream-55b0e261a41bb6b27c2df8669c3708e8c1ab70a4.tar.bz2
upstream-55b0e261a41bb6b27c2df8669c3708e8c1ab70a4.zip
ar71xx/lzma-loader: fix O32 ABI conformance
According to the calling convention of the o32 ABI the caller function must reserve stack space for $a0-$a3 registers in case the callee needs to save its arguments. The assembly code of the loader does not reserve stack space for these registers thus when the 'loader_main' function needs to save its arguments, those will be stored in the 'workspace' area instead of the stack. Because the workspace area is also used by other part of the code, the saved register values gets overwritten and this often leads to failed kernel boots. Fix the code to reserve stack space for the registers to avoid this error. Signed-off-by: Gabor Juhos <juhosg@openwrt.org> SVN-Revision: 48279
-rw-r--r--target/linux/ar71xx/image/lzma-loader/src/head.S3
1 files changed, 3 insertions, 0 deletions
diff --git a/target/linux/ar71xx/image/lzma-loader/src/head.S b/target/linux/ar71xx/image/lzma-loader/src/head.S
index 543996a0da..47a7c9bd63 100644
--- a/target/linux/ar71xx/image/lzma-loader/src/head.S
+++ b/target/linux/ar71xx/image/lzma-loader/src/head.S
@@ -109,6 +109,9 @@ __bss_check:
/* Setup new "C" stack */
la sp, _stack
+ /* reserve stack space for a0-a3 registers */
+ subu sp, 16
+
/* jump to the decompressor routine */
la t0, loader_main
jr t0