diff options
author | Jonas Bonn <jonas@southpole.se> | 2012-04-18 12:23:39 +0200 |
---|---|---|
committer | Jonas Bonn <jonas@southpole.se> | 2012-05-08 11:43:59 +0200 |
commit | 207e715ff17b42e022df107210dc8c736a703eb3 (patch) | |
tree | 284b3b25fe53905286444ad0beacbba0663771f8 | |
parent | 7b903e6c021a5462e26ea7a8f014fa60b6782bdb (diff) | |
download | op-kernel-dev-207e715ff17b42e022df107210dc8c736a703eb3.zip op-kernel-dev-207e715ff17b42e022df107210dc8c736a703eb3.tar.gz |
openrisc: use scratch regs in atomic syscall
The function sys_or1k_atomic was using call-saved registers without
restoring their value before returning. This is a faux pas: either
we need to restore their values or use scratch regs; the latter is
less code so that's the route this patch takes.
Thanks to David Hennerström for doing most of the heavy-lifting in
tracking this one down.
Reported-by: Davd Hennerström <david.hennerstrom@aacmicrotec.com>
Signed-off-by: Jonas Bonn <jonas@southpole.se>
-rw-r--r-- | arch/openrisc/kernel/entry.S | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/arch/openrisc/kernel/entry.S b/arch/openrisc/kernel/entry.S index 6e61af8..ddfcaa8 100644 --- a/arch/openrisc/kernel/entry.S +++ b/arch/openrisc/kernel/entry.S @@ -1117,10 +1117,10 @@ ENTRY(sys_rt_sigreturn) ENTRY(sys_or1k_atomic) /* FIXME: This ignores r3 and always does an XCHG */ DISABLE_INTERRUPTS(r17,r19) - l.lwz r30,0(r4) - l.lwz r28,0(r5) - l.sw 0(r4),r28 - l.sw 0(r5),r30 + l.lwz r29,0(r4) + l.lwz r27,0(r5) + l.sw 0(r4),r27 + l.sw 0(r5),r29 ENABLE_INTERRUPTS(r17) l.jr r9 l.or r11,r0,r0 |