Not my package, just spotted a typo below:
On Fri, Nov 17, 2023 at 17:59:49 +0800, Chao Li wrote:
> Since some ARCH or platform not require execute code on memory during
> PEI phase, some values may transferred via CPU registers.
>
> Adding PeiServcieTablePointerLibReg to allow set and get the PEI service
> table pointer depend by a CPU register, this library can accommodate lot
> of platforms who not require execte code on memory during PEI phase.
>
> Adding PeiServiceTablePointerLibReg to allows setting and getting the
> PEI service table pointer via CPU registers, and the library can
> accommodate many platforms that do not need to execute code on memory
> during the PEI phase.
>
> The idea of this library is derived from
> ArmPkg/Library/PeiServicesTablePointerLib/
>
> BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=4584
>
> Cc: Michael D Kinney <michael.d.kin...@intel.com>
> Cc: Liming Gao <gaolim...@byosoft.com.cn>
> Cc: Zhiguang Liu <zhiguang....@intel.com>
> Cc: Leif Lindholm <quic_llind...@quicinc.com>
> Cc: Ard Biesheuvel <ardb+tianoc...@kernel.org>
> Cc: Sami Mujawar <sami.muja...@arm.com>
> Cc: Laszlo Ersek <ler...@redhat.com>
> Cc: Sunil V L <suni...@ventanamicro.com>
> Signed-off-by: Chao Li <lic...@loongson.cn>
> ---
> .../Library/PeiServicesTablePointerLib.h | 37 +++++++-
> .../PeiServicesTablePointer.c | 86 +++++++++++++++++++
> .../PeiServicesTablePointerLib.uni | 20 +++++
> .../PeiServicesTablePointerLibReg.inf | 40 +++++++++
> MdePkg/MdePkg.dsc | 1 +
> 5 files changed, 180 insertions(+), 4 deletions(-)
> create mode 100644
> MdePkg/Library/PeiServicesTablePointerLibReg/PeiServicesTablePointer.c
> create mode 100644
> MdePkg/Library/PeiServicesTablePointerLibReg/PeiServicesTablePointerLib.uni
> create mode 100644
> MdePkg/Library/PeiServicesTablePointerLibReg/PeiServicesTablePointerLibReg.inf
>
> diff --git a/MdePkg/Include/Library/PeiServicesTablePointerLib.h
> b/MdePkg/Include/Library/PeiServicesTablePointerLib.h
> index 61635eff00..f5c764cb13 100644
> --- a/MdePkg/Include/Library/PeiServicesTablePointerLib.h
> +++ b/MdePkg/Include/Library/PeiServicesTablePointerLib.h
> @@ -52,10 +52,11 @@ SetPeiServicesTablePointer (
> immediately preceding the Interrupt Descriptor Table (IDT) in memory.
> For X64 CPUs, the PEI Services Table pointer is stored in the 8 bytes
> immediately preceding the Interrupt Descriptor Table (IDT) in memory.
> - For Itanium and ARM CPUs, a the PEI Services Table Pointer is stored in
> - a dedicated CPU register. This means that there is no memory storage
> - associated with storing the PEI Services Table pointer, so no additional
> - migration actions are required for Itanium or ARM CPUs.
> + For Itanium, ARM and LoongArch CPUs, a the PEI Services Table Pointer
> + is stored in a dedicated CPU register. This means that there is no
> + memory storage associated with storing the PEI Services Table pointer,
> + so no additional migration actions are required for Itanium, ARM and
> + LoongArch CPUs.
>
> **/
> VOID
> @@ -64,4 +65,32 @@ MigratePeiServicesTablePointer (
> VOID
> );
>
> +/**
> + Retrieves the cached value of the PEI Services Table pointer from a CPU
> register.
> +
> + Returns the cached value of the PEI Services Table pointer in a CPU
> specific manner
> + as specified in the CPU binding section of the Platform Initialization
> Pre-EFI
> + Initialization Core Interface Specification.
> +
> + @return The pointer to PeiServices.
> +**/
> +CONST EFI_PEI_SERVICES **
> +EFIAPI
> +GetPeiServicesTablePointerFromRegister (
> + VOID
> + );
> +
> +/**
> + Set the pointer PEI Service Table to a CPU register.
> +
> + Caches the pointer to the PEI Services Table specified by
> PeiServicesTablePointer
> + in a platform specific manner.
> +
> + @param PeiServicesTablePointer The address of PeiServices.
> +**/
> +VOID
> +EFIAPI
> +SetPeiServicesTablePointerToRegester (
SetPeiServicesTablePointerToRegester ->
SetPeiServicesTablePointerToRegister
Regester -> Register.
/
Leif
> + IN UINTN PeiServicesTablePointer
> + );
> #endif
> diff --git
> a/MdePkg/Library/PeiServicesTablePointerLibReg/PeiServicesTablePointer.c
> b/MdePkg/Library/PeiServicesTablePointerLibReg/PeiServicesTablePointer.c
> new file mode 100644
> index 0000000000..0227f98871
> --- /dev/null
> +++ b/MdePkg/Library/PeiServicesTablePointerLibReg/PeiServicesTablePointer.c
> @@ -0,0 +1,86 @@
> +/** @file
> + PEI Services Table Pointer Library For Reigseter Mechanism.
> +
> + This library is used for PEIM which does executed from flash device
> directly but
> + executed in memory.
> +
> + Copyright (c) 2006 - 2010, Intel Corporation. All rights reserved.<BR>
> + Copyright (c) 2011 Hewlett-Packard Corporation. All rights reserved.<BR>
> + Copyright (c) 2023 Loongson Technology Corporation Limited. All rights
> reserved.<BR>
> +
> + SPDX-License-Identifier: BSD-2-Clause-Patent
> +
> +**/
> +
> +#include <PiPei.h>
> +#include <Library/PeiServicesTablePointerLib.h>
> +#include <Library/DebugLib.h>
> +
> +/**
> + Caches a pointer PEI Services Table.
> +
> + Caches the pointer to the PEI Services Table specified by
> PeiServicesTablePointer
> + in a platform specific manner.
> +
> + If PeiServicesTablePointer is NULL, then ASSERT().
> +
> + @param PeiServicesTablePointer The address of PeiServices pointer.
> +**/
> +VOID
> +EFIAPI
> +SetPeiServicesTablePointer (
> + IN CONST EFI_PEI_SERVICES **PeiServicesTablePointer
> + )
> +{
> + ASSERT (PeiServicesTablePointer != NULL);
> + SetPeiServicesTablePointerToRegester ((UINTN)PeiServicesTablePointer);
> +}
> +
> +/**
> + Retrieves the cached value of the PEI Services Table pointer.
> +
> + Returns the cached value of the PEI Services Table pointer in a CPU
> specific manner
> + as specified in the CPU binding section of the Platform Initialization
> Pre-EFI
> + Initialization Core Interface Specification.
> +
> + If the cached PEI Services Table pointer is NULL, then ASSERT().
> +
> + @return The pointer to PeiServices.
> +
> +**/
> +CONST EFI_PEI_SERVICES **
> +EFIAPI
> +GetPeiServicesTablePointer (
> + VOID
> + )
> +{
> + CONST EFI_PEI_SERVICES **PeiServices;
> +
> + PeiServices = GetPeiServicesTablePointerFromRegister ();
> + ASSERT (PeiServices != NULL);
> + return PeiServices;
> +}
> +
> +/**
> + Perform CPU specific actions required to migrate the PEI Services Table
> + pointer from temporary RAM to permanent RAM.
> +
> + For IA32 CPUs, the PEI Services Table pointer is stored in the 4 bytes
> + immediately preceding the Interrupt Descriptor Table (IDT) in memory.
> + For X64 CPUs, the PEI Services Table pointer is stored in the 8 bytes
> + immediately preceding the Interrupt Descriptor Table (IDT) in memory.
> + For Itanium, ARM and LoongArch CPUs, a the PEI Services Table Pointer
> + is stored in a dedicated CPU register. This means that there is no
> + memory storage associated with storing the PEI Services Table pointer,
> + so no additional migration actions are required for Itanium, ARM and
> + LoongArch CPUs.
> +
> +**/
> +VOID
> +EFIAPI
> +MigratePeiServicesTablePointer (
> + VOID
> + )
> +{
> + return;
> +}
> diff --git
> a/MdePkg/Library/PeiServicesTablePointerLibReg/PeiServicesTablePointerLib.uni
> b/MdePkg/Library/PeiServicesTablePointerLibReg/PeiServicesTablePointerLib.uni
> new file mode 100644
> index 0000000000..937cf857d9
> --- /dev/null
> +++
> b/MdePkg/Library/PeiServicesTablePointerLibReg/PeiServicesTablePointerLib.uni
> @@ -0,0 +1,20 @@
> +// /** @file
> +// Instance of PEI Services Table Pointer Library using CPU register for the
> table pointer.
> +//
> +// PEI Services Table Pointer Library implementation that retrieves a
> pointer to the
> +// PEI Services Table from a CPU register. Applies to modules that execute
> from
> +// read-only memory.
> +//
> +// Copyright (c) 2007 - 2018, Intel Corporation. All rights reserved.<BR>
> +// Copyright (c) 2011 Hewlett-Packard Corporation. All rights reserved.<BR>
> +// Copyright (c) 2023 Loongson Technology Corporation Limited. All rights
> reserved.<BR>
> +//
> +// SPDX-License-Identifier: BSD-2-Clause-Patent
> +//
> +// **/
> +
> +
> +#string STR_MODULE_ABSTRACT #language en-US "Instance of PEI
> Services Table Pointer Library using CPU register for the table pointer"
> +
> +#string STR_MODULE_DESCRIPTION #language en-US "The PEI Services
> Table Pointer Library implementation that retrieves a pointer to the PEI
> Services Table from a CPU register. Applies to modules that execute from
> read-only memory."
> +
> diff --git
> a/MdePkg/Library/PeiServicesTablePointerLibReg/PeiServicesTablePointerLibReg.inf
>
> b/MdePkg/Library/PeiServicesTablePointerLibReg/PeiServicesTablePointerLibReg.inf
> new file mode 100644
> index 0000000000..22499e22ad
> --- /dev/null
> +++
> b/MdePkg/Library/PeiServicesTablePointerLibReg/PeiServicesTablePointerLibReg.inf
> @@ -0,0 +1,40 @@
> +## @file
> +# Instance of PEI Services Table Pointer Library using CPU register for the
> table pointer.
> +#
> +# PEI Services Table Pointer Library implementation that retrieves a pointer
> to the
> +# PEI Services Table from a CPU register. Applies to modules that execute
> from
> +# read-only memory.
> +#
> +# Copyright (c) 2007 - 2018, Intel Corporation. All rights reserved.<BR>
> +# Copyright (c) 2011 Hewlett-Packard Corporation. All rights reserved.<BR>
> +# Copyright (c) 2023 Loongson Technology Corporation Limited. All rights
> reserved.<BR>
> +#
> +# SPDX-License-Identifier: BSD-2-Clause-Patent
> +#
> +#
> +##
> +
> +[Defines]
> + INF_VERSION = 0x00010005
> + BASE_NAME = PeiServicesTablePointerLib
> + MODULE_UNI_FILE = PeiServicesTablePointerLib.uni
> + FILE_GUID = 619950D1-7C5F-EA1B-D6DD-2FF7B0A4A2B7
> + MODULE_TYPE = PEIM
> + VERSION_STRING = 1.0
> + LIBRARY_CLASS = PeiServicesTablePointerLib|PEIM PEI_CORE
> SEC
> +
> +#
> +# VALID_ARCHITECTURES = IA32 X64 AARCH64 RISCV64 LOONGARCH64
> +#
> +
> +[Sources]
> + PeiServicesTablePointer.c
> +
> +[Packages]
> + MdePkg/MdePkg.dec
> +
> +[LibraryClasses]
> + DebugLib
> +
> +[Pcd]
> +
> diff --git a/MdePkg/MdePkg.dsc b/MdePkg/MdePkg.dsc
> index 3abd1a1e23..2e9a3d4b4c 100644
> --- a/MdePkg/MdePkg.dsc
> +++ b/MdePkg/MdePkg.dsc
> @@ -103,6 +103,7 @@
> MdePkg/Library/PeiResourcePublicationLib/PeiResourcePublicationLib.inf
> MdePkg/Library/PeiServicesLib/PeiServicesLib.inf
> MdePkg/Library/PeiServicesTablePointerLib/PeiServicesTablePointerLib.inf
> +
> MdePkg/Library/PeiServicesTablePointerLibReg/PeiServicesTablePointerLibReg.inf
> MdePkg/Library/PeiSmbusLibSmbus2Ppi/PeiSmbusLibSmbus2Ppi.inf
> MdePkg/Library/PeiPciLibPciCfg2/PeiPciLibPciCfg2.inf
> MdePkg/Library/PeiPciSegmentLibPciCfg2/PeiPciSegmentLibPciCfg2.inf
> --
> 2.27.0
>
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#111393): https://edk2.groups.io/g/devel/message/111393
Mute This Topic: https://groups.io/mt/102644754/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe:
https://edk2.groups.io/g/devel/leave/9847357/21656/1706620634/xyzzy
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
