Re: [SeaBIOS] [Qemu-devel] [PATCH] piix: do not reset APIC base address (0x80) on piix4_reset.

2013-12-19 Thread Kevin O'Connor
On Thu, Dec 19, 2013 at 08:03:15PM +0200, Marcel Apfelbaum wrote: > On Thu, 2013-12-19 at 11:06 -0500, Kevin O'Connor wrote: > > On Wed, Dec 18, 2013 at 06:55:24PM +0200, Marcel Apfelbaum wrote: > > > On Wed, 2013-12-18 at 11:34 -0500, Paolo Bonzini wrote: > > > > Or put an array of (bdf, offset, s

Re: [SeaBIOS] [Qemu-devel] [PATCH] piix: do not reset APIC base address (0x80) on piix4_reset.

2013-12-19 Thread Marcel Apfelbaum
On Thu, 2013-12-19 at 11:06 -0500, Kevin O'Connor wrote: > On Wed, Dec 18, 2013 at 06:55:24PM +0200, Marcel Apfelbaum wrote: > > On Wed, 2013-12-18 at 11:34 -0500, Paolo Bonzini wrote: > > > Or put an array of (bdf, offset, size, value) tuples somewhere in low > > > memory, > > > fill it at startu

Re: [SeaBIOS] [Qemu-devel] [PATCH] piix: do not reset APIC base address (0x80) on piix4_reset.

2013-12-19 Thread Kevin O'Connor
On Wed, Dec 18, 2013 at 06:55:24PM +0200, Marcel Apfelbaum wrote: > On Wed, 2013-12-18 at 11:34 -0500, Paolo Bonzini wrote: > > Or put an array of (bdf, offset, size, value) tuples somewhere in low > > memory, > > fill it at startup, and reproduce it blindly at S3 resume time. This is > > simila

Re: [SeaBIOS] [Qemu-devel] [PATCH] piix: do not reset APIC base address (0x80) on piix4_reset.

2013-12-19 Thread Michael S. Tsirkin
On Thu, Dec 19, 2013 at 11:49:21AM +0200, Marcel Apfelbaum wrote: > > > Basically something like the below (warning: completely untested, > > > sorry). > > I tested and it works fine for both windows and linux guests! > Actually it doesn't work :( > I was testing a "working hack", not the master br

Re: [SeaBIOS] [Qemu-devel] [PATCH] piix: do not reset APIC base address (0x80) on piix4_reset.

2013-12-19 Thread Marcel Apfelbaum
On Thu, 2013-12-19 at 11:37 +0200, Marcel Apfelbaum wrote: > On Wed, 2013-12-18 at 19:20 +0200, Michael S. Tsirkin wrote: > > On Wed, Dec 18, 2013 at 06:49:24PM +0200, Marcel Apfelbaum wrote: > > > On Wed, 2013-12-18 at 18:33 +0200, Michael S. Tsirkin wrote: > > > > On Wed, Dec 18, 2013 at 06:27:12

Re: [SeaBIOS] [Qemu-devel] [PATCH] piix: do not reset APIC base address (0x80) on piix4_reset.

2013-12-19 Thread Marcel Apfelbaum
On Wed, 2013-12-18 at 19:20 +0200, Michael S. Tsirkin wrote: > On Wed, Dec 18, 2013 at 06:49:24PM +0200, Marcel Apfelbaum wrote: > > On Wed, 2013-12-18 at 18:33 +0200, Michael S. Tsirkin wrote: > > > On Wed, Dec 18, 2013 at 06:27:12PM +0200, Marcel Apfelbaum wrote: > > > > On Wed, 2013-12-18 at 17:

Re: [SeaBIOS] [Qemu-devel] [PATCH] piix: do not reset APIC base address (0x80) on piix4_reset.

2013-12-18 Thread Paolo Bonzini
Il 18/12/2013 23:10, Laszlo Ersek ha scritto: > > So, if PEI must do something after S3 resume that is independent of any > DXE drivers, it can simply do it. The boot script is only necessary when > the S3 resume PEI actions (in step 6) need to depend on earlier actions > during DXE (step 3). In

Re: [SeaBIOS] [Qemu-devel] [PATCH] piix: do not reset APIC base address (0x80) on piix4_reset.

2013-12-18 Thread Laszlo Ersek
On 12/18/13 23:10, Laszlo Ersek wrote: > 1. SEC after cold boot > 2. PEI after cold boot > 2.5 DXE IPL PEIM loads DXE core > 3. DXE after cold boot > 4. BDS after cold boot > 5. runtime (OSPM), normal entry > 6. PEI after S3 resume > 6.5 DXE IPL PEIM branches to S3 resume PEIM > 7. runtime (OSPM),

Re: [SeaBIOS] [Qemu-devel] [PATCH] piix: do not reset APIC base address (0x80) on piix4_reset.

