diff options
| author | Jonas Bonn <jonas@southpole.se> | 2012-03-02 10:05:24 +0100 |
|---|---|---|
| committer | Jonas Bonn <jonas@southpole.se> | 2012-03-06 10:37:00 +0100 |
| commit | 6cbe5e95267449ea0b79c0b049342409949da3ac (patch) | |
| tree | 21243a6c899b27f67338d77a28ccbe0f473272bc /arch/openrisc/include/asm/syscall.h | |
| parent | openrisc: fix virt_addr_valid (diff) | |
| download | linux-6cbe5e95267449ea0b79c0b049342409949da3ac.tar.xz linux-6cbe5e95267449ea0b79c0b049342409949da3ac.zip | |
openrisc: sanitize use of orig_gpr11
The pt_regs struct had both a 'syscallno' field and an 'orig_gpr11' field
and it wasn't really clear how these were supposed to be used. This patch
removes the syscallno field altogether and makes orig_gpr11 work more
like other architectures: keep track of syscall number in progress or
hold -1 for non-syscall exceptions.
Signed-off-by: Jonas Bonn <jonas@southpole.se>
Diffstat (limited to 'arch/openrisc/include/asm/syscall.h')
| -rw-r--r-- | arch/openrisc/include/asm/syscall.h | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/arch/openrisc/include/asm/syscall.h b/arch/openrisc/include/asm/syscall.h index 9f0337055d26..b752bb67891d 100644 --- a/arch/openrisc/include/asm/syscall.h +++ b/arch/openrisc/include/asm/syscall.h @@ -25,7 +25,7 @@ static inline int syscall_get_nr(struct task_struct *task, struct pt_regs *regs) { - return regs->syscallno ? regs->syscallno : -1; + return regs->orig_gpr11; } static inline void @@ -50,10 +50,7 @@ static inline void syscall_set_return_value(struct task_struct *task, struct pt_regs *regs, int error, long val) { - if (error) - regs->gpr[11] = -error; - else - regs->gpr[11] = val; + regs->gpr[11] = (long) error ?: val; } static inline void |