From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:33209) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Yrydu-0000HC-8j for qemu-devel@nongnu.org; Mon, 11 May 2015 21:05:55 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Yrydr-0008AB-3U for qemu-devel@nongnu.org; Mon, 11 May 2015 21:05:54 -0400 Received: from mail-qg0-f41.google.com ([209.85.192.41]:34162) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Yrydq-0008A7-Vy for qemu-devel@nongnu.org; Mon, 11 May 2015 21:05:51 -0400 Received: by qgfi89 with SMTP id i89so77475521qgf.1 for ; Mon, 11 May 2015 18:05:50 -0700 (PDT) MIME-Version: 1.0 Sender: peter.crosthwaite@petalogix.com In-Reply-To: <554B3188.30308@msgid.tls.msk.ru> References: <5549C424.3080203@msgid.tls.msk.ru> <554B024B.90101@msgid.tls.msk.ru> <554B0A67.5050107@msgid.tls.msk.ru> <554B3188.30308@msgid.tls.msk.ru> Date: Mon, 11 May 2015 18:05:50 -0700 Message-ID: From: Peter Crosthwaite Content-Type: text/plain; charset=UTF-8 Subject: Re: [Qemu-devel] Bug report - Windows XP guest failure List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Michael Tokarev Cc: G 3 , qemu-devel qemu-devel , Paolo Bonzini On Thu, May 7, 2015 at 2:34 AM, Michael Tokarev wrote: > 07.05.2015 09:47, Michael Tokarev wrote: >> 07.05.2015 09:12, Michael Tokarev wrote: >>> 07.05.2015 04:11, G 3 wrote: >>>> Did you boot Windows XP to the desktop? I have tested Windows 95, Windows 2000, and Windows XP. All of them fail to boot to the desktop. >>> >>> Yes, booted to desktop and did some minimal work in there, >>> installnig one update or two. >>> >>>> Command used: >>>> ./i386-softmmu/qemu-system-i386 -boot c -hda "Windows XP Hard Drive.img" >>> >>> Aha. You run without kvm, in tcg mode. I don't usually do that, >>> lemme try... >> >> Ok, I can reproduce this, winXP BSODs on boot in tcg mode. >> Git bisect points to this: >> >> commit 23820dbfc79d1c9dce090b4c555994f2bb6a69b3 >> Author: Peter Crosthwaite >> Date: Mon Mar 16 22:35:54 2015 -0700 >> >> exec: Respect as_translate_internal length clamp >> >> address_space_translate_internal will clamp the *plen length argument >> based on the size of the memory region being queried. The iommu walker >> logic in addresss_space_translate was ignoring this by discarding the >> post fn call value of *plen. Fix by just always using *plen as the >> length argument throughout the fn, removing the len local variable. >> >> This fixes a bootloader bug when a single elf section spans multiple >> QEMU memory regions. >> >> Signed-off-by: Peter Crosthwaite >> Message-Id: <1426570554-15940-1-git-send-email-peter.crosthwaite@xilinx.com> >> Signed-off-by: Paolo Bonzini > > This winXP BSOD happens on x86_64 target too. Reverting the > above commit from git master fixes the BSOD. > Any useful info about IO addresses on that BSOD? The last issue with this patch was IOPort code relying on the bug that this patch fixed. This could be similar and if we can track the failure to a particular address we can fix properly rather than another revert of that patch. Regards, Peter > Thanks, > > /mjt > >