From nobody Fri Nov 1 12:22:50 2024 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.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 1516679278156282.9148286140519; Mon, 22 Jan 2018 19:47:58 -0800 (PST) Received: from [127.0.0.1] (localhost [IPv6:::1]) by ml01.01.org (Postfix) with ESMTP id 8D7BC22333794; Mon, 22 Jan 2018 19:42:30 -0800 (PST) Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) (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 665B622333762 for ; Mon, 22 Jan 2018 19:42:28 -0800 (PST) Received: from orsmga008.jf.intel.com ([10.7.209.65]) by orsmga105.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 22 Jan 2018 19:47:55 -0800 Received: from shwdeopenpsi114.ccr.corp.intel.com ([10.239.157.135]) by orsmga008.jf.intel.com with ESMTP; 22 Jan 2018 19:47:54 -0800 X-Original-To: edk2-devel@lists.01.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; Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=134.134.136.100; helo=mga07.intel.com; envelope-from=dandan.bi@intel.com; receiver=edk2-devel@lists.01.org X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.46,399,1511856000"; d="scan'208";a="12262471" From: Dandan Bi To: edk2-devel@lists.01.org Date: Tue, 23 Jan 2018 11:47:28 +0800 Message-Id: <1516679255-12328-2-git-send-email-dandan.bi@intel.com> X-Mailer: git-send-email 1.9.5.msysgit.1 In-Reply-To: <1516679255-12328-1-git-send-email-dandan.bi@intel.com> References: <1516679255-12328-1-git-send-email-dandan.bi@intel.com> Subject: [edk2] [patch 1/8] MdeModulePkg:Add definitions for new Performance infrastructure X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Star Zeng , "Gao, Liming" 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: "Gao, Liming" Cc: Liming Gao Cc: Star Zeng Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Signed-off-by: Dandan Bi --- .../Include/Guid/ExtendedFirmwarePerformance.h | 291 +++++++++++++++++= ++++ MdeModulePkg/MdeModulePkg.dec | 11 +- MdeModulePkg/MdeModulePkg.uni | 8 +- 3 files changed, 308 insertions(+), 2 deletions(-) create mode 100644 MdeModulePkg/Include/Guid/ExtendedFirmwarePerformance.h diff --git a/MdeModulePkg/Include/Guid/ExtendedFirmwarePerformance.h b/MdeM= odulePkg/Include/Guid/ExtendedFirmwarePerformance.h new file mode 100644 index 0000000..05588a4 --- /dev/null +++ b/MdeModulePkg/Include/Guid/ExtendedFirmwarePerformance.h @@ -0,0 +1,291 @@ +/** @file + This file defines edk2 extended firmware performance records. + These records will be added into ACPI FPDT Firmware Basic Boot Performan= ce Table. + +Copyright (c) 2018, Intel Corporation. All rights reserved.
+This program and the accompanying materials are licensed and made availabl= e under +the terms and conditions of the BSD License that accompanies this distribu= tion. +The full text of the license may be found at +http://opensource.org/licenses/bsd-license.php. + +THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, +WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLI= ED. + +**/ + +#ifndef __EXTENDED_FIRMWARE_PERFORMANCE_H__ +#define __EXTENDED_FIRMWARE_PERFORMANCE_H__ + +#include + +// +// Known performance tokens +// +#define SEC_TOK "SEC" ///< SEC Phase +#define DXE_TOK "DXE" ///< DXE Phase +#define PEI_TOK "PEI" ///< PEI Phase +#define BDS_TOK "BDS" ///< BDS Phase +#define DRIVERBINDING_START_TOK "DB:Start:" ///< Driver Bind= ing Start() function call +#define DRIVERBINDING_SUPPORT_TOK "DB:Support:" ///< Driver Bind= ing Support() function call +#define DRIVERBINDING_STOP_TOK "DB:Stop:" ///< Driver Bind= ing Stop() function call +#define LOAD_IMAGE_TOK "LoadImage:" ///< Load a disp= atched module +#define START_IMAGE_TOK "StartImage:" ///< Dispatched = Modules Entry Point execution +#define PEIM_TOK "PEIM" ///< PEIM Module= s Entry Point execution + +// +// Public Progress Identifiers for Event Records to map the above known to= ken +// +#define MODULE_START_ID 0x01 +#define MODULE_END_ID 0x02 +#define MODULE_LOADIMAGE_START_ID 0x03 +#define MODULE_LOADIMAGE_END_ID 0x04 +#define MODULE_DB_START_ID 0x05 +#define MODULE_DB_END_ID 0x06 +#define MODULE_DB_SUPPORT_START_ID 0x07 +#define MODULE_DB_SUPPORT_END_ID 0x08 +#define MODULE_DB_STOP_START_ID 0x09 +#define MODULE_DB_STOP_END_ID 0x0A + +#define PERF_EVENTSIGNAL_START_ID 0x10 +#define PERF_EVENTSIGNAL_END_ID 0x11 +#define PERF_CALLBACK_START_ID 0x20 +#define PERF_CALLBACK_END_ID 0x21 +#define PERF_FUNCTION_START_ID 0x30 +#define PERF_FUNCTION_END_ID 0x31 +#define PERF_INMODULE_START_ID 0x40 +#define PERF_INMODULE_END_ID 0x41 +#define PERF_CROSSMODULE_START_ID 0x50 +#define PERF_CROSSMODULE_END_ID 0x51 + +// +// Misc defines +// +#define RECORD_REVISION_1 (0x01) + +// +// Length field in EFI_ACPI_5_0_FPDT_PERFORMANCE_RECORD_HEADER is a UINT8,= thus: +// +#define MAX_PERF_RECORD_SIZE (0xFF) + +// +// FPDT Record Types +// +#define GUID_EVENT_TYPE 0x1010 +#define DYNAMIC_STRING_EVENT_TYPE 0x1011 +#define DUAL_GUID_STRING_EVENT_TYPE 0x1012 +#define GUID_QWORD_EVENT_TYPE 0x1013 +#define GUID_QWORD_STRING_EVENT_TYPE 0x1014 + +// +// EDKII extended Fpdt record structures +// +#define EDKII_STRING_EVENT_RECORD_NAME_LENGTH 24 + +#pragma pack(1) +// +// FPDT Boot Performance Guid Event Record Structure +// +typedef struct { + EFI_ACPI_5_0_FPDT_PERFORMANCE_RECORD_HEADER Header; + /// + /// ProgressID < 0x10 are reserved for core performance entries. + /// Start measurement point shall have lowered one nibble set to zero and + /// corresponding end points shall have lowered one nibble set to non-ze= ro value; + /// keeping other nibbles same as start point. + /// + UINT16 ProgressID; + /// + /// APIC ID for the processor in the system used as a timestamp clock so= urce. + /// If only one timestamp clock source is used, this field is Reserved a= nd populated as 0. + /// + UINT32 ApicID; + /// + /// 64-bit value (nanosecond) describing elapsed time since the most rec= ent deassertion of processor reset. + /// + UINT64 Timestamp; + /// + /// If ProgressID < 0x10, GUID of the referenced module; otherwise, GUID= of the module logging the event. + /// + EFI_GUID Guid; +} GUID_EVENT_RECORD; + +// +// FPDT Boot Performance Dynamic String Event Record Structure +// +typedef struct { + EFI_ACPI_5_0_FPDT_PERFORMANCE_RECORD_HEADER Header; + /// + /// ProgressID < 0x10 are reserved for core performance entries. + /// Start measurement point shall have lowered one nibble set to zero and + /// corresponding end points shall have lowered one nibble set to non-ze= ro value; + /// keeping other nibbles same as start point. + /// + UINT16 ProgressID; + /// + /// APIC ID for the processor in the system used as a timestamp clock so= urce. + /// If only one timestamp clock source is used, this field is Reserved a= nd populated as 0. + /// + UINT32 ApicID; + /// + /// 64-bit value (nanosecond) describing elapsed time since the most rec= ent deassertion of processor reset. + /// + UINT64 Timestamp; + /// + /// If ProgressID < 0x10, GUID of the referenced module; otherwise, GUID= of the module logging the event. + /// + EFI_GUID Guid; + /// + /// ASCII string describing the module. Padding supplied at the end if n= ecessary with null characters (0x00). + /// It may be module name, function name, or token name. + /// + CHAR8 String[0]; +} DYNAMIC_STRING_EVENT_RECORD; + +// +// FPDT Boot Performance Dual GUID String Event Record Structure +// +typedef struct { + EFI_ACPI_5_0_FPDT_PERFORMANCE_RECORD_HEADER Header; + /// + /// ProgressID < 0x10 are reserved for core performance entries. + /// Start measurement point shall have lowered one nibble set to zero and + /// corresponding end points shall have lowered one nibble set to non-ze= ro value; + /// keeping other nibbles same as start point. + /// + UINT16 ProgressID; + /// + /// APIC ID for the processor in the system used as a timestamp clock so= urce. + /// If only one timestamp clock source is used, this field is Reserved a= nd populated as 0. + /// + UINT32 ApicID; + /// + /// 64-bit value (nanosecond) describing elapsed time since the most rec= ent deassertion of processor reset. + /// + UINT64 Timestamp; + /// + /// GUID of the module logging the event. + /// + EFI_GUID Guid1; + /// + /// Event or Ppi or Protocol GUID for Callback. + /// + EFI_GUID Guid2; + /// + /// ASCII string describing the module. Padding supplied at the end if n= ecessary with null characters (0x00). + /// It is the function name. + /// + CHAR8 String[0]; +} DUAL_GUID_STRING_EVENT_RECORD; + +// +// FPDT Boot Performance GUID Qword Event Record Structure +// +typedef struct { + EFI_ACPI_5_0_FPDT_PERFORMANCE_RECORD_HEADER Header; + /// + /// ProgressID < 0x10 are reserved for core performance entries. + /// Start measurement point shall have lowered one nibble set to zero and + /// corresponding end points shall have lowered one nibble set to non-ze= ro value; + /// keeping other nibbles same as start point. + /// + UINT16 ProgressID; + /// + /// APIC ID for the processor in the system used as a timestamp clock so= urce. + /// If only one timestamp clock source is used, this field is Reserved a= nd populated as 0. + /// + UINT32 ApicID; + /// + /// 64-bit value (nanosecond) describing elapsed time since the most rec= ent deassertion of processor reset. + /// + UINT64 Timestamp; + /// + /// GUID of the module logging the event + /// + EFI_GUID Guid; + /// + /// Qword of misc data, meaning depends on the ProgressId + /// + UINT64 Qword; +} GUID_QWORD_EVENT_RECORD; + +// +// FPDT Boot Performance GUID Qword String Event Record Structure +// +typedef struct { + EFI_ACPI_5_0_FPDT_PERFORMANCE_RECORD_HEADER Header; + /// + /// ProgressID < 0x10 are reserved for core performance entries. + /// Start measurement point shall have lowered one nibble set to zero and + /// corresponding end points shall have lowered one nibble set to non-ze= ro value; + /// keeping other nibbles same as start point. + /// + UINT16 ProgressID; + /// + /// APIC ID for the processor in the system used as a timestamp clock so= urce. + /// If only one timestamp clock source is used, this field is Reserved a= nd populated as 0. + /// + UINT32 ApicID; + /// + /// 64-bit value (nanosecond) describing elapsed time since the most rec= ent deassertion of processor reset. + /// + UINT64 Timestamp; + /// + /// GUID of the module logging the event + /// + EFI_GUID Guid; + /// + /// Qword of misc data, meaning depends on the ProgressId + /// + UINT64 Qword; + /// + /// ASCII string describing the module. Padding supplied at the end if n= ecessary with null characters (0x00). + /// + CHAR8 String[0]; +} GUID_QWORD_STRING_EVENT_RECORD; + +#pragma pack() + +typedef struct { + UINT16 ProgressID; + UINT16 Type; + UINT8 RecordSize; +} BASIC_RECORD_INFO; + +// +// Union of all FPDT records +// +typedef union { + EFI_ACPI_5_0_FPDT_PERFORMANCE_RECORD_HEADER RecordHeader; + GUID_EVENT_RECORD GuidEvent; + DYNAMIC_STRING_EVENT_RECORD DynamicStringEvent; + DUAL_GUID_STRING_EVENT_RECORD DualGuidStringEvent; + GUID_QWORD_EVENT_RECORD GuidQwordEvent; + GUID_QWORD_STRING_EVENT_RECORD GuidQwordStringEvent; +} FPDT_RECORD; + +// +// Union of all pointers to FPDT records +// +typedef union { + EFI_ACPI_5_0_FPDT_PERFORMANCE_RECORD_HEADER *RecordHeader; + GUID_EVENT_RECORD *GuidEvent; + DYNAMIC_STRING_EVENT_RECORD *DynamicStringEvent; + DUAL_GUID_STRING_EVENT_RECORD *DualGuidStringEvent; + GUID_QWORD_EVENT_RECORD *GuidQwordEvent; + GUID_QWORD_STRING_EVENT_RECORD *GuidQwordStringEvent; +} FPDT_RECORD_PTR; + +/// +/// Hob: +/// GUID - gEdkiiExtendedFirmwarePerformanceGuid +/// Data - PEI_EXT_FIRMWARE_PERF_HEADER + one or more FPDT records +/// +typedef struct { + UINT32 SizeOfAllEntries; + UINT32 LoadImageCount; + UINT32 HobIsFull; +} PEI_EXT_FIRMWARE_PERF_HEADER; + +extern EFI_GUID gEdkiiExtendedFirmwarePerformanceGuid; + +#endif diff --git a/MdeModulePkg/MdeModulePkg.dec b/MdeModulePkg/MdeModulePkg.dec index 491fb27..24230be 100644 --- a/MdeModulePkg/MdeModulePkg.dec +++ b/MdeModulePkg/MdeModulePkg.dec @@ -1,11 +1,11 @@ ## @file MdeModulePkg.dec # This package provides the modules that conform to UEFI/PI Industry stand= ards. # It also provides the definitions(including PPIs/PROTOCOLs/GUIDs and libr= ary classes) # and libraries instances, which are used for those modules. # -# Copyright (c) 2007 - 2017, Intel Corporation. All rights reserved.
+# Copyright (c) 2007 - 2018, Intel Corporation. All rights reserved.
# Copyright (c) 2016, Linaro Ltd. All rights reserved.
# (C) Copyright 2016 Hewlett Packard Enterprise Development LP
# Copyright (c) 2017, AMD Incorporated. All rights reserved.
# # This program and the accompanying materials are licensed and made availa= ble under @@ -390,10 +390,13 @@ gEdkiiNonDiscoverableXhciDeviceGuid =3D { 0xB1BE0BC5, 0x6C28, 0x442D, {0= xAA, 0x37, 0x15, 0x1B, 0x42, 0x57, 0xBD, 0x78 } } =20 ## Include/Guid/PlatformHasAcpi.h gEdkiiPlatformHasAcpiGuid =3D { 0xf0966b41, 0xc23f, 0x41b9, { 0x96, 0x04= , 0x0f, 0xf7, 0xe1, 0x11, 0x96, 0x5a } } =20 + ## Include/Guid/ExtendedFirmwarePerformance.h + gEdkiiExtendedFirmwarePerformanceGuid =3D { 0x3b387bfd, 0x7abc, 0x4cf2, = { 0xa0, 0xca, 0xb6, 0xa1, 0x6c, 0x1b, 0x1b, 0x25 } } + ## Include/Guid/EndofS3Resume.h gEdkiiEndOfS3ResumeGuid =3D { 0x96f5296d, 0x05f7, 0x4f3c, {0x84, 0x67, 0= xe4, 0x56, 0x89, 0x0e, 0x0c, 0xb5 } } =20 [Ppis] ## Include/Ppi/AtaController.h @@ -1404,10 +1407,16 @@ =20 ## CapsuleMax value in capsule report variable. # @Prompt CapsuleMax value in capsule report variable. gEfiMdeModulePkgTokenSpaceGuid.PcdCapsuleMax|0xFFFF|UINT16|0x00000107 =20 + ## Control which FPDT record format will be used to store the performanc= e entry. + # On TRUE, the string FPDT record will be used to store every performanc= e entry. + # On FALSE, the different FPDT record will be used to store the differen= t performance entries. + # @Prompt String FPDT Record Enable Only + gEfiMdeModulePkgTokenSpaceGuid.PcdEdkiiFpdtStringRecordEnableOnly|FALSE|= BOOLEAN|0x00000109 + [PcdsPatchableInModule, PcdsDynamic, PcdsDynamicEx] ## This PCD defines the Console output row. The default value is 25 acco= rding to UEFI spec. # This PCD could be set to 0 then console output would be at max column= and max row. # @Prompt Console output row. gEfiMdeModulePkgTokenSpaceGuid.PcdConOutRow|25|UINT32|0x40000006 diff --git a/MdeModulePkg/MdeModulePkg.uni b/MdeModulePkg/MdeModulePkg.uni index 0e06842..dacd671 100644 --- a/MdeModulePkg/MdeModulePkg.uni +++ b/MdeModulePkg/MdeModulePkg.uni @@ -2,11 +2,11 @@ // This package provides the modules that conform to UEFI/PI Industry stan= dards. // // It also provides the definitions(including PPIs/PROTOCOLs/GUIDs and lib= rary classes) // and libraries instances, which are used for those modules. // -// Copyright (c) 2007 - 2016, Intel Corporation. All rights reserved.
+// Copyright (c) 2007 - 2018, Intel Corporation. All rights reserved.
// // This program and the accompanying materials are licensed and made avail= able under // the terms and conditions of the BSD License that accompanies this distr= ibution. // The full text of the license may be found at // http://opensource.org/licenses/bsd-license.php. @@ -1230,5 +1230,11 @@ = " When its value is set in PEI, it will trig the default setting= to be applied as the default EFI variable.\n" =20 #string STR_gEfiMdeModulePkgTokenSpaceGuid_PcdNvStoreDefaultValueBuffer_PR= OMPT #language en-US "NV Storage Default Value Buffer" =20 #string STR_gEfiMdeModulePkgTokenSpaceGuid_PcdNvStoreDefaultValueBuffer_HE= LP #language en-US "This dynamic PCD holds the DynamicHii PCD value. Its= value is the auto generated.\n" + +#string STR_gEfiMdeModulePkgTokenSpaceGuid_PcdEdkiiFpdtStringRecordEnableO= nly_PROMPT #language en-US "String FPDT Record Enable Only" + +#string STR_gEfiMdeModulePkgTokenSpaceGuid_PcdEdkiiFpdtStringRecordEnableO= nly_HELP #language en-US "Control which FPDT record format will be used = to store the performance entry.\n" + = "On TRUE, the string FPDT record will be used t= o store every performance entry.\n" + = "On FALSE, the different FPDT record will be us= ed to store the different performance entries." --=20 1.9.5.msysgit.1 _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel From nobody Fri Nov 1 12:22:50 2024 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.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 1516679284866473.1488539690823; Mon, 22 Jan 2018 19:48:04 -0800 (PST) Received: from [127.0.0.1] (localhost [IPv6:::1]) by ml01.01.org (Postfix) with ESMTP id 06AB622333797; Mon, 22 Jan 2018 19:42:37 -0800 (PST) Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) (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 46FC622333762 for ; Mon, 22 Jan 2018 19:42:36 -0800 (PST) Received: from orsmga008.jf.intel.com ([10.7.209.65]) by orsmga102.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 22 Jan 2018 19:48:01 -0800 Received: from shwdeopenpsi114.ccr.corp.intel.com ([10.239.157.135]) by orsmga008.jf.intel.com with ESMTP; 22 Jan 2018 19:48:00 -0800 X-Original-To: edk2-devel@lists.01.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; Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=134.134.136.24; helo=mga09.intel.com; envelope-from=dandan.bi@intel.com; receiver=edk2-devel@lists.01.org X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.46,399,1511856000"; d="scan'208";a="12262504" From: Dandan Bi To: edk2-devel@lists.01.org Date: Tue, 23 Jan 2018 11:47:29 +0800 Message-Id: <1516679255-12328-3-git-send-email-dandan.bi@intel.com> X-Mailer: git-send-email 1.9.5.msysgit.1 In-Reply-To: <1516679255-12328-1-git-send-email-dandan.bi@intel.com> References: <1516679255-12328-1-git-send-email-dandan.bi@intel.com> Subject: [edk2] [patch 2/8] MdeModulePkg/PeiPerformance:Updated to track FPDT record in PEI phase X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Star Zeng , Liming Gao 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" Updated to convert Pref entry to FPDT record in PEI phase and then export records to FPDT table. Cc: Liming Gao Cc: Star Zeng Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Dandan Bi --- .../Library/PeiPerformanceLib/PeiPerformanceLib.c | 563 ++++++++++++++---= ---- .../PeiPerformanceLib/PeiPerformanceLib.inf | 14 +- 2 files changed, 378 insertions(+), 199 deletions(-) diff --git a/MdeModulePkg/Library/PeiPerformanceLib/PeiPerformanceLib.c b/M= deModulePkg/Library/PeiPerformanceLib/PeiPerformanceLib.c index 62527b2..f78c349 100644 --- a/MdeModulePkg/Library/PeiPerformanceLib/PeiPerformanceLib.c +++ b/MdeModulePkg/Library/PeiPerformanceLib/PeiPerformanceLib.c @@ -5,11 +5,11 @@ performance logging GUIDed HOB on the first performance logging and then= logs the performance data to the GUIDed HOB. Due to the limitation of temporary R= AM, the maximum number of performance logging entry is specified by PcdMaxPeiPerformance= LogEntries or=20 PcdMaxPeiPerformanceLogEntries16. =20 -Copyright (c) 2006 - 2016, Intel Corporation. All rights reserved.
+Copyright (c) 2006 - 2018, 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 BSD = License which accompanies this distribution. The full text of the license may be = found at http://opensource.org/licenses/bsd-license.php @@ -20,215 +20,444 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EI= THER EXPRESS OR IMPLIED. **/ =20 =20 #include =20 -#include +#include =20 #include #include #include #include #include #include #include =20 +#define STRING_SIZE (EDKII_STRING_EVENT_RECORD_NAME_LENGTH * s= izeof (CHAR8)) +#define MAX_RECORD_SIZE (sizeof (FPDT_RECORD) + STRING_SIZE) =20 /** - Gets the GUID HOB for PEI performance. +Check whether the Token is a known one which is uesed by core. =20 - This internal function searches for the GUID HOB for PEI performance. - If that GUID HOB is not found, it will build a new one. - It outputs the data area of that GUID HOB to record performance log. +@param Token Pointer to a Null-terminated ASCII string =20 - @param PeiPerformanceLog Pointer to Pointer to PEI performa= nce log header. - @param PeiPerformanceIdArray Pointer to Pointer to PEI performa= nce identifier array. +@retval TRUE Is a known one used by core. +@retval FALSE Not a known one. =20 **/ -VOID -InternalGetPerformanceHobLog ( - OUT PEI_PERFORMANCE_LOG_HEADER **PeiPerformanceLog, - OUT UINT32 **PeiPerformanceIdArray +BOOLEAN +IsKnownTokens ( + IN CONST CHAR8 *Token ) { - EFI_HOB_GUID_TYPE *GuidHob; - UINTN PeiPerformanceSize; - UINT16 PeiPerformanceLogEntries; + if (AsciiStrCmp (Token, SEC_TOK) =3D=3D 0 || + AsciiStrCmp (Token, PEI_TOK) =3D=3D 0 || + AsciiStrCmp (Token, DXE_TOK) =3D=3D 0 || + AsciiStrCmp (Token, BDS_TOK) =3D=3D 0 || + AsciiStrCmp (Token, DRIVERBINDING_START_TOK) =3D=3D 0 || + AsciiStrCmp (Token, DRIVERBINDING_SUPPORT_TOK) =3D=3D 0 || + AsciiStrCmp (Token, DRIVERBINDING_STOP_TOK) =3D=3D 0 || + AsciiStrCmp (Token, LOAD_IMAGE_TOK) =3D=3D 0 || + AsciiStrCmp (Token, START_IMAGE_TOK) =3D=3D 0 || + AsciiStrCmp (Token, PEIM_TOK) =3D=3D 0) { + return TRUE; + } else { + return FALSE; + } +} =20 - ASSERT (PeiPerformanceLog !=3D NULL); - ASSERT (PeiPerformanceIdArray !=3D NULL); +/** +Check whether the ID is a known one which map to the known Token. =20 - PeiPerformanceLogEntries =3D (UINT16) (PcdGet16 (PcdMaxPeiPerformanceLog= Entries16) !=3D 0 ? - PcdGet16 (PcdMaxPeiPerformanceLogEn= tries16) : - PcdGet8 (PcdMaxPeiPerformanceLogEnt= ries)); - GuidHob =3D GetFirstGuidHob (&gPerformanceProtocolGuid); +@param Identifier 32-bit identifier. =20 - if (GuidHob !=3D NULL) { - // - // PEI Performance HOB was found, then return the existing one. - // - *PeiPerformanceLog =3D GET_GUID_HOB_DATA (GuidHob); +@retval TRUE Is a known one used by core. +@retval FALSE Not a known one. =20 - GuidHob =3D GetFirstGuidHob (&gPerformanceExProtocolGuid); - ASSERT (GuidHob !=3D NULL); - *PeiPerformanceIdArray =3D GET_GUID_HOB_DATA (GuidHob); +**/ +BOOLEAN +IsKnownID ( + IN UINT32 Identifier + ) +{ + if (Identifier =3D=3D MODULE_START_ID || + Identifier =3D=3D MODULE_END_ID || + Identifier =3D=3D MODULE_LOADIMAGE_START_ID || + Identifier =3D=3D MODULE_LOADIMAGE_END_ID || + Identifier =3D=3D MODULE_DB_START_ID || + Identifier =3D=3D MODULE_DB_END_ID || + Identifier =3D=3D MODULE_DB_SUPPORT_START_ID || + Identifier =3D=3D MODULE_DB_SUPPORT_END_ID || + Identifier =3D=3D MODULE_DB_STOP_START_ID || + Identifier =3D=3D MODULE_DB_STOP_END_ID) { + return TRUE; } else { - // - // PEI Performance HOB was not found, then build one. - // - PeiPerformanceSize =3D sizeof (PEI_PERFORMANCE_LOG_HEADER) + - sizeof (PEI_PERFORMANCE_LOG_ENTRY) * PeiPerfo= rmanceLogEntries; - *PeiPerformanceLog =3D BuildGuidHob (&gPerformanceProtocolGuid, Pe= iPerformanceSize); - *PeiPerformanceLog =3D ZeroMem (*PeiPerformanceLog, PeiPerformance= Size); - - PeiPerformanceSize =3D sizeof (UINT32) * PeiPerformanceLogEntries; - *PeiPerformanceIdArray =3D BuildGuidHob (&gPerformanceExProtocolGuid, = PeiPerformanceSize); - *PeiPerformanceIdArray =3D ZeroMem (*PeiPerformanceIdArray, PeiPerform= anceSize); + return FALSE; } } =20 /** - Searches in the log array with keyword Handle, Token, Module and Identif= ier. - - This internal function searches for the log entry in the log array. - If there is an entry that exactly matches the given keywords - and its end time stamp is zero, then the index of that log entry is retu= rned; - otherwise, the the number of log entries in the array is returned. + Get the FPDT record info. =20 - @param PeiPerformanceLog Pointer to the data structure containing= PEI=20 - performance data. - @param PeiPerformanceIdArray Pointer to PEI performance identifier ar= ray. + @param IsStart TRUE if the performance log is start log. @param Handle Pointer to environment specific context = used to identify the component being measured. @param Token Pointer to a Null-terminated ASCII string that identifies the component being meas= ured. @param Module Pointer to a Null-terminated ASCII string that identifies the module being measure= d. - @param Identifier 32-bit identifier. + @param RecordInfo On return, pointer to the info of the re= cord. =20 - @retval The index of log entry in the array. + @retval EFI_SUCCESS Get record info successfully. + @retval EFI_UNSUPPORTED No matched FPDT record. =20 **/ -UINT32 -InternalSearchForLogEntry ( - IN PEI_PERFORMANCE_LOG_HEADER *PeiPerformanceLog, - IN UINT32 *PeiPerformanceIdArray, - IN CONST VOID *Handle, OPTIONAL - IN CONST CHAR8 *Token, OPTIONAL - IN CONST CHAR8 *Module, OPTIONAL - IN UINT32 Identifier +EFI_STATUS +GetFpdtRecordInfo ( + IN BOOLEAN IsStart, + IN CONST VOID *Handle, + IN CONST CHAR8 *Token, + IN CONST CHAR8 *Module, + OUT BASIC_RECORD_INFO *RecordInfo ) { - UINT32 Index; - UINT32 Index2; - UINT32 NumberOfEntries; - PEI_PERFORMANCE_LOG_ENTRY *LogEntryArray; + UINTN StringSize; + UINT16 RecordType; =20 + RecordType =3D DYNAMIC_STRING_EVENT_TYPE; =20 - if (Token =3D=3D NULL) { - Token =3D ""; - } - if (Module =3D=3D NULL) { - Module =3D ""; + // + // Get the ProgressID based on the Token. + // When PcdEdkiiFpdtStringRecordEnableOnly is TRUE, all records are with= type of DYNAMIC_STRING_EVENT_TYPE. + // + if (Token !=3D NULL) { + if (AsciiStrCmp (Token, LOAD_IMAGE_TOK) =3D=3D 0) { // "= LoadImage:" + if (IsStart) { + RecordInfo->ProgressID =3D MODULE_LOADIMAGE_START_ID; + } else { + RecordInfo->ProgressID =3D MODULE_LOADIMAGE_END_ID; + } + if(!PcdGetBool (PcdEdkiiFpdtStringRecordEnableOnly)) { + RecordType =3D GUID_QWORD_EVENT_TYPE; + } + } else if (AsciiStrCmp (Token, SEC_TOK) =3D=3D 0 || // "= SEC" + AsciiStrCmp (Token, PEI_TOK) =3D=3D 0) { // "= PEI" + if (IsStart) { + RecordInfo->ProgressID =3D PERF_CROSSMODULE_START_ID; + } else { + RecordInfo->ProgressID =3D PERF_CROSSMODULE_END_ID; + } + } else if (AsciiStrCmp (Token, PEIM_TOK) =3D=3D 0) { // "= PEIM" + if (IsStart) { + RecordInfo->ProgressID =3D MODULE_START_ID; + } else { + RecordInfo->ProgressID =3D MODULE_END_ID; + } + if(!PcdGetBool (PcdEdkiiFpdtStringRecordEnableOnly)) { + RecordType =3D GUID_EVENT_TYPE; + } + } else { //Pref u= sed in Modules. + if (IsStart) { + RecordInfo->ProgressID =3D PERF_INMODULE_START_ID; + } else { + RecordInfo->ProgressID =3D PERF_INMODULE_END_ID; + } + } + } else if (Module !=3D NULL || Handle !=3D NULL) { //Pr= ef used in Modules. + if (IsStart) { + RecordInfo->ProgressID =3D PERF_INMODULE_START_ID; + } else { + RecordInfo->ProgressID =3D PERF_INMODULE_END_ID; + } + } else { + return EFI_UNSUPPORTED; } - NumberOfEntries =3D PeiPerformanceLog->NumberOfEntries; - LogEntryArray =3D (PEI_PERFORMANCE_LOG_ENTRY *) (PeiPerformanceLog + 1= ); - - Index2 =3D 0; - - for (Index =3D 0; Index < NumberOfEntries; Index++) { - Index2 =3D NumberOfEntries - 1 - Index; - if (LogEntryArray[Index2].EndTimeStamp =3D=3D 0 && - (LogEntryArray[Index2].Handle =3D=3D (EFI_PHYSICAL_ADDRESS) (UINTN= ) Handle) && - AsciiStrnCmp (LogEntryArray[Index2].Token, Token, PEI_PERFORMANCE_= STRING_LENGTH) =3D=3D 0 && - AsciiStrnCmp (LogEntryArray[Index2].Module, Module, PEI_PERFORMANC= E_STRING_LENGTH) =3D=3D 0) { - Index =3D Index2; + + // + // Get the Guid and string. + // + if(PcdGetBool (PcdEdkiiFpdtStringRecordEnableOnly)) { + RecordInfo->RecordSize =3D sizeof (DYNAMIC_STRING_EVENT_RECORD) + STRI= NG_SIZE; + } else { + switch (RecordType) { + case GUID_EVENT_TYPE: + RecordInfo->RecordSize =3D sizeof (GUID_EVENT_RECORD); break; + + case GUID_QWORD_EVENT_TYPE: + RecordInfo->RecordSize =3D sizeof (GUID_QWORD_EVENT_RECORD); + break; + + case DYNAMIC_STRING_EVENT_TYPE: + if (Token !=3D NULL) { + StringSize =3D AsciiStrSize (Token); + } else if (Module !=3D NULL) { + StringSize =3D AsciiStrSize (Module); + } else { + StringSize =3D STRING_SIZE; + } + RecordInfo->RecordSize =3D (UINT8)(sizeof (DYNAMIC_STRING_EVENT_REC= ORD) + StringSize); + break; + + default: + // + // Other type is unsupported in PEI phase yet, return EFI_UNSUPPORTED + // + return EFI_UNSUPPORTED; } } - return Index; + RecordInfo->Type =3D RecordType; + return EFI_SUCCESS; } =20 /** - Creates a record for the beginning of a performance measurement. - - Creates a record that contains the Handle, Token, Module and Identifier. - If TimeStamp is not zero, then TimeStamp is added to the record as the s= tart time. - If TimeStamp is zero, then this function reads the current time stamp - and adds that time stamp value to the record as the start time. + Convert PEI performance log to FPDT String boot record. =20 + @param IsStart TRUE if the performance log is start log. @param Handle Pointer to environment specific context = used to identify the component being measured. @param Token Pointer to a Null-terminated ASCII string that identifies the component being meas= ured. @param Module Pointer to a Null-terminated ASCII string that identifies the module being measure= d. - @param TimeStamp 64-bit time stamp. + @param Ticker 64-bit time stamp. @param Identifier 32-bit identifier. If the value is 0, th= e created record - is same as the one created by StartPerfo= rmanceMeasurement. + is same as the one created by StartGauge= of PERFORMANCE_PROTOCOL. =20 - @retval RETURN_SUCCESS The start of the measurement was recorde= d. - @retval RETURN_OUT_OF_RESOURCES There are not enough resources to record= the measurement. + @retval EFI_SUCCESS Add FPDT boot record. + @retval EFI_OUT_OF_RESOURCES There are not enough resources to recor= d the measurement. + @retval EFI_UNSUPPORTED No matched FPDT record. =20 **/ -RETURN_STATUS -EFIAPI -StartPerformanceMeasurementEx ( +EFI_STATUS +InsertPeiFpdtMeasurement ( + IN BOOLEAN IsStart, IN CONST VOID *Handle, OPTIONAL IN CONST CHAR8 *Token, OPTIONAL IN CONST CHAR8 *Module, OPTIONAL - IN UINT64 TimeStamp, + IN UINT64 Ticker, IN UINT32 Identifier ) { - PEI_PERFORMANCE_LOG_HEADER *PeiPerformanceLog; - UINT32 *PeiPerformanceIdArray; - PEI_PERFORMANCE_LOG_ENTRY *LogEntryArray; - UINT32 Index; - UINT16 PeiPerformanceLogEntries; + EFI_HOB_GUID_TYPE *GuidHob; + UINTN PeiPerformanceSize; + UINT8 *PeiFirmwarePerformance; + PEI_EXT_FIRMWARE_PERF_HEADER *PeiPerformanceLogHeader; + FPDT_RECORD_PTR FpdtRecordPtr; + BASIC_RECORD_INFO RecordInfo; + CONST VOID *ModuleGuid; + UINTN DestMax; + UINTN StrLength; + CONST CHAR8 *StringPtr; + EFI_STATUS Status; + UINT16 PeiPerformanceLogEntries; + UINT64 TimeStamp; + + StringPtr =3D NULL; + + // + // Get record info (type, size, ProgressID and Module Guid). + // + Status =3D GetFpdtRecordInfo (IsStart, Handle, Token, Module, &RecordInf= o); + if (EFI_ERROR (Status)) { + return Status; + } + + // + // If PERF_START()/PERF_END() have specified the ProgressID,it has high = priority. + // !!! Note: If the Pref is not the known Token used in the core but hav= e same + // ID with the core Token, this case will not be supported. + // And in currtnt usage mode, for the unkown ID, there is a general rule: + // If it is start pref: the lower 4 bits of the ID should be 0. + // If it is end pref: the lower 4 bits of the ID should not be 0. + // If input ID doesn't follow the rule, we will adjust it. + // + if ((Identifier !=3D 0) && (IsKnownID (Identifier)) && (!IsKnownTokens (= Token))) { + return EFI_UNSUPPORTED; + } else if ((Identifier !=3D 0) && (!IsKnownID (Identifier)) && (!IsKnown= Tokens (Token))) { + if (IsStart && ((Identifier & 0x000F) !=3D 0)) { + Identifier &=3D 0xFFF0; + } else if ((!IsStart) && ((Identifier & 0x000F) =3D=3D 0)) { + Identifier +=3D 1; + } + RecordInfo.ProgressID =3D (UINT16)Identifier; + } =20 + // + // Get the number of PeiPerformanceLogEntries form PCD. + // PeiPerformanceLogEntries =3D (UINT16) (PcdGet16 (PcdMaxPeiPerformanceLog= Entries16) !=3D 0 ? PcdGet16 (PcdMaxPeiPerformanceLogEn= tries16) : PcdGet8 (PcdMaxPeiPerformanceLogEnt= ries)); =20 - InternalGetPerformanceHobLog (&PeiPerformanceLog, &PeiPerformanceIdArray= ); + // + // Create GUID HOB Data. + // + GuidHob =3D GetFirstGuidHob (&gEdkiiExtendedFirmwarePerformanceGuid); + PeiFirmwarePerformance =3D NULL; + while (GuidHob !=3D NULL) { + // + // PEI Performance HOB was found, then return the existing one. + // + PeiFirmwarePerformance =3D (UINT8*)GET_GUID_HOB_DATA (GuidHob); + PeiPerformanceLogHeader =3D (PEI_EXT_FIRMWARE_PERF_HEADER *)PeiFirmwar= ePerformance; + if (!PeiPerformanceLogHeader->HobIsFull && PeiPerformanceLogHeader->Si= zeOfAllEntries + RecordInfo.RecordSize > PeiPerformanceLogEntries * MAX_REC= ORD_SIZE) { + PeiPerformanceLogHeader->HobIsFull =3D TRUE; + } + if (!PeiPerformanceLogHeader->HobIsFull && PeiPerformanceLogHeader->Si= zeOfAllEntries + RecordInfo.RecordSize <=3D PeiPerformanceLogEntries * MAX_= RECORD_SIZE) { + FpdtRecordPtr.RecordHeader =3D (EFI_ACPI_5_0_FPDT_PERFORMANCE_RECORD= _HEADER *)(PeiFirmwarePerformance + sizeof (PEI_EXT_FIRMWARE_PERF_HEADER) += PeiPerformanceLogHeader->SizeOfAllEntries); + break; + } + // + // Previous HOB is used, then find next one. + // + GuidHob =3D GetNextGuidHob (&gEdkiiExtendedFirmwarePerformanceGuid, GE= T_NEXT_HOB (GuidHob)); + } =20 - if (PeiPerformanceLog->NumberOfEntries >=3D PeiPerformanceLogEntries) { - DEBUG ((DEBUG_ERROR, "PEI performance log array out of resources\n")); - return RETURN_OUT_OF_RESOURCES; + if (GuidHob =3D=3D NULL) { + // + // PEI Performance HOB was not found, then build one. + // + PeiPerformanceSize =3D sizeof (PEI_EXT_FIRMWARE_PERF_HEADER) + + MAX_RECORD_SIZE * PeiPerformanceLogEntries; + PeiFirmwarePerformance =3D (UINT8*)BuildGuidHob (&gEdkiiExtendedFirmw= arePerformanceGuid, PeiPerformanceSize); + if (PeiFirmwarePerformance !=3D NULL) { + ZeroMem (PeiFirmwarePerformance, PeiPerformanceSize); + } + PeiPerformanceLogHeader =3D (PEI_EXT_FIRMWARE_PERF_HEADER *)PeiFirmwar= ePerformance; + FpdtRecordPtr.RecordHeader =3D (EFI_ACPI_5_0_FPDT_PERFORMANCE_RECORD_H= EADER *)(PeiFirmwarePerformance + sizeof (PEI_EXT_FIRMWARE_PERF_HEADER)); } - Index =3D PeiPerformanceLog->NumberOfEntries++; - LogEntryArray =3D (PEI_PERFORMANCE_LOG_ENTRY *) (PeiPerfor= manceLog + 1); - LogEntryArray[Index].Handle =3D (EFI_PHYSICAL_ADDRESS) (UINTN) Handle; =20 - if (Token !=3D NULL) { - AsciiStrnCpyS (LogEntryArray[Index].Token, PEI_PERFORMANCE_STRING_SIZE= , Token, PEI_PERFORMANCE_STRING_LENGTH); + if (PeiFirmwarePerformance =3D=3D NULL) { + // + // there is no enough resource to store performance data + // + return EFI_OUT_OF_RESOURCES; + } + + // + // Get the TimeStamp. + // + if (Ticker =3D=3D 0) { + Ticker =3D GetPerformanceCounter (); + TimeStamp =3D GetTimeInNanoSecond (Ticker); + } else if (Ticker =3D=3D 1) { + TimeStamp =3D 0; + } else { + TimeStamp =3D GetTimeInNanoSecond (Ticker); } - if (Module !=3D NULL) { - AsciiStrnCpyS (LogEntryArray[Index].Module, PEI_PERFORMANCE_STRING_SIZ= E, Module, PEI_PERFORMANCE_STRING_LENGTH); + + // + // Get the ModuleGuid. + // + if (Handle !=3D NULL) { + ModuleGuid =3D Handle; + } else { + ModuleGuid =3D &gEfiCallerIdGuid; } =20 - LogEntryArray[Index].EndTimeStamp =3D 0; - PeiPerformanceIdArray[Index] =3D Identifier; + switch (RecordInfo.Type) { + case GUID_EVENT_TYPE: + FpdtRecordPtr.GuidEvent->Header.Type =3D GUID_EVENT_TYPE; + FpdtRecordPtr.GuidEvent->Header.Length =3D sizeof (GUID_EVENT_RECO= RD); + FpdtRecordPtr.GuidEvent->Header.Revision =3D RECORD_REVISION_1; + FpdtRecordPtr.GuidEvent->ProgressID =3D RecordInfo.ProgressID; + FpdtRecordPtr.GuidEvent->Timestamp =3D TimeStamp; + CopyMem (&FpdtRecordPtr.GuidEvent->Guid, ModuleGuid, sizeof (EFI_GUID)= ); + PeiPerformanceLogHeader->SizeOfAllEntries +=3D sizeof (GUID_EVENT_RECO= RD); + break; + + case GUID_QWORD_EVENT_TYPE: + FpdtRecordPtr.GuidQwordEvent->Header.Type =3D GUID_QWORD_EVENT_TYP= E; + FpdtRecordPtr.GuidQwordEvent->Header.Length =3D sizeof (GUID_QWORD_E= VENT_RECORD); + FpdtRecordPtr.GuidQwordEvent->Header.Revision =3D RECORD_REVISION_1; + FpdtRecordPtr.GuidQwordEvent->ProgressID =3D RecordInfo.ProgressI= D; + FpdtRecordPtr.GuidQwordEvent->Timestamp =3D TimeStamp; + PeiPerformanceLogHeader->LoadImageCount++; + FpdtRecordPtr.GuidQwordEvent->Qword =3D PeiPerformanceLogHea= der->LoadImageCount; + CopyMem (&FpdtRecordPtr.GuidQwordEvent->Guid, ModuleGuid, sizeof (EFI_= GUID)); + PeiPerformanceLogHeader->SizeOfAllEntries +=3D sizeof (GUID_QWORD_EVEN= T_RECORD); + break; + + case DYNAMIC_STRING_EVENT_TYPE: + FpdtRecordPtr.DynamicStringEvent->Header.Type =3D DYNAMIC_STRING= _EVENT_TYPE; + FpdtRecordPtr.DynamicStringEvent->Header.Length =3D RecordInfo.Rec= ordSize; + FpdtRecordPtr.DynamicStringEvent->Header.Revision =3D RECORD_REVISIO= N_1; + FpdtRecordPtr.DynamicStringEvent->ProgressID =3D RecordInfo.Pro= gressID; + FpdtRecordPtr.DynamicStringEvent->Timestamp =3D TimeStamp; + CopyMem (&FpdtRecordPtr.DynamicStringEvent->Guid, ModuleGuid, sizeof (= EFI_GUID)); + PeiPerformanceLogHeader->SizeOfAllEntries +=3D RecordInfo.RecordSize; + + if (Token !=3D NULL) { + StringPtr =3D Token; + } else if (Module !=3D NULL) { + StringPtr =3D Module; + } + if (StringPtr !=3D NULL && AsciiStrLen (StringPtr) !=3D 0) { + DestMax =3D (RecordInfo.RecordSize - sizeof (D= YNAMIC_STRING_EVENT_RECORD)) / sizeof (CHAR8); + StrLength =3D AsciiStrLen (StringPtr); + if (StrLength >=3D DestMax) { + StrLength =3D DestMax -1; + } + AsciiStrnCpyS (FpdtRecordPtr.DynamicStringEvent->String, DestMax, St= ringPtr, StrLength); + } + break; =20 - if (TimeStamp =3D=3D 0) { - TimeStamp =3D GetPerformanceCounter (); + default: + // + // Record is not supported in current PEI phase, return EFI_ABORTED + // + return EFI_UNSUPPORTED; } - LogEntryArray[Index].StartTimeStamp =3D TimeStamp; =20 - return RETURN_SUCCESS; + return EFI_SUCCESS; } =20 /** - Fills in the end time of a performance measurement. + Creates a record for the beginning of a performance measurement. + + If TimeStamp is zero, then this function reads the current time stamp + and adds that time stamp value to the record as the start time. + + If TimeStamp is one, then this function reads 0 as the start time. =20 - Looks up the record that matches Handle, Token and Module. - If the record can not be found then return RETURN_NOT_FOUND. - If the record is found and TimeStamp is not zero, - then TimeStamp is added to the record as the end time. - If the record is found and TimeStamp is zero, then this function reads - the current time stamp and adds that time stamp value to the record as t= he end time. + If TimeStamp is other value, then TimeStamp is added to the record as th= e start time. + + @param Handle Pointer to environment specific context = used + to identify the component being measured. + @param Token Pointer to a Null-terminated ASCII string + that identifies the component being meas= ured. + @param Module Pointer to a Null-terminated ASCII string + that identifies the module being measure= d. + @param TimeStamp 64-bit time stamp. + @param Identifier 32-bit identifier. If the value is 0, th= e created record + is same as the one created by StartPerfo= rmanceMeasurement. + + @retval RETURN_SUCCESS The start of the measurement was recorde= d. + @retval RETURN_OUT_OF_RESOURCES There are not enough resources to record= the measurement. + +**/ +RETURN_STATUS +EFIAPI +StartPerformanceMeasurementEx ( + IN CONST VOID *Handle, OPTIONAL + IN CONST CHAR8 *Token, OPTIONAL + IN CONST CHAR8 *Module, OPTIONAL + IN UINT64 TimeStamp, + IN UINT32 Identifier + ) +{ + return InsertPeiFpdtMeasurement (TRUE, Handle, Token, Module, TimeStamp,= Identifier); +} + +/** + + Creates a record for the end of a performance measurement. + + If the TimeStamp is not zero or one, then TimeStamp is added to the reco= rd as the end time. + If the TimeStamp is zero, then this function reads the current time stam= p and adds that time stamp value to the record as the end time. + If the TimeStamp is one, then this function reads 0 as the end time. =20 @param Handle Pointer to environment specific context = used to identify the component being measured. @param Token Pointer to a Null-terminated ASCII string that identifies the component being meas= ured. @@ -250,28 +479,11 @@ EndPerformanceMeasurementEx ( IN CONST CHAR8 *Module, OPTIONAL IN UINT64 TimeStamp, IN UINT32 Identifier ) { - PEI_PERFORMANCE_LOG_HEADER *PeiPerformanceLog; - UINT32 *PeiPerformanceIdArray; - PEI_PERFORMANCE_LOG_ENTRY *LogEntryArray; - UINT32 Index; - - if (TimeStamp =3D=3D 0) { - TimeStamp =3D GetPerformanceCounter (); - } - - InternalGetPerformanceHobLog (&PeiPerformanceLog, &PeiPerformanceIdArray= ); - Index =3D InternalSearchForLogEntry (PeiPerformanceLog, PeiP= erformanceIdArray, Handle, Token, Module, Identifier); - if (Index >=3D PeiPerformanceLog->NumberOfEntries) { - return RETURN_NOT_FOUND; - } - LogEntryArray =3D (PEI_PERFORMANCE_LOG_ENTRY *) (PeiPerformanceLog += 1); - LogEntryArray[Index].EndTimeStamp =3D TimeStamp; - - return RETURN_SUCCESS; + return InsertPeiFpdtMeasurement (FALSE, Handle, Token, Module, TimeStamp= , Identifier); } =20 /** Attempts to retrieve a performance measurement log entry from the perfor= mance measurement log. It can also retrieve the log created by StartPerformanceMeasurement and = EndPerformanceMeasurement, @@ -293,10 +505,12 @@ EndPerformanceMeasurementEx ( If Module is NULL, then ASSERT(). If StartTimeStamp is NULL, then ASSERT(). If EndTimeStamp is NULL, then ASSERT(). If Identifier is NULL, then ASSERT(). =20 + !!!NOT Support yet!!! + @param LogEntryKey On entry, the key of the performance mea= surement log entry to retrieve. 0, then the first performance measuremen= t log entry is retrieved. On exit, the key of the next performance= of entry entry. @param Handle Pointer to environment specific context = used to identify the component being measured. @@ -323,56 +537,24 @@ GetPerformanceMeasurementEx ( OUT UINT64 *StartTimeStamp, OUT UINT64 *EndTimeStamp, OUT UINT32 *Identifier ) { - PEI_PERFORMANCE_LOG_HEADER *PeiPerformanceLog; - UINT32 *PeiPerformanceIdArray; - PEI_PERFORMANCE_LOG_ENTRY *CurrentLogEntry; - PEI_PERFORMANCE_LOG_ENTRY *LogEntryArray; - UINTN NumberOfEntries; - - ASSERT (Handle !=3D NULL); - ASSERT (Token !=3D NULL); - ASSERT (Module !=3D NULL); - ASSERT (StartTimeStamp !=3D NULL); - ASSERT (EndTimeStamp !=3D NULL); - ASSERT (Identifier !=3D NULL); - - InternalGetPerformanceHobLog (&PeiPerformanceLog, &PeiPerformanceIdArray= ); - - NumberOfEntries =3D (UINTN) (PeiPerformanceLog->NumberOfEntries); - LogEntryArray =3D (PEI_PERFORMANCE_LOG_ENTRY *) (PeiPerformanceLog += 1); - // - // Make sure that LogEntryKey is a valid log entry key. - // - ASSERT (LogEntryKey <=3D NumberOfEntries); - - if (LogEntryKey =3D=3D NumberOfEntries) { - return 0; - } - - CurrentLogEntry =3D &(LogEntryArray[LogEntryKey]); - - *Handle =3D (VOID *) (UINTN) (CurrentLogEntry->Handle); - *Token =3D CurrentLogEntry->Token; - *Module =3D CurrentLogEntry->Module; - *StartTimeStamp =3D CurrentLogEntry->StartTimeStamp; - *EndTimeStamp =3D CurrentLogEntry->EndTimeStamp; - *Identifier =3D PeiPerformanceIdArray[LogEntryKey++]; - - return LogEntryKey; + return 0; } =20 /** Creates a record for the beginning of a performance measurement. =20 - Creates a record that contains the Handle, Token, and Module. - If TimeStamp is not zero, then TimeStamp is added to the record as the s= tart time. If TimeStamp is zero, then this function reads the current time stamp and adds that time stamp value to the record as the start time. =20 + If TimeStamp is one, then this function reads 0 as the start time. + + If TimeStamp is other value, then TimeStamp is added to the record as th= e start time. + + @param Handle Pointer to environment specific context = used to identify the component being measured. @param Token Pointer to a Null-terminated ASCII string that identifies the component being meas= ured. @param Module Pointer to a Null-terminated ASCII string @@ -390,22 +572,20 @@ StartPerformanceMeasurement ( IN CONST CHAR8 *Token, OPTIONAL IN CONST CHAR8 *Module, OPTIONAL IN UINT64 TimeStamp ) { - return StartPerformanceMeasurementEx (Handle, Token, Module, TimeStamp, = 0); + return InsertPeiFpdtMeasurement (TRUE, Handle, Token, Module, TimeStamp,= 0); } =20 /** - Fills in the end time of a performance measurement. =20 - Looks up the record that matches Handle, Token, and Module. - If the record can not be found then return RETURN_NOT_FOUND. - If the record is found and TimeStamp is not zero, - then TimeStamp is added to the record as the end time. - If the record is found and TimeStamp is zero, then this function reads - the current time stamp and adds that time stamp value to the record as t= he end time. + Creates a record for the end of a performance measurement. + + If the TimeStamp is not zero or one, then TimeStamp is added to the reco= rd as the end time. + If the TimeStamp is zero, then this function reads the current time stam= p and adds that time stamp value to the record as the end time. + If the TimeStamp is one, then this function reads 0 as the end time. =20 @param Handle Pointer to environment specific context = used to identify the component being measured. @param Token Pointer to a Null-terminated ASCII string that identifies the component being meas= ured. @@ -424,11 +604,11 @@ EndPerformanceMeasurement ( IN CONST CHAR8 *Token, OPTIONAL IN CONST CHAR8 *Module, OPTIONAL IN UINT64 TimeStamp ) { - return EndPerformanceMeasurementEx (Handle, Token, Module, TimeStamp, 0); + return InsertPeiFpdtMeasurement (FALSE, Handle, Token, Module, TimeStamp= , 0); } =20 /** Attempts to retrieve a performance measurement log entry from the perfor= mance measurement log. It can also retrieve the log created by StartPerformanceMeasurementEx an= d EndPerformanceMeasurementEx, @@ -449,10 +629,12 @@ EndPerformanceMeasurement ( If Token is NULL, then ASSERT(). If Module is NULL, then ASSERT(). If StartTimeStamp is NULL, then ASSERT(). If EndTimeStamp is NULL, then ASSERT(). =20 + NOT Support yet. + @param LogEntryKey On entry, the key of the performance mea= surement log entry to retrieve. 0, then the first performance measuremen= t log entry is retrieved. On exit, the key of the next performance= of entry entry. @param Handle Pointer to environment specific context = used to identify the component being measured. @@ -477,12 +659,11 @@ GetPerformanceMeasurement ( OUT CONST CHAR8 **Module, OUT UINT64 *StartTimeStamp, OUT UINT64 *EndTimeStamp ) { - UINT32 Identifier; - return GetPerformanceMeasurementEx (LogEntryKey, Handle, Token, Module, = StartTimeStamp, EndTimeStamp, &Identifier); + return 0; } =20 /** Returns TRUE if the performance measurement macros are enabled. =20 diff --git a/MdeModulePkg/Library/PeiPerformanceLib/PeiPerformanceLib.inf b= /MdeModulePkg/Library/PeiPerformanceLib/PeiPerformanceLib.inf index 08aa064..7239673 100644 --- a/MdeModulePkg/Library/PeiPerformanceLib/PeiPerformanceLib.inf +++ b/MdeModulePkg/Library/PeiPerformanceLib/PeiPerformanceLib.inf @@ -3,11 +3,11 @@ # =20 # This library provides the performance measurement interfaces in PEI pha= se, it creates # and consumes GUIDed HOB for performance logging. The GUIDed HOB is pass= ed to DXE phase # so that it can be taken over by DxeCorePerformanceLib. =20 # =20 -# Copyright (c) 2006 - 2014, Intel Corporation. All rights reserved.
+# Copyright (c) 2006 - 2018, Intel Corporation. All rights reserved.
# (C) Copyright 2016 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 # which accompanies this distribution. The full text of the license may = be found at # http://opensource.org/licenses/bsd-license.php @@ -51,14 +51,12 @@ =20 =20 [Guids] ## PRODUCES ## HOB ## CONSUMES ## HOB - gPerformanceProtocolGuid - ## PRODUCES ## HOB - ## CONSUMES ## HOB - gPerformanceExProtocolGuid + gEdkiiExtendedFirmwarePerformanceGuid =20 [Pcd] - gEfiMdeModulePkgTokenSpaceGuid.PcdMaxPeiPerformanceLogEntries ## CONSU= MES - gEfiMdeModulePkgTokenSpaceGuid.PcdMaxPeiPerformanceLogEntries16 ## CONSU= MES - gEfiMdePkgTokenSpaceGuid.PcdPerformanceLibraryPropertyMask ## CONSU= MES + gEfiMdeModulePkgTokenSpaceGuid.PcdMaxPeiPerformanceLogEntries ## CON= SUMES + gEfiMdeModulePkgTokenSpaceGuid.PcdMaxPeiPerformanceLogEntries16 ## CON= SUMES + gEfiMdePkgTokenSpaceGuid.PcdPerformanceLibraryPropertyMask ## CON= SUMES + gEfiMdeModulePkgTokenSpaceGuid.PcdEdkiiFpdtStringRecordEnableOnly ## CON= SUMES --=20 1.9.5.msysgit.1 _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel From nobody Fri Nov 1 12:22:50 2024 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.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 151667929221916.43654379307145; Mon, 22 Jan 2018 19:48:12 -0800 (PST) Received: from [127.0.0.1] (localhost [IPv6:::1]) by ml01.01.org (Postfix) with ESMTP id 6A1462233379B; Mon, 22 Jan 2018 19:42:44 -0800 (PST) Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) (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 68A3F22333762 for ; Mon, 22 Jan 2018 19:42:42 -0800 (PST) Received: from orsmga008.jf.intel.com ([10.7.209.65]) by fmsmga101.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 22 Jan 2018 19:48:08 -0800 Received: from shwdeopenpsi114.ccr.corp.intel.com ([10.239.157.135]) by orsmga008.jf.intel.com with ESMTP; 22 Jan 2018 19:48:07 -0800 X-Original-To: edk2-devel@lists.01.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; Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=192.55.52.88; helo=mga01.intel.com; envelope-from=dandan.bi@intel.com; receiver=edk2-devel@lists.01.org X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.46,399,1511856000"; d="scan'208";a="12262534" From: Dandan Bi To: edk2-devel@lists.01.org Date: Tue, 23 Jan 2018 11:47:30 +0800 Message-Id: <1516679255-12328-4-git-send-email-dandan.bi@intel.com> X-Mailer: git-send-email 1.9.5.msysgit.1 In-Reply-To: <1516679255-12328-1-git-send-email-dandan.bi@intel.com> References: <1516679255-12328-1-git-send-email-dandan.bi@intel.com> Subject: [edk2] [patch 3/8] MdeModulePkg/DxeCorePerformanceLib:Track FPDT record in DXE phase X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Star Zeng , Liming Gao 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" Updated to convert Pref enrty to FPDT record in DXE phase and then export records to FPDT table. Cc: Liming Gao Cc: Star Zeng Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Dandan Bi --- .../DxeCorePerformanceLib/DxeCorePerformanceLib.c | 1130 ++++++++++++++--= ---- .../DxeCorePerformanceLib.inf | 14 +- .../DxeCorePerformanceLibInternal.h | 14 +- 3 files changed, 839 insertions(+), 319 deletions(-) diff --git a/MdeModulePkg/Library/DxeCorePerformanceLib/DxeCorePerformanceL= ib.c b/MdeModulePkg/Library/DxeCorePerformanceLib/DxeCorePerformanceLib.c index 7c0e207..6c7dd35 100644 --- a/MdeModulePkg/Library/DxeCorePerformanceLib/DxeCorePerformanceLib.c +++ b/MdeModulePkg/Library/DxeCorePerformanceLib/DxeCorePerformanceLib.c @@ -8,11 +8,11 @@ which are consumed by DxePerformanceLib to logging performance data in D= XE phase. =20 This library is mainly used by DxeCore to start performance logging to e= nsure that Performance Protocol is installed at the very beginning of DXE phase. =20 -Copyright (c) 2006 - 2017, Intel Corporation. All rights reserved.
+Copyright (c) 2006 - 2018, Intel Corporation. All rights reserved.
(C) Copyright 2016 Hewlett Packard Enterprise Development LP
This program and the accompanying materials are licensed and made available under the terms and conditions of the BSD = License which accompanies this distribution. The full text of the license may be = found at http://opensource.org/licenses/bsd-license.php @@ -23,27 +23,36 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITH= ER EXPRESS OR IMPLIED. **/ =20 =20 #include "DxeCorePerformanceLibInternal.h" =20 - -// -// The data structure to hold global performance data. -// -GAUGE_DATA_HEADER *mGaugeData; - -// -// The current maximum number of logging entries. If current number of=20 -// entries exceeds this value, it will re-allocate a larger array and -// migration the old data to the larger array. -// -UINT32 mMaxGaugeRecords; - // -// The handle to install Performance Protocol instance. +// Data for FPDT performance records. // -EFI_HANDLE mHandle =3D NULL; +#define STRING_SIZE (EDKII_STRING_EVENT_RECORD_NAME_LENGTH * s= izeof (CHAR8)) +#define MAX_RECORD_SIZE (sizeof (FPDT_RECORD) + STRING_SIZE) +#define FIRMWARE_RECORD_BUFFER 0x100000 +#define CACHE_HANDLE_GUID_COUNT 0x1000 + +typedef struct { + EFI_HANDLE Handle; + CHAR8 NameString[EDKII_STRING_EVENT_RECORD_NAME_LENGTH]; + EFI_GUID ModuleGuid; +} HANDLE_GUID_MAP; + +HANDLE_GUID_MAP mCacheHandleGuidTable[CACHE_HANDLE_GUID_COUNT]; +UINTN mCachePairCount =3D 0; + +UINT32 mLoadImageCount =3D 0; +UINTN mPerformanceLength =3D 0; +UINTN mMaxPerformanceLength =3D 0; +BOOLEAN mFpdtDataIsReported =3D FALSE; +BOOLEAN mStartFpdtDataReport =3D FALSE; +CHAR8 *mPlatformLanguage =3D NULL; +UINT8 *mPerformancePointer =3D NULL; + +EFI_DEVICE_PATH_TO_TEXT_PROTOCOL *mDevicePathToText =3D NULL; =20 // // Interfaces for Performance Protocol. // PERFORMANCE_PROTOCOL mPerformanceInterface =3D { @@ -61,179 +70,837 @@ PERFORMANCE_EX_PROTOCOL mPerformanceExInterface =3D { GetGaugeEx }; =20 PERFORMANCE_PROPERTY mPerformanceProperty; =20 -// -// Gauge record lock to avoid data corruption or even memory overflow -// -STATIC EFI_LOCK mPerfRecordLock =3D EFI_INITIALIZE_LOCK_VARIABLE (TPL_NOTI= FY); +/** +Check whether the Token is a known one which is uesed by core. + +@param Token Pointer to a Null-terminated ASCII string + +@retval TRUE Is a known one used by core. +@retval FALSE Not a known one. + +**/ +BOOLEAN +IsKnownTokens ( + IN CONST CHAR8 *Token + ) +{ + if (AsciiStrCmp (Token, SEC_TOK) =3D=3D 0 || + AsciiStrCmp (Token, PEI_TOK) =3D=3D 0 || + AsciiStrCmp (Token, DXE_TOK) =3D=3D 0 || + AsciiStrCmp (Token, BDS_TOK) =3D=3D 0 || + AsciiStrCmp (Token, DRIVERBINDING_START_TOK) =3D=3D 0 || + AsciiStrCmp (Token, DRIVERBINDING_SUPPORT_TOK) =3D=3D 0 || + AsciiStrCmp (Token, DRIVERBINDING_STOP_TOK) =3D=3D 0 || + AsciiStrCmp (Token, LOAD_IMAGE_TOK) =3D=3D 0 || + AsciiStrCmp (Token, START_IMAGE_TOK) =3D=3D 0 || + AsciiStrCmp (Token, PEIM_TOK) =3D=3D 0) { + return TRUE; + } else { + return FALSE; + } +} =20 /** - Searches in the gauge array with keyword Handle, Token, Module and Ident= ifier. +Check whether the ID is a known one which map to the known Token. + +@param Identifier 32-bit identifier. + +@retval TRUE Is a known one used by core. +@retval FALSE Not a known one. + +**/ +BOOLEAN +IsKnownID ( + IN UINT32 Identifier + ) +{ + if (Identifier =3D=3D MODULE_START_ID || + Identifier =3D=3D MODULE_END_ID || + Identifier =3D=3D MODULE_LOADIMAGE_START_ID || + Identifier =3D=3D MODULE_LOADIMAGE_END_ID || + Identifier =3D=3D MODULE_DB_START_ID || + Identifier =3D=3D MODULE_DB_END_ID || + Identifier =3D=3D MODULE_DB_SUPPORT_START_ID || + Identifier =3D=3D MODULE_DB_SUPPORT_END_ID || + Identifier =3D=3D MODULE_DB_STOP_START_ID || + Identifier =3D=3D MODULE_DB_STOP_END_ID) { + return TRUE; + } else { + return FALSE; + } +} + +/** + Get a human readable module name and module guid for the given image han= dle. + If module name can't be found, "" string will return. + If module guid can't be found, Zero Guid will return. + + @param Handle Image handle or Controller handle. + @param NameString The ascii string will be filled into it. If not = found, null string will return. + @param BufferSize Size of the input NameString buffer. + @param ModuleGuid Point to the guid buffer to store the got module= guid value. + + @retval EFI_SUCCESS Successfully get module name and guid. + @retval EFI_INVALID_PARAMETER The input parameter NameString is NULL. + @retval other value Module Name can't be got. +**/ +EFI_STATUS +GetModuleInfoFromHandle ( + IN EFI_HANDLE Handle, + OUT CHAR8 *NameString, + IN UINTN BufferSize, + OUT EFI_GUID *ModuleGuid OPTIONAL + ) +{ + EFI_STATUS Status; + EFI_LOADED_IMAGE_PROTOCOL *LoadedImage; + EFI_DRIVER_BINDING_PROTOCOL *DriverBinding; + CHAR8 *PdbFileName; + EFI_GUID *TempGuid; + UINTN StartIndex; + UINTN Index; + INTN Count; + BOOLEAN ModuleGuidIsGet; + UINTN StringSize; + CHAR16 *StringPtr; + EFI_COMPONENT_NAME2_PROTOCOL *ComponentName2; + MEDIA_FW_VOL_FILEPATH_DEVICE_PATH *FvFilePath; + + if (NameString =3D=3D NULL || BufferSize =3D=3D 0) { + return EFI_INVALID_PARAMETER; + } + // + // Try to get the ModuleGuid and name string form the caached array. + // + if (mCachePairCount > 0) { + for (Count =3D mCachePairCount -1; Count >=3D 0; Count--) { + if (Handle =3D=3D mCacheHandleGuidTable[Count].Handle) { + CopyGuid (ModuleGuid, &mCacheHandleGuidTable[Count].ModuleGuid); + AsciiStrCpyS (NameString, EDKII_STRING_EVENT_RECORD_NAME_LENGTH, m= CacheHandleGuidTable[Count].NameString); + return EFI_SUCCESS; + } + } + } + + Status =3D EFI_INVALID_PARAMETER; + LoadedImage =3D NULL; + ModuleGuidIsGet =3D FALSE; + + // + // Initialize GUID as zero value. + // + TempGuid =3D &gZeroGuid; + // + // Initialize it as "" string. + // + NameString[0] =3D 0; + + if (Handle !=3D NULL) { + // + // Try Handle as ImageHandle. + // + Status =3D gBS->HandleProtocol ( + Handle, + &gEfiLoadedImageProtocolGuid, + (VOID**) &LoadedImage + ); + + if (EFI_ERROR (Status)) { + // + // Try Handle as Controller Handle + // + Status =3D gBS->OpenProtocol ( + Handle, + &gEfiDriverBindingProtocolGuid, + (VOID **) &DriverBinding, + NULL, + NULL, + EFI_OPEN_PROTOCOL_GET_PROTOCOL + ); + if (!EFI_ERROR (Status)) { + // + // Get Image protocol from ImageHandle + // + Status =3D gBS->HandleProtocol ( + DriverBinding->ImageHandle, + &gEfiLoadedImageProtocolGuid, + (VOID**) &LoadedImage + ); + } + } + } + + if (!EFI_ERROR (Status) && LoadedImage !=3D NULL) { + // + // Get Module Guid from DevicePath. + // + if (LoadedImage->FilePath !=3D NULL && + LoadedImage->FilePath->Type =3D=3D MEDIA_DEVICE_PATH && + LoadedImage->FilePath->SubType =3D=3D MEDIA_PIWG_FW_FILE_DP + ) { + // + // Determine GUID associated with module logging performance + // + ModuleGuidIsGet =3D TRUE; + FvFilePath =3D (MEDIA_FW_VOL_FILEPATH_DEVICE_PATH *) LoadedImag= e->FilePath; + TempGuid =3D &FvFilePath->FvFileName; + } + + // + // Method 1 Get Module Name from PDB string. + // + PdbFileName =3D PeCoffLoaderGetPdbPointer (LoadedImage->ImageBase); + if (PdbFileName !=3D NULL && BufferSize > 0) { + StartIndex =3D 0; + for (Index =3D 0; PdbFileName[Index] !=3D 0; Index++) { + if ((PdbFileName[Index] =3D=3D '\\') || (PdbFileName[Index] =3D=3D= '/')) { + StartIndex =3D Index + 1; + } + } + // + // Copy the PDB file name to our temporary string. + // If the length is bigger than BufferSize, trim the redudant charac= ters to avoid overflow in array boundary. + // + for (Index =3D 0; Index < BufferSize - 1; Index++) { + NameString[Index] =3D PdbFileName[Index + StartIndex]; + if (NameString[Index] =3D=3D 0 || NameString[Index] =3D=3D '.') { + NameString[Index] =3D 0; + break; + } + } + + if (Index =3D=3D BufferSize - 1) { + NameString[Index] =3D 0; + } + // + // Module Name is got. + // + goto Done; + } + } + + // + // Method 2: Get the name string from ComponentName2 protocol + // + Status =3D gBS->HandleProtocol ( + Handle, + &gEfiComponentName2ProtocolGuid, + (VOID **) &ComponentName2 + ); + if (!EFI_ERROR (Status)) { + // + // Get the current platform language setting + // + if (mPlatformLanguage =3D=3D NULL) { + GetEfiGlobalVariable2 (L"PlatformLang", (VOID **) &mPlatformLanguage= , NULL); + } + if (mPlatformLanguage !=3D NULL) { + Status =3D ComponentName2->GetDriverName ( + ComponentName2, + mPlatformLanguage !=3D NULL ? mPlatformLa= nguage : "en-US", + &StringPtr + ); + if (!EFI_ERROR (Status)) { + for (Index =3D 0; Index < BufferSize - 1 && StringPtr[Index] !=3D = 0; Index++) { + NameString[Index] =3D (CHAR8) StringPtr[Index]; + } + NameString[Index] =3D 0; + // + // Module Name is got. + // + goto Done; + } + } + } + + if (ModuleGuidIsGet) { + // + // Method 3 Try to get the image's FFS UI section by image GUID + // + StringPtr =3D NULL; + StringSize =3D 0; + Status =3D GetSectionFromAnyFv ( + TempGuid, + EFI_SECTION_USER_INTERFACE, + 0, + (VOID **) &StringPtr, + &StringSize + ); + + if (!EFI_ERROR (Status)) { + // + // Method 3. Get the name string from FFS UI section + // + for (Index =3D 0; Index < BufferSize - 1 && StringPtr[Index] !=3D 0;= Index++) { + NameString[Index] =3D (CHAR8) StringPtr[Index]; + } + NameString[Index] =3D 0; + FreePool (StringPtr); + } + } + +Done: + // + // Copy Module Guid + // + if (ModuleGuid !=3D NULL) { + CopyGuid (ModuleGuid, TempGuid); + if (IsZeroGuid(TempGuid) && (Handle !=3D NULL) && !ModuleGuidIsGet) { + // Handle is GUID + CopyGuid (ModuleGuid, (EFI_GUID *) Handle); + } + } =20 - This internal function searches for the gauge entry in the gauge array. - If there is an entry that exactly matches the given keywords - and its end time stamp is zero, then the index of that gauge entry is re= turned; - otherwise, the the number of gauge entries in the array is returned. + // + // Cache the Handle and Guid pairs. + // + if (mCachePairCount < CACHE_HANDLE_GUID_COUNT) { + mCacheHandleGuidTable[mCachePairCount].Handle =3D Handle; + CopyGuid (&mCacheHandleGuidTable[mCachePairCount].ModuleGuid, ModuleGu= id); + AsciiStrCpyS (mCacheHandleGuidTable[mCachePairCount].NameString, EDKII= _STRING_EVENT_RECORD_NAME_LENGTH, NameString); + mCachePairCount ++; + } + + return Status; +} =20 +/** + Get the FPDT record info. + + @param IsStart TRUE if the performance log is start log. @param Handle Pointer to environment specific context = used to identify the component being measured. @param Token Pointer to a Null-terminated ASCII string that identifies the component being meas= ured. @param Module Pointer to a Null-terminated ASCII string that identifies the module being measure= d. - @param Identifier 32-bit identifier. + @param RecordInfo On return, pointer to the info of the re= cord. + @param UseModuleName Only useful for DYNAMIC_STRING_EVENT_TYP= E, indicate that whether need use + Module name to fill the string field in = the DYNAMIC_STRING_EVENT_RECORD. =20 - @retval The index of gauge entry in the array. + @retval EFI_SUCCESS Get record info successfully. + @retval EFI_UNSUPPORTED No matched FPDT record. =20 **/ -UINT32 -InternalSearchForGaugeEntry ( - IN CONST VOID *Handle, OPTIONAL - IN CONST CHAR8 *Token, OPTIONAL - IN CONST CHAR8 *Module, OPTIONAL - IN UINT32 Identifier +EFI_STATUS +GetFpdtRecordInfo ( + IN BOOLEAN IsStart, + IN CONST VOID *Handle, + IN CONST CHAR8 *Token, + IN CONST CHAR8 *Module, + OUT BASIC_RECORD_INFO *RecordInfo, + IN OUT BOOLEAN *UseModuleName ) { - UINT32 Index; - UINT32 Index2; - UINT32 NumberOfEntries; - GAUGE_DATA_ENTRY_EX *GaugeEntryExArray; + UINT16 RecordType; + UINTN StringSize; =20 - if (Token =3D=3D NULL) { - Token =3D ""; - } - if (Module =3D=3D NULL) { - Module =3D ""; + RecordType =3D DYNAMIC_STRING_EVENT_TYPE; + + // + // Token to Type and Id. + // + if (Token !=3D NULL) { + if (AsciiStrCmp (Token, START_IMAGE_TOK) =3D=3D 0) { //= "StartImage:" + *UseModuleName =3D TRUE; + RecordType =3D GUID_EVENT_TYPE; + if (IsStart) { + RecordInfo->ProgressID =3D MODULE_START_ID; + } else { + RecordInfo->ProgressID =3D MODULE_END_ID; + } + } else if (AsciiStrCmp (Token, LOAD_IMAGE_TOK) =3D=3D 0) { //= "LoadImage:" + *UseModuleName =3D TRUE; + RecordType =3D GUID_QWORD_EVENT_TYPE; + if (IsStart) { + RecordInfo->ProgressID =3D MODULE_LOADIMAGE_START_ID; + } else { + RecordInfo->ProgressID =3D MODULE_LOADIMAGE_END_ID; + } + } else if (AsciiStrCmp (Token, DRIVERBINDING_START_TOK) =3D=3D 0) { /= / "DB:Start:" + *UseModuleName =3D TRUE; + if (IsStart) { + RecordInfo->ProgressID =3D MODULE_DB_START_ID; + RecordType =3D GUID_QWORD_EVENT_TYPE; + } else { + RecordInfo->ProgressID =3D MODULE_DB_END_ID; + RecordType =3D GUID_QWORD_STRING_EVENT_TYPE; + } + } else if (AsciiStrCmp (Token, DRIVERBINDING_SUPPORT_TOK) =3D=3D 0) { = // "DB:Support:" + *UseModuleName =3D TRUE; + if (PcdGetBool (PcdEdkiiFpdtStringRecordEnableOnly)) { + return RETURN_UNSUPPORTED; + } + RecordType =3D GUID_QWORD_EVENT_TYPE; + if (IsStart) { + RecordInfo->ProgressID =3D MODULE_DB_SUPPORT_START_ID; + } else { + RecordInfo->ProgressID =3D MODULE_DB_SUPPORT_END_ID; + } + } else if (AsciiStrCmp (Token, DRIVERBINDING_STOP_TOK) =3D=3D 0) { = // "DB:Stop:" + *UseModuleName =3D TRUE; + if (PcdGetBool (PcdEdkiiFpdtStringRecordEnableOnly)) { + return RETURN_UNSUPPORTED; + } + RecordType =3D GUID_QWORD_EVENT_TYPE; + if (IsStart) { + RecordInfo->ProgressID =3D MODULE_DB_STOP_START_ID; + } else { + RecordInfo->ProgressID =3D MODULE_DB_STOP_END_ID; + } + } else if (AsciiStrCmp (Token, PEI_TOK) =3D=3D 0 || = // "PEI" + AsciiStrCmp (Token, DXE_TOK) =3D=3D 0 || = // "DXE" + AsciiStrCmp (Token, BDS_TOK) =3D=3D 0) { = // "BDS" + if (IsStart) { + RecordInfo->ProgressID =3D PERF_CROSSMODULE_START_ID; + } else { + RecordInfo->ProgressID =3D PERF_CROSSMODULE_END_ID; + } + } else { // P= ref used in Modules. + if (IsStart) { + RecordInfo->ProgressID =3D PERF_INMODULE_START_ID; + } else { + RecordInfo->ProgressID =3D PERF_INMODULE_END_ID; + } + } + } else if (Handle!=3D NULL || Module !=3D NULL) { = // Pref used in Modules. + if (IsStart) { + RecordInfo->ProgressID =3D PERF_INMODULE_START_ID; + } else { + RecordInfo->ProgressID =3D PERF_INMODULE_END_ID; + } + } else { + return EFI_UNSUPPORTED; } =20 - NumberOfEntries =3D mGaugeData->NumberOfEntries; - GaugeEntryExArray =3D (GAUGE_DATA_ENTRY_EX *) (mGaugeData + 1); + // + // Get Record size baesed on the record type. + // When PcdEdkiiFpdtStringRecordEnableOnly is TRUE, all records are with= type of DYNAMIC_STRING_EVENT_TYPE. + // + if (PcdGetBool (PcdEdkiiFpdtStringRecordEnableOnly)) { + RecordType =3D DYNAMIC_STRING_EVENT_TYPE; + RecordInfo->RecordSize =3D sizeof (DYNAMIC_STRING_EVENT_RECORD) + ST= RING_SIZE; + } else { + switch (RecordType) { + case GUID_EVENT_TYPE: + RecordInfo->RecordSize =3D sizeof (GUID_EVENT_RECORD); + break; =20 - Index2 =3D 0; + case DYNAMIC_STRING_EVENT_TYPE: + if (*UseModuleName) { + StringSize =3D STRING_SIZE; + } else if (Token !=3D NULL) { + StringSize =3D AsciiStrSize (Token); + } else if (Module !=3D NULL) { + StringSize =3D AsciiStrSize (Module); + } else { + StringSize =3D STRING_SIZE; + } + if (StringSize > STRING_SIZE) { + StringSize =3D STRING_SIZE; + } + RecordInfo->RecordSize =3D (UINT8)(sizeof (DYNAMIC_STRING_EVENT_RECO= RD) + StringSize); + break; =20 - for (Index =3D 0; Index < NumberOfEntries; Index++) { - Index2 =3D NumberOfEntries - 1 - Index; - if (GaugeEntryExArray[Index2].EndTimeStamp =3D=3D 0 && - (GaugeEntryExArray[Index2].Handle =3D=3D (EFI_PHYSICAL_ADDRESS) (U= INTN) Handle) && - AsciiStrnCmp (GaugeEntryExArray[Index2].Token, Token, DXE_PERFORMA= NCE_STRING_LENGTH) =3D=3D 0 && - AsciiStrnCmp (GaugeEntryExArray[Index2].Module, Module, DXE_PERFOR= MANCE_STRING_LENGTH) =3D=3D 0) { - Index =3D Index2; + case GUID_QWORD_EVENT_TYPE: + RecordInfo->RecordSize =3D (UINT8)sizeof (GUID_QWORD_EVENT_RECORD); break; + + case GUID_QWORD_STRING_EVENT_TYPE: + RecordInfo->RecordSize =3D (UINT8)sizeof (GUID_QWORD_STRING_EVENT_RE= CORD); + break; + + default: + // + // Record is unsupported yet, return EFI_UNSUPPORTED + // + return EFI_UNSUPPORTED; } } =20 - return Index; + RecordInfo->Type =3D RecordType; + return EFI_SUCCESS; } =20 /** - Adds a record at the end of the performance measurement log - that records the start time of a performance measurement. - - Adds a record to the end of the performance measurement log - that contains the Handle, Token, Module and Identifier. - The end time of the new record must be set to zero. - If TimeStamp is not zero, then TimeStamp is used to fill in the start ti= me in the record. - If TimeStamp is zero, the start time in the record is filled in with the= value - read from the current time stamp. + Add performance log to FPDT boot record table. =20 + @param IsStart TRUE if the performance log is start log. @param Handle Pointer to environment specific context = used to identify the component being measured. @param Token Pointer to a Null-terminated ASCII string that identifies the component being meas= ured. @param Module Pointer to a Null-terminated ASCII string that identifies the module being measure= d. - @param TimeStamp 64-bit time stamp. + @param Ticker 64-bit time stamp. @param Identifier 32-bit identifier. If the value is 0, th= e created record is same as the one created by StartGauge= of PERFORMANCE_PROTOCOL. =20 - @retval EFI_SUCCESS The data was read correctly from the dev= ice. + @retval EFI_SUCCESS Add FPDT boot record. @retval EFI_OUT_OF_RESOURCES There are not enough resources to record= the measurement. + @retval EFI_UNSUPPORTED No matched FPDT record. =20 **/ EFI_STATUS -EFIAPI -StartGaugeEx ( +InsertFpdtMeasurement ( + IN BOOLEAN IsStart, IN CONST VOID *Handle, OPTIONAL IN CONST CHAR8 *Token, OPTIONAL IN CONST CHAR8 *Module, OPTIONAL - IN UINT64 TimeStamp, + IN UINT64 Ticker, IN UINT32 Identifier ) { - GAUGE_DATA_ENTRY_EX *GaugeEntryExArray; - UINTN GaugeDataSize; - GAUGE_DATA_HEADER *NewGaugeData; - UINTN OldGaugeDataSize; - GAUGE_DATA_HEADER *OldGaugeData; - UINT32 Index; - EFI_STATUS Status; + EFI_GUID ModuleGuid; + CHAR8 ModuleName[EDKII_STRING_EVENT_RECORD_NAME_L= ENGTH]; + UINT8 FpdtRecord[MAX_RECORD_SIZE]; + EFI_STATUS Status; + FPDT_RECORD_PTR FpdtRecordPtr; + BASIC_RECORD_INFO RecordInfo; + UINT64 TimeStamp; + UINTN DestMax; + UINTN StrLength; + CONST CHAR8 *StringPtr; + BOOLEAN UseModuleName; + + StringPtr =3D NULL; + UseModuleName =3D FALSE; + ZeroMem (ModuleName, sizeof (ModuleName)); + ZeroMem (FpdtRecord, sizeof (FpdtRecord)); =20 - Status =3D EfiAcquireLockOrFail (&mPerfRecordLock); + // + // Get record info (type, size, ProgressID and Module Guid). + // + Status =3D GetFpdtRecordInfo (IsStart, Handle, Token, Module, &RecordInf= o, &UseModuleName); if (EFI_ERROR (Status)) { return Status; } =20 - Index =3D mGaugeData->NumberOfEntries; - if (Index >=3D mMaxGaugeRecords) { + // + // If PERF_START()/PERF_END() have specified the ProgressID,it has high = priority. + // !!! Note: If the Pref is not the known Token used in the core but hav= e same + // ID with the core Token, this case will not be supported. + // And in currtnt usage mode, for the unkown ID, there is a general rule: + // If it is start pref: the lower 4 bits of the ID should be 0. + // If it is end pref: the lower 4 bits of the ID should not be 0. + // If input ID doesn't follow the rule, we will adjust it. + // + if ((Identifier !=3D 0) && (IsKnownID (Identifier)) && (!IsKnownTokens (= Token))) { + return EFI_UNSUPPORTED; + } else if ((Identifier !=3D 0) && (!IsKnownID (Identifier)) && (!IsKnown= Tokens (Token))) { + if (IsStart && ((Identifier & 0x000F) !=3D 0)) { + Identifier &=3D 0xFFF0; + } else if ((!IsStart) && ((Identifier & 0x000F) =3D=3D 0)) { + Identifier +=3D 1; + } + RecordInfo.ProgressID =3D (UINT16)Identifier; + } + + if (mFpdtDataIsReported) { + FpdtRecordPtr.RecordHeader =3D (EFI_ACPI_5_0_FPDT_PERFORMANCE_RECORD_H= EADER *)FpdtRecord; + } else { + if (mStartFpdtDataReport) { + return EFI_UNSUPPORTED; + } + // + // Check if pre-allocated buffer is full + // + if (mPerformanceLength + RecordInfo.RecordSize > mMaxPerformanceLength= ) { + mPerformancePointer =3D ReallocatePool ( + mPerformanceLength, + mPerformanceLength + RecordInfo.RecordSize += FIRMWARE_RECORD_BUFFER, + mPerformancePointer + ); + + if (mPerformancePointer =3D=3D NULL) { + return EFI_OUT_OF_RESOURCES; + } + mMaxPerformanceLength =3D mPerformanceLength + RecordInfo.RecordSize= + FIRMWARE_RECORD_BUFFER; + } // - // Try to enlarge the scale of gauge array. + // Covert buffer to FPDT Ptr Union type. // - OldGaugeData =3D mGaugeData; - OldGaugeDataSize =3D sizeof (GAUGE_DATA_HEADER) + sizeof (GAUGE_DATA_= ENTRY_EX) * mMaxGaugeRecords; + FpdtRecordPtr.RecordHeader =3D (EFI_ACPI_5_0_FPDT_PERFORMANCE_RECORD_H= EADER *)(mPerformancePointer + mPerformanceLength); + } + FpdtRecordPtr.RecordHeader->Length =3D 0; =20 - GaugeDataSize =3D sizeof (GAUGE_DATA_HEADER) + sizeof (GAUGE_DATA_= ENTRY_EX) * mMaxGaugeRecords * 2; + // + // Get the TimeStamp. + // + if (Ticker =3D=3D 0) { + Ticker =3D GetPerformanceCounter (); + TimeStamp =3D GetTimeInNanoSecond (Ticker); + } else if (Ticker =3D=3D 1) { + TimeStamp =3D 0; + } else { + TimeStamp =3D GetTimeInNanoSecond (Ticker); + } =20 - NewGaugeData =3D AllocateZeroPool (GaugeDataSize); - if (NewGaugeData =3D=3D NULL) { - EfiReleaseLock (&mPerfRecordLock); - return EFI_OUT_OF_RESOURCES; + // + // Get the ModuleName and ModuleGuid form the handle. + // + GetModuleInfoFromHandle ((EFI_HANDLE *)Handle, ModuleName, sizeof (Modul= eName), &ModuleGuid); + + // + // Fill in the record information. + // + switch (RecordInfo.Type) { + case GUID_EVENT_TYPE: + FpdtRecordPtr.GuidEvent->Header.Type =3D GUID_EVENT_TYP= E; + FpdtRecordPtr.GuidEvent->Header.Length =3D RecordInfo.Rec= ordSize; + FpdtRecordPtr.GuidEvent->Header.Revision =3D RECORD_REVISIO= N_1; + FpdtRecordPtr.GuidEvent->ProgressID =3D RecordInfo.Pro= gressID; + FpdtRecordPtr.GuidEvent->Timestamp =3D TimeStamp; + CopyMem (&FpdtRecordPtr.GuidEvent->Guid, &ModuleGuid, sizeof (FpdtReco= rdPtr.GuidEvent->Guid)); + break; + + case DYNAMIC_STRING_EVENT_TYPE: + FpdtRecordPtr.DynamicStringEvent->Header.Type =3D DYNAMIC_STRING= _EVENT_TYPE; + FpdtRecordPtr.DynamicStringEvent->Header.Length =3D RecordInfo.Rec= ordSize; + FpdtRecordPtr.DynamicStringEvent->Header.Revision =3D RECORD_REVISIO= N_1; + FpdtRecordPtr.DynamicStringEvent->ProgressID =3D RecordInfo.Pro= gressID; + FpdtRecordPtr.DynamicStringEvent->Timestamp =3D TimeStamp; + CopyMem (&FpdtRecordPtr.DynamicStringEvent->Guid, &ModuleGuid, sizeof = (FpdtRecordPtr.DynamicStringEvent->Guid)); + + if (UseModuleName) { + StringPtr =3D ModuleName; + } else if (Token !=3D NULL) { + StringPtr =3D Token; + } else if (Module !=3D NULL) { + StringPtr =3D Module; + } else if (ModuleName !=3D NULL) { + StringPtr =3D ModuleName; + } + if (StringPtr !=3D NULL && AsciiStrLen (StringPtr) !=3D 0) { + StrLength =3D AsciiStrLen (StringPtr); + DestMax =3D (RecordInfo.RecordSize - sizeof (DYNAMIC_STRING_EV= ENT_RECORD)) / sizeof (CHAR8); + if (StrLength >=3D DestMax) { + StrLength =3D DestMax -1; + } + AsciiStrnCpyS (FpdtRecordPtr.DynamicStringEvent->String, DestMax, St= ringPtr, StrLength); + } + break; + + case GUID_QWORD_EVENT_TYPE: + FpdtRecordPtr.GuidQwordEvent->Header.Type =3D GUID_QWORD_EVE= NT_TYPE; + FpdtRecordPtr.GuidQwordEvent->Header.Length =3D RecordInfo.Rec= ordSize; + FpdtRecordPtr.GuidQwordEvent->Header.Revision =3D RECORD_REVISIO= N_1; + FpdtRecordPtr.GuidQwordEvent->ProgressID =3D RecordInfo.Pro= gressID; + FpdtRecordPtr.GuidQwordEvent->Timestamp =3D TimeStamp; + CopyMem (&FpdtRecordPtr.GuidQwordEvent->Guid, &ModuleGuid, sizeof (Fpd= tRecordPtr.GuidQwordEvent->Guid)); + if ((MODULE_LOADIMAGE_START_ID =3D=3D RecordInfo.ProgressID) && AsciiS= trCmp (Token, LOAD_IMAGE_TOK) =3D=3D 0) { + mLoadImageCount++; + FpdtRecordPtr.GuidQwordEvent->Qword =3D mLoadImageCoun= t; } + break; + + case GUID_QWORD_STRING_EVENT_TYPE: + FpdtRecordPtr.GuidQwordStringEvent->Header.Type =3D GUID_QWORD_STR= ING_EVENT_TYPE; + FpdtRecordPtr.GuidQwordStringEvent->Header.Length =3D RecordInfo.Rec= ordSize; + FpdtRecordPtr.GuidQwordStringEvent->Header.Revision =3D RECORD_REVISIO= N_1; + FpdtRecordPtr.GuidQwordStringEvent->ProgressID =3D RecordInfo.Pro= gressID; + FpdtRecordPtr.GuidQwordStringEvent->Timestamp =3D TimeStamp; + CopyMem (&FpdtRecordPtr.GuidQwordStringEvent->Guid, &ModuleGuid, sizeo= f (FpdtRecordPtr.GuidQwordStringEvent->Guid)); + break; + + default: + // + // Record is not supported in current DXE phase, return EFI_ABORTED + // + return EFI_UNSUPPORTED; + } + + // + // Report record one by one after records have been reported together. + // + if (mFpdtDataIsReported) { + REPORT_STATUS_CODE_EX ( + EFI_PROGRESS_CODE, + EFI_SOFTWARE_DXE_BS_DRIVER, + 0, + NULL, + &gEfiFirmwarePerformanceGuid, + FpdtRecordPtr.RecordHeader, + FpdtRecordPtr.RecordHeader->Length + ); + } else { + // + // Update the cached FPDT record buffer. + // + mPerformanceLength +=3D FpdtRecordPtr.RecordHeader->Length; + } =20 - mGaugeData =3D NewGaugeData; - mMaxGaugeRecords *=3D 2; + return EFI_SUCCESS; +} =20 +/** + Dumps all the PEI performance. + + @param HobStart A pointer to a Guid. + + This internal function dumps all the PEI performance log to the DXE perf= ormance gauge array. + It retrieves the optional GUID HOB for PEI performance and then saves th= e performance data + to DXE performance data structures. + +**/ +VOID +InternalGetPeiPerformance ( + VOID *HobStart + ) +{ + UINT8 *FirmwarePerformanceHob; + PEI_EXT_FIRMWARE_PERF_HEADER *PeiPerformanceLogHeader; + UINT8 *EventRec; + EFI_HOB_GUID_TYPE *GuidHob; + + GuidHob =3D GetNextGuidHob (&gEdkiiExtendedFirmwarePerformanceGuid, HobS= tart); + while (GuidHob !=3D NULL) { + FirmwarePerformanceHob =3D GET_GUID_HOB_DATA (GuidHob); + PeiPerformanceLogHeader =3D (PEI_EXT_FIRMWARE_PERF_HEADER *)FirmwarePe= rformanceHob; + + if (mPerformanceLength + PeiPerformanceLogHeader->SizeOfAllEntries > m= MaxPerformanceLength) { + mPerformancePointer =3D ReallocatePool ( + mPerformanceLength, + mPerformanceLength + + (UINTN)PeiPerformanceLogHeader->SizeOfAllEnt= ries + + FIRMWARE_RECORD_BUFFER, + mPerformancePointer + ); + ASSERT (mPerformancePointer !=3D NULL); + mMaxPerformanceLength =3D mPerformanceLength + + (UINTN)(PeiPerformanceLogHeader->SizeOfAllEn= tries) + + FIRMWARE_RECORD_BUFFER; + } + + EventRec =3D mPerformancePointer + mPerformanceLength; + CopyMem (EventRec, FirmwarePerformanceHob + sizeof (PEI_EXT_FIRMWARE_P= ERF_HEADER), (UINTN)(PeiPerformanceLogHeader->SizeOfAllEntries)); // - // Initialize new data array and migrate old data one. + // Update the used buffer size. // - mGaugeData =3D CopyMem (mGaugeData, OldGaugeData, OldGaugeDataSize); + mPerformanceLength +=3D (UINTN)(PeiPerformanceLogHeader->SizeOfAllEntr= ies); + mLoadImageCount +=3D PeiPerformanceLogHeader->LoadImageCount; =20 - FreePool (OldGaugeData); + // + // Get next performance guid hob + // + GuidHob =3D GetNextGuidHob (&gEdkiiExtendedFirmwarePerformanceGuid, GE= T_NEXT_HOB (GuidHob)); } +} =20 - GaugeEntryExArray =3D (GAUGE_DATA_ENTRY_EX *) (mGaugeData = + 1); - GaugeEntryExArray[Index].Handle =3D (EFI_PHYSICAL_ADDRESS) (UINTN) Handl= e; +/** + Report all FPDT record as report status code. =20 - if (Token !=3D NULL) { - AsciiStrnCpyS (GaugeEntryExArray[Index].Token, DXE_PERFORMANCE_STRING_= SIZE, Token, DXE_PERFORMANCE_STRING_LENGTH); - } - if (Module !=3D NULL) { - AsciiStrnCpyS (GaugeEntryExArray[Index].Module, DXE_PERFORMANCE_STRING= _SIZE, Module, DXE_PERFORMANCE_STRING_LENGTH); + @param Event The event of notify protocol. + @param Context Notify event context. + +**/ +VOID +EFIAPI +ReportFpdtRecordData ( + IN EFI_EVENT Event, + IN VOID *Context + ) +{ + UINT8 *PerfBuffer; + UINTN MaxUint16; + + PerfBuffer =3D mPerformancePointer; + MaxUint16 =3D 0xFFFF; + + if (mPerformanceLength =3D=3D 0 || mPerformancePointer =3D=3D NULL) { + return; } =20 - GaugeEntryExArray[Index].EndTimeStamp =3D 0; - GaugeEntryExArray[Index].Identifier =3D Identifier; + mStartFpdtDataReport =3D TRUE; =20 - if (TimeStamp =3D=3D 0) { - TimeStamp =3D GetPerformanceCounter (); + while (mPerformanceLength > MaxUint16) { + // + // Report extension data size is UINT16. So, the size of report data c= an't exceed 0xFFFF. + // + REPORT_STATUS_CODE_EX ( + EFI_PROGRESS_CODE, + EFI_SOFTWARE_DXE_BS_DRIVER, + 0, + NULL, + &gEfiFirmwarePerformanceGuid, + PerfBuffer, + MaxUint16 + ); + mPerformanceLength =3D mPerformanceLength - MaxUint16; + PerfBuffer =3D PerfBuffer + MaxUint16; } - GaugeEntryExArray[Index].StartTimeStamp =3D TimeStamp; =20 - mGaugeData->NumberOfEntries++; + REPORT_STATUS_CODE_EX ( + EFI_PROGRESS_CODE, + EFI_SOFTWARE_DXE_BS_DRIVER, + 0, + NULL, + &gEfiFirmwarePerformanceGuid, + PerfBuffer, + mPerformanceLength + ); + // + // Free Cached FPDT record Buffer + // + FreePool (mPerformancePointer); + mPerformancePointer =3D NULL; + mPerformanceLength =3D 0; + mMaxPerformanceLength =3D 0; + // + // Set FPDT report state to TRUE. + // + mFpdtDataIsReported =3D TRUE; + mStartFpdtDataReport =3D FALSE; +} =20 - EfiReleaseLock (&mPerfRecordLock); +/** + Adds a record at the end of the performance measurement log + that records the start time of a performance measurement. =20 - return EFI_SUCCESS; + Adds a record to the end of the performance measurement log + that contains the Handle, Token, Module and Identifier. + The end time of the new record must be set to zero. + If TimeStamp is not zero, then TimeStamp is used to fill in the start ti= me in the record. + If TimeStamp is zero, the start time in the record is filled in with the= value + read from the current time stamp. + + @param Handle Pointer to environment specific context = used + to identify the component being measured. + @param Token Pointer to a Null-terminated ASCII string + that identifies the component being meas= ured. + @param Module Pointer to a Null-terminated ASCII string + that identifies the module being measure= d. + @param TimeStamp 64-bit time stamp. + @param Identifier 32-bit identifier. If the value is 0, th= e created record + is same as the one created by StartGauge= of PERFORMANCE_PROTOCOL. + + @retval EFI_SUCCESS The data was read correctly from the dev= ice. + @retval EFI_OUT_OF_RESOURCES There are not enough resources to record= the measurement. + +**/ +EFI_STATUS +EFIAPI +StartGaugeEx ( + IN CONST VOID *Handle, OPTIONAL + IN CONST CHAR8 *Token, OPTIONAL + IN CONST CHAR8 *Module, OPTIONAL + IN UINT64 TimeStamp, + IN UINT32 Identifier + ) +{ + return InsertFpdtMeasurement (TRUE, Handle, Token, Module, TimeStamp, Id= entifier); } =20 /** Searches the performance measurement log from the beginning of the log for the first matching record that contains a zero end time and fills in= a valid end time. =20 Searches the performance measurement log from the beginning of the log - for the first record that matches Handle, Token and Module and has an en= d time value of zero. + for the first record that matches Handle, Token, Module and Identifier a= nd has an end time value of zero. If the record can not be found then return EFI_NOT_FOUND. If the record is found and TimeStamp is not zero, then the end time in the record is filled in with the value specified by= TimeStamp. If the record is found and TimeStamp is zero, then the end time in the m= atching record is filled in with the current time stamp value. @@ -260,89 +927,19 @@ EndGaugeEx ( IN CONST CHAR8 *Module, OPTIONAL IN UINT64 TimeStamp, IN UINT32 Identifier ) { - GAUGE_DATA_ENTRY_EX *GaugeEntryExArray; - UINT32 Index; - EFI_STATUS Status; - - Status =3D EfiAcquireLockOrFail (&mPerfRecordLock); - if (EFI_ERROR (Status)) { - return Status; - } - - if (TimeStamp =3D=3D 0) { - TimeStamp =3D GetPerformanceCounter (); - } - - Index =3D InternalSearchForGaugeEntry (Handle, Token, Module, Identifier= ); - if (Index >=3D mGaugeData->NumberOfEntries) { - EfiReleaseLock (&mPerfRecordLock); - return EFI_NOT_FOUND; - } - GaugeEntryExArray =3D (GAUGE_DATA_ENTRY_EX *) (mGaugeData + 1); - GaugeEntryExArray[Index].EndTimeStamp =3D TimeStamp; - - EfiReleaseLock (&mPerfRecordLock); - return EFI_SUCCESS; + return InsertFpdtMeasurement (FALSE, Handle, Token, Module, TimeStamp, I= dentifier); } =20 /** Retrieves a previously logged performance measurement. It can also retrieve the log created by StartGauge and EndGauge of PERFO= RMANCE_PROTOCOL, and then assign the Identifier with 0. =20 - Retrieves the performance log entry from the performance log specified b= y LogEntryKey. - If it stands for a valid entry, then EFI_SUCCESS is returned and - GaugeDataEntryEx stores the pointer to that entry. - - This internal function is added to avoid releasing lock before each retu= rn statement. - - @param LogEntryKey The key for the previous performance mea= surement log entry. - If 0, then the first performance measure= ment log entry is retrieved. - @param GaugeDataEntryEx The indirect pointer to the extended gau= ge data entry specified by LogEntryKey - if the retrieval is successful. - - @retval EFI_SUCCESS The GuageDataEntryEx is successfully fou= nd based on LogEntryKey. - @retval EFI_NOT_FOUND The LogEntryKey is the last entry (equal= s to the total entry number). - @retval EFI_INVALIDE_PARAMETER The LogEntryKey is not a valid entry (gr= eater than the total entry number). - @retval EFI_INVALIDE_PARAMETER GaugeDataEntryEx is NULL. - -**/ -EFI_STATUS -EFIAPI -InternalGetGaugeEx ( - IN UINTN LogEntryKey, - OUT GAUGE_DATA_ENTRY_EX **GaugeDataEntryEx - ) -{ - UINTN NumberOfEntries; - GAUGE_DATA_ENTRY_EX *GaugeEntryExArray; - - NumberOfEntries =3D (UINTN) (mGaugeData->NumberOfEntries); - if (LogEntryKey > NumberOfEntries) { - return EFI_INVALID_PARAMETER; - } - if (LogEntryKey =3D=3D NumberOfEntries) { - return EFI_NOT_FOUND; - } - - GaugeEntryExArray =3D (GAUGE_DATA_ENTRY_EX *) (mGaugeData + 1); - - if (GaugeDataEntryEx =3D=3D NULL) { - return EFI_INVALID_PARAMETER; - } - *GaugeDataEntryEx =3D &GaugeEntryExArray[LogEntryKey]; - - return EFI_SUCCESS; -} - -/** - Retrieves a previously logged performance measurement. - It can also retrieve the log created by StartGauge and EndGauge of PERFO= RMANCE_PROTOCOL, - and then assign the Identifier with 0. + !!! Not support!!! =20 Retrieves the performance log entry from the performance log specified b= y LogEntryKey. If it stands for a valid entry, then EFI_SUCCESS is returned and GaugeDataEntryEx stores the pointer to that entry. =20 @@ -362,22 +959,11 @@ EFIAPI GetGaugeEx ( IN UINTN LogEntryKey, OUT GAUGE_DATA_ENTRY_EX **GaugeDataEntryEx ) { - EFI_STATUS Status; - - Status =3D EfiAcquireLockOrFail (&mPerfRecordLock); - if (EFI_ERROR (Status)) { - return Status; - } - - Status =3D InternalGetGaugeEx (LogEntryKey, GaugeDataEntryEx); - - EfiReleaseLock (&mPerfRecordLock); - - return Status; + return EFI_UNSUPPORTED; } =20 /** Adds a record at the end of the performance measurement log that records the start time of a performance measurement. @@ -452,10 +1038,12 @@ EndGauge ( /** Retrieves a previously logged performance measurement. It can also retrieve the log created by StartGaugeEx and EndGaugeEx of P= ERFORMANCE_EX_PROTOCOL, and then eliminate the Identifier. =20 + !!! Not support!!! + Retrieves the performance log entry from the performance log specified b= y LogEntryKey. If it stands for a valid entry, then EFI_SUCCESS is returned and GaugeDataEntry stores the pointer to that entry. =20 @param LogEntryKey The key for the previous performance mea= surement log entry. @@ -474,81 +1062,13 @@ EFIAPI GetGauge ( IN UINTN LogEntryKey, OUT GAUGE_DATA_ENTRY **GaugeDataEntry ) { - EFI_STATUS Status; - GAUGE_DATA_ENTRY_EX *GaugeEntryEx; - - GaugeEntryEx =3D NULL; - - Status =3D GetGaugeEx (LogEntryKey, &GaugeEntryEx); - if (EFI_ERROR (Status)) { - return Status; - } - - if (GaugeDataEntry =3D=3D NULL) { - return EFI_INVALID_PARAMETER; - } - - *GaugeDataEntry =3D (GAUGE_DATA_ENTRY *) GaugeEntryEx; - - return EFI_SUCCESS; + return EFI_UNSUPPORTED; } =20 -/** - Dumps all the PEI performance log to DXE performance gauge array. - - This internal function dumps all the PEI performance log to the DXE perf= ormance gauge array. - It retrieves the optional GUID HOB for PEI performance and then saves th= e performance data - to DXE performance data structures. - -**/ -VOID -InternalGetPeiPerformance ( - VOID - ) -{ - EFI_HOB_GUID_TYPE *GuidHob; - PEI_PERFORMANCE_LOG_HEADER *LogHob; - PEI_PERFORMANCE_LOG_ENTRY *LogEntryArray; - UINT32 *LogIdArray; - GAUGE_DATA_ENTRY_EX *GaugeEntryExArray; - UINT32 Index; - UINT32 NumberOfEntries; - - NumberOfEntries =3D 0; - GaugeEntryExArray =3D (GAUGE_DATA_ENTRY_EX *) (mGaugeData + 1); - - // - // Dump PEI Log Entries to DXE Guage Data structure. - // - GuidHob =3D GetFirstGuidHob (&gPerformanceProtocolGuid); - if (GuidHob !=3D NULL) { - LogHob =3D GET_GUID_HOB_DATA (GuidHob); - LogEntryArray =3D (PEI_PERFORMANCE_LOG_ENTRY *) (LogHob + 1); - - NumberOfEntries =3D LogHob->NumberOfEntries; - for (Index =3D 0; Index < NumberOfEntries; Index++) { - GaugeEntryExArray[Index].Handle =3D LogEntryArray[Index].Han= dle; - AsciiStrCpyS (GaugeEntryExArray[Index].Token, DXE_PERFORMANCE_STRIN= G_SIZE, LogEntryArray[Index].Token); - AsciiStrCpyS (GaugeEntryExArray[Index].Module, DXE_PERFORMANCE_STRIN= G_SIZE, LogEntryArray[Index].Module); - GaugeEntryExArray[Index].StartTimeStamp =3D LogEntryArray[Index].Sta= rtTimeStamp; - GaugeEntryExArray[Index].EndTimeStamp =3D LogEntryArray[Index].End= TimeStamp; - GaugeEntryExArray[Index].Identifier =3D 0; - } - - GuidHob =3D GetFirstGuidHob (&gPerformanceExProtocolGuid); - if (GuidHob !=3D NULL) { - LogIdArray =3D GET_GUID_HOB_DATA (GuidHob); - for (Index =3D 0; Index < NumberOfEntries; Index++) { - GaugeEntryExArray[Index].Identifier =3D LogIdArray[Index]; - } - } - } - mGaugeData->NumberOfEntries =3D NumberOfEntries; -} =20 /** The constructor function initializes Performance infrastructure for DXE = phase. =20 The constructor function publishes Performance and PerformanceEx protoco= l, allocates memory to log DXE performance @@ -567,40 +1087,53 @@ DxeCorePerformanceLibConstructor ( IN EFI_HANDLE ImageHandle, IN EFI_SYSTEM_TABLE *SystemTable ) { EFI_STATUS Status; + EFI_HANDLE Handle; + EFI_EVENT ReadyToBootEvent; PERFORMANCE_PROPERTY *PerformanceProperty; =20 - if (!PerformanceMeasurementEnabled ()) { // // Do not initialize performance infrastructure if not required. // return EFI_SUCCESS; } + + // + // Dump normal PEI performance records + // + InternalGetPeiPerformance (GetHobList()); + // - // Install the protocol interfaces. + // Install the protocol interfaces for DXE performance library instance. // + Handle =3D NULL; Status =3D gBS->InstallMultipleProtocolInterfaces ( - &mHandle, + &Handle, &gPerformanceProtocolGuid, &mPerformanceInterface, &gPerformanceExProtocolGuid, &mPerformanceExInterface, NULL ); ASSERT_EFI_ERROR (Status); =20 - mMaxGaugeRecords =3D INIT_DXE_GAUGE_DATA_ENTRIES + (UINT16) (PcdGet16 (P= cdMaxPeiPerformanceLogEntries16) !=3D 0 ? - PcdGet16 (Pcd= MaxPeiPerformanceLogEntries16) : - PcdGet8 (PcdM= axPeiPerformanceLogEntries)); - - mGaugeData =3D AllocateZeroPool (sizeof (GAUGE_DATA_HEADER) + (sizeof (G= AUGE_DATA_ENTRY_EX) * mMaxGaugeRecords)); - ASSERT (mGaugeData !=3D NULL); + // + // Register End of DXE event to report StatusCode data + // + Status =3D gBS->CreateEventEx ( + EVT_NOTIFY_SIGNAL, + TPL_CALLBACK, + ReportFpdtRecordData, + NULL, + &gEfiEventReadyToBootGuid, + &ReadyToBootEvent + ); =20 - InternalGetPeiPerformance (); + ASSERT_EFI_ERROR (Status); =20 Status =3D EfiGetSystemConfigurationTable (&gPerformanceProtocolGuid, (V= OID **) &PerformanceProperty); if (EFI_ERROR (Status)) { // // Install configuration table for performance property. @@ -651,19 +1184,19 @@ StartPerformanceMeasurementEx ( IN CONST CHAR8 *Module, OPTIONAL IN UINT64 TimeStamp, IN UINT32 Identifier ) { - return (RETURN_STATUS) StartGaugeEx (Handle, Token, Module, TimeStamp, I= dentifier); + return InsertFpdtMeasurement (TRUE, Handle, Token, Module, TimeStamp, Id= entifier); } =20 /** Searches the performance measurement log from the beginning of the log for the first matching record that contains a zero end time and fills in= a valid end time. =20 Searches the performance measurement log from the beginning of the log - for the first record that matches Handle, Token and Module and has an en= d time value of zero. + for the first record that matches Handle, Token, Module and Identifier a= nd has an end time value of zero. If the record can not be found then return RETURN_NOT_FOUND. If the record is found and TimeStamp is not zero, then the end time in the record is filled in with the value specified by= TimeStamp. If the record is found and TimeStamp is zero, then the end time in the m= atching record is filled in with the current time stamp value. @@ -690,18 +1223,20 @@ EndPerformanceMeasurementEx ( IN CONST CHAR8 *Module, OPTIONAL IN UINT64 TimeStamp, IN UINT32 Identifier ) { - return (RETURN_STATUS) EndGaugeEx (Handle, Token, Module, TimeStamp, Ide= ntifier); + return InsertFpdtMeasurement (FALSE, Handle, Token, Module, TimeStamp, I= dentifier); } =20 /** Attempts to retrieve a performance measurement log entry from the perfor= mance measurement log. It can also retrieve the log created by StartPerformanceMeasurement and = EndPerformanceMeasurement, and then assign the Identifier with 0. =20 + !!! Not support!!! + Attempts to retrieve the performance log entry specified by LogEntryKey.= If LogEntryKey is zero on entry, then an attempt is made to retrieve the first entry from = the performance log, and the key for the second entry in the log is returned. If the perform= ance log is empty, then no entry is retrieved and zero is returned. If LogEntryKey is not = zero, then the performance log entry associated with LogEntryKey is retrieved, and the key for the = next entry in the log is @@ -729,63 +1264,29 @@ EndPerformanceMeasurementEx ( being measured. @param StartTimeStamp Pointer to the 64-bit time stamp that wa= s recorded when the measurement was started. @param EndTimeStamp Pointer to the 64-bit time stamp that wa= s recorded when the measurement was ended. - @param Identifier Pointer to the 32-bit identifier that wa= s recorded. + @param Identifier Pointer to the 32-bit identifier that wa= s recorded when the measurement + was ended. =20 @return The key for the next performance log entry (in general case). =20 **/ UINTN EFIAPI GetPerformanceMeasurementEx ( - IN UINTN LogEntryKey,=20 + IN UINTN LogEntryKey, OUT CONST VOID **Handle, OUT CONST CHAR8 **Token, OUT CONST CHAR8 **Module, OUT UINT64 *StartTimeStamp, OUT UINT64 *EndTimeStamp, OUT UINT32 *Identifier ) { - EFI_STATUS Status; - GAUGE_DATA_ENTRY_EX *GaugeData; - - GaugeData =3D NULL; - =20 - ASSERT (Handle !=3D NULL); - ASSERT (Token !=3D NULL); - ASSERT (Module !=3D NULL); - ASSERT (StartTimeStamp !=3D NULL); - ASSERT (EndTimeStamp !=3D NULL); - ASSERT (Identifier !=3D NULL); - - Status =3D GetGaugeEx (LogEntryKey++, &GaugeData); - - // - // Make sure that LogEntryKey is a valid log entry key, - // - ASSERT (Status !=3D EFI_INVALID_PARAMETER); - - if (EFI_ERROR (Status)) { - // - // The LogEntryKey is the last entry (equals to the total entry number= ). - // - return 0; - } - - ASSERT (GaugeData !=3D NULL); - - *Handle =3D (VOID *) (UINTN) GaugeData->Handle; - *Token =3D GaugeData->Token; - *Module =3D GaugeData->Module; - *StartTimeStamp =3D GaugeData->StartTimeStamp; - *EndTimeStamp =3D GaugeData->EndTimeStamp; - *Identifier =3D GaugeData->Identifier; - - return LogEntryKey; + return 0; } =20 /** Adds a record at the end of the performance measurement log that records the start time of a performance measurement. @@ -816,11 +1317,11 @@ StartPerformanceMeasurement ( IN CONST CHAR8 *Token, OPTIONAL IN CONST CHAR8 *Module, OPTIONAL IN UINT64 TimeStamp ) { - return StartPerformanceMeasurementEx (Handle, Token, Module, TimeStamp, = 0); + return InsertFpdtMeasurement (TRUE, Handle, Token, Module, TimeStamp, 0); } =20 /** Searches the performance measurement log from the beginning of the log for the first matching record that contains a zero end time and fills in= a valid end time. @@ -852,18 +1353,20 @@ EndPerformanceMeasurement ( IN CONST CHAR8 *Token, OPTIONAL IN CONST CHAR8 *Module, OPTIONAL IN UINT64 TimeStamp ) { - return EndPerformanceMeasurementEx (Handle, Token, Module, TimeStamp, 0); + return InsertFpdtMeasurement (FALSE, Handle, Token, Module, TimeStamp, 0= ); } =20 /** Attempts to retrieve a performance measurement log entry from the perfor= mance measurement log. It can also retrieve the log created by StartPerformanceMeasurementEx an= d EndPerformanceMeasurementEx, and then eliminate the Identifier. =20 + !!! Not support!!! + Attempts to retrieve the performance log entry specified by LogEntryKey.= If LogEntryKey is zero on entry, then an attempt is made to retrieve the first entry from = the performance log, and the key for the second entry in the log is returned. If the perform= ance log is empty, then no entry is retrieved and zero is returned. If LogEntryKey is not = zero, then the performance log entry associated with LogEntryKey is retrieved, and the key for the = next entry in the log is @@ -905,12 +1408,11 @@ GetPerformanceMeasurement ( OUT CONST CHAR8 **Module, OUT UINT64 *StartTimeStamp, OUT UINT64 *EndTimeStamp ) { - UINT32 Identifier; - return GetPerformanceMeasurementEx (LogEntryKey, Handle, Token, Module, = StartTimeStamp, EndTimeStamp, &Identifier); + return 0; } =20 /** Returns TRUE if the performance measurement macros are enabled. =20 diff --git a/MdeModulePkg/Library/DxeCorePerformanceLib/DxeCorePerformanceL= ib.inf b/MdeModulePkg/Library/DxeCorePerformanceLib/DxeCorePerformanceLib.i= nf index 5b89ce2..c2e5ccf 100644 --- a/MdeModulePkg/Library/DxeCorePerformanceLib/DxeCorePerformanceLib.inf +++ b/MdeModulePkg/Library/DxeCorePerformanceLib/DxeCorePerformanceLib.inf @@ -7,11 +7,11 @@ # It initializes DXE phase performance logging by publishing the Performa= nce and PerformanceEx Protocol, # which is consumed by DxePerformanceLib to logging performance data in D= XE phase. # This library is mainly used by DxeCore to start performance logging to = ensure that # Performance and PerformanceEx Protocol are installed at the very beginn= ing of DXE phase. # =20 -# Copyright (c) 2006 - 2017, Intel Corporation. All rights reserved.
+# Copyright (c) 2006 - 2018, Intel Corporation. All rights reserved.
# (C) Copyright 2016 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 # which accompanies this distribution. The full text of the license may = be found at # http://opensource.org/licenses/bsd-license.php @@ -55,20 +55,26 @@ BaseMemoryLib BaseLib HobLib DebugLib UefiLib + ReportStatusCodeLib + DxeServicesLib + PeCoffGetEntryPointLib =20 =20 [Guids] ## SOMETIMES_CONSUMES ## HOB ## PRODUCES ## UNDEFINED # Install protocol ## PRODUCES ## SystemTable gPerformanceProtocolGuid ## SOMETIMES_CONSUMES ## HOB ## PRODUCES ## UNDEFINED # Install protocol gPerformanceExProtocolGuid + gZeroGuid ## SOMETIMES_CONSUMES ## G= UID + gEfiFirmwarePerformanceGuid ## SOMETIMES_PRODUCES ## U= NDEFINED # StatusCode Data + gEdkiiExtendedFirmwarePerformanceGuid ## SOMETIMES_CONSUMES ## H= OB + gEfiEventReadyToBootGuid ## CONSUMES ## E= vent =20 [Pcd] - gEfiMdeModulePkgTokenSpaceGuid.PcdMaxPeiPerformanceLogEntries ## CONSU= MES - gEfiMdeModulePkgTokenSpaceGuid.PcdMaxPeiPerformanceLogEntries16 ## CONSU= MES - gEfiMdePkgTokenSpaceGuid.PcdPerformanceLibraryPropertyMask ## CONSU= MES + gEfiMdePkgTokenSpaceGuid.PcdPerformanceLibraryPropertyMask ## CO= NSUMES + gEfiMdeModulePkgTokenSpaceGuid.PcdEdkiiFpdtStringRecordEnableOnly ## CO= NSUMES diff --git a/MdeModulePkg/Library/DxeCorePerformanceLib/DxeCorePerformanceL= ibInternal.h b/MdeModulePkg/Library/DxeCorePerformanceLib/DxeCorePerformanc= eLibInternal.h index f1540d8..42fce9b 100644 --- a/MdeModulePkg/Library/DxeCorePerformanceLib/DxeCorePerformanceLibInter= nal.h +++ b/MdeModulePkg/Library/DxeCorePerformanceLib/DxeCorePerformanceLibInter= nal.h @@ -2,11 +2,11 @@ Master header files for DxeCorePerformanceLib instance. =20 This header file holds the prototypes of the Performance and Performance= Ex Protocol published by this library instance at its constructor. =20 -Copyright (c) 2006 - 2017, Intel Corporation. All rights reserved.
+Copyright (c) 2006 - 2018, 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 which accompanies this distribution. The full text of the license may be = found at http://opensource.org/licenses/bsd-license.php =20 @@ -20,10 +20,19 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITH= ER EXPRESS OR IMPLIED. =20 =20 #include =20 #include +#include +#include +#include +#include + +#include +#include +#include +#include =20 #include #include #include #include @@ -31,10 +40,13 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITH= ER EXPRESS OR IMPLIED. #include #include #include #include #include +#include +#include +#include =20 // // Interface declarations for PerformanceEx Protocol. // /** --=20 1.9.5.msysgit.1 _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel From nobody Fri Nov 1 12:22:50 2024 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.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 1516679294631616.3165385681285; Mon, 22 Jan 2018 19:48:14 -0800 (PST) Received: from [127.0.0.1] (localhost [IPv6:::1]) by ml01.01.org (Postfix) with ESMTP id D835F2233379C; Mon, 22 Jan 2018 19:42:46 -0800 (PST) Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) (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 721B02233379C for ; Mon, 22 Jan 2018 19:42:44 -0800 (PST) Received: from orsmga008.jf.intel.com ([10.7.209.65]) by fmsmga101.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 22 Jan 2018 19:48:10 -0800 Received: from shwdeopenpsi114.ccr.corp.intel.com ([10.239.157.135]) by orsmga008.jf.intel.com with ESMTP; 22 Jan 2018 19:48:09 -0800 X-Original-To: edk2-devel@lists.01.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; Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=192.55.52.88; helo=mga01.intel.com; envelope-from=dandan.bi@intel.com; receiver=edk2-devel@lists.01.org X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.46,399,1511856000"; d="scan'208";a="12262539" From: Dandan Bi To: edk2-devel@lists.01.org Date: Tue, 23 Jan 2018 11:47:31 +0800 Message-Id: <1516679255-12328-5-git-send-email-dandan.bi@intel.com> X-Mailer: git-send-email 1.9.5.msysgit.1 In-Reply-To: <1516679255-12328-1-git-send-email-dandan.bi@intel.com> References: <1516679255-12328-1-git-send-email-dandan.bi@intel.com> Subject: [edk2] [patch 4/8] MdeModulePkg/SmmCorePerformanceLib:Track FPDT record in SMM phase X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Star Zeng , Liming Gao 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" Updated to convert Pref entry to FPDT record in SMM phase and then export records to FPDT table. Cc: Liming Gao Cc: Star Zeng Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Dandan Bi --- .../SmmCorePerformanceLib/SmmCorePerformanceLib.c | 1094 ++++++++++++----= ---- .../SmmCorePerformanceLib.inf | 10 +- .../SmmCorePerformanceLibInternal.h | 11 +- 3 files changed, 681 insertions(+), 434 deletions(-) diff --git a/MdeModulePkg/Library/SmmCorePerformanceLib/SmmCorePerformanceL= ib.c b/MdeModulePkg/Library/SmmCorePerformanceLib/SmmCorePerformanceLib.c index cd1f1a5..cb880ef 100644 --- a/MdeModulePkg/Library/SmmCorePerformanceLib/SmmCorePerformanceLib.c +++ b/MdeModulePkg/Library/SmmCorePerformanceLib/SmmCorePerformanceLib.c @@ -14,11 +14,11 @@ This external input must be validated carefully to avoid security issue l= ike buffer overflow, integer overflow. =20 SmmPerformanceHandlerEx(), SmmPerformanceHandler() will receive untrusted= input and do basic validation. =20 -Copyright (c) 2011 - 2017, Intel Corporation. All rights reserved.
+Copyright (c) 2011 - 2018, 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 which accompanies this distribution. The full text of the license may be = found at http://opensource.org/licenses/bsd-license.php =20 @@ -28,30 +28,32 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITH= ER EXPRESS OR IMPLIED. **/ =20 =20 #include "SmmCorePerformanceLibInternal.h" =20 -// -// The data structure to hold global performance data. -// -GAUGE_DATA_HEADER *mGaugeData; - -// -// The current maximum number of logging entries. If current number of=20 -// entries exceeds this value, it will re-allocate a larger array and -// migration the old data to the larger array. -// -UINT32 mMaxGaugeRecords; - -// -// The handle to install Performance Protocol instance. -// -EFI_HANDLE mHandle =3D NULL; - -BOOLEAN mPerformanceMeasurementEnabled; - -SPIN_LOCK mSmmPerfLock; +#define STRING_SIZE (EDKII_STRING_EVENT_RECORD_NAME_= LENGTH * sizeof (CHAR8)) +#define MAX_RECORD_SIZE (sizeof (FPDT_RECORD) + STRING_S= IZE) +#define FIRMWARE_RECORD_BUFFER 0x1000 +#define CACHE_HANDLE_GUID_COUNT 0x1000 + +typedef struct { + EFI_HANDLE Handle; + CHAR8 NameString[EDKII_STRING_EVENT_RECORD_NAME_LENGTH]; + EFI_GUID ModuleGuid; +} HANDLE_GUID_MAP; + +HANDLE_GUID_MAP mCacheHandleGuidTable[CACHE_HANDLE_GUID_COUNT]; +UINTN mCachePairCount =3D 0; + +UINT8 *mPerformancePointer =3D NULL; +UINTN mPerformanceLength =3D 0; +UINTN mMaxPerformanceLength =3D 0; +BOOLEAN mFpdtDataIsReported =3D FALSE; +BOOLEAN mStartFpdtDataReport =3D FALSE; +CHAR8 *mPlatformLanguage =3D NULL; +SPIN_LOCK mSmmFpdtLock; +PERFORMANCE_PROPERTY mPerformanceProperty; =20 // // Interfaces for SMM Performance Protocol. // PERFORMANCE_PROTOCOL mPerformanceInterface =3D { @@ -67,172 +69,634 @@ PERFORMANCE_EX_PROTOCOL mPerformanceExInterface =3D { StartGaugeEx, EndGaugeEx, GetGaugeEx }; =20 -PERFORMANCE_PROPERTY mPerformanceProperty; +/** +Check whether the Token is a known one which is uesed by core. + +@param Token Pointer to a Null-terminated ASCII string + +@retval TRUE Is a known one used by core. +@retval FALSE Not a known one. + +**/ +BOOLEAN +IsKnownTokens ( + IN CONST CHAR8 *Token + ) +{ + if (AsciiStrCmp (Token, SEC_TOK) =3D=3D 0 || + AsciiStrCmp (Token, PEI_TOK) =3D=3D 0 || + AsciiStrCmp (Token, DXE_TOK) =3D=3D 0 || + AsciiStrCmp (Token, BDS_TOK) =3D=3D 0 || + AsciiStrCmp (Token, DRIVERBINDING_START_TOK) =3D=3D 0 || + AsciiStrCmp (Token, DRIVERBINDING_SUPPORT_TOK) =3D=3D 0 || + AsciiStrCmp (Token, DRIVERBINDING_STOP_TOK) =3D=3D 0 || + AsciiStrCmp (Token, LOAD_IMAGE_TOK) =3D=3D 0 || + AsciiStrCmp (Token, START_IMAGE_TOK) =3D=3D 0 || + AsciiStrCmp (Token, PEIM_TOK) =3D=3D 0) { + return TRUE; + } else { + return FALSE; + } +} =20 /** - Searches in the gauge array with keyword Handle, Token, Module and Ident= fier. +Check whether the ID is a known one which map to the known Token. + +@param Identifier 32-bit identifier. =20 - This internal function searches for the gauge entry in the gauge array. - If there is an entry that exactly matches the given keywords - and its end time stamp is zero, then the index of that gauge entry is re= turned; - otherwise, the the number of gauge entries in the array is returned. +@retval TRUE Is a known one used by core. +@retval FALSE Not a known one. =20 +**/ +BOOLEAN +IsKnownID ( + IN UINT32 Identifier + ) +{ + if (Identifier =3D=3D MODULE_START_ID || + Identifier =3D=3D MODULE_END_ID || + Identifier =3D=3D MODULE_LOADIMAGE_START_ID || + Identifier =3D=3D MODULE_LOADIMAGE_END_ID || + Identifier =3D=3D MODULE_DB_START_ID || + Identifier =3D=3D MODULE_DB_END_ID || + Identifier =3D=3D MODULE_DB_SUPPORT_START_ID || + Identifier =3D=3D MODULE_DB_SUPPORT_END_ID || + Identifier =3D=3D MODULE_DB_STOP_START_ID || + Identifier =3D=3D MODULE_DB_STOP_END_ID) { + return TRUE; + } else { + return FALSE; + } +} + +/** + Get the FPDT record info. + + @param IsStart TRUE if the performance log is start log. @param Handle Pointer to environment specific context = used to identify the component being measured. @param Token Pointer to a Null-terminated ASCII string that identifies the component being meas= ured. @param Module Pointer to a Null-terminated ASCII string that identifies the module being measure= d. - @param Identifier 32-bit identifier. + @param RecordInfo On return, pointer to the info of the re= cord. + @param UseModuleName Only useful for DYNAMIC_STRING_EVENT_TYP= E, indicate that whether need use + Module name to fill the string field in = the DYNAMIC_STRING_EVENT_RECORD. =20 - @retval The index of gauge entry in the array. + @retval EFI_SUCCESS Get record info successfully. + @retval EFI_UNSUPPORTED No matched FPDT record. =20 **/ -UINT32 -SmmSearchForGaugeEntry ( - IN CONST VOID *Handle, OPTIONAL - IN CONST CHAR8 *Token, OPTIONAL - IN CONST CHAR8 *Module, OPTIONAL - IN CONST UINT32 Identifier +EFI_STATUS +GetFpdtRecordInfo ( + IN BOOLEAN IsStart, + IN CONST VOID *Handle, + IN CONST CHAR8 *Token, + IN CONST CHAR8 *Module, + OUT BASIC_RECORD_INFO *RecordInfo, + IN OUT BOOLEAN *UseModuleName ) { - UINT32 Index; - UINT32 Index2; - UINT32 NumberOfEntries; - GAUGE_DATA_ENTRY_EX *GaugeEntryExArray; + UINT16 RecordType; + UINTN StringSize; =20 - if (Token =3D=3D NULL) { - Token =3D ""; - } - if (Module =3D=3D NULL) { - Module =3D ""; + RecordType =3D DYNAMIC_STRING_EVENT_TYPE; + + // + // Token to Type and Id. + // + if (Token !=3D NULL) { + if (AsciiStrCmp (Token, START_IMAGE_TOK) =3D=3D 0) { // "= StartImage:" + *UseModuleName =3D TRUE; + RecordType =3D GUID_EVENT_TYPE; + if (IsStart) { + RecordInfo->ProgressID =3D MODULE_START_ID; + } else { + RecordInfo->ProgressID =3D MODULE_END_ID; + } + } else if (AsciiStrCmp (Token, LOAD_IMAGE_TOK) =3D=3D 0) { // "= LoadImage:" + *UseModuleName =3D TRUE; + RecordType =3D GUID_QWORD_EVENT_TYPE; + if (IsStart) { + RecordInfo->ProgressID =3D MODULE_LOADIMAGE_START_ID; + } else { + RecordInfo->ProgressID =3D MODULE_LOADIMAGE_END_ID; + } + } else { // Pref = used in Modules + if (IsStart) { + RecordInfo->ProgressID =3D PERF_INMODULE_START_ID; + } else { + RecordInfo->ProgressID =3D PERF_INMODULE_END_ID; + } + } + } else if (Handle !=3D NULL || Module !=3D NULL) { // Pr= ef used in Modules + if (IsStart) { + RecordInfo->ProgressID =3D PERF_INMODULE_START_ID; + } else { + RecordInfo->ProgressID =3D PERF_INMODULE_END_ID; + } + } else { + return EFI_UNSUPPORTED; } =20 - NumberOfEntries =3D mGaugeData->NumberOfEntries; - GaugeEntryExArray =3D (GAUGE_DATA_ENTRY_EX *) (mGaugeData + 1); + if (PcdGetBool (PcdEdkiiFpdtStringRecordEnableOnly)) { + RecordType =3D DYNAMIC_STRING_EVENT_TYPE; + RecordInfo->RecordSize =3D sizeof (DYNAMIC_STRING_EVENT_RECORD) + ST= RING_SIZE; + } else { + switch (RecordType) { + case GUID_EVENT_TYPE: + RecordInfo->RecordSize =3D sizeof (GUID_EVENT_RECORD); + break; =20 - Index2 =3D 0; + case DYNAMIC_STRING_EVENT_TYPE: + if (*UseModuleName) { + StringSize =3D STRING_SIZE; + } else if (Token !=3D NULL) { + StringSize =3D AsciiStrSize (Token); + } else if (Module !=3D NULL) { + StringSize =3D AsciiStrSize (Module); + } else { + StringSize =3D STRING_SIZE; + } + if (StringSize > STRING_SIZE) { + StringSize =3D STRING_SIZE; + } + RecordInfo->RecordSize =3D (UINT8)(sizeof (DYNAMIC_STRING_EVENT_REC= ORD) + StringSize); + break; =20 - for (Index =3D 0; Index < NumberOfEntries; Index++) { - Index2 =3D NumberOfEntries - 1 - Index; - if (GaugeEntryExArray[Index2].EndTimeStamp =3D=3D 0 && - (GaugeEntryExArray[Index2].Handle =3D=3D (EFI_PHYSICAL_ADDRESS) (U= INTN) Handle) && - AsciiStrnCmp (GaugeEntryExArray[Index2].Token, Token, SMM_PERFORMA= NCE_STRING_LENGTH) =3D=3D 0 && - AsciiStrnCmp (GaugeEntryExArray[Index2].Module, Module, SMM_PERFOR= MANCE_STRING_LENGTH) =3D=3D 0) { - Index =3D Index2; + case GUID_QWORD_EVENT_TYPE: + RecordInfo->RecordSize =3D (UINT8)sizeof (GUID_QWORD_EVENT_RECORD); break; + + default: + // + // Record type is unsupported in SMM phase. + // + return EFI_UNSUPPORTED; } } =20 - return Index; + RecordInfo->Type =3D RecordType; + return EFI_SUCCESS; } =20 /** - Adds a record at the end of the performance measurement log - that records the start time of a performance measurement. + Get a human readable module name and module guid for the given image han= dle. + If module name can't be found, "" string will return. + If module guid can't be found, Zero Guid will return. + + @param Handle Image handle or Controller handle. + @param NameString The ascii string will be filled into it. If not = found, null string will return. + @param BufferSize Size of the input NameString buffer. + @param ModuleGuid Point to the guid buffer to store the got module= guid value. + + @retval EFI_SUCCESS Successfully get module name and guid. + @retval EFI_INVALID_PARAMETER The input parameter NameString is NULL. + @retval other value Module Name can't be got. +**/ +EFI_STATUS +EFIAPI +GetModuleInfoFromHandle ( + IN EFI_HANDLE Handle, + OUT CHAR8 *NameString, + IN UINTN BufferSize, + OUT EFI_GUID *ModuleGuid OPTIONAL + ) +{ + EFI_STATUS Status; + EFI_LOADED_IMAGE_PROTOCOL *LoadedImage; + EFI_DRIVER_BINDING_PROTOCOL *DriverBinding; + CHAR8 *PdbFileName; + EFI_GUID *TempGuid; + UINTN StartIndex; + UINTN Index; + INTN Count; + BOOLEAN ModuleGuidIsGet; + UINTN StringSize; + CHAR16 *StringPtr; + MEDIA_FW_VOL_FILEPATH_DEVICE_PATH *FvFilePath; + + if (NameString =3D=3D NULL || BufferSize =3D=3D 0) { + return EFI_INVALID_PARAMETER; + } =20 - Adds a record to the end of the performance measurement log - that contains the Handle, Token, Module and Identifier. - The end time of the new record must be set to zero. - If TimeStamp is not zero, then TimeStamp is used to fill in the start ti= me in the record. - If TimeStamp is zero, the start time in the record is filled in with the= value - read from the current time stamp. + // + // Try to get the ModuleGuid and name string form the caached array. + // + if (mCachePairCount > 0) { + for (Count =3D mCachePairCount - 1; Count >=3D 0; Count--) { + if (Handle =3D=3D mCacheHandleGuidTable[Count].Handle) { + CopyGuid (ModuleGuid, &mCacheHandleGuidTable[Count].ModuleGuid); + AsciiStrCpyS (NameString, EDKII_STRING_EVENT_RECORD_NAME_LENGTH, m= CacheHandleGuidTable[Count].NameString); + return EFI_SUCCESS; + } + } + } + + Status =3D EFI_INVALID_PARAMETER; + LoadedImage =3D NULL; + ModuleGuidIsGet =3D FALSE; + + // + // Initialize GUID as zero value. + // + TempGuid =3D &gZeroGuid; + // + // Initialize it as "" string. + // + NameString[0] =3D 0; =20 + if (Handle !=3D NULL) { + // + // Try Handle as ImageHandle. + // + Status =3D gBS->HandleProtocol ( + Handle, + &gEfiLoadedImageProtocolGuid, + (VOID**) &LoadedImage + ); + + if (EFI_ERROR (Status)) { + // + // Try Handle as Controller Handle + // + Status =3D gBS->OpenProtocol ( + Handle, + &gEfiDriverBindingProtocolGuid, + (VOID **) &DriverBinding, + NULL, + NULL, + EFI_OPEN_PROTOCOL_GET_PROTOCOL + ); + if (!EFI_ERROR (Status)) { + // + // Get Image protocol from ImageHandle + // + Status =3D gBS->HandleProtocol ( + DriverBinding->ImageHandle, + &gEfiLoadedImageProtocolGuid, + (VOID**) &LoadedImage + ); + } + } + } + + if (!EFI_ERROR (Status) && LoadedImage !=3D NULL) { + // + // Get Module Guid from DevicePath. + // + if (LoadedImage->FilePath !=3D NULL && + LoadedImage->FilePath->Type =3D=3D MEDIA_DEVICE_PATH && + LoadedImage->FilePath->SubType =3D=3D MEDIA_PIWG_FW_FILE_DP + ) { + // + // Determine GUID associated with module logging performance + // + ModuleGuidIsGet =3D TRUE; + FvFilePath =3D (MEDIA_FW_VOL_FILEPATH_DEVICE_PATH *) LoadedImag= e->FilePath; + TempGuid =3D &FvFilePath->FvFileName; + } + + // + // Method 1 Get Module Name from PDB string. + // + PdbFileName =3D PeCoffLoaderGetPdbPointer (LoadedImage->ImageBase); + if (PdbFileName !=3D NULL && BufferSize > 0) { + StartIndex =3D 0; + for (Index =3D 0; PdbFileName[Index] !=3D 0; Index++) { + if ((PdbFileName[Index] =3D=3D '\\') || (PdbFileName[Index] =3D=3D= '/')) { + StartIndex =3D Index + 1; + } + } + // + // Copy the PDB file name to our temporary string. + // If the length is bigger than BufferSize, trim the redudant charac= ters to avoid overflow in array boundary. + // + for (Index =3D 0; Index < BufferSize - 1; Index++) { + NameString[Index] =3D PdbFileName[Index + StartIndex]; + if (NameString[Index] =3D=3D 0 || NameString[Index] =3D=3D '.') { + NameString[Index] =3D 0; + break; + } + } + + if (Index =3D=3D BufferSize - 1) { + NameString[Index] =3D 0; + } + // + // Module Name is got. + // + goto Done; + } + } + + if (ModuleGuidIsGet) { + // + // Method 2 Try to get the image's FFS UI section by image GUID + // + StringPtr =3D NULL; + StringSize =3D 0; + Status =3D GetSectionFromAnyFv ( + TempGuid, + EFI_SECTION_USER_INTERFACE, + 0, + (VOID **) &StringPtr, + &StringSize + ); + + if (!EFI_ERROR (Status)) { + // + // Method 3. Get the name string from FFS UI section + // + for (Index =3D 0; Index < BufferSize - 1 && StringPtr[Index] !=3D 0;= Index++) { + NameString[Index] =3D (CHAR8) StringPtr[Index]; + } + NameString[Index] =3D 0; + FreePool (StringPtr); + } + } + +Done: + // + // Copy Module Guid + // + if (ModuleGuid !=3D NULL) { + CopyGuid (ModuleGuid, TempGuid); + if (IsZeroGuid(TempGuid) && (Handle !=3D NULL) && !ModuleGuidIsGet) { + // Handle is GUID + CopyGuid (ModuleGuid, (EFI_GUID *) Handle); + } + } + + // + // Cache the Handle and Guid pairs. + // + if (mCachePairCount < CACHE_HANDLE_GUID_COUNT) { + mCacheHandleGuidTable[mCachePairCount].Handle =3D Handle; + CopyGuid (&mCacheHandleGuidTable[mCachePairCount].ModuleGuid, ModuleGu= id); + AsciiStrCpyS (mCacheHandleGuidTable[mCachePairCount].NameString, EDKII= _STRING_EVENT_RECORD_NAME_LENGTH, NameString); + mCachePairCount ++; + } + + return Status; +} + +/** + Add performance log to FPDT boot record table. + + @param IsStart TRUE if the performance log is start log. @param Handle Pointer to environment specific context = used to identify the component being measured. @param Token Pointer to a Null-terminated ASCII string that identifies the component being meas= ured. @param Module Pointer to a Null-terminated ASCII string that identifies the module being measure= d. - @param TimeStamp 64-bit time stamp. + @param Ticker 64-bit time stamp. @param Identifier 32-bit identifier. If the value is 0, th= e created record is same as the one created by StartGauge= of PERFORMANCE_PROTOCOL. =20 - @retval EFI_SUCCESS The data was read correctly from the dev= ice. + @retval EFI_SUCCESS Add FPDT boot record. @retval EFI_OUT_OF_RESOURCES There are not enough resources to record= the measurement. + @retval EFI_UNSUPPORTED No matched FPDT record. =20 **/ EFI_STATUS -EFIAPI -StartGaugeEx ( +InsertFpdtMeasurement ( + IN BOOLEAN IsStart, IN CONST VOID *Handle, OPTIONAL IN CONST CHAR8 *Token, OPTIONAL IN CONST CHAR8 *Module, OPTIONAL - IN UINT64 TimeStamp, + IN UINT64 Ticker, IN UINT32 Identifier ) { - GAUGE_DATA_ENTRY_EX *GaugeEntryExArray; - UINTN GaugeDataSize; - GAUGE_DATA_HEADER *NewGaugeData; - UINTN OldGaugeDataSize; - GAUGE_DATA_HEADER *OldGaugeData; - UINT32 Index; + EFI_GUID ModuleGuid; + CHAR8 ModuleName[EDKII_STRING_EVENT_RECORD_NAME_L= ENGTH]; + UINT8 FpdtRecord[MAX_RECORD_SIZE]; + EFI_STATUS Status; + FPDT_RECORD_PTR FpdtRecordPtr; + UINT64 TimeStamp; + BASIC_RECORD_INFO RecordInfo; + UINTN DestMax; + UINTN StrLength; + CONST CHAR8 *StringPtr; + BOOLEAN UseModuleName; + + StringPtr =3D NULL; + UseModuleName =3D FALSE; + ZeroMem (ModuleName, sizeof (ModuleName)); + ZeroMem (FpdtRecord, sizeof (FpdtRecord)); =20 - AcquireSpinLock (&mSmmPerfLock); + // + // Get record info includes type, size, ProgressID. + // + Status =3D GetFpdtRecordInfo (IsStart, Handle, Token, Module, &RecordInf= o, &UseModuleName); + if (EFI_ERROR (Status)) { + return Status; + } + + // + // If PERF_START()/PERF_END() have specified the ProgressID,it has high = priority. + // !!! Note: If the Pref is not the known Token used in the core but hav= e same + // ID with the core Token, this case will not be supported. + // And in currtnt usage mode, for the unkown ID, there is a general rule: + // If it is start pref: the lower 4 bits of the ID should be 0. + // If it is end pref: the lower 4 bits of the ID should not be 0. + // If input ID doesn't follow the rule, we will adjust it. + // + if ((Identifier !=3D 0) && (IsKnownID (Identifier)) && (!IsKnownTokens (= Token))) { + return EFI_UNSUPPORTED; + } else if ((Identifier !=3D 0) && (!IsKnownID (Identifier)) && (!IsKnown= Tokens (Token))) { + if (IsStart && ((Identifier & 0x000F) !=3D 0)) { + Identifier &=3D 0xFFF0; + } else if ((!IsStart) && ((Identifier & 0x000F) =3D=3D 0)) { + Identifier +=3D 1; + } + RecordInfo.ProgressID =3D (UINT16)Identifier; + } =20 - Index =3D mGaugeData->NumberOfEntries; - if (Index >=3D mMaxGaugeRecords) { + if (mFpdtDataIsReported) { // - // Try to enlarge the scale of gauge array. + // Cached FPDT data has been reported. Now, report FPDT record one by = one. // - OldGaugeData =3D mGaugeData; - OldGaugeDataSize =3D sizeof (GAUGE_DATA_HEADER) + sizeof (GAUGE_DATA_= ENTRY_EX) * mMaxGaugeRecords; - - GaugeDataSize =3D sizeof (GAUGE_DATA_HEADER) + sizeof (GAUGE_DATA_= ENTRY_EX) * mMaxGaugeRecords * 2; - - NewGaugeData =3D AllocateZeroPool (GaugeDataSize); - if (NewGaugeData =3D=3D NULL) { - ReleaseSpinLock (&mSmmPerfLock); - return EFI_OUT_OF_RESOURCES; + FpdtRecordPtr.RecordHeader =3D (EFI_ACPI_5_0_FPDT_PERFORMANCE_RECORD_H= EADER *)FpdtRecord; + } else { + if (mStartFpdtDataReport) { + return EFI_UNSUPPORTED; + } + // + // Check if pre-allocated buffer is full + // + if (mPerformanceLength + RecordInfo.RecordSize > mMaxPerformanceLength= ) { + mPerformancePointer =3D ReallocatePool ( + mPerformanceLength, + mPerformanceLength + RecordInfo.RecordSize += FIRMWARE_RECORD_BUFFER, + mPerformancePointer + ); + + if (mPerformancePointer =3D=3D NULL) { + return EFI_OUT_OF_RESOURCES; + } + mMaxPerformanceLength =3D mPerformanceLength + RecordInfo.RecordSize= + FIRMWARE_RECORD_BUFFER; } - - mGaugeData =3D NewGaugeData; - mMaxGaugeRecords *=3D 2; - // - // Initialize new data array and migrate old data one. + // Covert buffer to FPDT Ptr Union type. // - mGaugeData =3D CopyMem (mGaugeData, OldGaugeData, OldGaugeDataSize); + FpdtRecordPtr.RecordHeader =3D (EFI_ACPI_5_0_FPDT_PERFORMANCE_RECORD_H= EADER *)(mPerformancePointer + mPerformanceLength); + } + FpdtRecordPtr.RecordHeader->Length =3D 0; =20 - FreePool (OldGaugeData); + // + // Get the TimeStamp. + // + if (Ticker =3D=3D 0) { + Ticker =3D GetPerformanceCounter (); + TimeStamp =3D GetTimeInNanoSecond (Ticker); + } else if (Ticker =3D=3D 1) { + TimeStamp =3D 0; + } else { + TimeStamp =3D GetTimeInNanoSecond (Ticker); } =20 - GaugeEntryExArray =3D (GAUGE_DATA_ENTRY_EX *) (mGaugeData = + 1); - GaugeEntryExArray[Index].Handle =3D (EFI_PHYSICAL_ADDRESS) (UINTN) Handl= e; + // + // Get the ModuleName and ModuleGuid form the handle. + // + GetModuleInfoFromHandle ((EFI_HANDLE *)Handle, ModuleName, sizeof (Modul= eName), &ModuleGuid); =20 - if (Token !=3D NULL) { - AsciiStrnCpyS (GaugeEntryExArray[Index].Token, SMM_PERFORMANCE_STRING_= SIZE, Token, SMM_PERFORMANCE_STRING_LENGTH); + // + // Fill in the record information. + // + switch (RecordInfo.Type) { + case GUID_EVENT_TYPE: + FpdtRecordPtr.GuidEvent->Header.Type =3D GUID_EVENT_TYP= E; + FpdtRecordPtr.GuidEvent->Header.Length =3D RecordInfo.Rec= ordSize; + FpdtRecordPtr.GuidEvent->Header.Revision =3D RECORD_REVISIO= N_1; + FpdtRecordPtr.GuidEvent->ProgressID =3D RecordInfo.Pro= gressID; + FpdtRecordPtr.GuidEvent->Timestamp =3D TimeStamp; + CopyMem (&FpdtRecordPtr.GuidEvent->Guid, &ModuleGuid, sizeof (FpdtReco= rdPtr.GuidEvent->Guid)); + break; + + case DYNAMIC_STRING_EVENT_TYPE: + FpdtRecordPtr.DynamicStringEvent->Header.Type =3D DYNAMIC_STRING= _EVENT_TYPE; + FpdtRecordPtr.DynamicStringEvent->Header.Length =3D RecordInfo.Rec= ordSize; + FpdtRecordPtr.DynamicStringEvent->Header.Revision =3D RECORD_REVISIO= N_1; + FpdtRecordPtr.DynamicStringEvent->ProgressID =3D RecordInfo.Pro= gressID; + FpdtRecordPtr.DynamicStringEvent->Timestamp =3D TimeStamp; + CopyMem (&FpdtRecordPtr.DynamicStringEvent->Guid, &ModuleGuid, sizeof = (FpdtRecordPtr.DynamicStringEvent->Guid)); + + if (UseModuleName) { + StringPtr =3D ModuleName; + } else if (Token !=3D NULL) { + StringPtr =3D Token; + } else if (Module !=3D NULL) { + StringPtr =3D Module; + } else if (ModuleName !=3D NULL) { + StringPtr =3D ModuleName; + } + if (StringPtr !=3D NULL && AsciiStrLen (StringPtr) !=3D 0) { + StrLength =3D AsciiStrLen (StringPtr); + DestMax =3D (RecordInfo.RecordSize - sizeof (DYNAMIC_STRING_EV= ENT_RECORD)) / sizeof (CHAR8); + if (StrLength >=3D DestMax) { + StrLength =3D DestMax -1; + } + AsciiStrnCpyS (FpdtRecordPtr.DynamicStringEvent->String, DestMax, St= ringPtr, StrLength); + } + break; + + case GUID_QWORD_EVENT_TYPE: + FpdtRecordPtr.GuidQwordEvent->Header.Type =3D GUID_QWORD_EVE= NT_TYPE; + FpdtRecordPtr.GuidQwordEvent->Header.Length =3D RecordInfo.Rec= ordSize; + FpdtRecordPtr.GuidQwordEvent->Header.Revision =3D RECORD_REVISIO= N_1; + FpdtRecordPtr.GuidQwordEvent->ProgressID =3D RecordInfo.Pro= gressID; + FpdtRecordPtr.GuidQwordEvent->Timestamp =3D TimeStamp; + CopyMem (&FpdtRecordPtr.GuidQwordEvent->Guid, &ModuleGuid, sizeof (Fpd= tRecordPtr.GuidQwordEvent->Guid)); + break; + + default: + // + // Record is not supported in current SMM phase, return EFI_UNSUPPORTED + // + return EFI_UNSUPPORTED; } - if (Module !=3D NULL) { - AsciiStrnCpyS (GaugeEntryExArray[Index].Module, SMM_PERFORMANCE_STRING= _SIZE, Module, SMM_PERFORMANCE_STRING_LENGTH); + // + // Report record one by one after records have been reported together. + // + if (mFpdtDataIsReported) { + REPORT_STATUS_CODE_EX ( + EFI_PROGRESS_CODE, + EFI_SOFTWARE_SMM_DRIVER, + 0, + NULL, + &gEfiFirmwarePerformanceGuid, + FpdtRecordPtr.RecordHeader, + FpdtRecordPtr.RecordHeader->Length + ); + } else { + // + // Update the cached FPDT record buffer. + // + mPerformanceLength +=3D FpdtRecordPtr.RecordHeader->Length; } =20 - GaugeEntryExArray[Index].EndTimeStamp =3D 0; - GaugeEntryExArray[Index].Identifier =3D Identifier; + return EFI_SUCCESS; +} =20 - if (TimeStamp =3D=3D 0) { - TimeStamp =3D GetPerformanceCounter (); - } - GaugeEntryExArray[Index].StartTimeStamp =3D TimeStamp; +/** + Adds a record at the end of the performance measurement log + that records the start time of a performance measurement. =20 - mGaugeData->NumberOfEntries++; + Adds a record to the end of the performance measurement log + that contains the Handle, Token, Module and Identifier. + The end time of the new record must be set to zero. + If TimeStamp is not zero, then TimeStamp is used to fill in the start ti= me in the record. + If TimeStamp is zero, the start time in the record is filled in with the= value + read from the current time stamp. =20 - ReleaseSpinLock (&mSmmPerfLock); + @param Handle Pointer to environment specific context = used + to identify the component being measured. + @param Token Pointer to a Null-terminated ASCII string + that identifies the component being meas= ured. + @param Module Pointer to a Null-terminated ASCII string + that identifies the module being measure= d. + @param TimeStamp 64-bit time stamp. + @param Identifier 32-bit identifier. If the value is 0, th= e created record + is same as the one created by StartGauge= of PERFORMANCE_PROTOCOL. =20 - return EFI_SUCCESS; + @retval EFI_SUCCESS The data was read correctly from the dev= ice. + @retval EFI_OUT_OF_RESOURCES There are not enough resources to record= the measurement. + +**/ +EFI_STATUS +EFIAPI +StartGaugeEx ( + IN CONST VOID *Handle, OPTIONAL + IN CONST CHAR8 *Token, OPTIONAL + IN CONST CHAR8 *Module, OPTIONAL + IN UINT64 TimeStamp, + IN UINT32 Identifier + ) +{ + EFI_STATUS Status; + + AcquireSpinLock (&mSmmFpdtLock); + + Status =3D InsertFpdtMeasurement (TRUE, Handle, Token, Module, TimeStamp= , Identifier); + + ReleaseSpinLock (&mSmmFpdtLock); + + return Status; } =20 /** Searches the performance measurement log from the beginning of the log for the first matching record that contains a zero end time and fills in= a valid end time. =20 Searches the performance measurement log from the beginning of the log - for the first record that matches Handle, Token and Module and has an en= d time value of zero. + for the first record that matches Handle, Token, Module and Identifier a= nd has an end time value of zero. If the record can not be found then return EFI_NOT_FOUND. If the record is found and TimeStamp is not zero, then the end time in the record is filled in with the value specified by= TimeStamp. If the record is found and TimeStamp is zero, then the end time in the m= atching record is filled in with the current time stamp value. @@ -259,37 +723,28 @@ EndGaugeEx ( IN CONST CHAR8 *Module, OPTIONAL IN UINT64 TimeStamp, IN UINT32 Identifier ) { - GAUGE_DATA_ENTRY_EX *GaugeEntryExArray; - UINT32 Index; + EFI_STATUS Status; =20 - AcquireSpinLock (&mSmmPerfLock); + AcquireSpinLock (&mSmmFpdtLock); =20 - if (TimeStamp =3D=3D 0) { - TimeStamp =3D GetPerformanceCounter (); - } - - Index =3D SmmSearchForGaugeEntry (Handle, Token, Module, Identifier); - if (Index >=3D mGaugeData->NumberOfEntries) { - ReleaseSpinLock (&mSmmPerfLock); - return EFI_NOT_FOUND; - } - GaugeEntryExArray =3D (GAUGE_DATA_ENTRY_EX *) (mGaugeData + 1); - GaugeEntryExArray[Index].EndTimeStamp =3D TimeStamp; + Status =3D InsertFpdtMeasurement (FALSE, Handle, Token, Module, TimeStam= p, Identifier); =20 - ReleaseSpinLock (&mSmmPerfLock); + ReleaseSpinLock (&mSmmFpdtLock); =20 - return EFI_SUCCESS; + return Status; } =20 /** Retrieves a previously logged performance measurement. It can also retrieve the log created by StartGauge and EndGauge of PERFO= RMANCE_PROTOCOL, and then assign the Identifier with 0. =20 + !!! Not Support!!! + Retrieves the performance log entry from the performance log specified b= y LogEntryKey. If it stands for a valid entry, then EFI_SUCCESS is returned and GaugeDataEntryEx stores the pointer to that entry. =20 @param LogEntryKey The key for the previous performance mea= surement log entry. @@ -308,29 +763,11 @@ EFIAPI GetGaugeEx ( IN UINTN LogEntryKey, OUT GAUGE_DATA_ENTRY_EX **GaugeDataEntryEx ) { - UINTN NumberOfEntries; - GAUGE_DATA_ENTRY_EX *GaugeEntryExArray; - - NumberOfEntries =3D (UINTN) (mGaugeData->NumberOfEntries); - if (LogEntryKey > NumberOfEntries) { - return EFI_INVALID_PARAMETER; - } - if (LogEntryKey =3D=3D NumberOfEntries) { - return EFI_NOT_FOUND; - } - - GaugeEntryExArray =3D (GAUGE_DATA_ENTRY_EX *) (mGaugeData + 1); - - if (GaugeDataEntryEx =3D=3D NULL) { - return EFI_INVALID_PARAMETER; - } - *GaugeDataEntryEx =3D &GaugeEntryExArray[LogEntryKey]; - - return EFI_SUCCESS; + return EFI_UNSUPPORTED; } =20 /** Adds a record at the end of the performance measurement log that records the start time of a performance measurement. @@ -405,10 +842,12 @@ EndGauge ( /** Retrieves a previously logged performance measurement. It can also retrieve the log created by StartGaugeEx and EndGaugeEx of P= ERFORMANCE_EX_PROTOCOL, and then eliminate the Identifier. =20 + !!! Not Support!!! + Retrieves the performance log entry from the performance log specified b= y LogEntryKey. If it stands for a valid entry, then EFI_SUCCESS is returned and GaugeDataEntry stores the pointer to that entry. =20 @param LogEntryKey The key for the previous performance mea= surement log entry. @@ -427,256 +866,90 @@ EFIAPI GetGauge ( IN UINTN LogEntryKey, OUT GAUGE_DATA_ENTRY **GaugeDataEntry ) { - EFI_STATUS Status; - GAUGE_DATA_ENTRY_EX *GaugeEntryEx; + return EFI_UNSUPPORTED; +} =20 - GaugeEntryEx =3D NULL; =20 - Status =3D GetGaugeEx (LogEntryKey, &GaugeEntryEx); - if (EFI_ERROR (Status)) { - return Status; - } +/** + SMM END_OF_DXE protocol notification event handler. =20 - if (GaugeDataEntry =3D=3D NULL) { - return EFI_INVALID_PARAMETER; - } + @param Protocol Points to the protocol's unique identifier + @param Interface Points to the interface instance + @param Handle The handle on which the interface was installed =20 - *GaugeDataEntry =3D (GAUGE_DATA_ENTRY *) GaugeEntryEx; + @retval EFI_SUCCESS SmmEndOfDxeCallback runs successfully =20 - return EFI_SUCCESS; -} - -/** - Communication service SMI Handler entry. - - This SMI handler provides services for the performance wrapper driver. - =20 - Caution: This function may receive untrusted input. - Communicate buffer and buffer size are external input, so this function= will do basic validation. - - @param[in] DispatchHandle The unique handle assigned to this handle= r by SmiHandlerRegister(). - @param[in] RegisterContext Points to an optional handler context whi= ch was specified when the - handler was registered. - @param[in, out] CommBuffer A pointer to a collection of data in memo= ry that will - be conveyed from a non-SMM environment in= to an SMM environment. - @param[in, out] CommBufferSize The size of the CommBuffer. - - @retval EFI_SUCCESS The interrupt was handled an= d quiesced. No other handlers=20 - should still be called. - @retval EFI_WARN_INTERRUPT_SOURCE_QUIESCED The interrupt has been quies= ced but other handlers should=20 - still be called. - @retval EFI_WARN_INTERRUPT_SOURCE_PENDING The interrupt is still pendi= ng and other handlers should still=20 - be called. - @retval EFI_INTERRUPT_PENDING The interrupt could not be q= uiesced. **/ EFI_STATUS EFIAPI -SmmPerformanceHandlerEx ( - IN EFI_HANDLE DispatchHandle, - IN CONST VOID *RegisterContext, - IN OUT VOID *CommBuffer, - IN OUT UINTN *CommBufferSize +SmmReportFpdtRecordData ( + IN CONST EFI_GUID *Protocol, + IN VOID *Interface, + IN EFI_HANDLE Handle ) { - EFI_STATUS Status; - SMM_PERF_COMMUNICATE_EX *SmmPerfCommData; - GAUGE_DATA_ENTRY_EX *GaugeEntryExArray; - UINT64 DataSize; - UINTN Index; - GAUGE_DATA_ENTRY_EX *GaugeDataEx; - UINTN NumberOfEntries; - UINTN LogEntryKey; - UINTN TempCommBufferSize; + UINT8 *PerfBuffer; + UINTN MaxUint16; =20 - GaugeEntryExArray =3D NULL; + PerfBuffer =3D mPerformancePointer; + MaxUint16 =3D 0xFFFF; =20 - // - // If input is invalid, stop processing this SMI - // - if (CommBuffer =3D=3D NULL || CommBufferSize =3D=3D NULL) { + if ( mPerformanceLength=3D=3D 0 || mPerformancePointer =3D=3D NULL) { return EFI_SUCCESS; } =20 - TempCommBufferSize =3D *CommBufferSize; + mStartFpdtDataReport =3D TRUE; =20 - if(TempCommBufferSize < sizeof (SMM_PERF_COMMUNICATE_EX)) { - return EFI_SUCCESS; - } - - if (!SmmIsBufferOutsideSmmValid ((UINTN)CommBuffer, TempCommBufferSize))= { - DEBUG ((EFI_D_ERROR, "SmmPerformanceHandlerEx: SMM communcation data b= uffer in SMRAM or overflow!\n")); - return EFI_SUCCESS; - } - =20 - SmmPerfCommData =3D (SMM_PERF_COMMUNICATE_EX *)CommBuffer; - - switch (SmmPerfCommData->Function) { - case SMM_PERF_FUNCTION_GET_GAUGE_ENTRY_NUMBER : - SmmPerfCommData->NumberOfEntries =3D mGaugeData->NumberOfEntries; - Status =3D EFI_SUCCESS; - break; - - case SMM_PERF_FUNCTION_GET_GAUGE_DATA : - GaugeDataEx =3D SmmPerfCommData->GaugeDataEx; - NumberOfEntries =3D SmmPerfCommData->NumberOfEntries; - LogEntryKey =3D SmmPerfCommData->LogEntryKey; - if (GaugeDataEx =3D=3D NULL || NumberOfEntries =3D=3D 0 || LogEntry= Key > mGaugeData->NumberOfEntries || - NumberOfEntries > mGaugeData->NumberOfEntries || LogEntryKey > = (mGaugeData->NumberOfEntries - NumberOfEntries)) { - Status =3D EFI_INVALID_PARAMETER; - break; - } - - // - // Sanity check - // - DataSize =3D MultU64x32 (NumberOfEntries, sizeof(GAUGE_DATA_ENTRY_E= X)); - if (!SmmIsBufferOutsideSmmValid ((UINTN) GaugeDataEx, DataSize)) { - DEBUG ((EFI_D_ERROR, "SmmPerformanceHandlerEx: SMM Performance Da= ta buffer in SMRAM or overflow!\n")); - Status =3D EFI_ACCESS_DENIED; - break; - } - - GaugeEntryExArray =3D (GAUGE_DATA_ENTRY_EX *) (mGaugeData + 1); - - for (Index =3D 0; Index < NumberOfEntries; Index++) { - CopyMem ( - (UINT8 *) &GaugeDataEx[Index], - (UINT8 *) &GaugeEntryExArray[LogEntryKey++], - sizeof (GAUGE_DATA_ENTRY_EX) - ); - } - Status =3D EFI_SUCCESS; - break; - - default: - Status =3D EFI_UNSUPPORTED; + while (mPerformanceLength > MaxUint16) { + // + // Report extension data size is UINT16. So, the size of report data c= an't exceed 0xFFFF. + // + REPORT_STATUS_CODE_EX ( + EFI_PROGRESS_CODE, + EFI_SOFTWARE_SMM_DRIVER, + 0, + NULL, + &gEfiFirmwarePerformanceGuid, + PerfBuffer, + MaxUint16 + ); + mPerformanceLength =3D mPerformanceLength - MaxUint16; + PerfBuffer =3D PerfBuffer + MaxUint16; } =20 - - SmmPerfCommData->ReturnStatus =3D Status; - =20 - return EFI_SUCCESS; -} - -/** - Communication service SMI Handler entry. - - This SMI handler provides services for the performance wrapper driver. - - Caution: This function may receive untrusted input. - Communicate buffer and buffer size are external input, so this function = will do basic validation. - - @param[in] DispatchHandle The unique handle assigned to this handle= r by SmiHandlerRegister(). - @param[in] RegisterContext Points to an optional handler context whi= ch was specified when the - handler was registered. - @param[in, out] CommBuffer A pointer to a collection of data in memo= ry that will - be conveyed from a non-SMM environment in= to an SMM environment. - @param[in, out] CommBufferSize The size of the CommBuffer. - - @retval EFI_SUCCESS The interrupt was handled an= d quiesced. No other handlers=20 - should still be called. - @retval EFI_WARN_INTERRUPT_SOURCE_QUIESCED The interrupt has been quies= ced but other handlers should=20 - still be called. - @retval EFI_WARN_INTERRUPT_SOURCE_PENDING The interrupt is still pendi= ng and other handlers should still=20 - be called. - @retval EFI_INTERRUPT_PENDING The interrupt could not be q= uiesced. -**/ -EFI_STATUS -EFIAPI -SmmPerformanceHandler ( - IN EFI_HANDLE DispatchHandle, - IN CONST VOID *RegisterContext, - IN OUT VOID *CommBuffer, - IN OUT UINTN *CommBufferSize - ) -{ - EFI_STATUS Status; - SMM_PERF_COMMUNICATE *SmmPerfCommData; - GAUGE_DATA_ENTRY_EX *GaugeEntryExArray; - UINT64 DataSize; - UINTN Index; - GAUGE_DATA_ENTRY *GaugeData; - UINTN NumberOfEntries; - UINTN LogEntryKey; - UINTN TempCommBufferSize; - - GaugeEntryExArray =3D NULL; + REPORT_STATUS_CODE_EX ( + EFI_PROGRESS_CODE, + EFI_SOFTWARE_SMM_DRIVER, + 0, + NULL, + &gEfiFirmwarePerformanceGuid, + PerfBuffer, + mPerformanceLength + ); =20 // - // If input is invalid, stop processing this SMI + // Free Cached FPDT record Buffer // - if (CommBuffer =3D=3D NULL || CommBufferSize =3D=3D NULL) { - return EFI_SUCCESS; - } - - TempCommBufferSize =3D *CommBufferSize; - - if(TempCommBufferSize < sizeof (SMM_PERF_COMMUNICATE)) { - return EFI_SUCCESS; - } - - if (!SmmIsBufferOutsideSmmValid ((UINTN)CommBuffer, TempCommBufferSize))= { - DEBUG ((EFI_D_ERROR, "SmmPerformanceHandler: SMM communcation data buf= fer in SMRAM or overflow!\n")); - return EFI_SUCCESS; - } - - SmmPerfCommData =3D (SMM_PERF_COMMUNICATE *)CommBuffer; - - switch (SmmPerfCommData->Function) { - case SMM_PERF_FUNCTION_GET_GAUGE_ENTRY_NUMBER : - SmmPerfCommData->NumberOfEntries =3D mGaugeData->NumberOfEntries; - Status =3D EFI_SUCCESS; - break; - - case SMM_PERF_FUNCTION_GET_GAUGE_DATA : - GaugeData =3D SmmPerfCommData->GaugeData; - NumberOfEntries =3D SmmPerfCommData->NumberOfEntries; - LogEntryKey =3D SmmPerfCommData->LogEntryKey; - if (GaugeData =3D=3D NULL || NumberOfEntries =3D=3D 0 || LogEntryKe= y > mGaugeData->NumberOfEntries || - NumberOfEntries > mGaugeData->NumberOfEntries || LogEntryKey > = (mGaugeData->NumberOfEntries - NumberOfEntries)) { - Status =3D EFI_INVALID_PARAMETER; - break; - } - - // - // Sanity check - // - DataSize =3D MultU64x32 (NumberOfEntries, sizeof(GAUGE_DATA_ENTRY)); - if (!SmmIsBufferOutsideSmmValid ((UINTN) GaugeData, DataSize)) { - DEBUG ((EFI_D_ERROR, "SmmPerformanceHandler: SMM Performance Data= buffer in SMRAM or overflow!\n")); - Status =3D EFI_ACCESS_DENIED; - break; - } - - GaugeEntryExArray =3D (GAUGE_DATA_ENTRY_EX *) (mGaugeData + 1); - - for (Index =3D 0; Index < NumberOfEntries; Index++) { - CopyMem ( - (UINT8 *) &GaugeData[Index], - (UINT8 *) &GaugeEntryExArray[LogEntryKey++], - sizeof (GAUGE_DATA_ENTRY) - ); - } - Status =3D EFI_SUCCESS; - break; - - default: - Status =3D EFI_UNSUPPORTED; - } - + FreePool (mPerformancePointer); + mPerformancePointer =3D NULL; + mPerformanceLength =3D 0; + mMaxPerformanceLength =3D 0; + // + // Set FPDT report state to TRUE. + // + mFpdtDataIsReported =3D TRUE; + mStartFpdtDataReport =3D FALSE; =20 - SmmPerfCommData->ReturnStatus =3D Status; - =20 return EFI_SUCCESS; } =20 /** - SmmBase2 protocol notify callback function, when SMST and SMM memory ser= vice get initialized=20 - this function is callbacked to initialize the Smm Performance Lib=20 + SmmBase2 protocol notify callback function, when SMST and SMM memory ser= vice get initialized + this function is callbacked to initialize the Smm Performance Lib =20 @param Event The event of notify protocol. @param Context Notify event context. =20 **/ @@ -685,52 +958,44 @@ EFIAPI InitializeSmmCorePerformanceLib ( IN EFI_EVENT Event, IN VOID *Context ) { - EFI_STATUS Status; EFI_HANDLE Handle; + EFI_STATUS Status; + VOID *SmmReadyToBootRegistration; PERFORMANCE_PROPERTY *PerformanceProperty; =20 // // Initialize spin lock // - InitializeSpinLock (&mSmmPerfLock); - - mMaxGaugeRecords =3D INIT_SMM_GAUGE_DATA_ENTRIES; + InitializeSpinLock (&mSmmFpdtLock); =20 - mGaugeData =3D AllocateZeroPool (sizeof (GAUGE_DATA_HEADER) + (sizeof (G= AUGE_DATA_ENTRY_EX) * mMaxGaugeRecords)); - ASSERT (mGaugeData !=3D NULL); - =20 // - // Install the protocol interfaces. + // Install the protocol interfaces for SMM performance library instance. // + Handle =3D NULL; Status =3D gSmst->SmmInstallProtocolInterface ( - &mHandle, + &Handle, &gSmmPerformanceProtocolGuid, EFI_NATIVE_INTERFACE, &mPerformanceInterface ); ASSERT_EFI_ERROR (Status); - Status =3D gSmst->SmmInstallProtocolInterface ( - &mHandle, + &Handle, &gSmmPerformanceExProtocolGuid, EFI_NATIVE_INTERFACE, &mPerformanceExInterface ); ASSERT_EFI_ERROR (Status); =20 - /// - /// Register SMM Performance SMI handler - /// - Handle =3D NULL; - Status =3D gSmst->SmiHandlerRegister (SmmPerformanceHandler, &gSmmPerfor= manceProtocolGuid, &Handle); - ASSERT_EFI_ERROR (Status); - Status =3D gSmst->SmiHandlerRegister (SmmPerformanceHandlerEx, &gSmmPerf= ormanceExProtocolGuid, &Handle); - ASSERT_EFI_ERROR (Status); - + Status =3D gSmst->SmmRegisterProtocolNotify ( + &gEdkiiSmmReadyToBootProtocolGuid, + SmmReportFpdtRecordData, + &SmmReadyToBootRegistration + ); Status =3D EfiGetSystemConfigurationTable (&gPerformanceProtocolGuid, (V= OID **) &PerformanceProperty); if (EFI_ERROR (Status)) { // // Install configuration table for performance property. // @@ -744,11 +1009,11 @@ InitializeSmmCorePerformanceLib ( ASSERT_EFI_ERROR (Status); } } =20 /** - The constructor function initializes the Performance Measurement Enable = flag and=20 + The constructor function initializes the Performance Measurement Enable = flag and registers SmmBase2 protocol notify callback. It will ASSERT() if one of these operations fails and it will always ret= urn EFI_SUCCESS. =20 @param ImageHandle The firmware allocated handle for the EFI image. @param SystemTable A pointer to the EFI System Table. @@ -765,12 +1030,11 @@ SmmCorePerformanceLibConstructor ( { EFI_STATUS Status; EFI_EVENT Event; VOID *Registration; =20 - mPerformanceMeasurementEnabled =3D (BOOLEAN) ((PcdGet8(PcdPerformanceLi= braryPropertyMask) & PERFORMANCE_LIBRARY_PROPERTY_MEASUREMENT_ENABLED) !=3D= 0); - if (!mPerformanceMeasurementEnabled) { + if (!PerformanceMeasurementEnabled ()) { // // Do not initialize performance infrastructure if not required. // return EFI_SUCCESS; } @@ -842,11 +1106,11 @@ StartPerformanceMeasurementEx ( /** Searches the performance measurement log from the beginning of the log for the first matching record that contains a zero end time and fills in= a valid end time. =20 Searches the performance measurement log from the beginning of the log - for the first record that matches Handle, Token and Module and has an en= d time value of zero. + for the first record that matches Handle, Token, Module and Identifier a= nd has an end time value of zero. If the record can not be found then return RETURN_NOT_FOUND. If the record is found and TimeStamp is not zero, then the end time in the record is filled in with the value specified by= TimeStamp. If the record is found and TimeStamp is zero, then the end time in the m= atching record is filled in with the current time stamp value. @@ -881,10 +1145,12 @@ EndPerformanceMeasurementEx ( /** Attempts to retrieve a performance measurement log entry from the perfor= mance measurement log. It can also retrieve the log created by StartPerformanceMeasurement and = EndPerformanceMeasurement, and then assign the Identifier with 0. =20 + !!! Not Support!!! + Attempts to retrieve the performance log entry specified by LogEntryKey.= If LogEntryKey is zero on entry, then an attempt is made to retrieve the first entry from = the performance log, and the key for the second entry in the log is returned. If the perform= ance log is empty, then no entry is retrieved and zero is returned. If LogEntryKey is not = zero, then the performance log entry associated with LogEntryKey is retrieved, and the key for the = next entry in the log is @@ -920,55 +1186,20 @@ EndPerformanceMeasurementEx ( =20 **/ UINTN EFIAPI GetPerformanceMeasurementEx ( - IN UINTN LogEntryKey,=20 + IN UINTN LogEntryKey, OUT CONST VOID **Handle, OUT CONST CHAR8 **Token, OUT CONST CHAR8 **Module, OUT UINT64 *StartTimeStamp, OUT UINT64 *EndTimeStamp, OUT UINT32 *Identifier ) { - EFI_STATUS Status; - GAUGE_DATA_ENTRY_EX *GaugeData; - - GaugeData =3D NULL; - =20 - ASSERT (Handle !=3D NULL); - ASSERT (Token !=3D NULL); - ASSERT (Module !=3D NULL); - ASSERT (StartTimeStamp !=3D NULL); - ASSERT (EndTimeStamp !=3D NULL); - ASSERT (Identifier !=3D NULL); - - Status =3D GetGaugeEx (LogEntryKey++, &GaugeData); - - // - // Make sure that LogEntryKey is a valid log entry key, - // - ASSERT (Status !=3D EFI_INVALID_PARAMETER); - - if (EFI_ERROR (Status)) { - // - // The LogEntryKey is the last entry (equals to the total entry number= ). - // - return 0; - } - - ASSERT (GaugeData !=3D NULL); - - *Handle =3D (VOID *) (UINTN) GaugeData->Handle; - *Token =3D GaugeData->Token; - *Module =3D GaugeData->Module; - *StartTimeStamp =3D GaugeData->StartTimeStamp; - *EndTimeStamp =3D GaugeData->EndTimeStamp; - *Identifier =3D GaugeData->Identifier; - - return LogEntryKey; + return 0; } =20 /** Adds a record at the end of the performance measurement log that records the start time of a performance measurement. @@ -999,11 +1230,11 @@ StartPerformanceMeasurement ( IN CONST CHAR8 *Token, OPTIONAL IN CONST CHAR8 *Module, OPTIONAL IN UINT64 TimeStamp ) { - return StartPerformanceMeasurementEx (Handle, Token, Module, TimeStamp, = 0); + return StartGaugeEx (Handle, Token, Module, TimeStamp, 0); } =20 /** Searches the performance measurement log from the beginning of the log for the first matching record that contains a zero end time and fills in= a valid end time. @@ -1035,18 +1266,20 @@ EndPerformanceMeasurement ( IN CONST CHAR8 *Token, OPTIONAL IN CONST CHAR8 *Module, OPTIONAL IN UINT64 TimeStamp ) { - return EndPerformanceMeasurementEx (Handle, Token, Module, TimeStamp, 0); + return EndGaugeEx (Handle, Token, Module, TimeStamp, 0); } =20 /** Attempts to retrieve a performance measurement log entry from the perfor= mance measurement log. It can also retrieve the log created by StartPerformanceMeasurementEx an= d EndPerformanceMeasurementEx, and then eliminate the Identifier. =20 + !!! Not Support!!! + Attempts to retrieve the performance log entry specified by LogEntryKey.= If LogEntryKey is zero on entry, then an attempt is made to retrieve the first entry from = the performance log, and the key for the second entry in the log is returned. If the perform= ance log is empty, then no entry is retrieved and zero is returned. If LogEntryKey is not = zero, then the performance log entry associated with LogEntryKey is retrieved, and the key for the = next entry in the log is @@ -1088,12 +1321,11 @@ GetPerformanceMeasurement ( OUT CONST CHAR8 **Module, OUT UINT64 *StartTimeStamp, OUT UINT64 *EndTimeStamp ) { - UINT32 Identifier; - return GetPerformanceMeasurementEx (LogEntryKey, Handle, Token, Module, = StartTimeStamp, EndTimeStamp, &Identifier); + return 0; } =20 /** Returns TRUE if the performance measurement macros are enabled. =20 @@ -1110,7 +1342,7 @@ BOOLEAN EFIAPI PerformanceMeasurementEnabled ( VOID ) { - return mPerformanceMeasurementEnabled; + return (BOOLEAN) ((PcdGet8(PcdPerformanceLibraryPropertyMask) & PERFORMA= NCE_LIBRARY_PROPERTY_MEASUREMENT_ENABLED) !=3D 0); } diff --git a/MdeModulePkg/Library/SmmCorePerformanceLib/SmmCorePerformanceL= ib.inf b/MdeModulePkg/Library/SmmCorePerformanceLib/SmmCorePerformanceLib.i= nf index 1b2fbd3..1d1841c 100644 --- a/MdeModulePkg/Library/SmmCorePerformanceLib/SmmCorePerformanceLib.inf +++ b/MdeModulePkg/Library/SmmCorePerformanceLib/SmmCorePerformanceLib.inf @@ -6,11 +6,11 @@ # It initializes SMM phase performance logging by publishing the SMM Perf= ormance and PerformanceEx Protocol, # which is consumed by SmmPerformanceLib to logging performance data in S= MM phase. # This library is mainly used by SMM Core to start performance logging to= ensure that # SMM Performance and PerformanceEx Protocol are installed at the very be= ginning of SMM phase. # =20 -# Copyright (c) 2011 - 2017, Intel Corporation. All rights reserved.
+# Copyright (c) 2011 - 2018, 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 # which accompanies this distribution. The full text of the license may = be found at # http://opensource.org/licenses/bsd-license.php # =20 @@ -56,21 +56,27 @@ DebugLib SynchronizationLib SmmServicesTableLib SmmMemLib UefiLib + ReportStatusCodeLib + PeCoffGetEntryPointLib =20 [Protocols] gEfiSmmBase2ProtocolGuid ## CONSUMES + gEdkiiSmmReadyToBootProtocolGuid ## NOTIFY =20 [Guids] ## PRODUCES ## UNDEFINED # Install protocol ## CONSUMES ## UNDEFINED # SmiHandlerRegister gSmmPerformanceProtocolGuid ## PRODUCES ## UNDEFINED # Install protocol ## CONSUMES ## UNDEFINED # SmiHandlerRegister gSmmPerformanceExProtocolGuid ## PRODUCES ## SystemTable gPerformanceProtocolGuid + gEfiFirmwarePerformanceGuid ## SOMETIMES_PRODUCES ## UNDEF= INED # StatusCode Data + gZeroGuid ## SOMETIMES_CONSUMES ## GUID =20 [Pcd] - gEfiMdePkgTokenSpaceGuid.PcdPerformanceLibraryPropertyMask ## CONSUMES + gEfiMdePkgTokenSpaceGuid.PcdPerformanceLibraryPropertyMask ## CON= SUMES + gEfiMdeModulePkgTokenSpaceGuid.PcdEdkiiFpdtStringRecordEnableOnly ## CON= SUMES diff --git a/MdeModulePkg/Library/SmmCorePerformanceLib/SmmCorePerformanceL= ibInternal.h b/MdeModulePkg/Library/SmmCorePerformanceLib/SmmCorePerformanc= eLibInternal.h index 8eb3032..9d24d9f 100644 --- a/MdeModulePkg/Library/SmmCorePerformanceLib/SmmCorePerformanceLibInter= nal.h +++ b/MdeModulePkg/Library/SmmCorePerformanceLib/SmmCorePerformanceLibInter= nal.h @@ -2,11 +2,11 @@ Master header files for SmmCorePerformanceLib instance. =20 This header file holds the prototypes of the SMM Performance and Perform= anceEx Protocol published by this library instance at its constructor. =20 -Copyright (c) 2011 - 2015, Intel Corporation. All rights reserved.
+Copyright (c) 2011 - 2018, 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 which accompanies this distribution. The full text of the license may be = found at http://opensource.org/licenses/bsd-license.php =20 @@ -18,10 +18,14 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITH= ER EXPRESS OR IMPLIED. #ifndef _SMM_CORE_PERFORMANCE_LIB_INTERNAL_H_ #define _SMM_CORE_PERFORMANCE_LIB_INTERNAL_H_ =20 =20 #include +#include +#include +#include +#include =20 #include #include #include #include @@ -31,12 +35,17 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITH= ER EXPRESS OR IMPLIED. #include =20 #include #include #include #include +#include +#include +#include =20 #include +#include +#include =20 // // Interface declarations for SMM PerformanceEx Protocol. // /** --=20 1.9.5.msysgit.1 _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel From nobody Fri Nov 1 12:22:50 2024 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.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 1516679297530263.175676450275; Mon, 22 Jan 2018 19:48:17 -0800 (PST) Received: from [127.0.0.1] (localhost [IPv6:::1]) by ml01.01.org (Postfix) with ESMTP id 45EF52034A8A2; Mon, 22 Jan 2018 19:42:49 -0800 (PST) Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) (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 4E0D122333762 for ; Mon, 22 Jan 2018 19:42:48 -0800 (PST) Received: from orsmga008.jf.intel.com ([10.7.209.65]) by fmsmga101.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 22 Jan 2018 19:48:14 -0800 Received: from shwdeopenpsi114.ccr.corp.intel.com ([10.239.157.135]) by orsmga008.jf.intel.com with ESMTP; 22 Jan 2018 19:48:13 -0800 X-Original-To: edk2-devel@lists.01.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; Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=192.55.52.88; helo=mga01.intel.com; envelope-from=dandan.bi@intel.com; receiver=edk2-devel@lists.01.org X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.46,399,1511856000"; d="scan'208";a="12262547" From: Dandan Bi To: edk2-devel@lists.01.org Date: Tue, 23 Jan 2018 11:47:32 +0800 Message-Id: <1516679255-12328-6-git-send-email-dandan.bi@intel.com> X-Mailer: git-send-email 1.9.5.msysgit.1 In-Reply-To: <1516679255-12328-1-git-send-email-dandan.bi@intel.com> References: <1516679255-12328-1-git-send-email-dandan.bi@intel.com> Subject: [edk2] [patch 5/8] MdeModulePkg/FirmwarePerformancePei:Add FPDT records for S3 phase X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Star Zeng , Liming Gao 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" Add FPDT records into boot performance table for S3 phase Cc: Liming Gao Cc: Star Zeng Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Dandan Bi --- .../FirmwarePerformancePei.c | 59 ++++++++++++++++++= +++- .../FirmwarePerformancePei.inf | 5 +- 2 files changed, 62 insertions(+), 2 deletions(-) diff --git a/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTablePei/Fi= rmwarePerformancePei.c b/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDat= aTablePei/FirmwarePerformancePei.c index e4800b7..e81f96c 100644 --- a/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTablePei/FirmwareP= erformancePei.c +++ b/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTablePei/FirmwareP= erformancePei.c @@ -3,11 +3,11 @@ Data Table in S3 resume boot mode. =20 This module register report status code listener to collect performance = data for S3 Resume Performance Record on S3 resume boot path. =20 - Copyright (c) 2011 - 2017, Intel Corporation. All rights reserved.
+ Copyright (c) 2011 - 2018, 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 http://opensource.org/licenses/bsd-license.php =20 @@ -17,20 +17,24 @@ **/ =20 #include =20 #include +#include =20 #include +#include +#include =20 #include #include #include #include #include #include #include +#include =20 /** Report status code listener for PEI. This is used to record the performa= nce data for S3 FullResume in FPDT. =20 @@ -68,10 +72,17 @@ FpdtStatusCodeListenerPei ( S3_PERFORMANCE_TABLE *AcpiS3PerformanceTable; EFI_ACPI_5_0_FPDT_S3_RESUME_RECORD *AcpiS3ResumeRecord; UINT64 S3ResumeTotal; EFI_ACPI_5_0_FPDT_S3_SUSPEND_RECORD S3SuspendRecord; EFI_ACPI_5_0_FPDT_S3_SUSPEND_RECORD *AcpiS3SuspendRecord; + EFI_PEI_READ_ONLY_VARIABLE2_PPI *VariableServices; + UINT8 *BootPerformanceTable; + FIRMWARE_PERFORMANCE_VARIABLE PerformanceVariable; + EFI_HOB_GUID_TYPE *GuidHob; + PEI_EXT_FIRMWARE_PERF_HEADER *PeiPerformanceLogHeader; + UINT8 *FirmwarePerformanceData; + UINT8 *FirmwarePerformanceTablePtr; =20 // // Check whether status code is what we are interested in. // if (((CodeType & EFI_STATUS_CODE_TYPE_MASK) !=3D EFI_PROGRESS_CODE) || @@ -128,10 +139,56 @@ FpdtStatusCodeListenerPei ( AcpiS3SuspendRecord->SuspendEnd =3D S3SuspendRecord.SuspendEnd; =20 DEBUG ((EFI_D_INFO, "FPDT: S3 Suspend Performance - SuspendStart =3D %ld= \n", AcpiS3SuspendRecord->SuspendStart)); DEBUG ((EFI_D_INFO, "FPDT: S3 Suspend Performance - SuspendEnd =3D %ld= \n", AcpiS3SuspendRecord->SuspendEnd)); =20 + Status =3D PeiServicesLocatePpi ( + &gEfiPeiReadOnlyVariable2PpiGuid, + 0, + NULL, + (VOID **) &VariableServices + ); + ASSERT_EFI_ERROR (Status); + + // + // Update S3 boot records into the basic boot performance table. + // + VarSize =3D sizeof (PerformanceVariable); + Status =3D VariableServices->GetVariable ( + VariableServices, + EFI_FIRMWARE_PERFORMANCE_VARIABLE_NAME, + &gEfiFirmwarePerformanceGuid, + NULL, + &VarSize, + &PerformanceVariable + ); + if (EFI_ERROR (Status)) { + return Status; + } + BootPerformanceTable =3D (UINT8*) (UINTN) PerformanceVariable.BootPerfor= manceTablePointer; + + // + // Dump PEI boot records + // + FirmwarePerformanceTablePtr =3D (BootPerformanceTable + sizeof (BOOT_PER= FORMANCE_TABLE)); + GuidHob =3D GetFirstGuidHob (&gEdkiiExtendedFirmwarePerformanceGuid); + while (GuidHob !=3D NULL) { + FirmwarePerformanceData =3D GET_GUID_HOB_DATA (GuidHob); + PeiPerformanceLogHeader =3D (PEI_EXT_FIRMWARE_PERF_HEADER *) FirmwareP= erformanceData; + + CopyMem (FirmwarePerformanceTablePtr, FirmwarePerformanceData + sizeof= (PEI_EXT_FIRMWARE_PERF_HEADER), (UINTN)(PeiPerformanceLogHeader->SizeOfAll= Entries)); + + GuidHob =3D GetNextGuidHob (&gEdkiiExtendedFirmwarePerformanceGuid, GE= T_NEXT_HOB (GuidHob)); + + FirmwarePerformanceTablePtr +=3D (UINTN)(PeiPerformanceLogHeader->Size= OfAllEntries); + } + + // + // Update Table length. + // + ((BOOT_PERFORMANCE_TABLE *) BootPerformanceTable)->Header.Length =3D (UI= NT32)((UINTN)FirmwarePerformanceTablePtr - (UINTN)BootPerformanceTable); + return EFI_SUCCESS; } =20 /** Main entry for Firmware Performance Data Table PEIM. diff --git a/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTablePei/Fi= rmwarePerformancePei.inf b/MdeModulePkg/Universal/Acpi/FirmwarePerformanceD= ataTablePei/FirmwarePerformancePei.inf index 53b45a2..703caf4 100644 --- a/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTablePei/FirmwareP= erformancePei.inf +++ b/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTablePei/FirmwareP= erformancePei.inf @@ -4,11 +4,11 @@ # In S3 resume boot mode, it updates S3 Resume Performance Record in ACPI= Firmware Performance Data Table. # # This module register report status code listener to collect performance= data # for S3 Resume Performance Record on S3 resume boot path. # -# Copyright (c) 2011 - 2017, Intel Corporation. All rights reserved.
+# Copyright (c) 2011 - 2018, 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 # which accompanies this distribution. The full text of the license may = be found at # http://opensource.org/licenses/bsd-license.php # =20 @@ -46,18 +46,21 @@ DebugLib TimerLib BaseMemoryLib LockBoxLib PcdLib + HobLib =20 [Ppis] gEfiPeiRscHandlerPpiGuid ## CONSUMES + gEfiPeiReadOnlyVariable2PpiGuid ## CONSUMES =20 [Guids] ## SOMETIMES_CONSUMES ## UNDEFINED # RestoreLockBox gEfiFirmwarePerformanceGuid gFirmwarePerformanceS3PointerGuid ## SOMETIMES_CONSUMES ## U= NDEFINED # RestoreLockBox + gEdkiiExtendedFirmwarePerformanceGuid ## SOMETIMES_CONSUMES ## H= OB =20 [FeaturePcd] gEfiMdeModulePkgTokenSpaceGuid.PcdFirmwarePerformanceDataTableS3Support = ## CONSUMES =20 [Depex] --=20 1.9.5.msysgit.1 _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel From nobody Fri Nov 1 12:22:50 2024 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.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 1516679300516167.71573558197122; Mon, 22 Jan 2018 19:48:20 -0800 (PST) Received: from [127.0.0.1] (localhost [IPv6:::1]) by ml01.01.org (Postfix) with ESMTP id AFBCB222CF1CA; Mon, 22 Jan 2018 19:42:51 -0800 (PST) Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) (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 4F55822333762 for ; Mon, 22 Jan 2018 19:42:50 -0800 (PST) Received: from orsmga008.jf.intel.com ([10.7.209.65]) by fmsmga101.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 22 Jan 2018 19:48:16 -0800 Received: from shwdeopenpsi114.ccr.corp.intel.com ([10.239.157.135]) by orsmga008.jf.intel.com with ESMTP; 22 Jan 2018 19:48:15 -0800 X-Original-To: edk2-devel@lists.01.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; Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=192.55.52.88; helo=mga01.intel.com; envelope-from=dandan.bi@intel.com; receiver=edk2-devel@lists.01.org X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.46,399,1511856000"; d="scan'208";a="12262560" From: Dandan Bi To: edk2-devel@lists.01.org Date: Tue, 23 Jan 2018 11:47:33 +0800 Message-Id: <1516679255-12328-7-git-send-email-dandan.bi@intel.com> X-Mailer: git-send-email 1.9.5.msysgit.1 In-Reply-To: <1516679255-12328-1-git-send-email-dandan.bi@intel.com> References: <1516679255-12328-1-git-send-email-dandan.bi@intel.com> Subject: [edk2] [patch 6/8] MdeModulePkg/FirmwarePerfDxe:Enhance for new pref infrastructure X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Star Zeng , Liming Gao 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" 1. Remove the macro EXTENSION_RECORD_SIZE, since the extension size can be got through PcdExtFpdtBootRecordPadSize. 2. Hook EFI_SW_DXE_BS_PC_READY_TO_BOOT_EVENT to install ACPI table 3. Copy SMM record according to the allocated size Cc: Liming Gao Cc: Star Zeng Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Dandan Bi --- .../FirmwarePerformanceDxe.c | 58 ++++++------------= ---- 1 file changed, 16 insertions(+), 42 deletions(-) diff --git a/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTableDxe/Fi= rmwarePerformanceDxe.c b/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDat= aTableDxe/FirmwarePerformanceDxe.c index b004cac..0de6bfa 100644 --- a/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTableDxe/FirmwareP= erformanceDxe.c +++ b/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTableDxe/FirmwareP= erformanceDxe.c @@ -3,11 +3,11 @@ =20 This module register report status code listener to collect performance = data for Firmware Basic Boot Performance Record and other boot performance re= cords,=20 and install FPDT to ACPI table. =20 - Copyright (c) 2011 - 2017, Intel Corporation. All rights reserved.
+ Copyright (c) 2011 - 2018, 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 http://opensource.org/licenses/bsd-license.php =20 @@ -40,12 +40,11 @@ #include #include #include #include =20 -#define EXTENSION_RECORD_SIZE 0x10000 -#define SMM_BOOT_RECORD_COMM_SIZE OFFSET_OF (EFI_SMM_COMMUNICATE_HEADER, D= ata) + sizeof(SMM_BOOT_RECORD_COMMUNICATE) +#define SMM_BOOT_RECORD_COMM_SIZE (OFFSET_OF (EFI_SMM_COMMUNICATE_HEADER, = Data) + sizeof(SMM_BOOT_RECORD_COMMUNICATE)) =20 EFI_RSC_HANDLER_PROTOCOL *mRscHandlerProtocol =3D NULL; =20 BOOLEAN mLockBoxReady =3D FALSE; EFI_EVENT mReadyToBootEvent; @@ -419,11 +418,15 @@ InstallFirmwarePerformanceDataTable ( SmmCommData->BootRecordSize =3D ReservedMemSize - SMM_BOOT_REC= ORD_COMM_SIZE; while (SmmCommData->BootRecordOffset < SmmBootRecordDataSize) { Status =3D Communication->Communicate (Communication, SmmBootR= ecordCommBuffer, &CommSize); ASSERT_EFI_ERROR (Status); ASSERT_EFI_ERROR(SmmCommData->ReturnStatus); - CopyMem ((UINT8 *) SmmBootRecordData + SmmCommData->BootRecord= Offset, SmmCommData->BootRecordData, SmmCommData->BootRecordSize); + if (SmmCommData->BootRecordOffset + SmmCommData->BootRecordSiz= e > SmmBootRecordDataSize) { + CopyMem ((UINT8 *) SmmBootRecordData + SmmCommData->BootReco= rdOffset, SmmCommData->BootRecordData, SmmBootRecordDataSize - SmmCommData-= >BootRecordOffset); + } else { + CopyMem ((UINT8 *) SmmBootRecordData + SmmCommData->BootReco= rdOffset, SmmCommData->BootRecordData, SmmCommData->BootRecordSize); + } SmmCommData->BootRecordOffset =3D SmmCommData->BootRecordOffse= t + SmmCommData->BootRecordSize; } } } } @@ -567,33 +570,10 @@ InstallFirmwarePerformanceDataTable ( =20 return EFI_SUCCESS; } =20 /** - Notify function for event group EFI_EVENT_GROUP_READY_TO_BOOT. This is u= sed to - install the Firmware Performance Data Table. - - @param[in] Event The Event that is being processed. - @param[in] Context The Event Context. - -**/ -VOID -EFIAPI -FpdtReadyToBootEventNotify ( - IN EFI_EVENT Event, - IN VOID *Context - ) -{ - if (mAcpiBootPerformanceTable =3D=3D NULL) { - // - // ACPI Firmware Performance Data Table not installed yet, install it = now. - // - InstallFirmwarePerformanceDataTable (); - } -} - -/** Report status code listener of FPDT. This is used to collect performance= data for OsLoaderLoadImageStart and OsLoaderStartImageStart in FPDT. =20 @param[in] CodeType Indicates the type of status code being = reported. @param[in] Value Describes the current status of a hardwa= re or software entity. @@ -697,22 +677,29 @@ FpdtStatusCodeListenerDxe ( DEBUG ((EFI_D_INFO, "FPDT: Boot Performance - ResetEnd = =3D %ld\n", mAcpiBootPerformanceTable->BasicBoot.ResetEnd)); DEBUG ((EFI_D_INFO, "FPDT: Boot Performance - OsLoaderLoadImageStart = =3D 0\n")); DEBUG ((EFI_D_INFO, "FPDT: Boot Performance - OsLoaderStartImageStart = =3D %ld\n", mAcpiBootPerformanceTable->BasicBoot.OsLoaderStartImageStart)); DEBUG ((EFI_D_INFO, "FPDT: Boot Performance - ExitBootServicesEntry = =3D 0\n")); DEBUG ((EFI_D_INFO, "FPDT: Boot Performance - ExitBootServicesExit = =3D 0\n")); + } else if (Value =3D=3D (EFI_SOFTWARE_DXE_BS_DRIVER | EFI_SW_DXE_BS_PC_R= EADY_TO_BOOT_EVENT)) { + if (mAcpiBootPerformanceTable =3D=3D NULL) { + // + // ACPI Firmware Performance Data Table not installed yet, install i= t now. + // + InstallFirmwarePerformanceDataTable (); + } } else if (Data !=3D NULL && CompareGuid (&Data->Type, &gEfiFirmwarePerf= ormanceGuid)) { // // Append one or more Boot records // if (mAcpiBootPerformanceTable =3D=3D NULL) { // // Append Boot records before FPDT ACPI table is installed.=20 // if (mBootRecordSize + Data->Size > mBootRecordMaxSize) { - mBootRecordBuffer =3D ReallocatePool (mBootRecordSize, mBootRecord= Size + Data->Size + EXTENSION_RECORD_SIZE, mBootRecordBuffer); + mBootRecordBuffer =3D ReallocatePool (mBootRecordSize, mBootRecord= Size + Data->Size + PcdGet32 (PcdExtFpdtBootRecordPadSize), mBootRecordBuff= er); ASSERT (mBootRecordBuffer !=3D NULL); - mBootRecordMaxSize =3D mBootRecordSize + Data->Size + EXTENSION_RE= CORD_SIZE; + mBootRecordMaxSize =3D mBootRecordSize + Data->Size + PcdGet32 (Pc= dExtFpdtBootRecordPadSize); } // // Save boot record into the temp memory space. // CopyMem (mBootRecordBuffer + mBootRecordSize, Data + 1, Data->Size); @@ -851,23 +838,10 @@ FirmwarePerformanceDxeEntryPoint ( &mExitBootServicesEvent ); ASSERT_EFI_ERROR (Status); =20 // - // Create ready to boot event to install ACPI FPDT table. - // - Status =3D gBS->CreateEventEx ( - EVT_NOTIFY_SIGNAL, - TPL_NOTIFY, - FpdtReadyToBootEventNotify, - NULL, - &gEfiEventReadyToBootGuid, - &mReadyToBootEvent - ); - ASSERT_EFI_ERROR (Status); - - // // Retrieve GUID HOB data that contains the ResetEnd. // GuidHob =3D GetFirstGuidHob (&gEfiFirmwarePerformanceGuid); if (GuidHob !=3D NULL) { Performance =3D (FIRMWARE_SEC_PERFORMANCE *) GET_GUID_HOB_DATA (GuidHo= b); --=20 1.9.5.msysgit.1 _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel From nobody Fri Nov 1 12:22:50 2024 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.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 1516679303857706.3738049813967; Mon, 22 Jan 2018 19:48:23 -0800 (PST) Received: from [127.0.0.1] (localhost [IPv6:::1]) by ml01.01.org (Postfix) with ESMTP id 31AC02232BE0E; Mon, 22 Jan 2018 19:42:56 -0800 (PST) Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) (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 CD2F222333762 for ; Mon, 22 Jan 2018 19:42:54 -0800 (PST) Received: from orsmga008.jf.intel.com ([10.7.209.65]) by fmsmga101.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 22 Jan 2018 19:48:21 -0800 Received: from shwdeopenpsi114.ccr.corp.intel.com ([10.239.157.135]) by orsmga008.jf.intel.com with ESMTP; 22 Jan 2018 19:48:20 -0800 X-Original-To: edk2-devel@lists.01.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; Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=192.55.52.88; helo=mga01.intel.com; envelope-from=dandan.bi@intel.com; receiver=edk2-devel@lists.01.org X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.46,399,1511856000"; d="scan'208";a="12262573" From: Dandan Bi To: edk2-devel@lists.01.org Date: Tue, 23 Jan 2018 11:47:34 +0800 Message-Id: <1516679255-12328-8-git-send-email-dandan.bi@intel.com> X-Mailer: git-send-email 1.9.5.msysgit.1 In-Reply-To: <1516679255-12328-1-git-send-email-dandan.bi@intel.com> References: <1516679255-12328-1-git-send-email-dandan.bi@intel.com> MIME-Version: 1.0 Subject: [edk2] [patch 7/8] MdeModulePkg/FirmwarePerfSmm: Add check for collecting SMM records X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Star Zeng , Liming Gao Content-Type: multipart/mixed; boundary="===============4553507035262864371==" Errors-To: edk2-devel-bounces@lists.01.org Sender: "edk2-devel" X-ZohoMail: RSF_4 Z_629925259 SPT_0 --===============4553507035262864371== Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" After DXE driver communicating with SMM driver to get the size and contents of the SMM records, make SMM=E3=80=80driver not to allocate buffer to collect new SMM records if the pre-allocate buffer is not enough. Cc: Liming Gao Cc: Star Zeng Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Dandan Bi --- .../FirmwarePerformanceSmm.c | 14 ++++++++++= ++-- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTableSmm/Fi= rmwarePerformanceSmm.c b/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDat= aTableSmm/FirmwarePerformanceSmm.c index c750331..160f6c0 100644 --- a/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTableSmm/FirmwareP= erformanceSmm.c +++ b/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTableSmm/FirmwareP= erformanceSmm.c @@ -9,11 +9,11 @@ This external input must be validated carefully to avoid security issue = like buffer overflow, integer overflow. =20 FpdtSmiHandler() will receive untrusted input and do basic validation. =20 - Copyright (c) 2011 - 2016, Intel Corporation. All rights reserved.
+ Copyright (c) 2011 - 2018, 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 http://opensource.org/licenses/bsd-license.php =20 @@ -49,10 +49,11 @@ UINT32 mBootRecordSize =3D 0; UINT32 mBootRecordMaxSize =3D 0; UINT8 *mBootRecordBuffer =3D NULL; =20 SPIN_LOCK mSmmFpdtLock; BOOLEAN mSmramIsOutOfResource =3D FALSE; +BOOLEAN mHaveReportedSmmRecord =3D FALSE; =20 /** Report status code listener for SMM. This is used to record the performa= nce data for S3 Suspend Start and S3 Suspend End in FPDT. =20 @@ -90,19 +91,27 @@ FpdtStatusCodeListenerSmm ( // Check whether status code is what we are interested in. // if ((CodeType & EFI_STATUS_CODE_TYPE_MASK) !=3D EFI_PROGRESS_CODE) { return EFI_UNSUPPORTED; } - =20 + // // Collect one or more Boot records in boot time // if (Data !=3D NULL && CompareGuid (&Data->Type, &gEfiFirmwarePerformance= Guid)) { AcquireSpinLock (&mSmmFpdtLock); =20 if (mBootRecordSize + Data->Size > mBootRecordMaxSize) { // + // If SMM records have been reported to DXE driver, don't re-allocat= e buffer + // to collect the new SMM records when pre-allocate buffer is not en= ough. + // + if (mHaveReportedSmmRecord) { + ReleaseSpinLock (&mSmmFpdtLock); + return EFI_OUT_OF_RESOURCES; + } + // // Try to allocate big SMRAM data to store Boot record.=20 // if (mSmramIsOutOfResource) { ReleaseSpinLock (&mSmmFpdtLock); return EFI_OUT_OF_RESOURCES; @@ -237,10 +246,11 @@ FpdtSmiHandler ( =20 Status =3D EFI_SUCCESS; =20 switch (SmmCommData->Function) { case SMM_FPDT_FUNCTION_GET_BOOT_RECORD_SIZE : + mHaveReportedSmmRecord =3D TRUE; SmmCommData->BootRecordSize =3D mBootRecordSize; break; =20 case SMM_FPDT_FUNCTION_GET_BOOT_RECORD_DATA : Status =3D EFI_UNSUPPORTED; --=20 1.9.5.msysgit.1 --===============4553507035262864371== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel --===============4553507035262864371==-- From nobody Fri Nov 1 12:22:50 2024 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.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 151667931153317.266333440462063; Mon, 22 Jan 2018 19:48:31 -0800 (PST) Received: from [127.0.0.1] (localhost [IPv6:::1]) by ml01.01.org (Postfix) with ESMTP id CAC0F22344321; Mon, 22 Jan 2018 19:43:03 -0800 (PST) Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) (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 2FDA12233379A for ; Mon, 22 Jan 2018 19:43:02 -0800 (PST) Received: from orsmga008.jf.intel.com ([10.7.209.65]) by orsmga101.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 22 Jan 2018 19:48:28 -0800 Received: from shwdeopenpsi114.ccr.corp.intel.com ([10.239.157.135]) by orsmga008.jf.intel.com with ESMTP; 22 Jan 2018 19:48:26 -0800 X-Original-To: edk2-devel@lists.01.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; Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=134.134.136.20; helo=mga02.intel.com; envelope-from=dandan.bi@intel.com; receiver=edk2-devel@lists.01.org X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.46,399,1511856000"; d="scan'208";a="12262586" From: Dandan Bi To: edk2-devel@lists.01.org Date: Tue, 23 Jan 2018 11:47:35 +0800 Message-Id: <1516679255-12328-9-git-send-email-dandan.bi@intel.com> X-Mailer: git-send-email 1.9.5.msysgit.1 In-Reply-To: <1516679255-12328-1-git-send-email-dandan.bi@intel.com> References: <1516679255-12328-1-git-send-email-dandan.bi@intel.com> Subject: [edk2] [patch 8/8] ShellPkg/Dp: Updated to dump perf log based on FPDT table X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Ruiyu Ni , Jaben Carsey , Star Zeng , Liming Gao 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" Cc: Liming Gao Cc: Star Zeng Cc: Ruiyu Ni Cc: Jaben Carsey Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Dandan Bi --- ShellPkg/DynamicCommand/DpDynamicCommand/Dp.c | 609 +++++++++++++++++= +++- ShellPkg/DynamicCommand/DpDynamicCommand/Dp.h | 7 +- ShellPkg/DynamicCommand/DpDynamicCommand/Dp.uni | 11 +- ShellPkg/DynamicCommand/DpDynamicCommand/DpApp.inf | 5 +- .../DpDynamicCommand/DpDynamicCommand.inf | 5 +- .../DynamicCommand/DpDynamicCommand/DpInternal.h | 9 +- ShellPkg/DynamicCommand/DpDynamicCommand/DpTrace.c | 92 +++- .../DynamicCommand/DpDynamicCommand/DpUtilities.c | 31 +- .../DynamicCommand/DpDynamicCommand/Literals.c | 24 +- .../DynamicCommand/DpDynamicCommand/Literals.h | 8 +- .../DpDynamicCommand/PerformanceTokens.h | 28 - 11 files changed, 713 insertions(+), 116 deletions(-) delete mode 100644 ShellPkg/DynamicCommand/DpDynamicCommand/PerformanceTok= ens.h diff --git a/ShellPkg/DynamicCommand/DpDynamicCommand/Dp.c b/ShellPkg/Dynam= icCommand/DpDynamicCommand/Dp.c index 3ecc753..6013d99 100644 --- a/ShellPkg/DynamicCommand/DpDynamicCommand/Dp.c +++ b/ShellPkg/DynamicCommand/DpDynamicCommand/Dp.c @@ -11,40 +11,64 @@ Measurement records contain identifying information (Handle, Token, Modu= le) and start and end time values. 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 - 2017, Intel Corporation. All rights reserved. + Copyright (c) 2009 - 2018, 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 which accompanies this distribution. The full text of the license may b= e found at http://opensource.org/licenses/bsd-license.php =20 THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMP= LIED. **/ =20 -#include "PerformanceTokens.h" #include "Dp.h" #include "Literals.h" #include "DpInternal.h" =20 +#pragma pack(1) + +typedef struct { + EFI_ACPI_DESCRIPTION_HEADER Header; + UINT32 Entry; +} RSDT_TABLE; + +typedef struct { + EFI_ACPI_DESCRIPTION_HEADER Header; + UINT64 Entry; +} XSDT_TABLE; + +#pragma pack() + EFI_HANDLE mDpHiiHandle; =20 +typedef struct { + EFI_HANDLE Handle; + EFI_GUID ModuleGuid; +} HANDLE_GUID_MAP; + +HANDLE_GUID_MAP *mCacheHandleGuidTable; +UINTN mCachePairCount =3D 0; + // /// Module-Global Variables ///@{ CHAR16 mGaugeString[DP_GAUGE_STRING_LENGTH + 1]; CHAR16 mUnicodeToken[DXE_PERFORMANCE_STRING_SIZE]; UINT64 mInterestThreshold; BOOLEAN mShowId =3D FALSE; +UINT8 *mBootPerformanceTable; +UINTN mBootPerformanceTableSize; +BOOLEAN mPeiPhase =3D FALSE; +BOOLEAN mDxePhase =3D FALSE; =20 PERF_SUMMARY_DATA SummaryData =3D { 0 }; ///< Create the SummaryData st= ructure and init. to ZERO. - -/// Timer Specific Information. -TIMER_INFO TimerInfo; +MEASUREMENT_RECORD *mMeasurementList =3D NULL; +UINTN mMeasurementNum =3D 0; =20 /// Items for which to gather cumulative statistics. PERF_CUM_DATA CumData[] =3D { PERF_INIT_CUM_DATA (LOAD_IMAGE_TOK), PERF_INIT_CUM_DATA (START_IMAGE_TOK), @@ -98,10 +122,540 @@ DumpStatistics( void ) SHELL_FREE_NON_NULL (StringPtr); SHELL_FREE_NON_NULL (StringPtrUnknown); } =20 /** + This function scan ACPI table in RSDT. + + @param Rsdt ACPI RSDT + @param Signature ACPI table signature + + @return ACPI table +**/ +VOID * +ScanTableInRSDT ( + IN RSDT_TABLE *Rsdt, + IN UINT32 Signature + ) +{ + UINTN Index; + UINT32 EntryCount; + UINT32 *EntryPtr; + EFI_ACPI_DESCRIPTION_HEADER *Table; + + EntryCount =3D (Rsdt->Header.Length - sizeof (EFI_ACPI_DESCRIPTION_HEADE= R)) / sizeof(UINT32); + + EntryPtr =3D &Rsdt->Entry; + for (Index =3D 0; Index < EntryCount; Index ++, EntryPtr ++) { + Table =3D (EFI_ACPI_DESCRIPTION_HEADER*)((UINTN)(*EntryPtr)); + if (Table->Signature =3D=3D Signature) { + return Table; + } + } + + return NULL; +} + +/** + This function scan ACPI table in XSDT. + + @param Xsdt ACPI XSDT + @param Signature ACPI table signature + + @return ACPI table +**/ +VOID * +ScanTableInXSDT ( + IN XSDT_TABLE *Xsdt, + IN UINT32 Signature + ) +{ + UINTN Index; + UINT32 EntryCount; + UINT64 EntryPtr; + UINTN BasePtr; + EFI_ACPI_DESCRIPTION_HEADER *Table; + + EntryCount =3D (Xsdt->Header.Length - sizeof (EFI_ACPI_DESCRIPTION_HEADE= R)) / sizeof(UINT64); + + BasePtr =3D (UINTN)(&(Xsdt->Entry)); + for (Index =3D 0; Index < EntryCount; Index ++) { + CopyMem (&EntryPtr, (VOID *)(BasePtr + Index * sizeof(UINT64)), sizeof= (UINT64)); + Table =3D (EFI_ACPI_DESCRIPTION_HEADER*)((UINTN)(EntryPtr)); + if (Table->Signature =3D=3D Signature) { + return Table; + } + } + + return NULL; +} + +/** + This function scan ACPI table in RSDP. + + @param Rsdp ACPI RSDP + @param Signature ACPI table signature + + @return ACPI table +**/ +VOID * +FindAcpiPtr ( + IN EFI_ACPI_5_0_ROOT_SYSTEM_DESCRIPTION_POINTER *Rsdp, + IN UINT32 Signature + ) +{ + EFI_ACPI_DESCRIPTION_HEADER *AcpiTable; + RSDT_TABLE *Rsdt; + XSDT_TABLE *Xsdt; + + AcpiTable =3D NULL; + + // + // Check ACPI2.0 table + // + Rsdt =3D (RSDT_TABLE *)(UINTN)Rsdp->RsdtAddress; + Xsdt =3D NULL; + if ((Rsdp->Revision >=3D 2) && (Rsdp->XsdtAddress < (UINT64)(UINTN)-1)) { + Xsdt =3D (XSDT_TABLE *)(UINTN)Rsdp->XsdtAddress; + } + // + // Check Xsdt + // + if (Xsdt !=3D NULL) { + AcpiTable =3D ScanTableInXSDT (Xsdt, Signature); + } + // + // Check Rsdt + // + if ((AcpiTable =3D=3D NULL) && (Rsdt !=3D NULL)) { + AcpiTable =3D ScanTableInRSDT (Rsdt, Signature); + } + + return AcpiTable; +} + +/** + Get Boot performance table form Acpi table. + +**/ +EFI_STATUS +GetBootPerformanceTable ( + ) +{ + EFI_STATUS Status; + VOID *AcpiTable; + FIRMWARE_PERFORMANCE_TABLE *FirmwarePerformanceTable; + + AcpiTable =3D NULL; + + Status =3D EfiGetSystemConfigurationTable ( + &gEfiAcpi20TableGuid, + &AcpiTable + ); + if (EFI_ERROR (Status)) { + Status =3D EfiGetSystemConfigurationTable ( + &gEfiAcpi10TableGuid, + &AcpiTable + ); + } + if (EFI_ERROR(Status)) { + return Status; + } + + FirmwarePerformanceTable =3D FindAcpiPtr ( + (EFI_ACPI_5_0_ROOT_SYSTEM_DESCRIPTION_POINTER *)Acpi= Table, + EFI_ACPI_5_0_FIRMWARE_PERFORMANCE_DATA_TABLE_SIGNATU= RE + ); + if (FirmwarePerformanceTable =3D=3D NULL) { + return EFI_NOT_FOUND; + } + + mBootPerformanceTable =3D (UINT8*) (UINTN)FirmwarePerformanceTable->Boot= PointerRecord.BootPerformanceTablePointer; + mBootPerformanceTableSize =3D ((BOOT_PERFORMANCE_TABLE *) mBootPerforman= ceTable)->Header.Length; + + return EFI_SUCCESS; +} + +/** + Get Handle form Module Guid. + + @param ModuleGuid Module Guid. + @param Handle The handle to be returned. + +**/ +VOID +GetHandleFormModuleGuid ( + IN EFI_GUID *ModuleGuid, + IN OUT EFI_HANDLE *Handle + ) +{ + UINTN Index; + + if (IsZeroGuid (ModuleGuid)) { + *Handle =3D NULL; + } + // + // Try to get the Handle form the caached array. + // + for (Index =3D 0; Index < mCachePairCount; Index++) { + if (CompareGuid (ModuleGuid, &mCacheHandleGuidTable[Index].ModuleGuid)= ) { + *Handle =3D mCacheHandleGuidTable[Index].Handle; + break; + } + } + if (Index >=3D mCachePairCount) { + *Handle =3D NULL; + } +} + +/** +Cache the GUID and handle mapping pairs. In order to save time for searchi= ng. + +**/ +EFI_STATUS +BuildCachedGuidHandleTable ( + VOID + ) +{ + EFI_STATUS Status; + EFI_HANDLE *HandleBuffer; + UINTN HandleCount; + UINTN Index; + EFI_LOADED_IMAGE_PROTOCOL *LoadedImage; + EFI_DRIVER_BINDING_PROTOCOL *DriverBinding; + EFI_GUID *TempGuid; + MEDIA_FW_VOL_FILEPATH_DEVICE_PATH *FvFilePath; + + Status =3D gBS->LocateHandleBuffer (AllHandles, NULL, NULL, &HandleCount= , &HandleBuffer); + if (EFI_ERROR (Status)) { + ShellPrintHiiEx (-1, -1, NULL, STRING_TOKEN (STR_DP_HANDLES_ERROR), mD= pHiiHandle, Status); + return Status; + } + + mCacheHandleGuidTable =3D AllocateZeroPool (HandleCount * sizeof (HANDLE= _GUID_MAP)); + if (mCacheHandleGuidTable =3D=3D NULL) { + return EFI_OUT_OF_RESOURCES; + } + + for (Index =3D 0; Index < HandleCount; Index++) { + // + // Try Handle as ImageHandle. + // + Status =3D gBS->HandleProtocol ( + HandleBuffer[Index], + &gEfiLoadedImageProtocolGuid, + (VOID**) &LoadedImage + ); + if (EFI_ERROR (Status)) { + // + // Try Handle as Controller Handle + // + Status =3D gBS->OpenProtocol ( + HandleBuffer[Index], + &gEfiDriverBindingProtocolGuid, + (VOID **) &DriverBinding, + NULL, + NULL, + EFI_OPEN_PROTOCOL_GET_PROTOCOL + ); + if (!EFI_ERROR (Status)) { + // + // Get Image protocol from ImageHandle + // + Status =3D gBS->HandleProtocol ( + DriverBinding->ImageHandle, + &gEfiLoadedImageProtocolGuid, + (VOID**) &LoadedImage + ); + } + } + + if (!EFI_ERROR (Status) && LoadedImage !=3D NULL) { + // + // Get Module Guid from DevicePath. + // + if (LoadedImage->FilePath !=3D NULL && + LoadedImage->FilePath->Type =3D=3D MEDIA_DEVICE_PATH && + LoadedImage->FilePath->SubType =3D=3D MEDIA_PIWG_FW_FILE_DP + ) { + FvFilePath =3D (MEDIA_FW_VOL_FILEPATH_DEVICE_PATH *) LoadedIm= age->FilePath; + TempGuid =3D &FvFilePath->FvFileName; + + mCacheHandleGuidTable[mCachePairCount].Handle =3D HandleBuffer[Ind= ex]; + CopyGuid (&mCacheHandleGuidTable[mCachePairCount].ModuleGuid, Temp= Guid); + mCachePairCount ++; + } + } + } + if (HandleBuffer !=3D NULL) { + FreePool (HandleBuffer); + HandleBuffer =3D NULL; + } + return Status; +} + +/** + Get Measurement form Fpdt records. + + @param RecordHeader Pointer to the start record. + @param IsStart Is start record or End record. + @param Measurement Pointer to the measurement which need to be = filled. + +**/ +VOID +GetMeasurementInfo ( + IN EFI_ACPI_5_0_FPDT_PERFORMANCE_RECORD_HEADER *RecordHeader, + IN BOOLEAN IsStart, + IN OUT MEASUREMENT_RECORD *Measurement + ) +{ + VOID *ModuleGuid; + EFI_HANDLE StartHandle; + + switch (RecordHeader->Type) { + case GUID_EVENT_TYPE: + ModuleGuid =3D &(((GUID_EVENT_RECORD *)RecordHeader= )->Guid); + Measurement->Identifier =3D ((UINT32)((GUID_EVENT_RECORD *)Recor= dHeader)->ProgressID); + if (IsStart) { + Measurement->StartTimeStamp =3D ((GUID_EVENT_RECORD *)RecordHeader)-= >Timestamp; + } else { + Measurement->EndTimeStamp =3D ((GUID_EVENT_RECORD *)RecordHeader)-= >Timestamp; + } + switch (Measurement->Identifier) { + case MODULE_START_ID: + case MODULE_END_ID: + if (mPeiPhase) { + Measurement->Token =3D ALit_PEIM; + Measurement->Module =3D ALit_PEIM; + } else if (mDxePhase) { + Measurement->Token =3D ALit_START_IMAGE; + Measurement->Module =3D ALit_START_IMAGE; + } + break; + default: + ASSERT(FALSE); + } + + if (AsciiStrnCmp (Measurement->Token, ALit_PEIM, PERF_TOKEN_LENGTH) = =3D=3D 0) { + Measurement->Handle =3D &(((GUID_EVENT_RECORD *)RecordHeader= )->Guid); + } else { + GetHandleFormModuleGuid(ModuleGuid, &StartHandle); + Measurement->Handle =3D StartHandle; + } + break; + + case DYNAMIC_STRING_EVENT_TYPE: + ModuleGuid =3D &(((DYNAMIC_STRING_EVENT_RECORD *)Re= cordHeader)->Guid); + Measurement->Identifier =3D ((UINT32)((DYNAMIC_STRING_EVENT_RECO= RD *)RecordHeader)->ProgressID); + if (IsStart) { + Measurement->StartTimeStamp =3D ((DYNAMIC_STRING_EVENT_RECORD *)Reco= rdHeader)->Timestamp; + } else { + Measurement->EndTimeStamp =3D ((DYNAMIC_STRING_EVENT_RECORD *)Reco= rdHeader)->Timestamp; + } + switch (Measurement->Identifier) { + case MODULE_START_ID: + case MODULE_END_ID: + if (mPeiPhase) { + Measurement->Token =3D ALit_PEIM; + } else if (mDxePhase) { + Measurement->Token =3D ALit_START_IMAGE; + } + break; + + case MODULE_LOADIMAGE_START_ID: + case MODULE_LOADIMAGE_END_ID: + Measurement->Token =3D ALit_LOAD_IMAGE; + break; + + case MODULE_DB_START_ID: + case MODULE_DB_END_ID: + Measurement->Token =3D ALit_DB_START; + break; + + case MODULE_DB_SUPPORT_START_ID: + case MODULE_DB_SUPPORT_END_ID: + Measurement->Token =3D ALit_DB_SUPPORT; + break; + + case MODULE_DB_STOP_START_ID: + case MODULE_DB_STOP_END_ID: + Measurement->Token =3D ALit_DB_STOP; + break; + + default: + Measurement->Token =3D ((DYNAMIC_STRING_EVENT_RECORD *)Reco= rdHeader)->String; + break; + } + + Measurement->Module =3D ((DYNAMIC_STRING_EVENT_RECORD *)Reco= rdHeader)->String; + + if (AsciiStrnCmp (Measurement->Token, ALit_PEIM, PERF_TOKEN_LENGTH) = =3D=3D 0) { + Measurement->Handle =3D &(((GUID_EVENT_RECORD *)RecordHeader= )->Guid); + } else { + GetHandleFormModuleGuid(ModuleGuid, &StartHandle); + Measurement->Handle =3D StartHandle; + } + break; + + case GUID_QWORD_EVENT_TYPE: + ModuleGuid =3D &(((GUID_QWORD_EVENT_RECORD *)Record= Header)->Guid); + Measurement->Identifier =3D ((UINT32)((GUID_QWORD_EVENT_RECORD *= )RecordHeader)->ProgressID); + if (IsStart) { + Measurement->StartTimeStamp =3D ((GUID_QWORD_EVENT_RECORD *)RecordHe= ader)->Timestamp; + } else { + Measurement->EndTimeStamp =3D ((GUID_QWORD_EVENT_RECORD *)RecordHe= ader)->Timestamp; + } + switch (Measurement->Identifier) { + case MODULE_DB_START_ID: + Measurement->Token =3D ALit_DB_START; + Measurement->Module =3D ALit_DB_START; + break; + + case MODULE_DB_SUPPORT_START_ID: + case MODULE_DB_SUPPORT_END_ID: + Measurement->Token =3D ALit_DB_SUPPORT; + Measurement->Module =3D ALit_DB_SUPPORT; + break; + + case MODULE_DB_STOP_START_ID: + case MODULE_DB_STOP_END_ID: + Measurement->Token =3D ALit_DB_STOP; + Measurement->Module =3D ALit_DB_STOP; + break; + + case MODULE_LOADIMAGE_START_ID: + case MODULE_LOADIMAGE_END_ID: + Measurement->Token =3D ALit_LOAD_IMAGE; + Measurement->Module =3D ALit_LOAD_IMAGE; + break; + + default: + ASSERT(FALSE); + } + GetHandleFormModuleGuid(ModuleGuid, &StartHandle); + Measurement->Handle =3D StartHandle; + break; + + case GUID_QWORD_STRING_EVENT_TYPE: + ModuleGuid =3D &(((GUID_QWORD_STRING_EVENT_RECORD *= )RecordHeader)->Guid); + Measurement->Identifier =3D ((UINT32)((GUID_QWORD_STRING_EVENT_R= ECORD *)RecordHeader)->ProgressID); + if (IsStart) { + Measurement->StartTimeStamp =3D ((GUID_QWORD_STRING_EVENT_RECORD*)Re= cordHeader)->Timestamp; + } else { + Measurement->EndTimeStamp =3D ((GUID_QWORD_STRING_EVENT_RECORD *)R= ecordHeader)->Timestamp; + } + // + // Currently only "DB:Start:" end record with GUID_QWORD_STRING_EVENT_= TYPE. + // + switch (Measurement->Identifier) { + case MODULE_DB_END_ID: + Measurement->Token =3D ALit_DB_START; + Measurement->Module =3D ALit_DB_START; + break; + default: + ASSERT(FALSE); + } + GetHandleFormModuleGuid(ModuleGuid, &StartHandle); + Measurement->Handle =3D StartHandle; + break; + + default: + break; + } +} + +/** + Search the start measurement in the mMeasurementList for the end measure= ment. + + @param EndMeasureMent Measurement for end record. + +**/ +VOID +SearchMeasurement ( + IN MEASUREMENT_RECORD *EndMeasureMent + ) +{ + INTN Index; + + for (Index =3D mMeasurementNum - 1; Index >=3D 0; Index--) { + if (AsciiStrnCmp (EndMeasureMent->Token, ALit_PEIM, PERF_TOKEN_LENGTH)= =3D=3D 0) { + if (mMeasurementList[Index].EndTimeStamp =3D=3D 0 && EndMeasureMent-= >Handle!=3D NULL && mMeasurementList[Index].Handle !=3D NULL&& + CompareGuid(mMeasurementList[Index].Handle, EndMeasureMent->Hand= le) && + (AsciiStrCmp (mMeasurementList[Index].Token, EndMeasureMent->Tok= en) =3D=3D 0) && + (AsciiStrCmp (mMeasurementList[Index].Module, EndMeasureMent->Mo= dule) =3D=3D 0)) { + mMeasurementList[Index].EndTimeStamp =3D EndMeasureMent->EndTimeSt= amp; + break; + } + } else { + if (mMeasurementList[Index].EndTimeStamp =3D=3D 0 && mMeasurementLis= t[Index].Handle =3D=3D EndMeasureMent->Handle && + (AsciiStrCmp (mMeasurementList[Index].Token, EndMeasureMent->Toke= n) =3D=3D 0) && + (AsciiStrCmp (mMeasurementList[Index].Module, EndMeasureMent->Mod= ule) =3D=3D 0)) { + mMeasurementList[Index].EndTimeStamp =3D EndMeasureMent->EndTimeSt= amp; + break; + } + } + } +} + +/** + Generate the measure record array. + +**/ +EFI_STATUS +BuildMeasurementList ( + ) +{ + EFI_ACPI_5_0_FPDT_PERFORMANCE_RECORD_HEADER *RecordHeader; + UINT8 *PerformanceTablePtr; + UINT16 StartProgressId; + UINTN TableLength; + UINT8 *StartRecordEvent; + MEASUREMENT_RECORD MeasureMent; + + mMeasurementList =3D AllocateZeroPool (mBootPerformanceTableSize); + if (mMeasurementList =3D=3D NULL) { + return EFI_OUT_OF_RESOURCES; + } + + TableLength =3D sizeof (BOOT_PERFORMANCE_TABLE); + PerformanceTablePtr =3D (mBootPerformanceTable + TableLength); + + while (TableLength < mBootPerformanceTableSize) { + RecordHeader =3D (EFI_ACPI_5_0_FPDT_PERFORMANCE_RECORD_HEADER*) P= erformanceTablePtr; + StartRecordEvent =3D (UINT8 *)RecordHeader; + StartProgressId =3D ((GUID_EVENT_RECORD *)StartRecordEvent)->Progres= sID; + + // + // If the record is the start record, fill the info to the measurement= in the mMeasurementList. + // If the record is the end record, find the related start measurement= in the mMeasurementList and fill the EndTimeStamp. + // + if (((StartProgressId >=3D PERF_EVENTSIGNAL_START_ID && ((StartProgres= sId & 0x000F) =3D=3D 0)) || + (StartProgressId < PERF_EVENTSIGNAL_START_ID && ((StartProgressId = & 0x0001) !=3D 0)))) { + // + // Since PEIM and StartImage has same Type and ID when PCD PcdEdkiiF= pdtStringRecordEnableOnly =3D FALSE + // So we need to identify these two kinds of record through differen= t phase. + // + if (AsciiStrCmp (((DYNAMIC_STRING_EVENT_RECORD *)StartRecordEvent)->= String, ALit_PEI) =3D=3D 0) { + mPeiPhase =3D TRUE; + } else if (AsciiStrCmp (((DYNAMIC_STRING_EVENT_RECORD *)StartRecordE= vent)->String, ALit_DXE) =3D=3D 0) { + mDxePhase =3D TRUE; + mPeiPhase =3D FALSE; + } + // Get measurement info form the start record to the mMeasurementLis= t. + GetMeasurementInfo (RecordHeader, TRUE, &(mMeasurementList[mMeasurem= entNum])); + mMeasurementNum ++; + } else { + GetMeasurementInfo (RecordHeader, FALSE, &MeasureMent); + SearchMeasurement (&MeasureMent); + } + TableLength +=3D RecordHeader->Length; + PerformanceTablePtr +=3D RecordHeader->Length; + } + return EFI_SUCCESS; +} + +/** Initialize the cumulative data. =20 **/ VOID InitCumulativeData ( @@ -153,10 +707,11 @@ RunDp ( BOOLEAN CumulativeMode; CONST CHAR16 *CustomCumulativeToken; PERF_CUM_DATA *CustomCumulativeData; UINTN NameSize; SHELL_STATUS ShellStatus; + TIMER_INFO TimerInfo; =20 StringPtr =3D NULL; SummaryMode =3D FALSE; VerboseMode =3D FALSE; AllMode =3D FALSE; @@ -173,10 +728,40 @@ RunDp ( // Status =3D ShellInitialize(); ASSERT_EFI_ERROR(Status); =20 // + // DP dump performance data by parsing FPDT table in ACPI table. + // Folloing 3 steps are to get the measurement form the FPDT table. + // + + // + //1. Get FPDT from ACPI table. + // + Status =3D GetBootPerformanceTable (); + if (EFI_ERROR(Status)) { + ShellPrintHiiEx (-1, -1, NULL, STRING_TOKEN (STR_DP_GET_BOOT_PERFORMAN= CE_TABLE_FAIL), mDpHiiHandle); + return Status; + } + + // + //2. Cache the ModuleGuid and hanlde mapping table. + // + Status =3D BuildCachedGuidHandleTable(); + if (EFI_ERROR (Status)) { + return Status; + } + + // + //3. Build the measurement array form the FPDT records. + // + Status =3D BuildMeasurementList (); + if (EFI_ERROR(Status)) { + return Status; + } + + // // Process Command Line arguments // Status =3D ShellCommandLineParse (ParamList, &ParamPackage, NULL, TRUE); if (EFI_ERROR(Status)) { ShellPrintHiiEx (-1, -1, NULL, STRING_TOKEN (STR_DP_INVALID_ARG), mDpH= iiHandle); @@ -265,15 +850,13 @@ RunDp ( ShellPrintHiiEx (-1, -1, NULL, STRING_TOKEN (STR_PERF_PROPERTY_NOT_FOU= ND), mDpHiiHandle); goto Done; } =20 TimerInfo.Frequency =3D (UINT32)DivU64x32 (PerformanceProperty->Frequen= cy, 1000); - TimerInfo.StartCount =3D PerformanceProperty->TimerStartValue; - TimerInfo.EndCount =3D PerformanceProperty->TimerEndValue; - - // Determine in which direction the performance counter counts. - TimerInfo.CountUp =3D (BOOLEAN) (TimerInfo.EndCount >=3D TimerInfo.Start= Count); + TimerInfo.StartCount =3D 0; + TimerInfo.EndCount =3D 0xFFFF; + TimerInfo.CountUp =3D TRUE; =20 // // Print header // // print DP's build version @@ -383,10 +966,16 @@ Done: if (CustomCumulativeData !=3D NULL) { SHELL_FREE_NON_NULL (CustomCumulativeData->Name); } SHELL_FREE_NON_NULL (CustomCumulativeData); =20 + SHELL_FREE_NON_NULL (mMeasurementList); + + SHELL_FREE_NON_NULL (mCacheHandleGuidTable); + + mMeasurementNum =3D 0; + mCachePairCount =3D 0; return ShellStatus; } =20 =20 /** diff --git a/ShellPkg/DynamicCommand/DpDynamicCommand/Dp.h b/ShellPkg/Dynam= icCommand/DpDynamicCommand/Dp.h index 4027b6b..9fd8857 100644 --- a/ShellPkg/DynamicCommand/DpDynamicCommand/Dp.h +++ b/ShellPkg/DynamicCommand/DpDynamicCommand/Dp.h @@ -1,9 +1,9 @@ /** @file Header file for 'dp' command functions. =20 - Copyright (c) 2009 - 2017, Intel Corporation. All rights reserved. + Copyright (c) 2009 - 2018, 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 http://opensource.org/licenses/bsd-license.php =20 @@ -17,10 +17,13 @@ =20 =20 #include =20 #include +#include +#include +#include =20 #include #include #include #include @@ -40,11 +43,11 @@ #include =20 extern EFI_HANDLE mDpHiiHandle; =20 #define DP_MAJOR_VERSION 2 -#define DP_MINOR_VERSION 4 +#define DP_MINOR_VERSION 5 =20 /** * The value assigned to DP_DEBUG controls which debug output * is generated. Set it to ZERO to disable. **/ diff --git a/ShellPkg/DynamicCommand/DpDynamicCommand/Dp.uni b/ShellPkg/Dyn= amicCommand/DpDynamicCommand/Dp.uni index b77c507..b6069ae 100644 --- a/ShellPkg/DynamicCommand/DpDynamicCommand/Dp.uni +++ b/ShellPkg/DynamicCommand/DpDynamicCommand/Dp.uni @@ -1,9 +1,9 @@ // *++ // // (C) Copyright 2014-2015 Hewlett-Packard Development Company, L.P.
-// Copyright (c) 2009 - 2017, Intel Corporation. All rights reserved.
+// Copyright (c) 2009 - 2018, 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 // which accompanies this distribution. The full text of the license may = be found at // http://opensource.org/licenses/bsd-license.php @@ -49,14 +49,14 @@ #string STR_DP_HANDLE_SECTION #language en-US "Index: Handle = Driver Name Description Time(us)\n" #string STR_DP_HANDLE_VARS #language en-US "%5d: [%3x] %36= s %11s %L8d\n" #string STR_DP_HANDLE_SECTION2 #language en-US "Index: Handle = Driver Name Description Time(us) ID\n" #string STR_DP_HANDLE_VARS2 #language en-US "%5d: [%3x] %36s = %11s %L8d %5d\n" #string STR_DP_SECTION_PEIMS #language en-US "PEIMs" -#string STR_DP_PEIM_SECTION #language en-US "Index: Pointer V= alue Instance GUID Token Time(us)\n" -#string STR_DP_PEIM_VARS #language en-US "%5d: 0x%11p %g= PEIM %L8d\n" -#string STR_DP_PEIM_SECTION2 #language en-US "Index: Pointer Va= lue Instance GUID Token Time(us) ID\n" -#string STR_DP_PEIM_VARS2 #language en-US "%5d: 0x%11p %g P= EIM %L8d %5d\n" +#string STR_DP_PEIM_SECTION #language en-US "Index: = Instance GUID Token Time(us)\n" +#string STR_DP_PEIM_VARS #language en-US "%5d: %g PEIM= %L8d\n" +#string STR_DP_PEIM_SECTION2 #language en-US "Index: = Instance GUID Token Time(us) ID\n" +#string STR_DP_PEIM_VARS2 #language en-US "%5d: %g PEIM %= L8d %5d\n" #string STR_DP_SECTION_GENERAL #language en-US "General" #string STR_DP_GLOBAL_SECTION #language en-US "Index = Name Description Time(us)\n" #string STR_DP_GLOBAL_VARS #language en-US "%5d:%25s %31s= %L8d\n" #string STR_DP_GLOBAL_SECTION2 #language en-US "Index = Name Description Time(us) ID\n" #string STR_DP_GLOBAL_VARS2 #language en-US "%5d:%25s %31s %L= 8d %5d\n" @@ -88,10 +88,11 @@ #string STR_DP_RAW_VARS2 #language en-US "%5d: %16LX %16LX = %16LX %31a %31a %5d\n" #string STR_DP_RAW_HEADR2 #language en-US "\nIndex Han= dle Start Count End Count Token = Module ID\n" #string STR_DP_INCOMPLETE #language en-US " I " #string STR_DP_COMPLETE #language en-US " " #string STR_ALIT_UNKNOWN #language en-US "Unknown" +#string STR_DP_GET_BOOT_PERFORMANCE_TABLE_FAIL #language en-US "= Fail to get boot performance table\n" =20 #string STR_GET_HELP_DP #language en-US "" ".TH dp 0 "Display performance metrics"\r\n" ".SH NAME\r\n" "Displays performance metrics that are stored in memory.\r\n" diff --git a/ShellPkg/DynamicCommand/DpDynamicCommand/DpApp.inf b/ShellPkg/= DynamicCommand/DpDynamicCommand/DpApp.inf index b0ed229..54fe001 100644 --- a/ShellPkg/DynamicCommand/DpDynamicCommand/DpApp.inf +++ b/ShellPkg/DynamicCommand/DpDynamicCommand/DpApp.inf @@ -1,9 +1,9 @@ ## @file # Provides Shell 'dp' standalone application. # -# Copyright (c) 2009 - 2017, Intel Corporation. All rights reserved. +# Copyright (c) 2009 - 2018, 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 # which accompanies this distribution. The full text of the license may b= e found at # http://opensource.org/licenses/bsd-license.php # THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, @@ -23,11 +23,10 @@ # This flag specifies whether HII resource section is generated into PE i= mage. # UEFI_HII_RESOURCE_SECTION =3D TRUE =20 [Sources.common] - PerformanceTokens.h Dp.uni Dp.c Dp.h Literals.h Literals.c @@ -59,10 +58,12 @@ DxeServicesLib PeCoffGetEntryPointLib =20 [Guids] gPerformanceProtocolGuid ## CONSUMES ## S= ystemTable + gEfiAcpi20TableGuid ## CONSUMES ## S= ystemTable + gEfiAcpi10TableGuid ## CONSUMES ## S= ystemTable =20 [Protocols] gEfiLoadedImageProtocolGuid ## CONSUMES gEfiDriverBindingProtocolGuid ## SOMETIMES_CON= SUMES gEfiComponentName2ProtocolGuid ## SOMETIMES_CON= SUMES diff --git a/ShellPkg/DynamicCommand/DpDynamicCommand/DpDynamicCommand.inf = b/ShellPkg/DynamicCommand/DpDynamicCommand/DpDynamicCommand.inf index 3164561..e906870 100644 --- a/ShellPkg/DynamicCommand/DpDynamicCommand/DpDynamicCommand.inf +++ b/ShellPkg/DynamicCommand/DpDynamicCommand/DpDynamicCommand.inf @@ -1,9 +1,9 @@ ## @file # Provides Shell 'dp' dynamic command. # -# Copyright (c) 2009 - 2017, Intel Corporation. All rights reserved. +# Copyright (c) 2009 - 2018, 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 # which accompanies this distribution. The full text of the license may b= e found at # http://opensource.org/licenses/bsd-license.php # THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, @@ -24,11 +24,10 @@ # This flag specifies whether HII resource section is generated into PE i= mage. # UEFI_HII_RESOURCE_SECTION =3D TRUE =20 [Sources.common] - PerformanceTokens.h Dp.uni Dp.c Dp.h Literals.h Literals.c @@ -60,10 +59,12 @@ DxeServicesLib PeCoffGetEntryPointLib =20 [Guids] gPerformanceProtocolGuid ## CONSUMES ## S= ystemTable + gEfiAcpi20TableGuid ## CONSUMES + gEfiAcpi10TableGuid ## CONSUMES =20 [Protocols] gEfiLoadedImageProtocolGuid ## CONSUMES gEfiDriverBindingProtocolGuid ## SOMETIMES_CON= SUMES gEfiComponentName2ProtocolGuid ## SOMETIMES_CON= SUMES diff --git a/ShellPkg/DynamicCommand/DpDynamicCommand/DpInternal.h b/ShellP= kg/DynamicCommand/DpDynamicCommand/DpInternal.h index ece1c23..eedc377 100644 --- a/ShellPkg/DynamicCommand/DpDynamicCommand/DpInternal.h +++ b/ShellPkg/DynamicCommand/DpDynamicCommand/DpInternal.h @@ -4,11 +4,11 @@ Declarations of data and functions which are private to the Dp applicati= on. This file should never be referenced by anything other than components o= f the Dp application. In addition to global data, function declarations for DpUtilities.c, DpTrace.c, and DpProfile.c are included here. =20 - Copyright (c) 2009 - 2017, Intel Corporation. All rights reserved. + Copyright (c) 2009 - 2018, 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 which accompanies this distribution. The full text of the license may b= e found at http://opensource.org/licenses/bsd-license.php @@ -27,16 +27,17 @@ extern EFI_HII_HANDLE mDpHiiHandle; extern CHAR16 mGaugeString[DP_GAUGE_STRING_LENGTH + 1]; extern CHAR16 mUnicodeToken[DXE_PERFORMANCE_STRING_SIZE]; extern UINT64 mInterestThreshold; extern BOOLEAN mShowId; +extern UINT8 *mBootPerformanceTable; +extern UINTN mBootPerformanceTableLength; +extern MEASUREMENT_RECORD *mMeasurementList; +extern UINTN mMeasurementNum; =20 extern PERF_SUMMARY_DATA SummaryData; ///< Create the SummaryData stru= cture and init. to ZERO. =20 -/// Timer Specific Information. -extern TIMER_INFO TimerInfo; - /// Items for which to gather cumulative statistics. extern PERF_CUM_DATA CumData[]; =20 /// Number of items for which we are gathering cumulative statistics. extern UINT32 const NumCum; diff --git a/ShellPkg/DynamicCommand/DpDynamicCommand/DpTrace.c b/ShellPkg/= DynamicCommand/DpDynamicCommand/DpTrace.c index bc882be..46ee0f8 100644 --- a/ShellPkg/DynamicCommand/DpDynamicCommand/DpTrace.c +++ b/ShellPkg/DynamicCommand/DpDynamicCommand/DpTrace.c @@ -1,9 +1,9 @@ /** @file Trace reporting for the Dp utility. =20 - Copyright (c) 2009 - 2017, Intel Corporation. All rights reserved. + Copyright (c) 2009 - 2018, 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 which accompanies this distribution. The full text of the license may b= e found at http://opensource.org/licenses/bsd-license.php @@ -21,17 +21,65 @@ #include #include #include #include =20 -#include - #include "Dp.h" #include "Literals.h" #include "DpInternal.h" =20 /** + Attempts to retrieve a performance measurement log entry from the perfor= mance measurement log. + + + @param LogEntryKey On entry, the key of the performance mea= surement log entry to retrieve. + 0, then the first performance measuremen= t log entry is retrieved. + On exit, the key of the next performance= log entry. + @param Handle Pointer to environment specific context = used to identify the component + being measured. + @param Token Pointer to a Null-terminated ASCII strin= g that identifies the component + being measured. + @param Module Pointer to a Null-terminated ASCII strin= g that identifies the module + being measured. + @param StartTimeStamp Pointer to the 64-bit time stamp that wa= s recorded when the measurement + was started. + @param EndTimeStamp Pointer to the 64-bit time stamp that wa= s recorded when the measurement + was ended. + @param Identifier Pointer to the 32-bit identifier that wa= s recorded when the measurement + was ended. + + @return The key for the next performance log entry (in general case). + +**/ +UINTN +GetPerformanceMeasurementRecord ( + IN UINTN LogEntryKey, + OUT CONST VOID **Handle, + OUT CONST CHAR8 **Token, + OUT CONST CHAR8 **Module, + OUT UINT64 *StartTimeStamp, + OUT UINT64 *EndTimeStamp, + OUT UINT32 *Identifier + ) +{ + if (LogEntryKey =3D=3D mMeasurementNum) { + return 0; + } + + *Handle =3D (VOID *) (UINTN) mMeasurementList[LogEntryKey].Handl= e; + *Token =3D mMeasurementList[LogEntryKey].Token; + *Module =3D mMeasurementList[LogEntryKey].Module; + *StartTimeStamp =3D mMeasurementList[LogEntryKey].StartTimeStamp; + *EndTimeStamp =3D mMeasurementList[LogEntryKey].EndTimeStamp; + *Identifier =3D mMeasurementList[LogEntryKey].Identifier; + + LogEntryKey ++; + + return LogEntryKey; +} + +/** Collect verbose statistics about the logged performance measurements. =20 General Summary information for all Trace measurements is gathered and stored within the SummaryData structure. This information is both used internally by subsequent reporting functions, and displayed @@ -55,11 +103,11 @@ GatherStatistics( UINT64 Duration; UINTN LogEntryKey; INTN TIndex; =20 LogEntryKey =3D 0; - while ((LogEntryKey =3D GetPerformanceMeasurementEx ( + while ((LogEntryKey =3D GetPerformanceMeasurementRecord ( LogEntryKey, &Measurement.Handle, &Measurement.Token, &Measurement.Module, &Measurement.StartTimeStamp, @@ -77,11 +125,11 @@ GatherStatistics( } =20 if (IsPhase( &Measurement)) { ++SummaryData.NumSummary; // Count the number of major phases } - else { // !IsPhase(... + else { // !IsPhase if(Measurement.Handle =3D=3D NULL) { ++SummaryData.NumGlobal; } } =20 @@ -188,11 +236,11 @@ DumpAllTrace( =20 LogEntryKey =3D 0; Count =3D 0; Index =3D 0; while ( WITHIN_LIMIT(Count, Limit) && - ((LogEntryKey =3D GetPerformanceMeasurementEx ( + ((LogEntryKey =3D GetPerformanceMeasurementRecord ( LogEntryKey, &Measurement.Handle, &Measurement.Token, &Measurement.Module, &Measurement.StartTimeStamp, @@ -329,11 +377,11 @@ DumpRawTrace( =20 LogEntryKey =3D 0; Count =3D 0; Index =3D 0; while ( WITHIN_LIMIT(Count, Limit) && - ((LogEntryKey =3D GetPerformanceMeasurementEx ( + ((LogEntryKey =3D GetPerformanceMeasurementRecord ( LogEntryKey, &Measurement.Handle, &Measurement.Token, &Measurement.Module, &Measurement.StartTimeStamp, @@ -418,11 +466,11 @@ ProcessPhases( (StringPtr =3D=3D NULL) ? StringPtrUnknown : StringPtr); FreePool (StringPtr); FreePool (StringPtrUnknown); =20 LogEntryKey =3D 0; - while ((LogEntryKey =3D GetPerformanceMeasurementEx ( + while ((LogEntryKey =3D GetPerformanceMeasurementRecord ( LogEntryKey, &Measurement.Handle, &Measurement.Token, &Measurement.Module, &Measurement.StartTimeStamp, @@ -460,45 +508,33 @@ ProcessPhases( } =20 // print PEI phase duration time // if (PeiTime > 0) { - ElapsedTime =3D DivU64x32 ( - PeiTime, - (UINT32)TimerInfo.Frequency - ); + ElapsedTime =3D DivU64x32 (PeiTime, 1000000); Total +=3D ElapsedTime; ShellPrintHiiEx (-1, -1, NULL, STRING_TOKEN (STR_DP_PHASE_DURATION), m= DpHiiHandle, ALit_PEI, ElapsedTime); } =20 // print DXE phase duration time // if (DxeTime > 0) { - ElapsedTime =3D DivU64x32 ( - DxeTime, - (UINT32)TimerInfo.Frequency - ); + ElapsedTime =3D DivU64x32 (DxeTime, 1000000); Total +=3D ElapsedTime; ShellPrintHiiEx (-1, -1, NULL, STRING_TOKEN (STR_DP_PHASE_DURATION), m= DpHiiHandle, ALit_DXE, ElapsedTime); } =20 // print BDS phase duration time // if (BdsTime > 0) { - ElapsedTime =3D DivU64x32 ( - BdsTime, - (UINT32)TimerInfo.Frequency - ); + ElapsedTime =3D DivU64x32 (BdsTime, 1000000); Total +=3D ElapsedTime; ShellPrintHiiEx (-1, -1, NULL, STRING_TOKEN (STR_DP_PHASE_DURATION), m= DpHiiHandle, ALit_BDS, ElapsedTime); } =20 if (BdsTimeoutValue > 0) { - ElapsedTime =3D DivU64x32 ( - BdsTimeoutValue, - (UINT32)TimerInfo.Frequency - ); + ElapsedTime =3D DivU64x32 (BdsTimeoutValue, 1000000); ShellPrintHiiEx (-1, -1, NULL, STRING_TOKEN (STR_DP_PHASE_BDSTO), mDpH= iiHandle, ALit_BdsTO, ElapsedTime); } =20 ShellPrintHiiEx (-1, -1, NULL, STRING_TOKEN (STR_DP_TOTAL_DURATION), mDp= HiiHandle, Total); } @@ -552,11 +588,11 @@ ProcessHandles( } ShellPrintHiiEx (-1, -1, NULL, STRING_TOKEN (STR_DP_DASHES), mDpHiiHan= dle); =20 LogEntryKey =3D 0; Count =3D 0; - while ((LogEntryKey =3D GetPerformanceMeasurementEx ( + while ((LogEntryKey =3D GetPerformanceMeasurementRecord ( LogEntryKey, &Measurement.Handle, &Measurement.Token, &Measurement.Module, &Measurement.StartTimeStamp, @@ -655,11 +691,11 @@ ProcessPeims( ShellPrintHiiEx (-1, -1, NULL, STRING_TOKEN (STR_DP_PEIM_SECTION), mDp= HiiHandle); } ShellPrintHiiEx (-1, -1, NULL, STRING_TOKEN (STR_DP_DASHES), mDpHiiHandl= e); TIndex =3D 0; LogEntryKey =3D 0; - while ((LogEntryKey =3D GetPerformanceMeasurementEx ( + while ((LogEntryKey =3D GetPerformanceMeasurementRecord ( LogEntryKey, &Measurement.Handle, &Measurement.Token, &Measurement.Module, &Measurement.StartTimeStamp, @@ -678,19 +714,17 @@ ProcessPeims( if (ElapsedTime >=3D mInterestThreshold) { // PEIM FILE Handle is the start address of its FFS file that contai= ns its file guid. if (mShowId) { ShellPrintHiiEx (-1, -1, NULL, STRING_TOKEN (STR_DP_PEIM_VARS2), m= DpHiiHandle, TIndex, // 1 based, Which measurement record is being prin= ted - Measurement.Handle, // base address Measurement.Handle, // file guid ElapsedTime, Measurement.Identifier ); } else { ShellPrintHiiEx (-1, -1, NULL, STRING_TOKEN (STR_DP_PEIM_VARS), mD= pHiiHandle, TIndex, // 1 based, Which measurement record is being prin= ted - Measurement.Handle, // base address Measurement.Handle, // file guid ElapsedTime ); } } @@ -744,11 +778,11 @@ ProcessGlobal( ShellPrintHiiEx (-1, -1, NULL, STRING_TOKEN (STR_DP_DASHES), mDpHiiHandl= e); =20 Index =3D 1; LogEntryKey =3D 0; =20 - while ((LogEntryKey =3D GetPerformanceMeasurementEx ( + while ((LogEntryKey =3D GetPerformanceMeasurementRecord ( LogEntryKey, &Measurement.Handle, &Measurement.Token, &Measurement.Module, &Measurement.StartTimeStamp, diff --git a/ShellPkg/DynamicCommand/DpDynamicCommand/DpUtilities.c b/Shell= Pkg/DynamicCommand/DpDynamicCommand/DpUtilities.c index b98ec4b..3d082e9 100644 --- a/ShellPkg/DynamicCommand/DpDynamicCommand/DpUtilities.c +++ b/ShellPkg/DynamicCommand/DpDynamicCommand/DpUtilities.c @@ -1,9 +1,9 @@ /** @file Utility functions used by the Dp application. =20 - Copyright (c) 2009 - 2017, Intel Corporation. All rights reserved. + Copyright (c) 2009 - 2018, 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 which accompanies this distribution. The full text of the license may b= e found at http://opensource.org/licenses/bsd-license.php @@ -37,11 +37,11 @@ =20 #include "Dp.h" #include "Literals.h" #include "DpInternal.h" =20 -/**=20 +/** Calculate an event's duration in timer ticks. =20 Given the count direction and the event's start and end timer values, calculate the duration of the event in timer ticks. Information for the current measurement is pointed to by the parameter. @@ -69,35 +69,21 @@ GetDuration ( =20 if (Measurement->EndTimeStamp =3D=3D 0) { return 0; } =20 - // PERF_START macros are called with a value of 1 to indicate - // the beginning of time. So, adjust the start ticker value - // to the real beginning of time. - // Assumes no wraparound. Even then, there is a very low probability - // of having a valid StartTicker value of 1. - if (Measurement->StartTimeStamp =3D=3D 1) { - Measurement->StartTimeStamp =3D TimerInfo.StartCount; - } - if (TimerInfo.CountUp) { - Duration =3D Measurement->EndTimeStamp - Measurement->StartTimeStamp; - Error =3D (BOOLEAN)(Duration > Measurement->EndTimeStamp); - } - else { - Duration =3D Measurement->StartTimeStamp - Measurement->EndTimeStamp; - Error =3D (BOOLEAN)(Duration > Measurement->StartTimeStamp); - } + Duration =3D Measurement->EndTimeStamp - Measurement->StartTimeStamp; + Error =3D (BOOLEAN)(Duration > Measurement->EndTimeStamp); =20 if (Error) { DEBUG ((EFI_D_ERROR, ALit_TimerLibError)); Duration =3D 0; } return Duration; } =20 -/**=20 +/** Determine whether the Measurement record is for an EFI Phase. =20 The Token and Module members of the measurement record are checked. Module must be empty and Token must be one of SEC, PEI, DXE, BDS, or SHE= LL. =20 @@ -111,11 +97,11 @@ IsPhase( IN MEASUREMENT_RECORD *Measurement ) { BOOLEAN RetVal; =20 - RetVal =3D (BOOLEAN)( ( *Measurement->Module =3D=3D '\0') = && + RetVal =3D (BOOLEAN)( ((AsciiStrnCmp (Measurement->Token, ALit_SEC, PERF_TOKEN_LENGT= H) =3D=3D 0) || (AsciiStrnCmp (Measurement->Token, ALit_PEI, PERF_TOKEN_LENGT= H) =3D=3D 0) || (AsciiStrnCmp (Measurement->Token, ALit_DXE, PERF_TOKEN_LENGT= H) =3D=3D 0) || (AsciiStrnCmp (Measurement->Token, ALit_BDS, PERF_TOKEN_LENGT= H) =3D=3D 0)) ); @@ -376,14 +362,11 @@ DpGetNameFromHandle ( UINT64 DurationInMicroSeconds ( IN UINT64 Duration ) { - UINT64 Temp; - - Temp =3D MultU64x32 (Duration, 1000); - return DivU64x32 (Temp, TimerInfo.Frequency); + return DivU64x32 (Duration, 1000); } =20 /**=20 Get index of Measurement Record's match in the CumData array. =20 diff --git a/ShellPkg/DynamicCommand/DpDynamicCommand/Literals.c b/ShellPkg= /DynamicCommand/DpDynamicCommand/Literals.c index c1cddfb..69dfc57 100644 --- a/ShellPkg/DynamicCommand/DpDynamicCommand/Literals.c +++ b/ShellPkg/DynamicCommand/DpDynamicCommand/Literals.c @@ -1,22 +1,28 @@ /** @file Definitions of ASCII string literals used by DP. =20 - Copyright (c) 2009 - 2017, Intel Corporation. All rights reserved.
+ Copyright (c) 2009 - 2018, 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 http://opensource.org/licenses/bsd-license.php =20 THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMP= LIED. **/ -#include +#include =20 // ASCII String literals which probably don't need translation -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_PEI[] =3D PEI_TOK; -CHAR8 const ALit_BDS[] =3D BDS_TOK; -CHAR8 const ALit_BdsTO[] =3D "BdsTimeOut"; -CHAR8 const ALit_PEIM[] =3D "PEIM"; +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_PEI[] =3D PEI_TOK; +CHAR8 const ALit_BDS[] =3D BDS_TOK; +CHAR8 const ALit_START_IMAGE[] =3D START_IMAGE_TOK; +CHAR8 const ALit_LOAD_IMAGE[] =3D LOAD_IMAGE_TOK; +CHAR8 const ALit_DB_START[] =3D DRIVERBINDING_START_TOK; +CHAR8 const ALit_DB_SUPPORT[] =3D DRIVERBINDING_SUPPORT_TOK; +CHAR8 const ALit_DB_STOP[] =3D DRIVERBINDING_STOP_TOK; + +CHAR8 const ALit_BdsTO[] =3D "BdsTimeOut"; +CHAR8 const ALit_PEIM[] =3D "PEIM"; diff --git a/ShellPkg/DynamicCommand/DpDynamicCommand/Literals.h b/ShellPkg= /DynamicCommand/DpDynamicCommand/Literals.h index 8aec09c..8695b1e 100644 --- a/ShellPkg/DynamicCommand/DpDynamicCommand/Literals.h +++ b/ShellPkg/DynamicCommand/DpDynamicCommand/Literals.h @@ -1,9 +1,9 @@ /** @file Declarations of ASCII string literals used by DP. =20 - Copyright (c) 2009 - 2010, Intel Corporation. All rights reserved.
+ Copyright (c) 2009 - 2018, 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 http://opensource.org/licenses/bsd-license.php =20 @@ -18,9 +18,15 @@ extern CHAR8 const ALit_TimerLibError[]; extern CHAR8 const ALit_SEC[]; extern CHAR8 const ALit_DXE[]; extern CHAR8 const ALit_SHELL[]; extern CHAR8 const ALit_PEI[]; extern CHAR8 const ALit_BDS[]; +extern CHAR8 const ALit_PEIM[]; +extern CHAR8 const ALit_START_IMAGE[]; +extern CHAR8 const ALit_LOAD_IMAGE[]; +extern CHAR8 const ALit_DB_START[]; +extern CHAR8 const ALit_DB_SUPPORT[]; +extern CHAR8 const ALit_DB_STOP[]; extern CHAR8 const ALit_BdsTO[]; extern CHAR8 const ALit_PEIM[]; =20 #endif // _LITERALS_H_ diff --git a/ShellPkg/DynamicCommand/DpDynamicCommand/PerformanceTokens.h b= /ShellPkg/DynamicCommand/DpDynamicCommand/PerformanceTokens.h deleted file mode 100644 index bbbc48d..0000000 --- a/ShellPkg/DynamicCommand/DpDynamicCommand/PerformanceTokens.h +++ /dev/null @@ -1,28 +0,0 @@ -/** @file - ASCII String Literals with special meaning to Performance measurement an= d the Dp utility. - -Copyright (c) 2009 - 2010, 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 -which accompanies this distribution. The full text of the license may be = found at -http://opensource.org/licenses/bsd-license.php - -THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, -WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLI= ED. - -**/ - -#ifndef __PERFORMANCE_TOKENS_H__ -#define __PERFORMANCE_TOKENS_H__ - -#define SEC_TOK "SEC" ///< SEC Phase -#define DXE_TOK "DXE" ///< DEC Phase -#define SHELL_TOK "SHELL" ///< Shell Phase -#define PEI_TOK "PEI" ///< PEI Phase -#define BDS_TOK "BDS" ///< BDS Phase -#define DRIVERBINDING_START_TOK "DB:Start:" ///< Driver Bind= ing Start() function call -#define DRIVERBINDING_SUPPORT_TOK "DB:Support:" ///< Driver Bind= ing Support() function call -#define LOAD_IMAGE_TOK "LoadImage:" ///< Load a disp= atched module -#define START_IMAGE_TOK "StartImage:" ///< Dispatched = Modules Entry Point execution - -#endif // __PERFORMANCE_TOKENS_H__ --=20 1.9.5.msysgit.1 _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel