From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from EUR02-VE1-obe.outbound.protection.outlook.com (mail-oln040092069078.outbound.protection.outlook.com [40.92.69.78]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 75C8321CB02F6 for ; Sun, 23 Jul 2017 03:10:17 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=outlook.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=WGkg/WprTWJSX0x/NenNz4zNLufz4XPhnPxYs2F7E3M=; b=rf+7zrCzcQ+l2AEbXjRDGzITAkbPTyS+vYzF52iNamL0dBNmYcH3BgtEwBP2KDFxNNLGpYc9LUYbNG9CbQyQ33cfwbKAxmHdFCfpetuWujnJK9Q9/5YeM1c/ELZTseWWniB5zVwipNG2oGp5sFK/ptFrgsVeovoF0TTawPRvLOImrzzraA5oavlu0gw2q6VndF8KP/wstq9SUxViGdm9ciCf5u5Z6TM7BmRS/aP9m79mK8DpUTnrcbBb1gw9kX1sHuai8GsuwgikCMyLIyY7b0gG8aoho1F56vnl0u8QGB1dzVd1+3rstR2MlP20siMqDMleHmYVIxBMlJBlGCHXXQ== Received: from VE1EUR02FT060.eop-EUR02.prod.protection.outlook.com (10.152.12.56) by VE1EUR02HT176.eop-EUR02.prod.protection.outlook.com (10.152.12.237) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P384) id 15.1.1240.9; Sun, 23 Jul 2017 10:12:10 +0000 Received: from AM4PR06MB1491.eurprd06.prod.outlook.com (10.152.12.58) by VE1EUR02FT060.mail.protection.outlook.com (10.152.13.140) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.1240.9 via Frontend Transport; Sun, 23 Jul 2017 10:12:10 +0000 Received: from AM4PR06MB1491.eurprd06.prod.outlook.com ([fe80::dcd0:60e6:90e0:871b]) by AM4PR06MB1491.eurprd06.prod.outlook.com ([fe80::dcd0:60e6:90e0:871b%13]) with mapi id 15.01.1282.017; Sun, 23 Jul 2017 10:12:10 +0000 From: =?iso-8859-1?Q?Marvin_H=E4user?= To: "edk2-devel@lists.01.org" CC: "michael.d.kinney@intel.com" , "liming.gao@intel.com" Thread-Topic: [PATCH] MdePkg/SecPeiDxeTimerLibCpu: Consume UefiCpuPkg LAPIC code. Thread-Index: AQHTA5wlcfBDCD5qrEmhamQ76DeIzA== Date: Sun, 23 Jul 2017 10:12:09 +0000 Message-ID: Accept-Language: de-DE, en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: lists.01.org; dkim=none (message not signed) header.d=none;lists.01.org; dmarc=none action=none header.from=outlook.com; x-incomingtopheadermarker: OriginalChecksum:6BB31F5EEA3EF9AD08D673338A74A99D0F88A82E3656525913BAC73AF9B85BC7; UpperCasedChecksum:F7DD6D0C6972A4766A78D8F3193CE9B29C755FDB17DFEBCCAF136B3E4FF39C77; SizeAsReceived:7271; Count:44 x-ms-exchange-messagesentrepresentingtype: 1 x-tmn: [nkMKJsFDA8XWeYScjrNnSRevYcUFC0r5] x-ms-publictraffictype: Email x-microsoft-exchange-diagnostics: 1; VE1EUR02HT176; 7:RK5YZQtFq+PZIyflT/MvcxwavjjTB2D4CVFSpDwzLKywwDHjaYsKQBN4zDlCL8dfntIr92Th16S4Cx/AOCr6uYApTv3qE01hzrIBumHaOv1Ujl+VY2TdDyvI4mJoVMSfI9mopU1Ok2qcjNFLt08xA2eBCKTQJ54ouDJ2Zf3Zx6lbLkJqfo9At457ZfN/fvnOIZUf33Nd+QHgGpnEd0RiI/O/wR1qhTKTGqeq4P5+HhN5BzHEXXiNEklDl/FmE+0lOP9ss0lvLj6MsI1SXI5M18nCLxseyU1t+dJjQMx6hjtRxxAeDNdlOwz5suApYLJXvGeMseEEAbaLBB99EH76C92PGLbB+rJfPUSQ4C3aQoE0BUEK5Bm1a+vwQS6sDnbMZHvzQKv00iCB5bhTus7cAO3cmaqaEetpCTXOYW4jrIAqrXyvip7wPKTnkl/AYM2ZeVI79uMVasddyMsjw1SoPHdnW5KPmbfO82R2+x1TGtgXeO2Ju7sDP7eSVgDAivST0iXE9FFa8MLBwxhVi8OkaaAb+Shq3Xmk0X7WvBmsTJcyvRskQWNhVZIwoitXHKVkz4fFKEIe6Qonp2XRVN4VeGx2Ojv50pU2faoojJjd+beWZXgWG9KgBGF5lzmymmhQjizJp+bgZgWCgzShe6sf1aAAJkWZhJdVKEO8BNEKXWe4hKApxmaNAFRD7hrWAsZ2LdXy6+QrfFyiZyzrQYHxnJ/z7jznZaV4lAokVgDgtnPSJylTdM9AoH+BCKJ15q9uuEGD6F+OfZySZ1BJwWWdDQ== x-incomingheadercount: 44 x-eopattributedmessage: 0 x-forefront-antispam-report: EFV:NLI; SFV:NSPM; SFS:(7070007)(98901004); DIR:OUT; SFP:1901; SCL:1; SRVR:VE1EUR02HT176; H:AM4PR06MB1491.eurprd06.prod.outlook.com; FPR:; SPF:None; LANG:en; x-ms-office365-filtering-correlation-id: 1abbe20e-8399-4866-e21e-08d4d1b34781 x-microsoft-antispam: UriScan:; BCL:0; PCL:0; RULEID:(300000500095)(300135000095)(300000501095)(300135300095)(22001)(300000502095)(300135100095)(300000503095)(300135400095)(201702061074)(5061506573)(5061507331)(1603103135)(2017031320274)(2017031324274)(2017031323274)(2017031322350)(1603101448)(1601125374)(1701031045)(300000504095)(300135200095)(300000505095)(300135600095)(300000506095)(300135500095); SRVR:VE1EUR02HT176; x-ms-traffictypediagnostic: VE1EUR02HT176: x-exchange-antispam-report-test: UriScan:(189930954265078); x-exchange-antispam-report-cfa-test: BCL:0; PCL:0; RULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(444000031); SRVR:VE1EUR02HT176; BCL:0; PCL:0; RULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095); SRVR:VE1EUR02HT176; x-forefront-prvs: 0377802854 spamdiagnosticoutput: 1:99 spamdiagnosticmetadata: NSPM MIME-Version: 1.0 X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-originalarrivaltime: 23 Jul 2017 10:12:09.9815 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Internet X-MS-Exchange-CrossTenant-id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa X-MS-Exchange-Transport-CrossTenantHeadersStamped: VE1EUR02HT176 Subject: [PATCH] MdePkg/SecPeiDxeTimerLibCpu: Consume UefiCpuPkg LAPIC code. X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 23 Jul 2017 10:10:18 -0000 Content-Language: en-US Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable X86TimerLib is changed to use UefiCpuPkg LAPIC register definitions and LocalApicLib to remove duplicated code. An implicite change is the value returned by InternalX86GetApicBase() as it now returns the result of GetLocalApicBaseAddress(), which is the full LAPIC address. This also implicitely fixes the incorrect mask value used previously, which did not only mask AcpiBase, but also the first nibble of AcpiBaseHi. This does not apply to 32-bit platforms. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Marvin Haeuser --- MdePkg/Library/SecPeiDxeTimerLibCpu/X86TimerLib.c | 35 +++++++-= ------------ MdePkg/Library/SecPeiDxeTimerLibCpu/SecPeiDxeTimerLibCpu.inf | 4 ++- MdePkg/MdePkg.dsc | 3 ++ 3 files changed, 18 insertions(+), 24 deletions(-) diff --git a/MdePkg/Library/SecPeiDxeTimerLibCpu/X86TimerLib.c b/MdePkg/Lib= rary/SecPeiDxeTimerLibCpu/X86TimerLib.c index 76c66fbce6fb..fa6e6f213029 100644 --- a/MdePkg/Library/SecPeiDxeTimerLibCpu/X86TimerLib.c +++ b/MdePkg/Library/SecPeiDxeTimerLibCpu/X86TimerLib.c @@ -1,7 +1,7 @@ /** @file Timer Library functions built upon local APIC on IA32/x64. =20 - Copyright (c) 2006 - 2015, Intel Corporation. All rights reserved.
+ Copyright (c) 2006 - 2017, Intel Corporation. All rights reserved.
This program and the accompanying materials are licensed and made available under the terms and conditions of the BS= D License which accompanies this distribution. The full text of the license may b= e found at @@ -13,18 +13,14 @@ **/ =20 #include +#include +#include #include #include #include #include #include =20 -#define APIC_SVR 0x0f0 -#define APIC_LVTERR 0x370 -#define APIC_TMICT 0x380 -#define APIC_TMCCT 0x390 -#define APIC_TDCR 0x3e0 - // // The following array is used in calculating the frequency of local APIC // timer. Refer to IA-32 developers' manual for more details. @@ -54,30 +50,21 @@ InternalX86GetApicBase ( VOID ) { - UINTN MsrValue; UINTN ApicBase; =20 - MsrValue =3D (UINTN) AsmReadMsr64 (27); - ApicBase =3D MsrValue & 0xffffff000ULL; - // - // Check the APIC Global Enable bit (bit 11) in IA32_APIC_BASE MSR. - // This bit will be 1, if local APIC is globally enabled. + // Verify local APIC is under XAPIC mode. // - ASSERT ((MsrValue & BIT11) !=3D 0); + ASSERT (GetApicMode () =3D=3D LOCAL_APIC_MODE_XAPIC); =20 - // - // Check the APIC Extended Mode bit (bit 10) in IA32_APIC_BASE MSR. - // This bit will be 0, if local APIC is under XAPIC mode. - // - ASSERT ((MsrValue & BIT10) =3D=3D 0); + ApicBase =3D GetLocalApicBaseAddress (); =20 // // Check the APIC Software Enable/Disable bit (bit 8) in Spurious-Interr= upt // Vector Register. // This bit will be 1, if local APIC is software enabled. // - ASSERT ((MmioRead32 (ApicBase + APIC_SVR) & BIT8) !=3D 0); + ASSERT ((MmioRead32 (ApicBase + XAPIC_SPURIOUS_VECTOR_OFFSET) & BIT8) != =3D 0); =20 return ApicBase; } @@ -98,7 +85,9 @@ InternalX86GetTimerFrequency ( { return PcdGet32(PcdFSBClock) / - mTimerLibLocalApicDivisor[MmioBitFieldRead32 (ApicBase + APIC_TDCR, 0,= 3)]; + mTimerLibLocalApicDivisor[ + MmioBitFieldRead32 (ApicBase + XAPIC_TIMER_DIVIDE_CONFIGURATION_OFFS= ET, 0, 3) + ]; } =20 /** @@ -115,7 +104,7 @@ InternalX86GetTimerTick ( IN UINTN ApicBase ) { - return MmioRead32 (ApicBase + APIC_TMCCT); + return MmioRead32 (ApicBase + XAPIC_TIMER_CURRENT_COUNT_OFFSET); } =20 /** @@ -131,7 +120,7 @@ InternalX86GetInitTimerCount ( IN UINTN ApicBase ) { - return MmioRead32 (ApicBase + APIC_TMICT); + return MmioRead32 (ApicBase + XAPIC_TIMER_INIT_COUNT_OFFSET); } =20 /** diff --git a/MdePkg/Library/SecPeiDxeTimerLibCpu/SecPeiDxeTimerLibCpu.inf b= /MdePkg/Library/SecPeiDxeTimerLibCpu/SecPeiDxeTimerLibCpu.inf index a00ebb0eeb64..286da09db174 100644 --- a/MdePkg/Library/SecPeiDxeTimerLibCpu/SecPeiDxeTimerLibCpu.inf +++ b/MdePkg/Library/SecPeiDxeTimerLibCpu/SecPeiDxeTimerLibCpu.inf @@ -13,7 +13,7 @@ # Note that for IA-32 and x64, this library only supports xAPIC mode. If x= 2APIC # support is desired, the SecPeiDxeTimerLibUefiCpu library can be used. # -# Copyright (c) 2007 - 2014, Intel Corporation. All rights reserved.
+# Copyright (c) 2007 - 2017, Intel Corporation. All rights reserved.
# # This program and the accompanying materials # are licensed and made available under the terms and conditions of the B= SD License @@ -48,6 +48,7 @@ [Sources.IPF] =20 [Packages] MdePkg/MdePkg.dec + UefiCpuPkg/UefiCpuPkg.dec =20 =20 [LibraryClasses] @@ -57,6 +58,7 @@ [LibraryClasses.IA32, LibraryClasses.X64] PcdLib IoLib DebugLib + LocalApicLib =20 [LibraryClasses.IPF] PalLib diff --git a/MdePkg/MdePkg.dsc b/MdePkg/MdePkg.dsc index 010ce533d7ea..8988d1947566 100644 --- a/MdePkg/MdePkg.dsc +++ b/MdePkg/MdePkg.dsc @@ -35,6 +35,9 @@ [PcdsFixedAtBuild] [PcdsFixedAtBuild.IPF] gEfiMdePkgTokenSpaceGuid.PcdIoBlockBaseAddressForIpf|0x0ffffc000000 =20 +[LibraryClasses] + LocalApicLib|UefiCpuPkg/Library/BaseXApicX2ApicLib/BaseXApicX2ApicLib.in= f + ##########################################################################= ######################### # # Components Section - list of the modules and components that will be pro= cessed by compilation --=20 2.12.2.windows.2