]> xenbits.xensource.com Git - ovmf.git/commitdiff
MdePkg/Include/Ppi: Remove Itanium leftover data structure
authorPaweł Poławski <ppolawsk@redhat.com>
Fri, 24 Mar 2023 14:36:59 +0000 (07:36 -0700)
committermergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Sun, 26 Mar 2023 02:03:50 +0000 (02:03 +0000)
Itanium support has been removed from EDK2 around 2019.
ITANIUM_HANDOFF_STATUS data structure looks to be
some leftover from that process.

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1560
GitHub: https://github.com/tianocore/edk2/commit/4e1daa60f5372c22a11503961061ffa569eaf873

There is also positive side effect of this data structure removal.
Due to HOB allocation type used in PEI stage there is a limit
how much data about virtual CPU can be hold. This limit result
in only 1024 vCPU can be used by VM.

With Itanium related data structure removed more allocated space
can be used for vCPU data and with current allocation limit
will change from 1024 to around 8k vCPUs.

Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Cc: Zhiguang Liu <zhiguang.liu@intel.com>
Signed-off-by: Paweł Poławski <ppolawsk@redhat.com>
Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com>
MdePkg/Include/Ppi/SecPlatformInformation.h

index 02b0711f189e0b9563d50d9143516a2df83a3965..c66183eb16778028cba869c8fdb4cd2e1eeed002 100644 (file)
@@ -84,57 +84,16 @@ typedef union {
 \r
 typedef EFI_HEALTH_FLAGS X64_HANDOFF_STATUS;\r
 typedef EFI_HEALTH_FLAGS IA32_HANDOFF_STATUS;\r
-///\r
-/// The hand-off status structure for Itanium architecture.\r
-///\r
-typedef struct {\r
-  ///\r
-  /// SALE_ENTRY state : 3 = Recovery_Check\r
-  /// and 0 = RESET or Normal_Boot phase.\r
-  ///\r
-  UINT8     BootPhase;\r
-  ///\r
-  /// Firmware status on entry to SALE.\r
-  ///\r
-  UINT8     FWStatus;\r
-  UINT16    Reserved1;\r
-  UINT32    Reserved2;\r
-  ///\r
-  /// Geographically significant unique processor ID assigned by PAL.\r
-  ///\r
-  UINT16    ProcId;\r
-  UINT16    Reserved3;\r
-  UINT8     IdMask;\r
-  UINT8     EidMask;\r
-  UINT16    Reserved4;\r
-  ///\r
-  /// Address to make PAL calls.\r
-  ///\r
-  UINT64    PalCallAddress;\r
-  ///\r
-  /// If the entry state is RECOVERY_CHECK, this contains the PAL_RESET\r
-  /// return address, and if entry state is RESET, this contains\r
-  /// address for PAL_authentication call.\r
-  ///\r
-  UINT64    PalSpecialAddress;\r
-  ///\r
-  /// GR35 from PALE_EXIT state.\r
-  ///\r
-  UINT64    SelfTestStatus;\r
-  ///\r
-  /// GR37 from PALE_EXIT state.\r
-  ///\r
-  UINT64    SelfTestControl;\r
-  UINT64    MemoryBufferRequired;\r
-} ITANIUM_HANDOFF_STATUS;\r
 \r
 ///\r
 /// EFI_SEC_PLATFORM_INFORMATION_RECORD.\r
+/// ItaniumHealthFlags has been removed from this union due to Itanium support\r
+/// being deprecated. The union has now diverged from the definition\r
+/// in the PI Specification.\r
 ///\r
 typedef union {\r
-  IA32_HANDOFF_STATUS       IA32HealthFlags;\r
-  X64_HANDOFF_STATUS        x64HealthFlags;\r
-  ITANIUM_HANDOFF_STATUS    ItaniumHealthFlags;\r
+  IA32_HANDOFF_STATUS    IA32HealthFlags;\r
+  X64_HANDOFF_STATUS     x64HealthFlags;\r
 } EFI_SEC_PLATFORM_INFORMATION_RECORD;\r
 \r
 /**\r