On Sun, Feb 10, 2013 at 10:30:49AM -0800, Richard Henderson wrote: > Signed-off-by: Richard Henderson <r...@twiddle.net> > --- > 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. > +# elif (TARGET_LONG_BITS == 32) && (HOST_LONG_BITS == 64) > unsigned long reserved_va = 0xf7000000; > -# endif > #else > unsigned long reserved_va; > #endif > -#endif > +#endif /* CONFIG_USE_GUEST_BASE */ > > static void usage(void); > > -- > 1.8.1.2 > > -- Aurelien Jarno GPG: 1024D/F1BCDB73 aurel...@aurel32.net http://www.aurel32.net