From patchwork Mon Apr 15 09:53:07 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Peter Chubb X-Patchwork-Id: 236551 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from lists.gnu.org (lists.gnu.org [208.118.235.17]) (using TLSv1 with cipher AES256-SHA (256/256 bits)) (Client did not present a certificate) by ozlabs.org (Postfix) with ESMTPS id D55762C00C3 for ; Mon, 15 Apr 2013 19:53:38 +1000 (EST) Received: from localhost ([::1]:32971 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1URg6S-0004qA-7u for incoming@patchwork.ozlabs.org; Mon, 15 Apr 2013 05:53:36 -0400 Received: from eggs.gnu.org ([208.118.235.92]:43516) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1URg6C-0004q1-3a for qemu-devel@nongnu.org; Mon, 15 Apr 2013 05:53:22 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1URg6A-0000O5-7P for qemu-devel@nongnu.org; Mon, 15 Apr 2013 05:53:19 -0400 Received: from lemon.ertos.nicta.com.au ([203.143.174.143]:47505) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1URg69-0000Nx-T2 for qemu-devel@nongnu.org; Mon, 15 Apr 2013 05:53:18 -0400 Received: from [2001:388:d000:800:224:d7ff:feaf:390] (port=49616 helo=Diprotodon.chubb.wattle.id.au) by lemon.ertos.nicta.com.au with esmtpsa (TLS1.0:DHE_RSA_AES_128_CBC_SHA1:16) (Exim 4.72) (envelope-from ) id 1URg6D-0007cn-9f; Mon, 15 Apr 2013 19:53:23 +1000 Date: Mon, 15 Apr 2013 19:53:07 +1000 Message-ID: <84mwt086ik.wl%peter@chubb.wattle.id.au> From: Peter Chubb To: Peter Maydell In-Reply-To: References: <84sj2s75yv.wl%peter@chubb.wattle.id.au> <84ppxw8d57.wl%peter@chubb.wattle.id.au> User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI/1.14.6 (Maruoka) FLIM/1.14.9 (=?UTF-8?B?R29qxY0=?=) APEL/10.8 Emacs/23.4 (x86_64-pc-linux-gnu) MULE/6.0 (HANACHIRUSATO) X-Face: GgFg(Z>fx((4\32hvXq<)|jndSniCH~~$D)Ka:P@e@JR1P%Vr}EwUdfwf-4j\rUs#JR{'h# !]])6%Jh~b$VA|ALhnpPiHu[-x~@<"@Iv&|%R)Fq[[, (&Z'O)Q)xCqe1\M[F8#9l8~}#u$S$Rm`S9% \'T@`:&8>Sb*c5d'=eDYI&GF`+t[LfDH="MP5rwOO]w>ALi7'=QJHz&y&C&TE_3j! Organization: NICTA MIME-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") X-SA-Exim-Connect-IP: 2001:388:d000:800:224:d7ff:feaf:390 X-SA-Exim-Mail-From: peter.chubb@nicta.com.au X-SA-Exim-Version: 4.2.1 (built Mon, 22 Mar 2010 06:52:44 +0000) X-SA-Exim-Scanned: Yes (on lemon.ertos.nicta.com.au) X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.6.x X-Received-From: 203.143.174.143 Cc: Japheth.Lim@nicta.com.au, Peter Chubb , qemu-devel@nongnu.org Subject: [Qemu-devel] [PATCH v2] target-arm: Reinsert SRS missing return statement. X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org Sender: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org Since patch 81465888c5306cd94abb9847e560796fd13d3c2f target-arm: factor out handling of SRS instruction the SRS instruction has not worked in QEMU. The problem is a missing return directive that was removed in the refactoring, so after decoding the instruction, qemu would do all the generic stuff that it should do for most instructions -- but not SRS. Signed-off-by: Peter Chubb Reviewed-by: Peter Maydell --- target-arm/translate.c | 1 + 1 file changed, 1 insertion(+) diff --git a/target-arm/translate.c b/target-arm/translate.c index 35a21be..a1b7b8c 100644 --- a/target-arm/translate.c +++ b/target-arm/translate.c @@ -6762,6 +6762,7 @@ static void disas_arm_insn(CPUARMState * env, DisasContext *s) } ARCH(6); gen_srs(s, (insn & 0x1f), (insn >> 23) & 3, insn & (1 << 21)); + return; } else if ((insn & 0x0e50ffe0) == 0x08100a00) { /* rfe */ int32_t offset;