Re: [SeaBIOS] [PATCH] Add acpi_reboot() function for invoking ACPI reset

2013-02-22 Thread David Woodhouse
On Tue, 2013-02-19 at 20:09 -0500, Kevin O'Connor wrote: > > Looks okay to me. I'd like to see the follow up patches that make use > of it before committing though. Sent. With Qemu fixed to actually *do* a hard reset, and OVMF fixed to pass in ACPI 2.0 tables with a RESET_REG filled in (thanks,

[SeaBIOS] [PATCH 4/4] Try pci_reboot() before i8042_reboot()

2013-02-22 Thread David Woodhouse
From: David Woodhouse The so-called "PCI" reboot at 0xCF9 is supposed to be a hard reset, while the keyboard controller is only a soft reset. So try pci_reboot() first. Signed-off-by: David Woodhouse --- src/resume.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src

[SeaBIOS] [PATCH 3/4] Add acpi_reboot() reset method using RESET_REG

2013-02-22 Thread David Woodhouse
From: David Woodhouse Signed-off-by: David Woodhouse --- src/acpi.c | 58 +++--- src/acpi.h | 14 ++ src/resume.c | 3 +++ 3 files changed, 64 insertions(+), 11 deletions(-) diff --git a/src/acpi.c b/src/acpi.c index 36bd39a.

[SeaBIOS] [PATCH 1/4] Fix return type of le64_to_cpu() and be64_to_cpu()

2013-02-22 Thread David Woodhouse
From: David Woodhouse Signed-off-by: David Woodhouse --- src/byteorder.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/byteorder.h b/src/byteorder.h index 5a8a64a..7362aeb 100644 --- a/src/byteorder.h +++ b/src/byteorder.h @@ -43,7 +43,7 @@ static inline u16 le16_t

[SeaBIOS] [PATCH 2/4] Rename find_pmtimer() to find_acpi_features()

2013-02-22 Thread David Woodhouse
From: David Woodhouse I'm about to make it do more than just the pmtimer... Signed-off-by: David Woodhouse --- src/acpi.c | 10 -- src/acpi.h | 2 +- src/coreboot.c | 4 ++-- src/csm.c | 4 ++-- src/xen.c | 4 ++-- 5 files changed, 11 insertions(+), 13 deletions(-

[SeaBIOS] [PATCH] Make Xen one of the top-level build target choices

2013-02-22 Thread David Woodhouse
I think we've fairly much decided not to do this, so I'll drop it from my tree for now. But for archival purposes here it is... Signed-off-by: David Woodhouse --- src/Kconfig| 16 +++- src/boot.c | 2 +- src/paravirt.c | 12 src/paravirt.h | 6 +- src/post.

Re: [SeaBIOS] [edk2] [PATCH 0/8] OvmfPkg: reset-related changes

2013-02-22 Thread David Woodhouse
On Fri, 2013-02-22 at 20:25 +0100, Laszlo Ersek wrote: > ... IOW, I think it's not important that SeaBIOS hasn't seen RESET_REG > and cannot set things right by doing a real hard reset: under the > circumstances you described, we shouldn't even *be* in SeaBIOS. Hm yes, thank you for the reminder.

Re: [SeaBIOS] [edk2] [PATCH 0/8] OvmfPkg: reset-related changes

2013-02-22 Thread Laszlo Ersek
On 02/22/13 19:13, Laszlo Ersek wrote: > On 02/22/13 15:55, David Woodhouse wrote: >> On Fri, 2013-02-22 at 04:23 +0100, Laszlo Ersek wrote: >>> Patches 1 to 6 upgrade the FADT to ACPI 2.0 and present the PIIX3 >>> Reset Control Register in it. >> >> Hm. I can still trigger the soft reset loop in S

Re: [SeaBIOS] [edk2] [PATCH 0/8] OvmfPkg: reset-related changes

2013-02-22 Thread Laszlo Ersek
On 02/22/13 15:55, David Woodhouse wrote: > On Fri, 2013-02-22 at 04:23 +0100, Laszlo Ersek wrote: >> Patches 1 to 6 upgrade the FADT to ACPI 2.0 and present the PIIX3 >> Reset Control Register in it. > > Hm. I can still trigger the soft reset loop in SeaBIOS, but perhaps not > in a circumstance t

Re: [SeaBIOS] [edk2] [PATCH 0/8] OvmfPkg: reset-related changes

2013-02-22 Thread David Woodhouse
On Fri, 2013-02-22 at 04:23 +0100, Laszlo Ersek wrote: > Patches 1 to 6 upgrade the FADT to ACPI 2.0 and present the PIIX3 > Reset Control Register in it. Hm. I can still trigger the soft reset loop in SeaBIOS, but perhaps not in a circumstance that we care about... I've fixed Qemu to do the prop

Re: [SeaBIOS] [RFC] SeaBIOS v1.7.2.1 stable release

2013-02-22 Thread Gerd Hoffmann
On 02/22/13 01:45, Kevin O'Connor wrote: > There's been requests for a stable release. Here's a list of patches > that have been requested / that I think would be useful: > > 616fb27f - Update tools/acpi_extract.py to handle iasl 20130117 release. > 7cac600a - USB-EHCI: Fix null pointer assignmen