From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:41706)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <crobinso@redhat.com>) id 1UwIuP-0003YJ-TD
	for qemu-devel@nongnu.org; Mon, 08 Jul 2013 17:23:47 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <crobinso@redhat.com>) id 1UwIuO-0007Oc-Fz
	for qemu-devel@nongnu.org; Mon, 08 Jul 2013 17:23:45 -0400
Received: from mx1.redhat.com ([209.132.183.28]:38129)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <crobinso@redhat.com>) id 1UwIuO-0007OK-8f
	for qemu-devel@nongnu.org; Mon, 08 Jul 2013 17:23:44 -0400
Message-ID: <51DB2DDC.3080001@redhat.com>
Date: Mon, 08 Jul 2013 17:23:40 -0400
From: Cole Robinson <crobinso@redhat.com>
MIME-Version: 1.0
References: <20130708034128.GA20623@morn.localdomain>
In-Reply-To: <20130708034128.GA20623@morn.localdomain>
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
Subject: [Qemu-devel] seabios 1.7.3 breaks booting windows 7? (was:
 [ANNOUNCE] SeaBIOS 1.7.3)
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Kevin O'Connor <kevin@koconnor.net>
Cc: seabios@seabios.org, qemu-devel@nongnu.org

On 07/07/2013 11:41 PM, Kevin O'Connor wrote:
> The 1.7.3 version of SeaBIOS has now been released.  For more
> information on the release, please see:
> 
> http://seabios.org/Releases
> 
> 
> New in this release:
> 
> * Initial support for using SeaBIOS as a UEFI Compatibility Support
>   Module (CSM)
> * Support for detecting and using ACPI reboot ports.
> * By default, all 16bit entry points now use an internal stack to
>   reduce stack footprint.
> * Floppy controller code has been rewritten to improve
>   compatibility. Non-standard floppy sizes now work again with recent
>   QEMU versions.
> * Several bug fixes and code cleanups
> 
> 

Just gave it a spin, breaks booting a Windows 7 VM for me. Using qemu-1.4.2
from Fedora 19 as well as qemu.git.

Here's the BSOD (F19 qemu and seabios 1.7.3):
http://crobinso.fedorapeople.org/misc/seabios_1.7.3_windows_err.png

I bisected to:

commit 76e58028d28e78431f9de3cee0b3c88d807fa39d
Author: Kevin O'Connor <kevin@koconnor.net>
Date:   Wed Mar 6 21:50:09 2013 -0500

    acpi: Eliminate BDAT parameter passing to DSDT code.


Reproducing command line:

x86_64-softmmu/qemu-system-x86_64 \
    -bios /usr/share/seabios/bios.bin \
    -machine pc-i440fx-1.4,accel=kvm,usb=off \
    -m 4096 -smp 4 \
    -sdl -vga std \
    win7.qcow2

Let me know if there's any more info I can provide.

Thanks,
Cole