From 36d58153b16c21a097b9a11a085dd064f5112a05 Mon Sep 17 00:00:00 2001 From: jyao1 Date: Fri, 11 Feb 2011 06:40:20 +0000 Subject: UEFI table also need to be in ACPI NVS memory, because some data field could be updated by OS present agent. For example, BufferPtrAddress in SMM communication ACPI table. git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@11303 6f19259b-4bc3-4df7-8a09-765794883524 --- MdeModulePkg/Universal/Acpi/AcpiTableDxe/AcpiTableProtocol.c | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) (limited to 'MdeModulePkg') diff --git a/MdeModulePkg/Universal/Acpi/AcpiTableDxe/AcpiTableProtocol.c b/MdeModulePkg/Universal/Acpi/AcpiTableDxe/AcpiTableProtocol.c index cb3dc89948..764ad86e61 100644 --- a/MdeModulePkg/Universal/Acpi/AcpiTableDxe/AcpiTableProtocol.c +++ b/MdeModulePkg/Universal/Acpi/AcpiTableDxe/AcpiTableProtocol.c @@ -1,7 +1,7 @@ /** @file ACPI Table Protocol Implementation - Copyright (c) 2006 - 2010, Intel Corporation. All rights reserved.
+ Copyright (c) 2006 - 2011, Intel Corporation. All rights reserved.
This program and the accompanying materials are licensed and made available under the terms and conditions of the BSD License which accompanies this distribution. The full text of the license may be found at @@ -614,12 +614,17 @@ AddTableToList ( // // Allocation memory type depends on the type of the table // - if (CurrentTableSignature == EFI_ACPI_2_0_FIRMWARE_ACPI_CONTROL_STRUCTURE_SIGNATURE || - CurrentTableSignature == EFI_ACPI_3_0_FIRMWARE_ACPI_CONTROL_STRUCTURE_SIGNATURE) { + if ((CurrentTableSignature == EFI_ACPI_2_0_FIRMWARE_ACPI_CONTROL_STRUCTURE_SIGNATURE) || + (CurrentTableSignature == EFI_ACPI_4_0_UEFI_ACPI_DATA_TABLE_SIGNATURE)) { // // Allocate memory for the FACS. This structure must be aligned // on a 64 byte boundary and must be ACPI NVS memory. // Using AllocatePages should ensure that it is always aligned. + // Do not change signature for new ACPI version because they are same. + // + // UEFI table also need to be in ACPI NVS memory, because some data field + // could be updated by OS present agent. For example, BufferPtrAddress in + // SMM communication ACPI table. // ASSERT ((EFI_PAGE_SIZE % 64) == 0); Status = gBS->AllocatePages ( -- cgit v1.2.3