]> rtime.felk.cvut.cz Git - lisovros/qemu_apohw.git/commit
tcg/arm: correctly save/restore registers in prologue/epilogue
authorAurelien Jarno <aurelien@aurel32.net>
Fri, 5 Mar 2010 07:35:07 +0000 (08:35 +0100)
committerAurelien Jarno <aurelien@aurel32.net>
Fri, 19 Mar 2010 22:02:37 +0000 (23:02 +0100)
commit5c6892078a2a9d3919a5b99942c9f0132e34080d
tree3d03de0af101e8932022cfe1df60ee3c123c27a6
parent18a21890ff2b24bc7f0cdc3807e2fb65e014522b
tcg/arm: correctly save/restore registers in prologue/epilogue

Since commit 6113d6d3169393c323ac4c82d756a850145a5e7a QEMU crashes
on ARM hosts. This is not a bug of this commit, but a latent bug
revealed by this commit.

The TCG code is called through a procedure call using the prologue
and epilogue code. This code does not save and restore enough registers.
The "Procedure Call Standard for the ARM Architecture" says:

  A subroutine must preserve the contents of the registers r4-r8, r10,
  r11 and SP (and r9 in PCS variants that designate r9 as v6).

The current code only saves and restores r9 to r11, and misses r4 to
r8. The patch fixes that by saving r4 to r12. Theoretically there is
no need to save and restore r12, but an even number of registers have
to be saved as per EABI.

Signed-off-by: Aurelien Jarno <aurelien@aurel32.net>
(cherry picked from commit 4e17eae9f2ee49833698aae2753c5bb041510870)
tcg/arm/tcg-target.c