From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1JRoc0-00083a-3C for qemu-devel@nongnu.org; Wed, 20 Feb 2008 08:03:48 -0500 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1JRoby-00082t-FR for qemu-devel@nongnu.org; Wed, 20 Feb 2008 08:03:47 -0500 Received: from [199.232.76.173] (helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1JRoby-00082p-As for qemu-devel@nongnu.org; Wed, 20 Feb 2008 08:03:46 -0500 Received: from wx-out-0506.google.com ([66.249.82.227]) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1JRobx-0002Pr-OT for qemu-devel@nongnu.org; Wed, 20 Feb 2008 08:03:45 -0500 Received: by wx-out-0506.google.com with SMTP id h31so2163474wxd.4 for ; Wed, 20 Feb 2008 05:03:39 -0800 (PST) Message-ID: <94a0d4530802200503ka8126eaq63ff9009862a19f0@mail.gmail.com> Date: Wed, 20 Feb 2008 15:03:39 +0200 From: "Felipe Contreras" Subject: Re: [Qemu-devel] [PATCH] possible mmap regression In-Reply-To: <20080216210715.GD7603@edgar.se.axis.com> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Content-Disposition: inline References: <94a0d4530802121142xd7987d7k31cd74b488882d19@mail.gmail.com> <20080216210715.GD7603@edgar.se.axis.com> Reply-To: qemu-devel@nongnu.org List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Edgar E. Iglesias" Cc: qemu-devel@nongnu.org Hi Edgar, On Feb 16, 2008 11:07 PM, Edgar E. Iglesias wrote: > > On Tue, Feb 12, 2008 at 09:42:15PM +0200, Felipe Contreras wrote: > > Hi, > > > > I don't know what I'm doing but this seems to fix the weird issue I was having. > > http://article.gmane.org/gmane.comp.emulators.qemu/23314 > > > > I've found out that this happens on linux 2.6.23, but not 2.6.24. > > > > Cheers. > > > > -- > > Felipe Contreras > > diff --git a/linux-user/mmap.c b/linux-user/mmap.c > > index 6292826..3050ad9 100644 > > --- a/linux-user/mmap.c > > +++ b/linux-user/mmap.c > > @@ -251,7 +251,7 @@ abi_long target_mmap(abi_ulong start, abi_ulong len, int prot, > > especially important if qemu_host_page_size > > > qemu_real_host_page_size */ > > p = mmap(g2h(mmap_start), > > - host_len, prot, flags | MAP_FIXED, fd, host_offset); > > + host_len, prot, flags, fd, host_offset); > > if (p == MAP_FAILED) > > return -1; > > /* update start so that it points to the file position at 'offset' */ > > Hello, > > Sorry, but I beleive your patch will break simulations where the targets pagesize is larger than the hosts. > > Would you mind trying the attach patched and let us know if it helps for you? > If not, it would be great if you could provide a small test case that trigs the bug you are seeing so we can debug the problem. > > Best regards > -- > Edgar E. Iglesias > Axis Communications AB > > diff --git a/linux-user/mmap.c b/linux-user/mmap.c > index 6292826..78a8162 100644 > --- a/linux-user/mmap.c > +++ b/linux-user/mmap.c > @@ -385,6 +385,9 @@ int target_munmap(abi_ulong start, abi_ulong len) > real_end -= qemu_host_page_size; > } > > + if (start < mmap_next_start) > + mmap_next_start = start; > + > /* unmap what we can */ > if (real_start < real_end) { > ret = munmap(g2h(real_start), real_end - real_start); > I tried your patch and it still crashes. I sent the details before: http://article.gmane.org/gmane.comp.emulators.qemu/23314 http://article.gmane.org/gmane.comp.emulators.qemu/23328 Basically it was triggered by this change: http://repo.or.cz/w/qemu.git?a=commitdiff;h=edbcc0b2eb1d4caee5f293e5c79f81023f3394e2 And it happens with some recursive Makefiles stuff. Best regards. -- Felipe Contreras