|
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [Xen-changelog] [xen master] ACPI/APEI: revert "accept validly sized ERST on Intel systems only for now"
commit 3920df230bbe9eb2fcb5147df97f7ab7ece22fc2
Author: Jan Beulich <jbeulich@xxxxxxxx>
AuthorDate: Thu Mar 28 09:06:19 2013 +0100
Commit: Jan Beulich <jbeulich@xxxxxxxx>
CommitDate: Thu Mar 28 09:06:19 2013 +0100
ACPI/APEI: revert "accept validly sized ERST on Intel systems only for now"
With the recent two fixes to ERST handling, this should no longer be
necessary.
Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
Tested-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
---
xen/drivers/acpi/apei/erst.c | 13 -------------
1 files changed, 0 insertions(+), 13 deletions(-)
diff --git a/xen/drivers/acpi/apei/erst.c b/xen/drivers/acpi/apei/erst.c
index 6519042..98f73d9 100644
--- a/xen/drivers/acpi/apei/erst.c
+++ b/xen/drivers/acpi/apei/erst.c
@@ -738,19 +738,6 @@ static int __init erst_check_table(struct acpi_table_erst
*erst_tab)
switch (erst_tab->header_length) {
case sizeof(*erst_tab) - sizeof(erst_tab->header):
-#ifdef CONFIG_X86
- /* XXX
- * While the rest of the ERST code appears to work on Intel
- * systems with properly sized tables, various AMD systems
- * appear to get hung (at boot time) by allowing this. Until
- * someone with access to suitable hardware can debug this,
- * disable the rest of the code by considering this case
- * invalid.
- */
- if (boot_cpu_data.x86_vendor != X86_VENDOR_INTEL)
- return -EINVAL;
- /* fall through */
-#endif
/*
* While invalid per specification, there are (early?) systems
* indicating the full header size here, so accept that value too.
--
generated by git-patchbot for /home/xen/git/xen.git#master
_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxx
http://lists.xensource.com/xen-changelog
|
![]() |
Lists.xenproject.org is hosted with RackSpace, monitoring our |