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

2012-12-05 Thread Gabriel L. Somlo
On Wed, Dec 05, 2012 at 07:50:17AM +0100, Gerd Hoffmann wrote: > Ok, so no IRQ declared for the RTC. We have IRQ 8 for both rtc and > hpet, which most likely is the root cause for the issue. You can try > simply dropping the line for testing. I'll try to come up with > something more clever as t

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

2012-12-04 Thread Gerd Hoffmann
On 12/04/12 18:14, Gabriel L. Somlo wrote: > On Tue, Dec 04, 2012 at 05:56:55PM +0100, Gerd Hoffmann wrote: >> Ok, and how does the RTC look like on your MacPro? > > Device (RTC) > { > Name (_HID, EisaId ("PNP0B00")) > Name (_CRS, ResourceTemplate () > { > IO (Decode16, >

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

2012-12-04 Thread Gabriel L. Somlo
On Tue, Dec 04, 2012 at 05:56:55PM +0100, Gerd Hoffmann wrote: > Ok, and how does the RTC look like on your MacPro? Device (RTC) { Name (_HID, EisaId ("PNP0B00")) Name (_CRS, ResourceTemplate () { IO (Decode16, 0x0070, // Range Minimum 0x0070

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

2012-12-04 Thread Gerd Hoffmann
Hi, > The complete definition for the HPET on my MacPro5,1 is below; along > with the 0xFED0 value gleaned from the previous SeaBIOS version of > _CRS, I came up with the above. Hopefully it works for WinXP too... > Device (HPET) { > Name (_HID, EisaId ("PNP0103")) > Name (BUF0, Res

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

2012-12-04 Thread Gerd Hoffmann
On 12/04/12 16:37, Gerd Hoffmann wrote: >> > I also noticed that on Windows 7, the 'IRQNoFlags' line above makes the RTC >> > clock complain that it does not have resources available. While removing >> > the >> > above line, removes that error. > Hmm. The IRQNoFlags for the RTC isn't new though,

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

2012-12-04 Thread Gabriel L. Somlo
On Tue, Dec 04, 2012 at 10:11:38AM -0500, jba...@redhat.com wrote: > > I don't have an XP install image anymore, but does replacing 2,8 with > > 0,8 help ? > > > > no. > > > On a few of my machines, this is actually coded as > > > > IRQNoFlags () > >

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

2012-12-04 Thread Gerd Hoffmann
Hi, >>> '-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. >> > > There was some discussion of trying to make q35 the default for 1.4,

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

2012-12-04 Thread jbaron
On Tue, Dec 04, 2012 at 09:42:44AM -0500, Gabriel L. Somlo wrote: > On Tue, 04 Dec 2012 08:40:27 +0100 Gerd Hoffmann wrote: > >On Mon, 3 Dec 2012 13:47:04 -0500, Jason Baron wrote: > >> 2) HPET ACPI error > >> > >> This line: 'IRQNoFlags () {2, 8}' in the HPET acpi table is causing the > >> folloi

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

2012-12-04 Thread Gabriel L. Somlo
On Tue, 04 Dec 2012 08:40:27 +0100 Gerd Hoffmann wrote: >On Mon, 3 Dec 2012 13:47:04 -0500, Jason Baron wrote: >> 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 ma

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

2012-12-04 Thread Jason Baron
On Tue, Dec 04, 2012 at 08:40:27AM +0100, Gerd Hoffmann wrote: > 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 a

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

2012-12-03 Thread Jan Kiszka
On 2012-12-04 08:40, Gerd Hoffmann wrote: > 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 >>

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

2012-12-03 Thread 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 dev

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

2012-12-03 Thread Jason Baron
On Mon, Dec 03, 2012 at 11:11:55AM +0100, Gerd Hoffmann wrote: > Hi, > > Most q35 seabios patches just landed upstream. So here we go with a > last-minute pull to plumb the missing q35 bits. It obviously updates > seabios again. It also adds autoloading for the acpi dsdt table. > > With this

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

2012-12-03 Thread Gerd Hoffmann
Hi, Most q35 seabios patches just landed upstream. So here we go with a last-minute pull to plumb the missing q35 bits. It obviously updates seabios again. It also adds autoloading for the acpi dsdt table. With this pull "qemu -M q35" JustWorks[tm]. please pull, Gerd The following change