From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from linux.microsoft.com (linux.microsoft.com [13.77.154.182]) by mx.groups.io with SMTP id smtpd.web10.348.1649698951318313685 for ; Mon, 11 Apr 2022 10:42:31 -0700 Authentication-Results: mx.groups.io; dkim=fail reason="body hash did not verify" header.i=@linux.microsoft.com header.s=default header.b=IIIa7NGC; spf=pass (domain: linux.microsoft.com, ip: 13.77.154.182, mailfrom: mikuback@linux.microsoft.com) Received: from localhost.localdomain (unknown [47.201.46.36]) by linux.microsoft.com (Postfix) with ESMTPSA id 8413C2058429; Mon, 11 Apr 2022 10:42:27 -0700 (PDT) DKIM-Filter: OpenDKIM Filter v2.11.0 linux.microsoft.com 8413C2058429 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.microsoft.com; s=default; t=1649698950; bh=90Qt2iNMDxEEQodcJchLJEGaFjA7P0k9oSpdGaPnLrw=; h=From:To:Cc:Subject:Date:From; b=IIIa7NGCRv0xHOmbpPkpIgM0agUjF/8NTltXjkw8QmSkpNXjp9fXdTo7NEbu/y5D+ 1gGHDEwHWKu1tmK4/loRwxUjBNHw5Xb9maOIei75papFuUUcvIevvIqLH5UYHiqnLe R5eMR2wsswxEMSZgyJu1A0UJCYXYORGdrddZNUU4= From: "Michael Kubacki" To: devel@edk2.groups.io Cc: Abner Chang , Andrew Fish , Anthony Perard , Ard Biesheuvel , Benjamin You , Brijesh Singh , Erdem Aktas , Gerd Hoffmann , Guo Dong , Hao A Wu , James Bottomley , Jian J Wang , Jiewen Yao , Jordan Justen , Julien Grall , Leif Lindholm , Liming Gao , Maurice Ma , Min Xu , Nickle Wang , Peter Grehan , Ray Ni , Rebecca Cran , Sami Mujawar , Sean Rhodes , Sebastien Boeuf , Tom Lendacky Subject: [PATCH v3 0/8] Add Variable Flash Info HOB Date: Mon, 11 Apr 2022 13:42:01 -0400 Message-Id: <20220411174209.2804-1-mikuback@linux.microsoft.com> X-Mailer: git-send-email 2.28.0.windows.1 MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable From: Michael Kubacki REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3D3479 The UEFI variable drivers such as VariableRuntimeDxe, VariableSmm, VariableStandaloneMm, etc. (and their dependent protocol/library stack), typically acquire UEFI variable store flash information with PCDs declared in MdeModulePkg. For example: [Pcd] gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageVariableBase gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageVariableBase64 gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageVariableSize These PCDs work as-is in the StandaloneMm driver if they are not dynamic such as Dynamic or DynamicEx because PCD services are not readily available in the Standalone MM environment. Platforms that use Standalone MM today, must define these PCDs as FixedAtBuild in their platform build. However, the PCDs do allow platforms to treat the PCDs as Dynamic/DynamicEx and being able to support that is currently a gap for Standalone MM. This patch series introduces a HOB that can be produced by the platform to provide the same information. The HOB list is available to Standalone MM. The PCD declarations are left as-is in MdeModulePkg for backward compatibility. This means unless a platform wants to use the HOB, their code will continue to work with no change (they do not need to produce the HOB). Only if the HOB is found, is its value used instead of the PCDs. Due to the large number of consumers of this information, access to the base address and size values is abstracted in a new library class (as requested in the v1 series) called VariableFlashInfoLib. The API of VariableFlashInfoLib does not bind the underlying data structure to the information returned to library users to allow flexibility in the library implementation in the future. V3 changes: 1. To better clarify usage, renamed the members "NvStorageBaseAddress" and "NvStorageLength" in "VARIABLE_FLASH_INFO" to "NvVariableBaseAddress" and "NvVariableLength". 2. Added description comments to the fields in "VARIABLE_FLASH_INFO". V2 changes: 1. Abstracted flash info data access with VariableFlashInfoLib. 2. Updated package builds in the repo that build the variable and FTW drivers to include VariableFlashInfoLib. 3. Removed a redundant variable assignment in VariableSmm.c. 4. Updated comments in FtwMisc.c and FaultTolerantWritePei.c to indicate driver assumption is UINTN (not UINT32) 5. Added a version field to the VARIABLE_FLASH_INFO structure. Cc: Abner Chang Cc: Andrew Fish Cc: Anthony Perard Cc: Ard Biesheuvel Cc: Benjamin You Cc: Brijesh Singh Cc: Erdem Aktas Cc: Gerd Hoffmann Cc: Guo Dong Cc: Hao A Wu Cc: James Bottomley Cc: Jian J Wang Cc: Jiewen Yao Cc: Jordan Justen Cc: Julien Grall Cc: Leif Lindholm Cc: Liming Gao Cc: Maurice Ma Cc: Min Xu Cc: Nickle Wang Cc: Peter Grehan Cc: Ray Ni Cc: Rebecca Cran Cc: Sami Mujawar Cc: Sean Rhodes Cc: Sebastien Boeuf Cc: Tom Lendacky Signed-off-by: Michael Kubacki Michael Kubacki (8): MdeModulePkg: Add Variable Flash Info HOB MdeModulePkg/VariableFlashInfoLib: Add initial library MdeModulePkg/Variable: Consume Variable Flash Info MdeModulePkg/FaultTolerantWrite: Consume Variable Flash Info ArmVirtPkg/ArmVirt.dsc.inc: Add VariableFlashInfoLib EmulatorPkg: Add VariableFlashInfoLib OvmfPkg: Add VariableFlashInfoLib UefiPayloadPkg: Add VariableFlashInfoLib MdeModulePkg/Library/BaseVariableFlashInfoLib/BaseVariableFlashInfoLib.c= | 178 ++++++++++++++++++++ MdeModulePkg/Universal/FaultTolerantWriteDxe/FtwMisc.c = | 41 +++-- MdeModulePkg/Universal/FaultTolerantWriteDxe/UpdateWorkingBlock.c = | 7 +- MdeModulePkg/Universal/FaultTolerantWritePei/FaultTolerantWritePei.c = | 28 +-- MdeModulePkg/Universal/Variable/Pei/Variable.c = | 14 +- MdeModulePkg/Universal/Variable/RuntimeDxe/VariableDxe.c = | 16 +- MdeModulePkg/Universal/Variable/RuntimeDxe/VariableNonVolatile.c = | 14 +- MdeModulePkg/Universal/Variable/RuntimeDxe/VariableSmm.c = | 17 +- ArmVirtPkg/ArmVirt.dsc.inc = | 1 + EmulatorPkg/EmulatorPkg.dsc = | 1 + MdeModulePkg/Include/Guid/VariableFlashInfo.h = | 78 +++++++++ MdeModulePkg/Include/Library/VariableFlashInfoLib.h = | 68 ++++++++ MdeModulePkg/Library/BaseVariableFlashInfoLib/BaseVariableFlashInfoLib.i= nf | 48 ++++++ MdeModulePkg/Library/BaseVariableFlashInfoLib/BaseVariableFlashInfoLib.u= ni | 12 ++ MdeModulePkg/MdeModulePkg.dec = | 8 + MdeModulePkg/MdeModulePkg.dsc = | 2 + MdeModulePkg/Universal/FaultTolerantWriteDxe/FaultTolerantWrite.h = | 7 +- MdeModulePkg/Universal/FaultTolerantWriteDxe/FaultTolerantWriteDxe.inf = | 10 +- MdeModulePkg/Universal/FaultTolerantWriteDxe/FaultTolerantWriteSmm.inf = | 10 +- MdeModulePkg/Universal/FaultTolerantWriteDxe/FaultTolerantWriteStandalon= eMm.inf | 10 +- MdeModulePkg/Universal/FaultTolerantWritePei/FaultTolerantWritePei.inf = | 10 +- MdeModulePkg/Universal/Variable/Pei/Variable.h = | 2 + MdeModulePkg/Universal/Variable/Pei/VariablePei.inf = | 5 +- MdeModulePkg/Universal/Variable/RuntimeDxe/Variable.h = | 7 +- MdeModulePkg/Universal/Variable/RuntimeDxe/VariableRuntimeDxe.inf = | 5 +- MdeModulePkg/Universal/Variable/RuntimeDxe/VariableSmm.inf = | 5 +- MdeModulePkg/Universal/Variable/RuntimeDxe/VariableStandaloneMm.inf = | 5 +- OvmfPkg/AmdSev/AmdSevX64.dsc = | 1 + OvmfPkg/Bhyve/BhyveX64.dsc = | 1 + OvmfPkg/CloudHv/CloudHvX64.dsc = | 1 + OvmfPkg/IntelTdx/IntelTdxX64.dsc = | 1 + OvmfPkg/Microvm/MicrovmX64.dsc = | 1 + OvmfPkg/OvmfPkgIa32.dsc = | 1 + OvmfPkg/OvmfPkgIa32X64.dsc = | 1 + OvmfPkg/OvmfPkgX64.dsc = | 1 + OvmfPkg/OvmfXen.dsc = | 1 + UefiPayloadPkg/UefiPayloadPkg.dsc = | 1 + 37 files changed, 525 insertions(+), 94 deletions(-) create mode 100644 MdeModulePkg/Library/BaseVariableFlashInfoLib/BaseVar= iableFlashInfoLib.c create mode 100644 MdeModulePkg/Include/Guid/VariableFlashInfo.h create mode 100644 MdeModulePkg/Include/Library/VariableFlashInfoLib.h create mode 100644 MdeModulePkg/Library/BaseVariableFlashInfoLib/BaseVar= iableFlashInfoLib.inf create mode 100644 MdeModulePkg/Library/BaseVariableFlashInfoLib/BaseVar= iableFlashInfoLib.uni --=20 2.28.0.windows.1