From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by mx.groups.io with SMTP id smtpd.web09.26153.1625933927168582420 for ; Sat, 10 Jul 2021 09:18:47 -0700 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: arm.com, ip: 217.140.110.172, mailfrom: omkar.kulkarni@arm.com) Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id BFD3B1063; Sat, 10 Jul 2021 09:18:41 -0700 (PDT) Received: from usa.arm.com (a077433.blr.arm.com [10.162.4.33]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPA id 50BC33F5A1; Sat, 10 Jul 2021 09:18:40 -0700 (PDT) From: "Omkar Anand Kulkarni" To: devel@edk2.groups.io Cc: Ard Biesheuvel , Sami Mujawar Subject: [PATCH v2 1/4] ArmPlatformPkg: Allow dynamic generation of HEST ACPI table Date: Sat, 10 Jul 2021 21:48:28 +0530 Message-Id: <20210710161831.30433-2-omkar.kulkarni@arm.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20210710161831.30433-1-omkar.kulkarni@arm.com> References: <20210710161831.30433-1-omkar.kulkarni@arm.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Introduce the HEST table generation protocol that allows platforms to build the table with multiple error source descriptors and install the table. The protocol provides two interfaces. The first interface allows for adding multiple error source descriptors into the HEST table. The second interface can then be used to dynamically install the fully populated HEST table. This allows multiple drivers and/or libraries to dynamically register error source descriptors into the HEST table. Co-authored-by: Thomas Abraham Signed-off-by: Omkar Anand Kulkarni --- ArmPlatformPkg/ArmPlatformPkg.dec | 4 + ArmPlatformPkg/Drivers/Apei/HestDxe/HestDxe.inf | 49 +++ ArmPlatformPkg/Include/Protocol/HestTable.h | 71 ++++ ArmPlatformPkg/Drivers/Apei/HestDxe/HestDxe.c | 354 ++++++++++++++++++= ++ 4 files changed, 478 insertions(+) diff --git a/ArmPlatformPkg/ArmPlatformPkg.dec b/ArmPlatformPkg/ArmPlatfo= rmPkg.dec index 3a25ddcdc8ca..e4afe5da8e11 100644 --- a/ArmPlatformPkg/ArmPlatformPkg.dec +++ b/ArmPlatformPkg/ArmPlatformPkg.dec @@ -127,3 +127,7 @@ gArmPlatformTokenSpaceGuid.PcdPL031RtcPpmAccuracy|300000000|UINT32|0x0= 0000022 =20 gArmPlatformTokenSpaceGuid.PcdWatchdogCount|0x0|UINT32|0x00000033 + +[Protocols.common] + ## Arm Platform HEST table generation protocol + gHestTableProtocolGuid =3D { 0x705bdcd9, 0x8c47, 0x457e, { 0xad, 0x0d,= 0xf7, 0x86, 0xf3, 0x4a, 0x0d, 0x63 } } diff --git a/ArmPlatformPkg/Drivers/Apei/HestDxe/HestDxe.inf b/ArmPlatfor= mPkg/Drivers/Apei/HestDxe/HestDxe.inf new file mode 100644 index 000000000000..91c7385bf7ff --- /dev/null +++ b/ArmPlatformPkg/Drivers/Apei/HestDxe/HestDxe.inf @@ -0,0 +1,49 @@ +## @file +# Dxe driver that creates and publishes the HEST table. +# +# This driver creates HEST header and provides protocol service to appe= nd +# and install the HEST table. +# +# Copyright (c) 2020 - 2021, ARM Limited. All rights reserved. +# SPDX-License-Identifier: BSD-2-Clause-Patent +# +## + +[Defines] + INF_VERSION =3D 0x0001001A + BASE_NAME =3D HestDxe + FILE_GUID =3D 933099a2-ef71-4e00-82aa-a79b1e0a3b3= 8 + MODULE_TYPE =3D DXE_DRIVER + VERSION_STRING =3D 1.0 + ENTRY_POINT =3D HestInitialize + +[Sources.Common] + HestDxe.c + +[Packages] + ArmPkg/ArmPkg.dec + ArmPlatformPkg/ArmPlatformPkg.dec + MdeModulePkg/MdeModulePkg.dec + MdePkg/MdePkg.dec + Platform/ARM/SgiPkg/SgiPlatform.dec + +[LibraryClasses] + ArmLib + BaseMemoryLib + DebugLib + UefiDriverEntryPoint + UefiLib + +[Protocols] + gEfiAcpiTableProtocolGuid ## PROTOCOL ALWAYS_CONSUMED + gHestTableProtocolGuid ## PRODUCES + +[FixedPcd] + gEfiMdeModulePkgTokenSpaceGuid.PcdAcpiDefaultCreatorId + gEfiMdeModulePkgTokenSpaceGuid.PcdAcpiDefaultCreatorRevision + gEfiMdeModulePkgTokenSpaceGuid.PcdAcpiDefaultOemId + gEfiMdeModulePkgTokenSpaceGuid.PcdAcpiDefaultOemRevision + gEfiMdeModulePkgTokenSpaceGuid.PcdAcpiDefaultOemTableId + +[Depex] + TRUE diff --git a/ArmPlatformPkg/Include/Protocol/HestTable.h b/ArmPlatformPkg= /Include/Protocol/HestTable.h new file mode 100644 index 000000000000..3b2e1f7d9203 --- /dev/null +++ b/ArmPlatformPkg/Include/Protocol/HestTable.h @@ -0,0 +1,71 @@ +/** @file + Builds and installs the HEST ACPI table. + + Define the protocol interface that allows HEST ACPI table to be create= d, + populated with error record descriptions and installation of the HEST = ACPI + table. + + Copyright (c) 2020 - 2021, ARM Limited. All rights reserved. + SPDX-License-Identifier: BSD-2-Clause-Patent +**/ + +#ifndef HEST_TABLE_H_ +#define HEST_TABLE_H_ + +#define HEST_TABLE_PROTOCOL_GUID \ + { \ + 0x705bdcd9, 0x8c47, 0x457e, \ + { 0xad, 0x0d, 0xf7, 0x86, 0xf3, 0x4a, 0x0d, 0x63 } \ + } + +/** + Append HEST error source descriptor protocol service. + + Protocol service used to append newly collected error source descripto= rs to + to an already created HEST table. + + @param[in] ErrorSourceDescriptorList List of Error Source Descri= ptors. + @param[in] ErrorSourceDescriptorListSize Total Size of Error Source + Descriptors. + @param[in] ErrorSourceDescriptorCount Total count of error source + descriptors. + + @retval EFI_SUCCESS Appending the error source descriptors + successful. + @retval EFI_OUT_OF_RESOURCES Buffer reallocation failed for the Hes= t + table. + @retval EFI_INVALID_PARAMETER Null ErrorSourceDescriptorList param o= r +**/ +typedef +EFI_STATUS +(EFIAPI *APPEND_ERROR_SOURCE_DESCRIPTOR) ( + IN CONST VOID *ErrorSourceDescriptorList, + IN UINTN ErrorSourceDescriptorListSize, + IN UINTN ErrorSourceDescriptorCount + ); + +/** + Install HEST table protocol service. + + Installs the HEST table that has been appended with the error source + descriptors. The checksum of this table is calculated and updated in + the table header. The HEST table is then installed. + + @retval EFI_SUCCESS HEST table is installed successfully. + @retval EFI_ABORTED HEST table is NULL. + @retval Other Install service call failed. +**/ +typedef +EFI_STATUS +(EFIAPI *INSTALL_HEST_TABLE) (VOID); + +// +// HEST_TABLE_PROTOCOL enables creation and installation of HEST table +// +typedef struct { + APPEND_ERROR_SOURCE_DESCRIPTOR AppendErrorSourceDescriptors; + INSTALL_HEST_TABLE InstallHestTable; +} HEST_TABLE_PROTOCOL; + +extern EFI_GUID gHestTableProtocolGuid; +#endif // HEST_TABLE_H_ diff --git a/ArmPlatformPkg/Drivers/Apei/HestDxe/HestDxe.c b/ArmPlatformP= kg/Drivers/Apei/HestDxe/HestDxe.c new file mode 100644 index 000000000000..b68e1a0c4e48 --- /dev/null +++ b/ArmPlatformPkg/Drivers/Apei/HestDxe/HestDxe.c @@ -0,0 +1,354 @@ +/** @file + Builds and installs the HEST ACPI table. + + This driver installs a protocol that can be used to create and install= a HEST + ACPI table. The protocol allows one or more error source producers to = add the + error source descriptors into the HEST table. It also allows the resul= ting + HEST table to be installed. + + Copyright (c) 2020 - 2021, ARM Limited. All rights reserved. + SPDX-License-Identifier: BSD-2-Clause-Patent + + @par Specification Reference: + - ACPI 6.3, Table 18-382, Hardware Error Source Table +**/ + +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include + +typedef struct { + VOID *HestTable; /// Pointer to HEST table. + UINT32 CurrentTableSize; /// Current size of HEST table. +} HEST_DXE_DRIVER_DATA; + +STATIC EFI_ACPI_TABLE_PROTOCOL *mAcpiTableProtocol =3D NULL; +STATIC HEST_DXE_DRIVER_DATA mHestDriverData; + +/** + Helper function to the driver. + + Function that reallocates memory for every new error source descriptor= info + added. + + @param[in] OldTableSize Old memory pool size. + @param[in] NewTableSize Required memory pool size. + @param[in, out] OldBuffer Current memory buffer pointer holding t= he Hest + table data. + + @return New pointer to reallocated memory pool. +**/ +STATIC +VOID* +ReallocateHestTableMemory ( + IN UINT32 OldTableSize, + IN UINT32 NewTableSize, + IN OUT VOID *OldBuffer + ) +{ + return ReallocateReservedPool ( + OldTableSize, + NewTableSize, + OldBuffer + ); +} + +/** + Allocate memory for the HEST table header and populate it. + + Helper function for this driver, populates the HEST table header. Call= ed once + in the beginning on first invocation of append error source descriptor + protocol service. + + @retval EFI_SUCCESS On successful creation of HEST header. + @retval EFI_OUT_OF_RESOURCES If system is out of memory recources. +**/ +STATIC +EFI_STATUS +BuildHestHeader (VOID) +{ + EFI_ACPI_6_3_HARDWARE_ERROR_SOURCE_TABLE_HEADER HestHeader; + UINT64 TempOemTableId; + + // + // Allocate memory for the HEST table header. + // + mHestDriverData.HestTable =3D + ReallocateHestTableMemory ( + 0, + sizeof (EFI_ACPI_6_3_HARDWARE_ERROR_SOURCE_TABLE_HEADER), + NULL + ); + if (mHestDriverData.HestTable =3D=3D NULL) { + return EFI_OUT_OF_RESOURCES; + } + + mHestDriverData.CurrentTableSize =3D + sizeof (EFI_ACPI_6_3_HARDWARE_ERROR_SOURCE_TABLE_HEADER); + + // + // Populate the values of the HEST table header. + // + HestHeader.Header.Signature =3D + EFI_ACPI_6_3_HARDWARE_ERROR_SOURCE_TABLE_SIGNATURE; + HestHeader.Header.Revision =3D + EFI_ACPI_6_3_HARDWARE_ERROR_SOURCE_TABLE_REVISION; + CopyMem ( + &HestHeader.Header.OemId, + FixedPcdGetPtr (PcdAcpiDefaultOemId), + sizeof (HestHeader.Header.OemId) + ); + TempOemTableId =3D FixedPcdGet64 (PcdAcpiDefaultOemTableId); + CopyMem ( + &HestHeader.Header.OemTableId, + &TempOemTableId, + sizeof (HestHeader.Header.OemTableId) + ); + HestHeader.Header.OemRevision =3D PcdGet32 (PcdAcpiDefaultOemRevision)= ; + HestHeader.Header.CreatorId =3D PcdGet32 (PcdAcpiDefaultCreatorId); + HestHeader.Header.CreatorRevision =3D PcdGet32 (PcdAcpiDefaultCreatorR= evision); + HestHeader.ErrorSourceCount =3D 0; + CopyMem (mHestDriverData.HestTable, &HestHeader, sizeof (HestHeader)); + + return EFI_SUCCESS; +} + +/** + Append HEST error source descriptor protocol service. + + Protocol service used to append newly collected error source descripto= rs to + to an already created HEST table. + + @param[in] ErrorSourceDescriptorList List of Error Source Descri= ptors. + @param[in] ErrorSourceDescriptorListSize Total Size of Error Source + Descriptors. + @param[in] ErrorSourceDescriptorCount Total count of error source + descriptors. + + @retval EFI_SUCCESS Appending the error source descriptors + successful. + @retval EFI_OUT_OF_RESOURCES Buffer reallocation failed for the Hes= t + table. + @retval EFI_INVALID_PARAMETER Null ErrorSourceDescriptorList param o= r +**/ +STATIC +EFI_STATUS +EFIAPI +AppendErrorSourceDescriptor ( + IN CONST VOID *ErrorSourceDescriptorList, + IN UINTN ErrorSourceDescriptorListSize, + IN UINTN ErrorSourceDescriptorCount + ) +{ + EFI_ACPI_6_3_HARDWARE_ERROR_SOURCE_TABLE_HEADER *HestHeaderPtr; + EFI_STATUS Status; + UINT32 NewTableSize; + VOID *ErrorDescriptorPtr; + + if ((ErrorSourceDescriptorList =3D=3D NULL) || + (ErrorSourceDescriptorListSize =3D=3D 0)) { + return EFI_INVALID_PARAMETER; + } + + // + // Create a HEST table header if not already created. + // + if (mHestDriverData.HestTable =3D=3D NULL) { + Status =3D BuildHestHeader (); + if (Status =3D=3D EFI_OUT_OF_RESOURCES) { + DEBUG (( + DEBUG_ERROR, + "%a: Failed to build HEST header, status: %r\n", + __FUNCTION__, + Status + )); + return Status; + } + } + + // + // Resize the existing HEST table buffer to accommodate the incoming e= rror + // source descriptors. + // + NewTableSize =3D mHestDriverData.CurrentTableSize + + ErrorSourceDescriptorListSize; + mHestDriverData.HestTable =3D ReallocateHestTableMemory ( + mHestDriverData.CurrentTableSize, + NewTableSize, + mHestDriverData.HestTable + ); + if (mHestDriverData.HestTable =3D=3D NULL) { + DEBUG (( + DEBUG_ERROR, + "%a: Failed to reallocate memory for HEST table\n", + __FUNCTION__ + )); + return EFI_OUT_OF_RESOURCES; + } + + // + // Copy the incoming error source descriptors into HEST table. + // + ErrorDescriptorPtr =3D (VOID *)mHestDriverData.HestTable + + mHestDriverData.CurrentTableSize; + HestHeaderPtr =3D (EFI_ACPI_6_3_HARDWARE_ERROR_SOURCE_TABLE_HEADER *) + mHestDriverData.HestTable; + CopyMem ( + ErrorDescriptorPtr, + ErrorSourceDescriptorList, + ErrorSourceDescriptorListSize + ); + mHestDriverData.CurrentTableSize =3D NewTableSize; + HestHeaderPtr->Header.Length =3D mHestDriverData.CurrentTableSize; + HestHeaderPtr->ErrorSourceCount +=3D ErrorSourceDescriptorCount; + + DEBUG (( + DEBUG_INFO, + "HestDxe: %d Error source descriptor(s) added \n", + ErrorSourceDescriptorCount + )); + return EFI_SUCCESS; +} + +/** + Install HEST table protocol service. + + Installs the HEST table that has been appended with the error source + descriptors. The checksum of this table is calculated and updated in + the table header. The HEST table is then installed. + + @retval EFI_SUCCESS HEST table is installed successfully. + @retval EFI_ABORTED HEST table is NULL. + @retval Other Install service call failed. +**/ +STATIC +EFI_STATUS +EFIAPI +InstallHestAcpiTable (VOID) +{ + EFI_ACPI_6_3_HARDWARE_ERROR_SOURCE_TABLE_HEADER *HestHeader; + EFI_STATUS Status; + UINTN AcpiTableHandle; + + // + // Check if we have valid HEST table data. If not, there no hardware e= rror + // sources supported by the platform and no HEST table to publish, ret= urn. + // + if (mHestDriverData.HestTable =3D=3D NULL) { + DEBUG (( + DEBUG_INFO, + "HestDxe: No data available to generate HEST table\n" + )); + return EFI_SUCCESS; + } + + // + // Valid data for HEST table found. Update the header checksum and ins= tall the + // HEST table. + // + HestHeader =3D (EFI_ACPI_6_3_HARDWARE_ERROR_SOURCE_TABLE_HEADER *) + mHestDriverData.HestTable; + HestHeader->Header.Checksum =3D CalculateCheckSum8 ( + (UINT8 *)HestHeader, + HestHeader->Header.Length + ); + + Status =3D mAcpiTableProtocol->InstallAcpiTable ( + mAcpiTableProtocol, + HestHeader, + HestHeader->Header.Length, + &AcpiTableHandle + ); + if (EFI_ERROR (Status)) { + DEBUG (( + DEBUG_ERROR, + "%a: HEST table installation failed, status: %r\n", + __FUNCTION__, + Status + )); + return Status; + } + + // + // Free the HEST table buffer. + // + FreePool (mHestDriverData.HestTable); + DEBUG (( + DEBUG_INFO, + "HestDxe: Installed HEST table \n" + )); + return EFI_SUCCESS; +} + +// +// HEST table generation protocol instance. +// +STATIC HEST_TABLE_PROTOCOL mHestProtocol =3D { + AppendErrorSourceDescriptor, + InstallHestAcpiTable +}; + +/** + The Entry Point for HEST Dxe driver. + + This function installs the HEST table creation and installation protoc= ol + services. + + @param[in] ImageHandle Handle to the Efi image. + @param[in] SystemTable A pointer to the Efi System Table. + + @retval EFI_SUCCESS On successful installation of protocol services= and + location the ACPI table protocol. + @retval Other On Failure to locate ACPI table protocol or ins= tall + of HEST table generation protocol. +**/ +EFI_STATUS +EFIAPI +HestInitialize ( + IN EFI_HANDLE ImageHandle, + IN EFI_SYSTEM_TABLE *SystemTable + ) +{ + EFI_HANDLE Handle =3D NULL; + EFI_STATUS Status; + + Status =3D gBS->LocateProtocol ( + &gEfiAcpiTableProtocolGuid, + NULL, + (VOID **)&mAcpiTableProtocol + ); + if (EFI_ERROR (Status)) { + DEBUG (( + DEBUG_ERROR, + "%a: Failed to locate ACPI table protocol, status: %r\n", + __FUNCTION__, + Status + )); + return Status; + } + + Status =3D gBS->InstallProtocolInterface ( + &Handle, + &gHestTableProtocolGuid, + EFI_NATIVE_INTERFACE, + &mHestProtocol + ); + if (EFI_ERROR (Status)) { + DEBUG (( + DEBUG_ERROR, + "%a: Failed to install HEST table generation protocol status: %r\n= ", + __FUNCTION__, + Status + )); + return Status; + } + + return EFI_SUCCESS; +} --=20 2.17.1