From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by mx.groups.io with SMTP id smtpd.web10.3140.1635499212785669457 for ; Fri, 29 Oct 2021 02:20:12 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=WQ8aIqnS; spf=pass (domain: kernel.org, ip: 198.145.29.99, mailfrom: ardb@kernel.org) Received: by mail.kernel.org (Postfix) with ESMTPSA id 0A53E610A0 for ; Fri, 29 Oct 2021 09:20:12 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1635499212; bh=kTiZGzckLBAAYWOKqnINZD3TkCRLTSFZXF6xdrM62ww=; h=References:In-Reply-To:From:Date:Subject:To:Cc:From; b=WQ8aIqnSxU8bTrg18mwLTLihI5AH6R5T8GRX6X06qVj/MQJhJ/AYTfFTDYWkne+QL iHEh85VT4SIolVEAXOCCHPfxQVsqQ8KMeXt0jcPwkxosbB2Li5B3WutmqCp0gG1vRM a76epPwcgNSjeHdSr/dwGLwHho1NA3yCkqKw5JrugFA4qJmugrZwXCnbRl8WAZ3T6a nW3+fHoQcZqeca/vIAK+ALnrvarfOkQdUcLnC4E5iyzA8TL+XPqVCUcZBR4FdfKAyX fP1lcDPhAdly+k1vkF6WZpbJ8xWoxSHVfh3KU5VjopAOA77KiTd8AmG2jrgPo8UJQA 60s8Bq7egRi2w== Received: by mail-oo1-f49.google.com with SMTP id q39-20020a4a962a000000b002b8bb100791so3173788ooi.0 for ; Fri, 29 Oct 2021 02:20:11 -0700 (PDT) X-Gm-Message-State: AOAM532OE1aLlwqX3tsUj2ucLuvTvIbIMv3Jwi+dKfA60BRpuKS2mRjW wE8ljTvG/f49vZ9+iDrRLi6Q3tmbKbnTUdk8eTc= X-Google-Smtp-Source: ABdhPJyk/qUraTQE9vDJPpum1nfUSn0LPiQSlVBtwV1rzKtqeFt20wztH9XgerKm47nEVO4zknZEdtWTbEfVFadM2C0= X-Received: by 2002:a4a:ea2e:: with SMTP id y14mr6821591ood.26.1635499211348; Fri, 29 Oct 2021 02:20:11 -0700 (PDT) MIME-Version: 1.0 References: In-Reply-To: From: "Ard Biesheuvel" Date: Fri, 29 Oct 2021 11:19:59 +0200 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH 0/2] Rename XenTimerDxe to LocalApicTimerDxe To: Min Xu Cc: edk2-devel-groups-io , Ard Biesheuvel , Jordan Justen , Brijesh Singh , Erdem Aktas , James Bottomley , Jiewen Yao , Tom Lendacky , Gerd Hoffmann Content-Type: text/plain; charset="UTF-8" On Fri, 29 Oct 2021 at 08:51, 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. > > 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. > For the series, Acked-by: Ard Biesheuvel > Code at: https://github.com/mxu9/edk2/tree/ovmf_lapic_timer > > Cc: Ard Biesheuvel > Cc: Jordan Justen > Cc: Brijesh Singh > Cc: Erdem Aktas > Cc: James Bottomley > Cc: Jiewen Yao > Cc: Tom Lendacky > Cc: Gerd Hoffmann > Signed-off-by: Min Xu > Min Xu (2): > OvmfPkg: Rename XenTimerDxe to LocalApicTimerDxe > OvmfPkg: Switch timer in build time for OvmfPkg > > OvmfPkg/AmdSev/AmdSevX64.dsc | 3 +-- > 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 | 6 +++++- > OvmfPkg/OvmfPkgIa32.fdf | 8 ++++++-- > OvmfPkg/OvmfPkgIa32X64.dsc | 6 +++++- > OvmfPkg/OvmfPkgIa32X64.fdf | 8 ++++++-- > OvmfPkg/OvmfPkgX64.dsc | 6 +++++- > OvmfPkg/OvmfPkgX64.fdf | 8 ++++++-- > OvmfPkg/OvmfXen.dsc | 2 +- > OvmfPkg/OvmfXen.fdf | 2 +- > 15 files changed, 47 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 >