From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:39148) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1SxQYW-00082B-Eb for qemu-devel@nongnu.org; Fri, 03 Aug 2012 18:41:17 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1SxQYV-000422-69 for qemu-devel@nongnu.org; Fri, 03 Aug 2012 18:41:16 -0400 Received: from mail-pb0-f45.google.com ([209.85.160.45]:58948) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1SxQYU-0003zU-85 for qemu-devel@nongnu.org; Fri, 03 Aug 2012 18:41:15 -0400 Received: by mail-pb0-f45.google.com with SMTP id ro12so1909554pbb.4 for ; Fri, 03 Aug 2012 15:41:13 -0700 (PDT) Sender: Richard Henderson From: Richard Henderson Date: Fri, 3 Aug 2012 15:40:55 -0700 Message-Id: <1344033657-9135-9-git-send-email-rth@twiddle.net> In-Reply-To: <1344033657-9135-1-git-send-email-rth@twiddle.net> References: <1344033657-9135-1-git-send-email-rth@twiddle.net> Subject: [Qemu-devel] [PATCH 08/10] alpha-linux-user: Fix a3 error return with v0 error bypass. List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org We were failing to initialize a3 for syscalls that bypass the negative return value error check. Signed-off-by: Richard Henderson --- linux-user/main.c | 12 +++++------- 1 files changed, 5 insertions(+), 7 deletions(-) diff --git a/linux-user/main.c b/linux-user/main.c index a21942c..53714de 100644 --- a/linux-user/main.c +++ b/linux-user/main.c @@ -2846,13 +2846,11 @@ void cpu_loop(CPUAlphaState *env) break; } /* Syscall writes 0 to V0 to bypass error check, similar - to how this is handled internal to Linux kernel. */ - if (env->ir[IR_V0] == 0) { - env->ir[IR_V0] = sysret; - } else { - env->ir[IR_V0] = (sysret < 0 ? -sysret : sysret); - env->ir[IR_A3] = (sysret < 0); - } + to how this is handled internal to Linux kernel. + (Ab)use trapnr temporarily as boolean indicating error. */ + trapnr = (env->ir[IR_V0] != 0 && sysret < 0); + env->ir[IR_V0] = (trapnr ? -sysret : sysret); + env->ir[IR_A3] = trapnr; break; case 0x86: /* IMB */ -- 1.7.7.6