From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:52870) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UKZtH-0004R2-6H for qemu-devel@nongnu.org; Tue, 26 Mar 2013 15:50:40 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UKZtF-0007MX-3y for qemu-devel@nongnu.org; Tue, 26 Mar 2013 15:50:39 -0400 Received: from hall.aurel32.net ([2001:470:1f15:c4f::1]:49475) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UKZtE-0007MO-U2 for qemu-devel@nongnu.org; Tue, 26 Mar 2013 15:50:37 -0400 Date: Tue, 26 Mar 2013 20:50:28 +0100 From: Aurelien Jarno Message-ID: <20130326195028.GD22106@hall.aurel32.net> References: <1360521050-29680-1-git-send-email-rth@twiddle.net> <1360521050-29680-10-git-send-email-rth@twiddle.net> <20130305140344.GG23040@ohm.aurel32.net> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-15 Content-Disposition: inline In-Reply-To: <20130305140344.GG23040@ohm.aurel32.net> Sender: Aurelien Jarno Subject: Re: [Qemu-devel] [PATCH 09/10] mipsn32-linux-user: Restrict address space to 31-bits. List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Richard Henderson Cc: peter.maydell@linaro.org, riku.voipio@iki.fi, qemu-devel@nongnu.org On Tue, Mar 05, 2013 at 03:03:44PM +0100, Aurelien Jarno wrote: > On Sun, Feb 10, 2013 at 10:30:49AM -0800, Richard Henderson wrote: > > Signed-off-by: Richard Henderson > > --- > > linux-user/main.c | 12 ++++++------ > > 1 file changed, 6 insertions(+), 6 deletions(-) > > > > diff --git a/linux-user/main.c b/linux-user/main.c > > index 8c4dffd..25491ca 100644 > > --- a/linux-user/main.c > > +++ b/linux-user/main.c > > @@ -46,10 +46,10 @@ int gdbstub_port; > > envlist_t *envlist; > > const char *cpu_model; > > unsigned long mmap_min_addr; > > + > > #if defined(CONFIG_USE_GUEST_BASE) > > unsigned long guest_base; > > int have_guest_base; > > -#if (TARGET_LONG_BITS == 32) && (HOST_LONG_BITS == 64) > > /* > > * When running 32-on-64 we should make sure we can fit all of the possible > > * guest address space into a contiguous chunk of virtual host memory. > > @@ -57,16 +57,16 @@ int have_guest_base; > > * This way we will never overlap with our own libraries or binaries or stack > > * or anything else that QEMU maps. > > */ > > -# ifdef TARGET_MIPS > > +# if HOST_LONG_BITS == 64 \ > > + && (defined(TARGET_ABI_MIPSO32) || defined(TARGET_ABI_MIPSN32)) > > /* MIPS only supports 31 bits of virtual address space for user space */ > > -unsigned long reserved_va = 0x77000000; > > -# else > > +unsigned long reserved_va = 0x7f000000; > > Is it really wanted to change 0x77 into 0x7f? If yes, I think the > commit log should explain why. > Ping? -- Aurelien Jarno GPG: 1024D/F1BCDB73 aurelien@aurel32.net http://www.aurel32.net