From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:40485) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1T4yGw-0007sI-1T for qemu-devel@nongnu.org; Fri, 24 Aug 2012 14:06:20 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1T4yGr-0007Ig-HO for qemu-devel@nongnu.org; Fri, 24 Aug 2012 14:06:17 -0400 Received: from hall.aurel32.net ([88.191.126.93]:52012) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1T4yGr-0007F1-BK for qemu-devel@nongnu.org; Fri, 24 Aug 2012 14:06:13 -0400 Date: Fri, 24 Aug 2012 20:05:57 +0200 From: Aurelien Jarno Message-ID: <20120824180557.GJ3553@ohm.aurel32.net> References: <20120824150557.GG1687@hall.aurel32.net> <5037A33D.3070907@suse.de> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-15 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <5037A33D.3070907@suse.de> Subject: Re: [Qemu-devel] [PATCH 2/5] softmmu templates: optionally pass CPUState to memory access functions List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Andreas =?iso-8859-15?Q?F=E4rber?= Cc: Blue Swirl , qemu-devel , Alexander Graf On Fri, Aug 24, 2012 at 05:52:29PM +0200, Andreas Färber wrote: > Am 24.08.2012 17:35, schrieb malc: > > On Fri, 24 Aug 2012, malc wrote: > > > >> On Fri, 24 Aug 2012, Aurelien Jarno wrote: > >> > >>> On Sun, Mar 11, 2012 at 10:24:03PM +0000, Blue Swirl wrote: > > > > [..snip..] > > > >>> - On 32 bit hosts, which usually need register alignments for 64-bit > >>> values (at least on arm and mips), given AREG0 is a 32-bit value this > >> ditto ppc32, erm.. with sysv abi that is > > ...which have been fixed in the v1.1 release cycle. You can take a look > at tcg/ppc/ for how we've fixed that with alignment macros and variable. The fix still includes one more constraint. Also the method used for PPC doesn't apply to ARM or MIPS, as they only have 4 registers to pass arguments, the remaining going on the stack. > Not opposed to changing the argument order, but given that we're inches > away from v1.2 (in Hard Freeze), it might be better to first get AREG0 > as first argument working for your favorite hosts as a bugfix and then > do any larger optimization for v1.3. It's what I tried to do first, but I don't think it is realistic to use such a code for v1.2, it is complex to support all cases, and thus likely full of bugs. Maybe we should simply disable ARM and MIPS support for this release. -- Aurelien Jarno GPG: 1024D/F1BCDB73 aurelien@aurel32.net http://www.aurel32.net