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

2013-07-09 Thread Cole Robinson
On 07/08/2013 08:41 PM, Kevin O'Connor wrote:
 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.
 

Indeed that was it, thanks Kevin, sorry for the noise.

- Cole




Re: [Qemu-devel] seabios 1.7.3 breaks booting windows 7? (was: [ANNOUNCE] SeaBIOS 1.7.3)

2013-07-08 Thread Kevin O'Connor
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.

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.

-Kevin



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

2013-07-08 Thread Cole Robinson
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?

 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.

- Cole



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

2013-07-08 Thread 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