From 9f9bf82209d2b7b2d0f9221c9e6c5eb901b9f772 Mon Sep 17 00:00:00 2001 From: Laszlo Ersek Date: Tue, 5 Mar 2024 12:38:36 +0100 Subject: [PATCH] OvmfPkg/IntelTdx: auto-gen & fix SEC ProcessLibraryConstructorList() decl declares a bogus ProcessLibraryConstructorList() for IntelTdx's SEC module. Rely on AutoGen for (properly) declaring ProcessLibraryConstructorList(). Update the call. Build-tested with: build -a X64 -b DEBUG -m OvmfPkg/IntelTdx/Sec/SecMain.inf \ -p OvmfPkg/IntelTdx/IntelTdxX64.dsc -t GCC5 Cc: Ard Biesheuvel Cc: Gerd Hoffmann Cc: Jiewen Yao Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=990 Signed-off-by: Laszlo Ersek Message-Id: <20240305113843.68812-4-lersek@redhat.com> Reviewed-by: Jiewen Yao Acked-by: Gerd Hoffmann --- OvmfPkg/IntelTdx/Sec/SecMain.c | 3 +-- OvmfPkg/IntelTdx/Sec/SecMain.inf | 2 +- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/OvmfPkg/IntelTdx/Sec/SecMain.c b/OvmfPkg/IntelTdx/Sec/SecMain.c index 42a587adfa..4e750755bf 100644 --- a/OvmfPkg/IntelTdx/Sec/SecMain.c +++ b/OvmfPkg/IntelTdx/Sec/SecMain.c @@ -11,7 +11,6 @@ #include -#include #include #include #include @@ -125,7 +124,7 @@ SecCoreStartupWithStack ( IdtDescriptor.Base = (UINTN)&IdtTableInStack.IdtTable; IdtDescriptor.Limit = (UINT16)(sizeof (IdtTableInStack.IdtTable) - 1); - ProcessLibraryConstructorList (NULL, NULL); + ProcessLibraryConstructorList (); // // Load the IDTR. diff --git a/OvmfPkg/IntelTdx/Sec/SecMain.inf b/OvmfPkg/IntelTdx/Sec/SecMain.inf index 7fb27941b7..cf95284f02 100644 --- a/OvmfPkg/IntelTdx/Sec/SecMain.inf +++ b/OvmfPkg/IntelTdx/Sec/SecMain.inf @@ -8,7 +8,7 @@ ## [Defines] - INF_VERSION = 0x00010005 + INF_VERSION = 1.30 BASE_NAME = SecMain FILE_GUID = 69d96630-eb64-4b33-8491-13a5cf023dcf MODULE_TYPE = SEC