[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 05/10] mips-linux-user: Fix n32 and n64 syscalls
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH 05/10] mips-linux-user: Fix n32 and n64 syscalls |
Date: |
Mon, 11 Feb 2013 17:33:55 +0000 |
On 10 February 2013 18:30, Richard Henderson <address@hidden> wrote:
> Signed-off-by: Richard Henderson <address@hidden>
Reviewed-by: Peter Maydell <address@hidden>
> - if ((unsigned int)ret >= (unsigned int)(-1133)) {
> + if ((abi_ulong)ret >= (abi_ulong)-1133) {
> env->active_tc.gpr[7] = 1; /* error flag */
> ret = -ret;
> } else {
You're only changing the cast here, so this isn't something you
should feel the need to fix in this patch, but do you know what's
special about -1133 ?
-- PMM
- Re: [Qemu-devel] [PATCH 01/10] mips-linux-user: Delete n32 and n64 signal stubs, (continued)
- [Qemu-devel] [PATCH 02/10] mips-linux-user: Share o32 code for n32 and n64 signals, Richard Henderson, 2013/02/10
- [Qemu-devel] [PATCH 03/10] mips-linux-user: Enable mips64 and mipsn32 linux-user targets, Richard Henderson, 2013/02/10
- [Qemu-devel] [PATCH 06/10] mips64-linux-user: Enable 64-bit address mode and fpu, Richard Henderson, 2013/02/10
- [Qemu-devel] [PATCH 05/10] mips-linux-user: Fix n32 and n64 syscalls, Richard Henderson, 2013/02/10
- [Qemu-devel] [PATCH 04/10] mips-linux-user: Save and restore fpu and dsp from sigcontext, Richard Henderson, 2013/02/10
- [Qemu-devel] [PATCH 08/10] target-mips: Fix check_cp1_64bitmode, Richard Henderson, 2013/02/10
- [Qemu-devel] [PATCH 07/10] mips64-linux-user: Use MIPS64R2-generic as default cpu., Richard Henderson, 2013/02/10