2013-12-18 Thread Laszlo Ersek
On 12/18/13 17:34, Paolo Bonzini wrote: > > > - Messaggio originale - >> Da: "Michael S. Tsirkin" >> A: "marcel a" >> Cc: "Paolo Bonzini" , "Gal Hammer" >> , seabios@seabios.org, >> qemu-de...@nongnu.org >> Inviato: Mercoledì, 18 dicembre 2013 17:33:06 >> Oggetto: Re: [Qemu-devel] [PAT

Re: [SeaBIOS] [Qemu-devel] [PATCH] piix: do not reset APIC base address (0x80) on piix4_reset.

2013-12-18 Thread Marcel Apfelbaum
On Wed, 2013-12-18 at 18:33 +0200, Michael S. Tsirkin wrote: > On Wed, Dec 18, 2013 at 06:27:12PM +0200, Marcel Apfelbaum wrote: > > On Wed, 2013-12-18 at 17:22 +0200, Michael S. Tsirkin wrote: > > > On Wed, Dec 18, 2013 at 03:22:59PM +0100, Paolo Bonzini wrote: > > > > Il 11/12/2013 10:21, Gal Ham

Re: [SeaBIOS] [Qemu-devel] [PATCH] piix: do not reset APIC base address (0x80) on piix4_reset.

2013-12-18 Thread Marcel Apfelbaum
On Wed, 2013-12-18 at 11:34 -0500, Paolo Bonzini wrote: > > - Messaggio originale - > > Da: "Michael S. Tsirkin" > > A: "marcel a" > > Cc: "Paolo Bonzini" , "Gal Hammer" > > , seabios@seabios.org, > > qemu-de...@nongnu.org > > Inviato: Mercoledì, 18 dicembre 2013 17:33:06 > > Oggetto: R

Re: [SeaBIOS] [Qemu-devel] [PATCH] piix: do not reset APIC base address (0x80) on piix4_reset.

2013-12-18 Thread Paolo Bonzini
- Messaggio originale - > Da: "Michael S. Tsirkin" > A: "marcel a" > Cc: "Paolo Bonzini" , "Gal Hammer" , > seabios@seabios.org, > qemu-de...@nongnu.org > Inviato: Mercoledì, 18 dicembre 2013 17:33:06 > Oggetto: Re: [Qemu-devel] [PATCH] piix: do not reset APIC base address (0x80) > on

Re: [SeaBIOS] [Qemu-devel] [PATCH] piix: do not reset APIC base address (0x80) on piix4_reset.

2013-12-18 Thread Michael S. Tsirkin
On Wed, Dec 18, 2013 at 06:55:24PM +0200, Marcel Apfelbaum wrote: > On Wed, 2013-12-18 at 11:34 -0500, Paolo Bonzini wrote: > > > > - Messaggio originale - > > > Da: "Michael S. Tsirkin" > > > A: "marcel a" > > > Cc: "Paolo Bonzini" , "Gal Hammer" > > > , seabios@seabios.org, > > > qemu

Re: [SeaBIOS] [Qemu-devel] [PATCH] piix: do not reset APIC base address (0x80) on piix4_reset.

2013-12-18 Thread Michael S. Tsirkin
On Wed, Dec 18, 2013 at 06:49:24PM +0200, Marcel Apfelbaum wrote: > On Wed, 2013-12-18 at 18:33 +0200, Michael S. Tsirkin wrote: > > On Wed, Dec 18, 2013 at 06:27:12PM +0200, Marcel Apfelbaum wrote: > > > On Wed, 2013-12-18 at 17:22 +0200, Michael S. Tsirkin wrote: > > > > On Wed, Dec 18, 2013 at 0

Re: [SeaBIOS] [Qemu-devel] [PATCH] piix: do not reset APIC base address (0x80) on piix4_reset.

2013-12-18 Thread Michael S. Tsirkin
On Wed, Dec 18, 2013 at 11:34:14AM -0500, Paolo Bonzini wrote: > > > - Messaggio originale - > > Da: "Michael S. Tsirkin" > > A: "marcel a" > > Cc: "Paolo Bonzini" , "Gal Hammer" > > , seabios@seabios.org, > > qemu-de...@nongnu.org > > Inviato: Mercoledì, 18 dicembre 2013 17:33:06 > >

Re: [SeaBIOS] [Qemu-devel] [PATCH] piix: do not reset APIC base address (0x80) on piix4_reset.

2013-12-18 Thread Michael S. Tsirkin
On Wed, Dec 18, 2013 at 06:27:12PM +0200, Marcel Apfelbaum wrote: > On Wed, 2013-12-18 at 17:22 +0200, Michael S. Tsirkin wrote: > > On Wed, Dec 18, 2013 at 03:22:59PM +0100, Paolo Bonzini wrote: > > > Il 11/12/2013 10:21, Gal Hammer ha scritto: > > > > Fix a bug that was introduced in commit c046e

Re: [SeaBIOS] [Qemu-devel] [PATCH] piix: do not reset APIC base address (0x80) on piix4_reset.

2013-12-18 Thread Marcel Apfelbaum
On Wed, 2013-12-18 at 17:22 +0200, Michael S. Tsirkin wrote: > On Wed, Dec 18, 2013 at 03:22:59PM +0100, Paolo Bonzini wrote: > > Il 11/12/2013 10:21, Gal Hammer ha scritto: > > > Fix a bug that was introduced in commit c046e8c4. QEMU fails to > > > resume from suspend mode (S3). > > > > > > Signe