ACPICA: Ignore ACPI table signature for Load() operator

Previous thread: ACPICA: Fix to allow zero-length ASL field declarations by Linux Kernel Mailing List on Wednesday, June 11, 2008 - 6:07 pm. (1 message)

Next thread: ACPICA: Fix for Load operator, load table at the namespace root by Linux Kernel Mailing List on Wednesday, June 11, 2008 - 6:08 pm. (1 message)
From: Linux Kernel Mailing List
Date: Wednesday, June 11, 2008 - 6:07 pm

Gitweb:     http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=bc45b1...
Commit:     bc45b1d39a925b56796bebf8a397a0491489d85c
Parent:     7aa7d4336df34e32195557a1ad422627bd69ef0b
Author:     Bob Moore <robert.moore@intel.com>
AuthorDate: Tue Jun 10 14:12:50 2008 +0800
Committer:  Len Brown <len.brown@intel.com>
CommitDate: Wed Jun 11 19:13:47 2008 -0400

    ACPICA: Ignore ACPI table signature for Load() operator
    
    Only "SSDT" is acceptable to the ACPI spec, but tables are
    seen with OEMx and null sigs. Therefore, signature validation
    is worthless.  Apparently MS ACPI accepts such signatures, ACPICA
    must be compatible.
    
    http://bugzilla.kernel.org/show_bug.cgi?id=10454
    
    Signed-off-by: Bob Moore <robert.moore@intel.com>
    Signed-off-by: Lin Ming <ming.m.lin@intel.com>
    Signed-off-by: Len Brown <len.brown@intel.com>
---
 drivers/acpi/tables/tbinstal.c |   25 +++++++------------------
 1 files changed, 7 insertions(+), 18 deletions(-)

diff --git a/drivers/acpi/tables/tbinstal.c b/drivers/acpi/tables/tbinstal.c
index 402f93e..5336ce8 100644
--- a/drivers/acpi/tables/tbinstal.c
+++ b/drivers/acpi/tables/tbinstal.c
@@ -123,24 +123,13 @@ acpi_tb_add_table(struct acpi_table_desc *table_desc,
 		}
 	}
 
-	/* The table must be either an SSDT or a PSDT or an OEMx */
-
-	if (!ACPI_COMPARE_NAME(table_desc->pointer->signature, ACPI_SIG_PSDT)&&
-	    !ACPI_COMPARE_NAME(table_desc->pointer->signature, ACPI_SIG_SSDT)&&
-	    strncmp(table_desc->pointer->signature, "OEM", 3)) {
-		/* Check for a printable name */
-		if (acpi_ut_valid_acpi_name(
-			*(u32 *) table_desc->pointer->signature)) {
-			ACPI_ERROR((AE_INFO, "Table has invalid signature "
-					"[%4.4s], must be SSDT or PSDT",
-				    table_desc->pointer->signature));
-		} else {
-			ACPI_ERROR((AE_INFO, "Table has invalid signature "
-					"(0x%8.8X), must be SSDT or PSDT",
-				    *(u32 *) ...
Previous thread: ACPICA: Fix to allow zero-length ASL field declarations by Linux Kernel Mailing List on Wednesday, June 11, 2008 - 6:07 pm. (1 message)

Next thread: ACPICA: Fix for Load operator, load table at the namespace root by Linux Kernel Mailing List on Wednesday, June 11, 2008 - 6:08 pm. (1 message)