From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:57420) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZRiUy-0007SF-AS for qemu-devel@nongnu.org; Tue, 18 Aug 2015 11:08:28 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZRiUt-0007PI-Rh for qemu-devel@nongnu.org; Tue, 18 Aug 2015 11:08:24 -0400 Sender: Paolo Bonzini References: <1439868498-2740-1-git-send-email-crosthwaite.peter@gmail.com> From: Paolo Bonzini Message-ID: <55D34A5A.3060109@redhat.com> Date: Tue, 18 Aug 2015 08:08:10 -0700 MIME-Version: 1.0 In-Reply-To: <1439868498-2740-1-git-send-email-crosthwaite.peter@gmail.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH] exec-all: Translate TCI return addresses backwards too List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Crosthwaite , qemu-devel@nongnu.org Cc: peter.maydell@linaro.org, rth@twiddle.net, Peter Crosthwaite , qemu-stable , sw@weilnetz.de On 17/08/2015 20:28, Peter Crosthwaite wrote: > This subtraction of return addresses applies directly to TCI as well as > host-TCG. This fixes Linux boots for at least Microblaze, CRIS, ARM and > SH4 when using TCI. > > Reviewed-by: Richard Henderson > Signed-off-by: Peter Crosthwaite > --- > Changed since v1: > Wrote commit message. > --- > include/exec/exec-all.h | 4 ---- > 1 file changed, 4 deletions(-) > > diff --git a/include/exec/exec-all.h b/include/exec/exec-all.h > index a6fce04..31c2405 100644 > --- a/include/exec/exec-all.h > +++ b/include/exec/exec-all.h > @@ -308,11 +308,7 @@ extern uintptr_t tci_tb_ptr; > to indicate the compressed mode; subtracting two works around that. It > is also the case that there are no host isas that contain a call insn > smaller than 4 bytes, so we don't worry about special-casing this. */ > -#if defined(CONFIG_TCG_INTERPRETER) > -# define GETPC_ADJ 0 > -#else > # define GETPC_ADJ 2 > -#endif > > #define GETPC() (GETRA() - GETPC_ADJ) > > Cc: qemu-stable@nongnu.org