tyler.durden
/
audk
mirror of https://github.com/acidanthera/audk.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

MdeModulePkg/IntelFrameworkModulePkg ACPI: Follow the new UEFI 2.4a spec to return EFI_ACCESS_DENIED for duplicated FADT, FACS or DSDT installation. 

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Star Zeng <star.zeng@intel.com>
Reviewed-by: Jiewen Yao <jiewen.yao@intel.com>

git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@15496 6f19259b-4bc3-4df7-8a09-765794883524
This commit is contained in:
Star Zeng 2014-05-06 02:11:23 +00:00 committed by lzeng14
parent a06ec3e2af
commit 5966402ed5
3 changed files with 46 additions and 137 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
IntelFrameworkModulePkg/Universal/Acpi/AcpiSupportDxe/AcpiSupportAcpiSupportProtocol.c
View File

@ -1,7 +1,7 @@
/** @file /** @file
ACPI Support Protocol implementation ACPI Support Protocol implementation
Copyright (c) 2006 - 2013, Intel Corporation. All rights reserved.<BR> Copyright (c) 2006 - 2014, Intel Corporation. All rights reserved.<BR>
This program and the accompanying materials This program and the accompanying materials
are licensed and made available under the terms and conditions are licensed and made available under the terms and conditions
@ -375,6 +375,9 @@ PublishTables (
and the size field embedded in the ACPI table pointed to by AcpiTableBuffer and the size field embedded in the ACPI table pointed to by AcpiTableBuffer
are not in sync. are not in sync.
@return EFI_OUT_OF_RESOURCES Insufficient resources exist to complete the request. @return EFI_OUT_OF_RESOURCES Insufficient resources exist to complete the request.
@retval EFI_ACCESS_DENIED The table signature matches a table already
present in the system and platform policy
does not allow duplicate tables of this type.
**/ **/
EFI_STATUS EFI_STATUS
@ -410,8 +413,8 @@ InstallAcpiTable (
// //
AcpiTableBufferConst = AllocateCopyPool (AcpiTableBufferSize, AcpiTableBuffer); AcpiTableBufferConst = AllocateCopyPool (AcpiTableBufferSize, AcpiTableBuffer);
*TableKey = 0; *TableKey = 0;
Status = AcpiSupport->SetAcpiTable ( Status = AddTableToList (
AcpiSupport, AcpiSupportInstance,
AcpiTableBufferConst, AcpiTableBufferConst,
TRUE, TRUE,
EFI_ACPI_TABLE_VERSION_1_0B | EFI_ACPI_TABLE_VERSION_2_0 | EFI_ACPI_TABLE_VERSION_3_0, EFI_ACPI_TABLE_VERSION_1_0B | EFI_ACPI_TABLE_VERSION_2_0 | EFI_ACPI_TABLE_VERSION_3_0,
@ -457,12 +460,10 @@ UninstallAcpiTable (
// //
// Uninstall the ACPI table by using ACPI support protocol // Uninstall the ACPI table by using ACPI support protocol
// //
Status = AcpiSupport->SetAcpiTable ( Status = RemoveTableFromList (
AcpiSupport, AcpiSupportInstance,
NULL,
FALSE,
EFI_ACPI_TABLE_VERSION_1_0B | EFI_ACPI_TABLE_VERSION_2_0 | EFI_ACPI_TABLE_VERSION_3_0, EFI_ACPI_TABLE_VERSION_1_0B | EFI_ACPI_TABLE_VERSION_2_0 | EFI_ACPI_TABLE_VERSION_3_0,
&TableKey TableKey
); );
if (!EFI_ERROR (Status)) { if (!EFI_ERROR (Status)) {
Status = AcpiSupport->PublishTables ( Status = AcpiSupport->PublishTables (
@ -588,8 +589,9 @@ ReallocateAcpiTableBuffer (
@return EFI_SUCCESS The function completed successfully. @return EFI_SUCCESS The function completed successfully.
@return EFI_OUT_OF_RESOURCES Could not allocate a required resource. @return EFI_OUT_OF_RESOURCES Could not allocate a required resource.
@return EFI_ABORTED The table is a duplicate of a table that is required @retval EFI_ACCESS_DENIED The table signature matches a table already
to be unique. present in the system and platform policy
does not allow duplicate tables of this type.
**/ **/
EFI_STATUS EFI_STATUS
AddTableToList ( AddTableToList (
@ -724,7 +726,7 @@ AddTableToList (
) { ) {
gBS->FreePages (CurrentTableList->PageAddress, CurrentTableList->NumberOfPages); gBS->FreePages (CurrentTableList->PageAddress, CurrentTableList->NumberOfPages);
gBS->FreePool (CurrentTableList); gBS->FreePool (CurrentTableList);
return EFI_ABORTED; return EFI_ACCESS_DENIED;
} }
// //
// Add the table to the appropriate table version // Add the table to the appropriate table version
@ -865,7 +867,7 @@ AddTableToList (
) { ) {
gBS->FreePages (CurrentTableList->PageAddress, CurrentTableList->NumberOfPages); gBS->FreePages (CurrentTableList->PageAddress, CurrentTableList->NumberOfPages);
gBS->FreePool (CurrentTableList); gBS->FreePool (CurrentTableList);
return EFI_ABORTED; return EFI_ACCESS_DENIED;
} }
// //
// FACS is referenced by FADT and is not part of RSDT // FACS is referenced by FADT and is not part of RSDT
@ -949,7 +951,7 @@ AddTableToList (
) { ) {
gBS->FreePages (CurrentTableList->PageAddress, CurrentTableList->NumberOfPages); gBS->FreePages (CurrentTableList->PageAddress, CurrentTableList->NumberOfPages);
gBS->FreePool (CurrentTableList); gBS->FreePool (CurrentTableList);
return EFI_ABORTED; return EFI_ACCESS_DENIED;
} }
// //
// DSDT is referenced by FADT and is not part of RSDT // DSDT is referenced by FADT and is not part of RSDT

127
MdeModulePkg/Universal/Acpi/AcpiTableDxe/AcpiTableProtocol.c
View File

@ -1,7 +1,7 @@
/** @file /** @file
ACPI Table Protocol Implementation ACPI Table Protocol Implementation
Copyright (c) 2006 - 2013, Intel Corporation. All rights reserved.<BR> Copyright (c) 2006 - 2014, Intel Corporation. All rights reserved.<BR>
This program and the accompanying materials This program and the accompanying materials
are licensed and made available under the terms and conditions of the BSD License 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 which accompanies this distribution. The full text of the license may be found at
@ -98,109 +98,6 @@ ChecksumCommonTables (
// Protocol function implementations. // Protocol function implementations.
// //
/**
This function adds, removes, or updates ACPI tables. If the address is not
null and the handle value is null, the table is added. If both the address and
handle are not null, the table at handle is updated with the table at address.
If the address is null and the handle is not, the table at handle is deleted.
@param AcpiTableInstance Instance of the protocol.
@param Table Pointer to a table.
@param Checksum Boolean indicating if the checksum should be calculated.
@param Version Version(s) to set.
@param Handle Handle of the table.
@return EFI_SUCCESS The function completed successfully.
@return EFI_INVALID_PARAMETER Both the Table and *Handle were NULL.
@return EFI_ABORTED Could not complete the desired request.
**/
EFI_STATUS
EFIAPI
SetAcpiTable (
IN EFI_ACPI_TABLE_INSTANCE *AcpiTableInstance,
IN VOID *Table OPTIONAL,
IN BOOLEAN Checksum,
IN EFI_ACPI_TABLE_VERSION Version,
IN OUT UINTN *Handle
)
{
UINTN SavedHandle;
EFI_STATUS Status;
//
// Check for invalid input parameters
//
ASSERT (Handle);
//
// Initialize locals
//
//
// Determine desired action
//
if (*Handle == 0) {
if (Table == NULL) {
//
// Invalid parameter combination
//
return EFI_INVALID_PARAMETER;
} else {
//
// Add table
//
Status = AddTableToList (AcpiTableInstance, Table, Checksum, Version, Handle);
}
} else {
if (Table != NULL) {
//
// Update table
//
//
// Delete the table list entry
//
Status = RemoveTableFromList (AcpiTableInstance, Version, *Handle);
if (EFI_ERROR (Status)) {
//
// Should not get an error here ever, but abort if we do.
//
return EFI_ABORTED;
}
//
// Set the handle to replace the table at the same handle
//
SavedHandle = AcpiTableInstance->CurrentHandle;
AcpiTableInstance->CurrentHandle = *Handle;
//
// Add the table
//
Status = AddTableToList (AcpiTableInstance, Table, Checksum, Version, Handle);
//
// Restore the saved current handle
//
AcpiTableInstance->CurrentHandle = SavedHandle;
} else {
//
// Delete table
//
Status = RemoveTableFromList (AcpiTableInstance, Version, *Handle);
}
}
if (EFI_ERROR (Status)) {
//
// Should not get an error here ever, but abort if we do.
//
return EFI_ABORTED;
}
//
// Done
//
return EFI_SUCCESS;
}
/** /**
This function publishes the specified versions of the ACPI tables by This function publishes the specified versions of the ACPI tables by
installing EFI configuration table entries for them. Any combination of installing EFI configuration table entries for them. Any combination of
@ -303,6 +200,9 @@ PublishTables (
and the size field embedded in the ACPI table pointed to by AcpiTableBuffer and the size field embedded in the ACPI table pointed to by AcpiTableBuffer
are not in sync. are not in sync.
@return EFI_OUT_OF_RESOURCES Insufficient resources exist to complete the request. @return EFI_OUT_OF_RESOURCES Insufficient resources exist to complete the request.
@retval EFI_ACCESS_DENIED The table signature matches a table already
present in the system and platform policy
does not allow duplicate tables of this type.
**/ **/
EFI_STATUS EFI_STATUS
@ -336,7 +236,7 @@ InstallAcpiTable (
// //
AcpiTableBufferConst = AllocateCopyPool (AcpiTableBufferSize,AcpiTableBuffer); AcpiTableBufferConst = AllocateCopyPool (AcpiTableBufferSize,AcpiTableBuffer);
*TableKey = 0; *TableKey = 0;
Status = SetAcpiTable ( Status = AddTableToList (
AcpiTableInstance, AcpiTableInstance,
AcpiTableBufferConst, AcpiTableBufferConst,
TRUE, TRUE,
@ -396,12 +296,10 @@ UninstallAcpiTable (
// //
// Uninstall the ACPI table // Uninstall the ACPI table
// //
Status = SetAcpiTable ( Status = RemoveTableFromList (
AcpiTableInstance, AcpiTableInstance,
NULL,
FALSE,
EFI_ACPI_TABLE_VERSION_1_0B | EFI_ACPI_TABLE_VERSION_2_0 | EFI_ACPI_TABLE_VERSION_3_0, EFI_ACPI_TABLE_VERSION_1_0B | EFI_ACPI_TABLE_VERSION_2_0 | EFI_ACPI_TABLE_VERSION_3_0,
&TableKey TableKey
); );
if (!EFI_ERROR (Status)) { if (!EFI_ERROR (Status)) {
Status = PublishTables ( Status = PublishTables (
@ -528,8 +426,9 @@ ReallocateAcpiTableBuffer (
@return EFI_SUCCESS The function completed successfully. @return EFI_SUCCESS The function completed successfully.
@return EFI_OUT_OF_RESOURCES Could not allocate a required resource. @return EFI_OUT_OF_RESOURCES Could not allocate a required resource.
@return EFI_ABORTED The table is a duplicate of a table that is required @retval EFI_ACCESS_DENIED The table signature matches a table already
to be unique. present in the system and platform policy
does not allow duplicate tables of this type.
**/ **/
EFI_STATUS EFI_STATUS
@ -665,7 +564,7 @@ AddTableToList (
) { ) {
gBS->FreePages (CurrentTableList->PageAddress, CurrentTableList->NumberOfPages); gBS->FreePages (CurrentTableList->PageAddress, CurrentTableList->NumberOfPages);
gBS->FreePool (CurrentTableList); gBS->FreePool (CurrentTableList);
return EFI_ABORTED; return EFI_ACCESS_DENIED;
} }
// //
// Add the table to the appropriate table version // Add the table to the appropriate table version
@ -800,7 +699,7 @@ AddTableToList (
) { ) {
gBS->FreePages (CurrentTableList->PageAddress, CurrentTableList->NumberOfPages); gBS->FreePages (CurrentTableList->PageAddress, CurrentTableList->NumberOfPages);
gBS->FreePool (CurrentTableList); gBS->FreePool (CurrentTableList);
return EFI_ABORTED; return EFI_ACCESS_DENIED;
} }
// //
// FACS is referenced by FADT and is not part of RSDT // FACS is referenced by FADT and is not part of RSDT
@ -884,7 +783,7 @@ AddTableToList (
) { ) {
gBS->FreePages (CurrentTableList->PageAddress, CurrentTableList->NumberOfPages); gBS->FreePages (CurrentTableList->PageAddress, CurrentTableList->NumberOfPages);
gBS->FreePool (CurrentTableList); gBS->FreePool (CurrentTableList);
return EFI_ABORTED; return EFI_ACCESS_DENIED;
} }
// //
// DSDT is referenced by FADT and is not part of RSDT // DSDT is referenced by FADT and is not part of RSDT

12
MdePkg/Include/Protocol/AcpiTable.h
View File

@ -2,7 +2,7 @@
The file provides the protocol to install or remove an ACPI The file provides the protocol to install or remove an ACPI
table from a platform. table from a platform.
Copyright (c) 2006 - 2011, Intel Corporation. All rights reserved.<BR> Copyright (c) 2006 - 2014, Intel Corporation. All rights reserved.<BR>
This program and the accompanying materials This program and the accompanying materials
are licensed and made available under the terms and conditions of the BSD License 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 which accompanies this distribution. The full text of the license may be found at
@ -31,7 +31,12 @@ typedef struct _EFI_ACPI_TABLE_PROTOCOL EFI_ACPI_TABLE_PROTOCOL;
copy into the RSDT/XSDT. InstallAcpiTable() must insert the new copy into the RSDT/XSDT. InstallAcpiTable() must insert the new
table at the end of the RSDT/XSDT. To prevent namespace table at the end of the RSDT/XSDT. To prevent namespace
collision, ACPI tables may be created using UEFI ACPI table collision, ACPI tables may be created using UEFI ACPI table
format. On successful output, TableKey is format. If this protocol is used to install a table with a
signature already present in the system, the new table will not
replace the existing table. It is a platform implementation
decision to add a new table with a signature matching an
existing table or disallow duplicate table signatures and
return EFI_ACCESS_DENIED. On successful output, TableKey is
initialized with a unique key. Its value may be used in a initialized with a unique key. Its value may be used in a
subsequent call to UninstallAcpiTable to remove an ACPI table. subsequent call to UninstallAcpiTable to remove an ACPI table.
If an EFI application is running at the time of this call, the If an EFI application is running at the time of this call, the
@ -61,6 +66,9 @@ typedef struct _EFI_ACPI_TABLE_PROTOCOL EFI_ACPI_TABLE_PROTOCOL;
@retval EFI_OUT_OF_RESOURCES Insufficient resources exist to @retval EFI_OUT_OF_RESOURCES Insufficient resources exist to
complete the request. complete the request.
@retval EFI_ACCESS_DENIED The table signature matches a table already
present in the system and platform policy
does not allow duplicate tables of this type.
**/ **/
typedef typedef