BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=3711
XenTimerDxe is a local Apic timer driver and it has nothing to do with Xen. So rename it to LocalApicTimerDxe. After renaming, LocalApicTimerDxe is used in OvmfPkg if CSM_ENABLE=FALSE. Otherwise 8254 timer is used. Since LocalApicTimerDxe doesn't depend on OvmfPkg, so it is moved to UefiCpuPkg. Patch #1: Rename XenTimerDxe to LocalApicTimerDxe Patch #2: Switch timer in build time for OvmfPkg. If CSM_ENABLE=TRUE, 8254 timer is used, otherwise the timer is LocalApicTimerDxe. Patch #3: Move LocalApicTimerDxe from OvmfPkg to UefiCpuPkg. Code at: https://github.com/mxu9/edk2/tree/ovmf_lapic_timer.v3 v3 changes: - Move LocalApicTimerDxe to UefiCpuPkg - Fix the errors in v2 patch-1 that OvmfXen.dsc / .fdf and MicrovmX64.dsc / .fdf are not updated for new name. - Add XenTimerDxe reviewers. v2 changes: - Add gEfiMdePkgTokenSpaceGuid.PcdFSBClock in *.dsc Cc: Ard Biesheuvel <ardb+tianoc...@kernel.org> Cc: Jordan Justen <jordan.l.jus...@intel.com> Cc: Brijesh Singh <brijesh.si...@amd.com> Cc: Erdem Aktas <erdemak...@google.com> Cc: James Bottomley <j...@linux.ibm.com> Cc: Jiewen Yao <jiewen....@intel.com> Cc: Tom Lendacky <thomas.lenda...@amd.com> Cc: Gerd Hoffmann <kra...@redhat.com> Cc: Eric Dong <eric.d...@intel.com> Cc: Ray Ni <ray...@intel.com> Signed-off-by: Min Xu <min.m...@intel.com> Min Xu (3): OvmfPkg: Rename XenTimerDxe to LocalApicTimerDxe OvmfPkg: Switch timer in build time for OvmfPkg OvmfPkg: Move LocalApicTimerDxe to UefiCpuPkg OvmfPkg/AmdSev/AmdSevX64.dsc | 5 +++-- OvmfPkg/AmdSev/AmdSevX64.fdf | 3 +-- OvmfPkg/Microvm/MicrovmX64.dsc | 2 +- OvmfPkg/Microvm/MicrovmX64.fdf | 2 +- OvmfPkg/OvmfPkgIa32.dsc | 10 +++++++++- OvmfPkg/OvmfPkgIa32.fdf | 8 ++++++-- OvmfPkg/OvmfPkgIa32X64.dsc | 10 +++++++++- OvmfPkg/OvmfPkgIa32X64.fdf | 8 ++++++-- OvmfPkg/OvmfPkgX64.dsc | 10 +++++++++- OvmfPkg/OvmfPkgX64.fdf | 8 ++++++-- OvmfPkg/OvmfXen.dsc | 2 +- OvmfPkg/OvmfXen.fdf | 2 +- .../LocalApicTimerDxe/LocalApicTimerDxe.c | 7 +++---- .../LocalApicTimerDxe/LocalApicTimerDxe.h | 4 ++-- .../LocalApicTimerDxe/LocalApicTimerDxe.inf | 11 +++++++---- 15 files changed, 65 insertions(+), 27 deletions(-) rename OvmfPkg/XenTimerDxe/XenTimerDxe.c => UefiCpuPkg/LocalApicTimerDxe/LocalApicTimerDxe.c (95%) rename OvmfPkg/XenTimerDxe/XenTimerDxe.h => UefiCpuPkg/LocalApicTimerDxe/LocalApicTimerDxe.h (96%) rename OvmfPkg/XenTimerDxe/XenTimerDxe.inf => UefiCpuPkg/LocalApicTimerDxe/LocalApicTimerDxe.inf (72%) -- 2.29.2.windows.2 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#83433): https://edk2.groups.io/g/devel/message/83433 Mute This Topic: https://groups.io/mt/86900362/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
