summaryrefslogtreecommitdiffstats
path: root/OvmfPkg/XenPlatformPei/XenPlatformPei.inf
diff options
context:
space:
mode:
authorAnthony PERARD <anthony.perard@citrix.com>2021-04-12 14:30:03 +0100
committermergify[bot] <37929162+mergify[bot]@users.noreply.github.com>2021-04-13 11:54:58 +0000
commit71cdb91f313380152d7bf38cfeebe76f5b2d39ac (patch)
tree551cd8bf2f74054f862e4c26386ff065203f6a4f /OvmfPkg/XenPlatformPei/XenPlatformPei.inf
parentc75c6405128e7c3f9cddb05180d55301f4d891a3 (diff)
downloadedk2-71cdb91f313380152d7bf38cfeebe76f5b2d39ac.tar.gz
edk2-71cdb91f313380152d7bf38cfeebe76f5b2d39ac.tar.bz2
edk2-71cdb91f313380152d7bf38cfeebe76f5b2d39ac.zip
OvmfPkg/OvmfXen: Set PcdFSBClock
Update gEfiMdePkgTokenSpaceGuid.PcdFSBClock so it can have the correct value when SecPeiDxeTimerLibCpu start to use it for the APIC timer. Currently, nothing appear to use the value in PcdFSBClock before XenPlatformPei had a chance to set it even though TimerLib is included in modules run before XenPlatformPei. XenPlatformPei doesn't use any of the functions that would use that value. No other modules in the PEI phase seems to use the TimerLib before PcdFSBClock is set. There are currently two other modules in the PEI phase that needs the TimerLib: - S3Resume2Pei, but only because LocalApicLib needs it, but nothing is using the value from PcdFSBClock. - CpuMpPei, but I believe it only runs after XenPlatformPei Before the PEI phase, there's the SEC phase, and SecMain needs TimerLib because of LocalApicLib. And it initialise the APIC timers for the debug agent. But I don't think any of the DebugLib that OvmfXen could use are actually using the *Delay functions in TimerLib, and so would not use the value from PcdFSBClock which would be uninitialised. A simple runtime test showed that TimerLib doesn't use PcdFSBClock value before it is set. Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2490 Signed-off-by: Anthony PERARD <anthony.perard@citrix.com> Reviewed-by: Laszlo Ersek <lersek@redhat.com> Message-Id: <20210412133003.146438-8-anthony.perard@citrix.com> [lersek@redhat.com: cast Freq to UINT32 for PcdSet32S(), not for ASSERT()]
Diffstat (limited to 'OvmfPkg/XenPlatformPei/XenPlatformPei.inf')
-rw-r--r--OvmfPkg/XenPlatformPei/XenPlatformPei.inf1
1 files changed, 1 insertions, 0 deletions
diff --git a/OvmfPkg/XenPlatformPei/XenPlatformPei.inf b/OvmfPkg/XenPlatformPei/XenPlatformPei.inf
index 5732d21888..87dd4b2467 100644
--- a/OvmfPkg/XenPlatformPei/XenPlatformPei.inf
+++ b/OvmfPkg/XenPlatformPei/XenPlatformPei.inf
@@ -85,6 +85,7 @@
gEfiMdeModulePkgTokenSpaceGuid.PcdDxeIplSwitchToLongMode
gEfiMdeModulePkgTokenSpaceGuid.PcdUse1GPageTable
gEfiMdeModulePkgTokenSpaceGuid.PcdPteMemoryEncryptionAddressOrMask
+ gEfiMdePkgTokenSpaceGuid.PcdFSBClock
gEfiSecurityPkgTokenSpaceGuid.PcdOptionRomImageVerificationPolicy
gUefiCpuPkgTokenSpaceGuid.PcdCpuLocalApicBaseAddress