From nobody Tue Feb 10 01:14:55 2026 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) client-ip=66.175.222.108; envelope-from=bounce+27952+84813+1787277+3901457@groups.io; helo=mail02.groups.io; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce+27952+84813+1787277+3901457@groups.io; dmarc=fail(p=none dis=none) header.from=intel.com ARC-Seal: i=1; a=rsa-sha256; t=1639489332; cv=none; d=zohomail.com; s=zohoarc; b=HhCULzwiBj66Ytao8toULTDqPOQ9HmU04gCRk+Vg0pPok9K/TskssvwhOD5iRc444m65uCnOXSnd02WOSK1MUrbkG28Jl4muQD4eZcK708w34SJeTbNEWCcM6vB1ElqcytddXX4fBL8qWYCJ6T7a6xclRZAHVvvL0aiKPIZ9g5k= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1639489332; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Id:List-Help:List-Unsubscribe:MIME-Version:Message-ID:Reply-To:References:Sender:Subject:To; bh=zqUWS1xOmdUmHJ6Xd45ZQ0oDrlx/Gm6Y3kSHHvG4Zo4=; b=br0sJUA27Ddb0Z3GTHiiyHrL7rSYB0ilmLYedU11spF6N60lQP+Kl+7jfnkdQ98S0tYofKFs8OQEag30fPEKp1xJBqNi+ZScY+8jGGrsr0Myc+2eoAIMRA00HJj7A/IacjUbg1IR5lfwuVjheVGYJXvlp1G9mSGLZBTi8BgvWlU= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce+27952+84813+1787277+3901457@groups.io; dmarc=fail header.from= (p=none dis=none) Received: from mail02.groups.io (mail02.groups.io [66.175.222.108]) by mx.zohomail.com with SMTPS id 1639489332758681.4751299332205; Tue, 14 Dec 2021 05:42:12 -0800 (PST) Return-Path: X-Received: by 127.0.0.2 with SMTP id hGXiYY1788612x3TDu40g0Ju; Tue, 14 Dec 2021 05:42:12 -0800 X-Received: from mga12.intel.com (mga12.intel.com [192.55.52.136]) by mx.groups.io with SMTP id smtpd.web08.25742.1639489321528370744 for ; Tue, 14 Dec 2021 05:42:11 -0800 X-IronPort-AV: E=McAfee;i="6200,9189,10197"; a="218993709" X-IronPort-AV: E=Sophos;i="5.88,205,1635231600"; d="scan'208";a="218993709" X-Received: from orsmga006.jf.intel.com ([10.7.209.51]) by fmsmga106.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 14 Dec 2021 05:42:11 -0800 X-IronPort-AV: E=Sophos;i="5.88,205,1635231600"; d="scan'208";a="465094293" X-Received: from mxu9-mobl1.ccr.corp.intel.com ([10.255.30.115]) by orsmga006-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 14 Dec 2021 05:42:08 -0800 From: "Min Xu" To: devel@edk2.groups.io Cc: Min Xu , Michael D Kinney , Brijesh Singh , Erdem Aktas , James Bottomley , Jiewen Yao , Tom Lendacky , Gerd Hoffmann Subject: [edk2-devel] [PATCH 09/10] OvmfPkg: Update DxeAcpiTimerLib to read HostBridgeDevId in PlatformInfoHob Date: Tue, 14 Dec 2021 21:41:25 +0800 Message-Id: <20211214134126.869-10-min.m.xu@intel.com> In-Reply-To: <20211214134126.869-1-min.m.xu@intel.com> References: <20211214134126.869-1-min.m.xu@intel.com> MIME-Version: 1.0 Precedence: Bulk List-Unsubscribe: List-Subscribe: List-Help: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Reply-To: devel@edk2.groups.io,min.m.xu@intel.com X-Gm-Message-State: 9FDHazqslMMq0AriMZyTGoN2x1787277AA= Content-Transfer-Encoding: quoted-printable DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=groups.io; q=dns/txt; s=20140610; t=1639489332; bh=JVxrWwX0iPsadeFPvzoIA2pXyJlUXA5ZpnKs/NyrBYc=; h=Cc:Date:From:Reply-To:Subject:To; b=hoeX0DBuFlRDjxR98d3Z4S7O2r9MzzSeUFATmfRvFuiVK4wj+bxvQ57WX9PG/0KnGJ0 mrSTUb1pd28QWvHstZrrMnBBNE/KhybaiibZgFwtGXWHHtkNxQT3nMsAEU7qRnpwrlq6Y LAFk90G9gBBuAhje+XHbhfGfFJUwGVlr6mI= X-ZohoMail-DKIM: pass (identity @groups.io) X-ZM-MESSAGEID: 1639489334329100001 Content-Type: text/plain; charset="utf-8" RFC: https://bugzilla.tianocore.org/show_bug.cgi?id=3D3429 The entrypoint of DxeAcpiTimerLib checks HostBridgeDevId by reading PcdOvmfHostBridgePciDevId. If the DevId is not set, ASSERT is triggered. Normally this DevId is set in PlatformPei which is executed in PEI phase. But Tdvf Config-B skips PEI phase so there is no chance to set this DevId. A word-round is that in Tdx guest HostBridgeDevId is set in PlatformInfoHob (in SecPlatformLibQemuTdx). So we can check if PlatformInfoHob exists and if HostBridgeDevId is set in this Hob. Cc: Michael D Kinney Cc: Brijesh Singh Cc: Erdem Aktas Cc: James Bottomley Cc: Jiewen Yao Cc: Tom Lendacky Cc: Gerd Hoffmann Signed-off-by: Min Xu --- .../Library/AcpiTimerLib/DxeAcpiTimerLib.c | 23 ++++++++++++++++--- .../Library/AcpiTimerLib/DxeAcpiTimerLib.inf | 4 ++++ 2 files changed, 24 insertions(+), 3 deletions(-) diff --git a/OvmfPkg/Library/AcpiTimerLib/DxeAcpiTimerLib.c b/OvmfPkg/Libra= ry/AcpiTimerLib/DxeAcpiTimerLib.c index 115846187455..5c7011f2dd3d 100644 --- a/OvmfPkg/Library/AcpiTimerLib/DxeAcpiTimerLib.c +++ b/OvmfPkg/Library/AcpiTimerLib/DxeAcpiTimerLib.c @@ -6,10 +6,16 @@ SPDX-License-Identifier: BSD-2-Clause-Patent **/ =20 +#include +#include +#include +#include +#include #include #include #include #include +#include #include =20 // @@ -36,13 +42,24 @@ AcpiTimerLibConstructor ( VOID ) { - UINT16 HostBridgeDevId; - UINTN Pmba; + UINT16 HostBridgeDevId; + UINTN Pmba; + EFI_HOB_GUID_TYPE *GuidHob; + EFI_HOB_PLATFORM_INFO *PlatformInfo =3D NULL; =20 // // Query Host Bridge DID to determine platform type + // Tdx guest stores the HostBridgePciDevId in a GuidHob. + // So we first check if this HOB exists // - HostBridgeDevId =3D PcdGet16 (PcdOvmfHostBridgePciDevId); + GuidHob =3D GetFirstGuidHob (&gUefiOvmfPkgTdxPlatformGuid); + if (GuidHob !=3D NULL) { + PlatformInfo =3D (EFI_HOB_PLATFORM_INFO *)GET_GUID_HOB_DATA (GuidHo= b); + HostBridgeDevId =3D PlatformInfo->HostBridgePciDevId; + } else { + HostBridgeDevId =3D PcdGet16 (PcdOvmfHostBridgePciDevId); + } + switch (HostBridgeDevId) { case INTEL_82441_DEVICE_ID: Pmba =3D POWER_MGMT_REGISTER_PIIX4 (PIIX4_PMBA); diff --git a/OvmfPkg/Library/AcpiTimerLib/DxeAcpiTimerLib.inf b/OvmfPkg/Lib= rary/AcpiTimerLib/DxeAcpiTimerLib.inf index e29872add3c7..44a4423a9ddd 100644 --- a/OvmfPkg/Library/AcpiTimerLib/DxeAcpiTimerLib.inf +++ b/OvmfPkg/Library/AcpiTimerLib/DxeAcpiTimerLib.inf @@ -33,3 +33,7 @@ BaseLib PciLib IoLib + HobLib + +[Guids] + gUefiOvmfPkgTdxPlatformGuid ## CONSUMES --=20 2.29.2.windows.2 -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#84813): https://edk2.groups.io/g/devel/message/84813 Mute This Topic: https://groups.io/mt/87720805/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-