Re: [Qemu-devel] seabios 1.7.3 breaks booting windows 7?

[Kevin O'Connor]

On Mon, Jul 08, 2013 at 08:18:42PM -0400, Cole Robinson wrote:
> On 07/08/2013 07:18 PM, Kevin O'Connor wrote:
> > On Mon, Jul 08, 2013 at 05:23:40PM -0400, Cole Robinson wrote:
> >> 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:
> >>
> >> 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
> > 
> > Thanks for testing.  I have a "Windows 7 Ultimate" beta (build 7100)
> > that I tested with and I don't see this problem.  (Nor do I see the
> > problem on WinXP.)
> > 
> > Can you try without smp and with a smaller memory size (eg, 512M) to
> > see if the behavior changes.
> > 
> 
> Hmm, my winxp 32 guest is reproducing as well. And I can reproduce with just
> the windows 7 media (mine is ultimate N, came off MSDN years ago):
> 
> ~/src/qemu/x86_64-softmmu/qemu-system-x86_64 -m 512 -sdl -cdrom
> /mnt/data/devel/media/win7_64.iso -boot d -bios ~/src/seabios/out/bios.bin
> 
> vista and windows 8 preview media are reproducing for me too.
> 
> Maybe it's a toolchain thing, anyone else on Fedora 19 that can independently
> confirm?

Ugh - make sure you aren't using an old acpi-dsdt.aml file.  I suggest
following the steps outlined in the SeaBIOS README file for creating a
mybiosdir/ directory and use the QEMU -L option to point to that
directory.


Otherwise...

I compile under FC17.  I have pre-packaged binaries available at:

http://code.coreboot.org/p/seabios/downloads/

Also, could you post the log output.  Add the following to qemu to
produce it:

qemu ... -chardev stdio,id=seabios \
    -device isa-debugcon,iobase=0x402,chardev=seabios ...


> > Otherwise, we'll need to iterate through each change in the 76e58028
> > commit and find out which part of the DSDT that your copy of Windows
> > doesn't like.
> >
> 
> I'll probably need some hand holding through that, I can't tell from looking
> at the patch what parts are independent.

You'll want to do something like:

--- a/src/acpi-dsdt-pci-crs.dsl
+++ b/src/acpi-dsdt-pci-crs.dsl
@@ -56,6 +56,8 @@ Scope(\_SB.PCI0) {
     })
 
     Method(_CRS, 0) {
+        Return (CRES)
+#if 0
         /* Fields provided by dynamically created ssdt */
         External(P0S, IntObj)
         External(P0E, IntObj)
@@ -86,5 +88,6 @@ Scope(\_SB.PCI0) {
         /* add window and return result */
         ConcatenateResTemplate(CRES, CR64, Local0)
         Return (Local0)
+#endif
     }
 }

and then move the "Return" and "#if 0" down until you find the
offending code.

-Kevin