From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:47457) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UsZt2-00019Y-ML for qemu-devel@nongnu.org; Fri, 28 Jun 2013 10:43:02 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UsZt1-0002bU-9z for qemu-devel@nongnu.org; Fri, 28 Jun 2013 10:42:56 -0400 Received: from cantor2.suse.de ([195.135.220.15]:41441 helo=mx2.suse.de) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UsZt1-0002bJ-3U for qemu-devel@nongnu.org; Fri, 28 Jun 2013 10:42:55 -0400 Message-ID: <51CDA0EC.6090508@suse.de> Date: Fri, 28 Jun 2013 16:42:52 +0200 From: =?UTF-8?B?QW5kcmVhcyBGw6RyYmVy?= MIME-Version: 1.0 References: <1367303766-9526-1-git-send-email-john.rigby@linaro.org> <517FAE24.1040000@suse.de> <517FB25F.7070501@suse.de> <51CD9D9D.2020207@suse.de> In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH v2 01/12] ARM: Export cpu_env List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Maydell Cc: Peter Crosthwaite , "Edgar E. Iglesias" , John Rigby , Alexander Graf , qemu-devel Am 28.06.2013 16:35, schrieb Peter Maydell: > On 28 June 2013 15:28, Andreas F=C3=A4rber wrote: >> I don't mind that cpu_env change getting committed as interim solution= , >> so far I did not come up with a better patch - we'd need to split out >> host parts from tcg/tcg.h first, for which I did not find time yet. >=20 > Interim solution on the path to where? I'm not convinced the > cpu_env variables should be visible outside each individual > decoder (any more than, for instance, the ARM cpu_V0, cpu_V1 > variables are). Admittedly the environment pointer is a bit > of a special case, but perhaps we should deal with it by making > it less of one? Well, I'm not so fond of the idea of having two static variables for the same thing. If that is just to shield the global symbols, we could rename the exported variable to arm_cpu_env and do #define cpu_env arm_cpu_env to avoid a large-scale renaming. I was thinking that the only thing this duplicated TCGv_ptr cpu_env depends on is host pointer size, TCG debug configuration (struct or integer) and host environment pointer register. So why duplicate it longterm rather than having one that can be shared by multiple targets to derive the target-specific register TCGv* variables? Cheers, Andreas --=20 SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 N=C3=BCrnberg, Germany GF: Jeff Hawn, Jennifer Guild, Felix Imend=C3=B6rffer; HRB 16746 AG N=C3=BC= rnberg