public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* [PATCH V2 0/2] Rename XenTimerDxe to LocalApicTimerDxe
@ 2021-11-01  8:46 Min Xu
  2021-11-01  8:46 ` [PATCH V2 1/2] OvmfPkg: " Min Xu
                   ` (3 more replies)
  0 siblings, 4 replies; 9+ messages in thread
From: Min Xu @ 2021-11-01  8:46 UTC (permalink / raw)
  To: devel
  Cc: Min Xu, Ard Biesheuvel, Jordan Justen, Brijesh Singh, Erdem Aktas,
	James Bottomley, Jiewen Yao, Tom Lendacky, Gerd Hoffmann

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.

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.

Code at: https://github.com/mxu9/edk2/tree/ovmf_lapic_timer.v2

v2 changes:
 - Add gEfiMdePkgTokenSpaceGuid.PcdFSBClock in *.dsc

Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
Cc: Jordan Justen <jordan.l.justen@intel.com>
Cc: Brijesh Singh <brijesh.singh@amd.com>
Cc: Erdem Aktas <erdemaktas@google.com>
Cc: James Bottomley <jejb@linux.ibm.com>
Cc: Jiewen Yao <jiewen.yao@intel.com>
Cc: Tom Lendacky <thomas.lendacky@amd.com>
Cc: Gerd Hoffmann <kraxel@redhat.com>
Signed-off-by: Min Xu <min.m.xu@intel.com>

Min Xu (2):
  OvmfPkg: Rename XenTimerDxe to LocalApicTimerDxe
  OvmfPkg: Switch timer in build time for OvmfPkg

 OvmfPkg/AmdSev/AmdSevX64.dsc                           |  5 +++--
 OvmfPkg/AmdSev/AmdSevX64.fdf                           |  3 +--
 .../LocalApicTimerDxe.c}                               |  7 +++----
 .../LocalApicTimerDxe.h}                               |  4 ++--
 .../LocalApicTimerDxe.inf}                             |  6 +++---
 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 +-
 15 files changed, 61 insertions(+), 26 deletions(-)
 rename OvmfPkg/{XenTimerDxe/XenTimerDxe.c => LocalApicTimerDxe/LocalApicTimerDxe.c} (95%)
 rename OvmfPkg/{XenTimerDxe/XenTimerDxe.h => LocalApicTimerDxe/LocalApicTimerDxe.h} (96%)
 rename OvmfPkg/{XenTimerDxe/XenTimerDxe.inf => LocalApicTimerDxe/LocalApicTimerDxe.inf} (86%)

-- 
2.29.2.windows.2


^ permalink raw reply	[flat|nested] 9+ messages in thread
end of thread, other threads:[~2021-11-02 10:24 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-11-01  8:46 [PATCH V2 0/2] Rename XenTimerDxe to LocalApicTimerDxe Min Xu
2021-11-01  8:46 ` [PATCH V2 1/2] OvmfPkg: " Min Xu
2021-11-01  8:46 ` [PATCH V2 2/2] OvmfPkg: Switch timer in build time for OvmfPkg Min Xu
2021-11-02  9:32 ` [PATCH V2 0/2] Rename XenTimerDxe to LocalApicTimerDxe Gerd Hoffmann
2021-11-02  9:38 ` Yao, Jiewen
2021-11-02  9:58   ` [edk2-devel] " Gerd Hoffmann
2021-11-02 10:05     ` Yao, Jiewen
2021-11-02 10:21       ` Gerd Hoffmann
2021-11-02 10:24         ` Yao, Jiewen

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox