On 11/6/23 04:30, Chao Li wrote: > Use a register to save PeiServiceTable pointer. This Library provides > PeiServiceTable pointer saveing and retrieval serivces. > > BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=4584 > > Cc: Ard Biesheuvel <ardb+tianoc...@kernel.org> > Cc: Jiewen Yao <jiewen....@intel.com> > Cc: Jordan Justen <jordan.l.jus...@intel.com> > Cc: Gerd Hoffmann <kra...@redhat.com> > Signed-off-by: Chao Li <lic...@loongson.cn> > Co-authored-by: Xianglai Li <lixiang...@loongson.cn> > Co-authored-by: Bibo Mao <maob...@loongson.cn> > --- > .../PeiServicesTablePointer.c | 75 +++++++++++++++++++ > .../PeiServicesTablePointerLib.inf | 31 ++++++++ > 2 files changed, 106 insertions(+) > create mode 100644 > OvmfPkg/LoongArchVirt/Library/PeiServicesTablePointerLib/PeiServicesTablePointer.c > create mode 100644 > OvmfPkg/LoongArchVirt/Library/PeiServicesTablePointerLib/PeiServicesTablePointerLib.inf
Why is this under OvmfPkg? Laszlo > > diff --git > a/OvmfPkg/LoongArchVirt/Library/PeiServicesTablePointerLib/PeiServicesTablePointer.c > > b/OvmfPkg/LoongArchVirt/Library/PeiServicesTablePointerLib/PeiServicesTablePointer.c > new file mode 100644 > index 0000000000..dcbcb50131 > --- /dev/null > +++ > b/OvmfPkg/LoongArchVirt/Library/PeiServicesTablePointerLib/PeiServicesTablePointer.c > @@ -0,0 +1,75 @@ > +/** @file > + PEI Services Table Pointer Library. > + > + 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> > +#include <Library/BaseLib.h> > +#include <Register/LoongArch64/Csr.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 > + ) > +{ > + CsrWrite (LOONGARCH_CSR_KS0, (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 > + ) > +{ > + return (CONST EFI_PEI_SERVICES **)(CsrRead (LOONGARCH_CSR_KS0)); > +} > + > +/** > +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 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. > +**/ > +VOID > +EFIAPI > +MigratePeiServicesTablePointer ( > + VOID > + ) > +{ > + return; > +} > diff --git > a/OvmfPkg/LoongArchVirt/Library/PeiServicesTablePointerLib/PeiServicesTablePointerLib.inf > > b/OvmfPkg/LoongArchVirt/Library/PeiServicesTablePointerLib/PeiServicesTablePointerLib.inf > new file mode 100644 > index 0000000000..274cb2f781 > --- /dev/null > +++ > b/OvmfPkg/LoongArchVirt/Library/PeiServicesTablePointerLib/PeiServicesTablePointerLib.inf > @@ -0,0 +1,31 @@ > +## @file > +# PEI Services Table Pointer Library. > +# > +# 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 > + FILE_GUID = 098B0DB0-AD01-8886-D409-90CBC7E89154 > + MODULE_TYPE = PEIM > + VERSION_STRING = 1.0 > + LIBRARY_CLASS = PeiServicesTablePointerLib|PEIM PEI_CORE > SEC > + > +# > +# VALID_ARCHITECTURES = LOONGARCH64 > +# > + > +[Sources] > + PeiServicesTablePointer.c > + > +[Packages] > + MdePkg/MdePkg.dec > + > +[LibraryClasses] > + BaseLib > + DebugLib > + > +[Pcd] -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#110932): https://edk2.groups.io/g/devel/message/110932 Mute This Topic: https://groups.io/mt/102413901/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/leave/9847357/21656/1706620634/xyzzy [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-