On Mon, Nov 08, 2021 at 02:07:57PM +0800, Min Xu wrote: > 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>
Reviewed-by: Gerd Hoffmann <kra...@redhat.com> -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#83486): https://edk2.groups.io/g/devel/message/83486 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] -=-=-=-=-=-=-=-=-=-=-=-
