Re: [Qemu-devel] [PULL for-1.3 0/3] seabios: q35 update

[Gerd Hoffmann]

  Hi,

> 1) legacy ide mode
> 
> I can currently create a ide controller on the command-line using
> '-device'. However, on the real h/w there is an IDE compatibility mode
> which essentially advertises an ide controller at the same location that
> the ahci lives at. In fact, it changes the PCI device id. To deal with
> the fact that AHCI has 6 ports and thus 6 possible devices, it then adds
> a second controller for the remaining 2 disks. This shouldn't be too
> hard to emulate. But I'm wondering what we want the qemu interface
> to look like? A -machine options such as:
> '-machine q35,diskmode=ahci,ide,raid'? 

I'm wondering whenever we want to deal with that at all?

"If your guest is too old to handle ahci natively, just stick to piix."
is a sensible policy IMHO.

> 2) HPET ACPI error
> 
> This line: 'IRQNoFlags () {2, 8}' in the HPET acpi table is causing the
> folloing ACPI message (removing it makes it go away):

Hmm.  That was added to make macos x happy and is also present on real
hardware, so I'm wondering what is going on here.

> 3) irq table
> 
> The irq table commit that makes windows 7 work, upsets Windows XP. If
> I back out seabios commit: 2114f50148c42e374586359d23b522483ca10e8d
> I do not get the following error:
> 
> "
> A problem has been detected and windows has been shut down To prevent damage
> to your computer.
> 
> If this is The first time you've seen this stop error screen,
> restart your computer. If this screen appears again, follow
> these steps:
> 
> The BIOS in this sysTem is not fully ACPI compliant. Please contact your
> system vendor for an updated BIOS.  If you are unable to obtain an
> updated BIOS or the latest BIOS supplied by your vendor is not ACPI
> compliant, you can turn off ACPI mode during textmode setup.  To do this,
> press The F7 key when you are prompted To install storage drivers. The
> system will not notify you that the F7 key was pressed - it will silently
> disable ACPI and allow you to continue your installation.
> 
> Technical informaTion:
> 
> *** STOP: 0x000000A5 
> (Ox0000000000010006,OxFFFFFADF9C461108,0x0000000000000000,0
> x0000000000000000)
> "

Jan, this is your commit, any idea?

cheers,
  Gerd