OvmfPkg: set a default value for the WorkAreaHeader PCD

SEC checks in IsSevGuest if the PCD defined WorkAreaHeader size
matches the size of the WorkAreaHeader struct definition. Set a
default value for the PCD to avoid unnecessary DSC/FDF file
changes in all OVMF DSC/FDF files.

Signed-off-by: Corvin Köhne <c.koehne@beckhoff.com>
Reviewed-by: Jiewen Yao <jiewen.yao@intel.com>

diff --git a/OvmfPkg/Include/WorkArea.h b/OvmfPkg/Include/WorkArea.h
index c16030e3ac0ae934333277c27b877189d666143f..721d0f1a8d68cfc73c9549fd3d3c45761fea5d58 100644 (file)
--- a/OvmfPkg/Include/WorkArea.h
+++ b/OvmfPkg/Include/WorkArea.h
@@ -25,6 +25,11 @@ typedef enum {
 // to the structure need to be kept in sync with the\r
 // PcdOvmfConfidentialComputingWorkAreaHeader.\r
 //\r
+// PcdOvmfConfidentialComputingWorkAreaHeader ==\r
+//   sizeof (CONFIDENTIAL_COMPUTING_WORK_AREA_HEADER)\r
+// PcdOvmfConfidentialComputingWorkAreaHeader defined in:\r
+//   OvmfPkg/OvmfPkg.dec\r
+//   OvmfPkg/OvmfPkgDefines.fdf.inc\r
 typedef struct _CONFIDENTIAL_COMPUTING_WORK_AREA_HEADER {\r
   UINT8                   GuestType;\r
   UINT8                   Reserved1[3];\r

diff --git a/OvmfPkg/OvmfPkg.dec b/OvmfPkg/OvmfPkg.dec
index c37dafad49bb4c72c1fec7517a2c50c97546c91b..1be8d5dccbc77821ca95116fadbcc483a884ae2c 100644 (file)
--- a/OvmfPkg/OvmfPkg.dec
+++ b/OvmfPkg/OvmfPkg.dec
@@ -338,7 +338,7 @@
   # The size of this header is used early boot, and is provided through\r
   # a fixed PCD. It need to be kept in sync with any changes to the\r
   # header definition.\r
-  gUefiOvmfPkgTokenSpaceGuid.PcdOvmfConfidentialComputingWorkAreaHeader|0|UINT32|0x51\r
+  gUefiOvmfPkgTokenSpaceGuid.PcdOvmfConfidentialComputingWorkAreaHeader|4|UINT32|0x51\r
 \r
 \r
 [PcdsDynamic, PcdsDynamicEx]\r