From nobody Sat Nov 2 16:26:35 2024 Delivered-To: importer@patchew.org Received-SPF: none (zoho.com: 198.145.21.10 is neither permitted nor denied by domain of lists.01.org) client-ip=198.145.21.10; envelope-from=edk2-devel-bounces@lists.01.org; helo=ml01.01.org; Authentication-Results: mx.zoho.com; spf=none (zoho.com: 198.145.21.10 is neither permitted nor denied by domain of lists.01.org) smtp.mailfrom=edk2-devel-bounces@lists.01.org; Return-Path: Received: from ml01.01.org (ml01.01.org [198.145.21.10]) by mx.zohomail.com with SMTPS id 1486097755818907.0037197585691; Thu, 2 Feb 2017 20:55:55 -0800 (PST) Received: from [127.0.0.1] (localhost [IPv6:::1]) by ml01.01.org (Postfix) with ESMTP id 2F28A82097; Thu, 2 Feb 2017 20:55:50 -0800 (PST) Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 831718208E for ; Thu, 2 Feb 2017 20:55:48 -0800 (PST) Received: from orsmga004.jf.intel.com ([10.7.209.38]) by orsmga104.jf.intel.com with ESMTP; 02 Feb 2017 20:55:48 -0800 Received: from mdkinney-mobl.amr.corp.intel.com ([10.252.133.81]) by orsmga004.jf.intel.com with ESMTP; 02 Feb 2017 20:55:47 -0800 X-Original-To: edk2-devel@lists.01.org X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.33,327,1477983600"; d="scan'208";a="54961018" From: Michael Kinney To: edk2-devel@lists.01.org Date: Thu, 2 Feb 2017 20:55:37 -0800 Message-Id: <1486097737-12816-4-git-send-email-michael.d.kinney@intel.com> X-Mailer: git-send-email 2.6.3.windows.1 In-Reply-To: <1486097737-12816-1-git-send-email-michael.d.kinney@intel.com> References: <1486097737-12816-1-git-send-email-michael.d.kinney@intel.com> Subject: [edk2] [PATCH v4 3/3] ShellPkg UefiDpLib: Remove TimerLib dependency X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Andrew Fish , Liming Gao , Jiewen Yao , Jaben Carsey , Star Zeng MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Errors-To: edk2-devel-bounces@lists.01.org Sender: "edk2-devel" X-ZohoMail: RSF_4 Z_629925259 SPT_0 Content-Type: text/plain; charset="utf-8" From: Star Zeng Current UefiDpLib implementation depends on TimerLib, as different platforms may implement and use their own TimerLib, it makes the dp command needs to be built by platform. The TimerLib dependency can be removed by using performance property configuration table to make UefiDpLib to be generic. Cc: Andrew Fish Cc: Michael Kinney Cc: Liming Gao Cc: Jiewen Yao Cc: Cinnamon Shia Cc: Jaben Carsey Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Star Zeng --- ShellPkg/Library/UefiDpLib/Dp.c | 29 +++++++++++------------------ ShellPkg/Library/UefiDpLib/DpInternal.h | 6 ++---- ShellPkg/Library/UefiDpLib/DpProfile.c | 3 +-- ShellPkg/Library/UefiDpLib/DpTrace.c | 25 ++----------------------- ShellPkg/Library/UefiDpLib/DpUtilities.c | 3 +-- ShellPkg/Library/UefiDpLib/Literals.c | 3 +-- ShellPkg/Library/UefiDpLib/UefiDpLib.inf | 6 ++++-- ShellPkg/Library/UefiDpLib/UefiDpLib.uni | 3 ++- ShellPkg/ShellPkg.dsc | 3 +-- 9 files changed, 25 insertions(+), 56 deletions(-) diff --git a/ShellPkg/Library/UefiDpLib/Dp.c b/ShellPkg/Library/UefiDpLib/D= p.c index 75c7d11..444c136 100644 --- a/ShellPkg/Library/UefiDpLib/Dp.c +++ b/ShellPkg/Library/UefiDpLib/Dp.c @@ -13,7 +13,7 @@ Dp uses this information to group records in different ways. It also us= es timer information to calculate elapsed time for each measurement. =20 - Copyright (c) 2009 - 2016, Intel Corporation. All rights reserved. + Copyright (c) 2009 - 2017, Intel Corporation. All rights reserved. (C) Copyright 2015-2016 Hewlett Packard Enterprise Development LP
This program and the accompanying materials are licensed and made available under the terms and conditions of the BS= D License @@ -25,16 +25,10 @@ **/ =20 #include "UefiDpLib.h" -#include -#include -#include -#include - #include #include #include #include -#include #include =20 #include @@ -153,8 +147,7 @@ ShellCommandRunDp ( CONST CHAR16 *CmdLineArg; EFI_STATUS Status; =20 - UINT64 Freq; - UINT64 Ticker; + PERFORMANCE_PROPERTY *PerformanceProperty; UINTN Number2Display; =20 EFI_STRING StringPtr; @@ -183,11 +176,6 @@ ShellCommandRunDp ( CustomCumulativeData =3D NULL; ShellStatus =3D SHELL_SUCCESS; =20 - // Get DP's entry time as soon as possible. - // This is used as the Shell-Phase end time. - // - Ticker =3D GetPerformanceCounter (); - // // initialize the shell lib (we must be in non-auto-init...) // @@ -283,10 +271,15 @@ ShellCommandRunDp ( // StartCount =3D Value loaded into the counter when it starts counti= ng // EndCount =3D Value counter counts to before it needs to be reset // - Freq =3D GetPerformanceCounterProperties (&TimerInfo.StartCount, &TimerI= nfo.EndCount); + Status =3D EfiGetSystemConfigurationTable (&gPerformanceProtocolGuid, &P= erformanceProperty); + if (EFI_ERROR (Status)) { + ShellPrintHiiEx (-1, -1, NULL, STRING_TOKEN (STR_PERF_PROPERTY_NOT_FOU= ND), gDpHiiHandle); + goto Done; + } =20 - // Convert the Frequency from Hz to KHz - TimerInfo.Frequency =3D (UINT32)DivU64x32 (Freq, 1000); + TimerInfo.Frequency =3D (UINT32)DivU64x32 (PerformanceProperty->Frequen= cy, 1000); + TimerInfo.StartCount =3D PerformanceProperty->TimerStartValue; + TimerInfo.EndCount =3D PerformanceProperty->TimerEndValue; =20 // Determine in which direction the performance counter counts. TimerInfo.CountUp =3D (BOOLEAN) (TimerInfo.EndCount >=3D TimerInfo.Start= Count); @@ -362,7 +355,7 @@ ShellCommandRunDp ( } else { //------------- Begin Cooked Mode Processing if (TraceMode) { - ProcessPhases ( Ticker ); + ProcessPhases (); if ( ! SummaryMode) { Status =3D ProcessHandles ( ExcludeMode); if (Status =3D=3D EFI_ABORTED) { diff --git a/ShellPkg/Library/UefiDpLib/DpInternal.h b/ShellPkg/Library/Uef= iDpLib/DpInternal.h index b5ec5f0..6de194a 100644 --- a/ShellPkg/Library/UefiDpLib/DpInternal.h +++ b/ShellPkg/Library/UefiDpLib/DpInternal.h @@ -6,7 +6,7 @@ Dp application. In addition to global data, function declarations for DpUtilities.c, DpTrace.c, and DpProfile.c are included here. =20 - Copyright (c) 2009 - 2013, Intel Corporation. All rights reserved. + Copyright (c) 2009 - 2017, Intel Corporation. All rights reserved. (C) Copyright 2015-2016 Hewlett Packard Enterprise Development LP
This program and the accompanying materials are licensed and made available under the terms and conditions of the BS= D License @@ -236,12 +236,10 @@ DumpRawTrace( /**=20 Gather and print Major Phase metrics. =20 - @param[in] Ticker The timer value for the END of Shell phase - =20 **/ VOID ProcessPhases( - IN UINT64 Ticker + VOID ); =20 =20 diff --git a/ShellPkg/Library/UefiDpLib/DpProfile.c b/ShellPkg/Library/Uefi= DpLib/DpProfile.c index 6458398..e443403 100644 --- a/ShellPkg/Library/UefiDpLib/DpProfile.c +++ b/ShellPkg/Library/UefiDpLib/DpProfile.c @@ -1,7 +1,7 @@ /** @file Measured Profiling reporting for the Dp utility. =20 - Copyright (c) 2009 - 2013, Intel Corporation. All rights reserved. + Copyright (c) 2009 - 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 @@ -16,7 +16,6 @@ #include #include #include -#include #include #include #include diff --git a/ShellPkg/Library/UefiDpLib/DpTrace.c b/ShellPkg/Library/UefiDp= Lib/DpTrace.c index eca2ef3..90a71ad 100644 --- a/ShellPkg/Library/UefiDpLib/DpTrace.c +++ b/ShellPkg/Library/UefiDpLib/DpTrace.c @@ -1,7 +1,7 @@ /** @file Trace reporting for the Dp utility. =20 - Copyright (c) 2009 - 2016, Intel Corporation. All rights reserved. + Copyright (c) 2009 - 2017, Intel Corporation. All rights reserved. (C) Copyright 2015-2016 Hewlett Packard Enterprise Development LP
This program and the accompanying materials are licensed and made available under the terms and conditions of the BS= D License @@ -17,7 +17,6 @@ #include #include #include -#include #include #include #include @@ -386,12 +385,10 @@ DumpRawTrace( /**=20 Gather and print Major Phase metrics. =20 - @param[in] Ticker The timer value for the END of Shell phase - =20 **/ VOID ProcessPhases( - IN UINT64 Ticker + VOID ) { MEASUREMENT_RECORD Measurement; @@ -400,7 +397,6 @@ ProcessPhases( UINT64 PeiTime; UINT64 DxeTime; UINT64 BdsTime; - UINT64 ShellTime; UINT64 ElapsedTime; UINT64 Duration; UINT64 Total; @@ -413,7 +409,6 @@ ProcessPhases( PeiTime =3D 0; DxeTime =3D 0; BdsTime =3D 0; - ShellTime =3D 0; =20 // // Get Execution Phase Statistics // @@ -434,9 +429,6 @@ ProcessPhases( &Measurement.EndTimeStamp, &Measurement.Identifier)) !=3D 0) { - if (AsciiStrnCmp (Measurement.Token, ALit_SHELL, PERF_TOKEN_LENGTH) = =3D=3D 0) { - Measurement.EndTimeStamp =3D Ticker; - } if (Measurement.EndTimeStamp =3D=3D 0) { // Skip "incomplete" records continue; } @@ -454,8 +446,6 @@ ProcessPhases( DxeTime =3D Duration; } else if (AsciiStrnCmp (Measurement.Token, ALit_BDS, PERF_TOKEN_LENGT= H) =3D=3D 0) { BdsTime =3D Duration; - } else if (AsciiStrnCmp (Measurement.Token, ALit_SHELL, PERF_TOKEN_LEN= GTH) =3D=3D 0) { - ShellTime =3D Duration; } } =20 @@ -510,17 +500,6 @@ ProcessPhases( ShellPrintHiiEx (-1, -1, NULL, STRING_TOKEN (STR_DP_PHASE_BDSTO), gDpH= iiHandle, ALit_BdsTO, ElapsedTime); } =20 - // print SHELL phase duration time - // - if (ShellTime > 0) { - ElapsedTime =3D DivU64x32 ( - ShellTime, - (UINT32)TimerInfo.Frequency - ); - Total +=3D ElapsedTime; - ShellPrintHiiEx (-1, -1, NULL, STRING_TOKEN (STR_DP_PHASE_DURATION), g= DpHiiHandle, ALit_SHELL, ElapsedTime); - } - ShellPrintHiiEx (-1, -1, NULL, STRING_TOKEN (STR_DP_TOTAL_DURATION), gDp= HiiHandle, Total); } =20 diff --git a/ShellPkg/Library/UefiDpLib/DpUtilities.c b/ShellPkg/Library/Ue= fiDpLib/DpUtilities.c index fbdd938..64f1830 100644 --- a/ShellPkg/Library/UefiDpLib/DpUtilities.c +++ b/ShellPkg/Library/UefiDpLib/DpUtilities.c @@ -1,7 +1,7 @@ /** @file Utility functions used by the Dp application. =20 - Copyright (c) 2009 - 2016, Intel Corporation. All rights reserved. + Copyright (c) 2009 - 2017, Intel Corporation. All rights reserved. (C) Copyright 2015-2016 Hewlett Packard Enterprise Development LP
This program and the accompanying materials are licensed and made available under the terms and conditions of the BS= D License @@ -17,7 +17,6 @@ #include #include #include -#include #include #include #include diff --git a/ShellPkg/Library/UefiDpLib/Literals.c b/ShellPkg/Library/UefiD= pLib/Literals.c index 68de0fb..c1cddfb 100644 --- a/ShellPkg/Library/UefiDpLib/Literals.c +++ b/ShellPkg/Library/UefiDpLib/Literals.c @@ -1,7 +1,7 @@ /** @file Definitions of ASCII string literals used by DP. =20 - Copyright (c) 2009 - 2010, Intel Corporation. All rights reserved.
+ Copyright (c) 2009 - 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 @@ -16,7 +16,6 @@ CHAR8 const ALit_TimerLibError[] =3D "Timer library instance error!\n"; CHAR8 const ALit_SEC[] =3D SEC_TOK; CHAR8 const ALit_DXE[] =3D DXE_TOK; -CHAR8 const ALit_SHELL[] =3D SHELL_TOK; CHAR8 const ALit_PEI[] =3D PEI_TOK; CHAR8 const ALit_BDS[] =3D BDS_TOK; CHAR8 const ALit_BdsTO[] =3D "BdsTimeOut"; diff --git a/ShellPkg/Library/UefiDpLib/UefiDpLib.inf b/ShellPkg/Library/Ue= fiDpLib/UefiDpLib.inf index 70d2163..0bce0ce 100644 --- a/ShellPkg/Library/UefiDpLib/UefiDpLib.inf +++ b/ShellPkg/Library/UefiDpLib/UefiDpLib.inf @@ -1,7 +1,7 @@ ## @file # Display Performance Application, Module information file. # -# Copyright (c) 2009 - 2015, Intel Corporation. All rights reserved. +# Copyright (c) 2009 - 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 @@ -48,7 +48,6 @@ MdeModulePkg/MdeModulePkg.dec =20 [LibraryClasses] - TimerLib PerformanceLib DxeServicesLib MemoryAllocationLib @@ -64,6 +63,9 @@ PrintLib DevicePathLib =20 +[Guids] + gPerformanceProtocolGuid ## CONSUMES ## S= ystemTable + [Protocols] gEfiLoadedImageProtocolGuid ## CONSUMES gEfiDriverBindingProtocolGuid ## SOMETIMES_CON= SUMES diff --git a/ShellPkg/Library/UefiDpLib/UefiDpLib.uni b/ShellPkg/Library/Ue= fiDpLib/UefiDpLib.uni index b092274..b77c507 100644 --- a/ShellPkg/Library/UefiDpLib/UefiDpLib.uni +++ b/ShellPkg/Library/UefiDpLib/UefiDpLib.uni @@ -1,7 +1,7 @@ // *++ // // (C) Copyright 2014-2015 Hewlett-Packard Development Company, L.P.
-// Copyright (c) 2009 - 2013, Intel Corporation. All rights reserved.
+// Copyright (c) 2009 - 2017, Intel Corporation. All rights reserved.
// (C) Copyright 2015 Hewlett Packard Enterprise Development LP
// This program and the accompanying materials // are licensed and made available under the terms and conditions of the B= SD License @@ -35,6 +35,7 @@ #string STR_DP_INVALID_ARG #language en-US "Invalid argument(= s)\n" #string STR_DP_HANDLES_ERROR #language en-US "Locate all handle= s error - %r\n" #string STR_DP_ERROR_NAME #language en-US "Unknown driver na= me" +#string STR_PERF_PROPERTY_NOT_FOUND #language en-US "Performance prope= rty not found\n" #string STR_DP_BUILD_REVISION #language en-US "\nDP Build Versio= n: %d.%d\n" #string STR_DP_KHZ #language en-US "System Performanc= e Timer Frequency: %,8d (KHz)\n" #string STR_DP_TIMER_PROPERTIES #language en-US "System Performanc= e Timer counts %s from 0x%Lx to 0x%Lx\n" diff --git a/ShellPkg/ShellPkg.dsc b/ShellPkg/ShellPkg.dsc index 6b7864b..5c01933 100644 --- a/ShellPkg/ShellPkg.dsc +++ b/ShellPkg/ShellPkg.dsc @@ -1,7 +1,7 @@ ## @file # Shell Package # -# Copyright (c) 2007 - 2016, 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= BSD License @@ -99,7 +99,6 @@ =20 ShellPkg/Library/UefiDpLib/UefiDpLib.inf { - TimerLib|MdePkg/Library/BaseTimerLibNullTemplate/BaseTimerLibNullTem= plate.inf PerformanceLib|MdePkg/Library/BasePerformanceLibNull/BasePerformance= LibNull.inf DxeServicesLib|MdePkg/Library/DxeServicesLib/DxeServicesLib.inf } --=20 2.6.3.windows.1 _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel