From: "Gao, Liming" <liming.gao@intel.com>
To: "Bi, Dandan" <dandan.bi@intel.com>,
"edk2-devel@lists.01.org" <edk2-devel@lists.01.org>
Cc: "Zeng, Star" <star.zeng@intel.com>
Subject: Re: [PATCH v5 0/8] Update EDKII Performance infrastructure based on ACPI FPDT table
Date: Wed, 7 Feb 2018 03:26:57 +0000 [thread overview]
Message-ID: <4A89E2EF3DFEDB4C8BFDE51014F606A14E1C8AAE@SHSMSX104.ccr.corp.intel.com> (raw)
In-Reply-To: <1517914607-9636-1-git-send-email-dandan.bi@intel.com>
Reviewed-by: Liming Gao <liming.gao@intel.com>
>-----Original Message-----
>From: Bi, Dandan
>Sent: Tuesday, February 06, 2018 6:57 PM
>To: edk2-devel@lists.01.org
>Cc: Gao, Liming <liming.gao@intel.com>; Zeng, Star <star.zeng@intel.com>
>Subject: [PATCH v5 0/8] Update EDKII Performance infrastructure based on
>ACPI FPDT table
>
>V5: Rename global variable LockInsertRecord to mLockInsertRecord.
>
>V4:
>a.Update the GUID for status code in DxeCorePerformanceLib and
>FirmwarePerformanceDxe.
>b. Add check for Insert FPDT record in DxeCorePerformanceLib to avoid re-
>entry case.
>
>V3:
>a. Add "FPDT_" prefix for related definitions in
>ExtendedFirmwarePerformance.h.
>b. Refine the code logic.
>
>V2:
>a. Update DxecorePerformanceLib/SmmCorePerformanceLib to report the
>boot performance table address instead of records contents.
>b. Update FirmwarePerformanceDxe/FirmwarePerformanceSmm to receive
>the address
>of performance records.
>
>This patch series also can be accessed at:
>https://github.com/dandanbi/edk2/tree/NewPerformanceInfrastructureV4
>
>These patches are to update EDKII performance infrastructure to log and
>dump
>the performance entry as FPDT record in ACPI FPDT table.This new
>infrastructure
>can support to dump performance data in UEFI Shell and OS both.
>(1)PeiPerformanceLib/DxeCorePerformanceLib/SmmCorePerformanceLib log
>the
>performance entry as FPDT record.
>(2)FirmwarePerformancePei/FirmwarePerformanceDxe/FirmwarePerforman
>ceSmm
>install the FPDT records to the ACPI table.
>(3)Update DP to dump the performance info from the FPDT records in
>FPDT table.
>
>Cc: Liming Gao <liming.gao@intel.com>
>Cc: Star Zeng <star.zeng@intel.com>
>Dandan Bi (7):
> MdeModulePkg/PeiPerformance:Updated to track FPDT record in PEI phase
> MdeModulePkg/DxeCorePerformanceLib:Track FPDT record in DXE phase
> MdeModulePkg/SmmCorePerformanceLib:Track FPDT record in SMM phase
> MdeModulePkg/FirmwarePerformancePei:Add FPDT records for S3 phase
> MdeModulePkg/FirmwarePerfDxe:Enhance for new pref infrastructure
> MdeModulePkg/FirmwarePerfSmm:Enhance for new pref infrastructure
> ShellPkg/Dp: Updated to dump perf log based on FPDT table
>
>Gao, Liming (1):
> MdeModulePkg:Add definitions for new Performance infrastructure
>
> .../Include/Guid/ExtendedFirmwarePerformance.h | 291 +++++
> MdeModulePkg/Include/Guid/FirmwarePerformance.h | 13 +-
> .../DxeCorePerformanceLib/DxeCorePerformanceLib.c | 1367
>+++++++++++++++-----
> .../DxeCorePerformanceLib.inf | 20 +-
> .../DxeCorePerformanceLibInternal.h | 17 +-
> .../Library/PeiPerformanceLib/PeiPerformanceLib.c | 567 +++++---
> .../PeiPerformanceLib/PeiPerformanceLib.inf | 14 +-
> .../SmmCorePerformanceLib/SmmCorePerformanceLib.c | 1068 ++++++++--
>-----
> .../SmmCorePerformanceLib.inf | 10 +-
> .../SmmCorePerformanceLibInternal.h | 11 +-
> MdeModulePkg/MdeModulePkg.dec | 11 +-
> MdeModulePkg/MdeModulePkg.uni | 8 +-
> .../FirmwarePerformanceDxe.c | 294 +----
> .../FirmwarePerformanceDxe.inf | 5 +-
> .../FirmwarePerformancePei.c | 59 +-
> .../FirmwarePerformancePei.inf | 5 +-
> .../FirmwarePerformanceSmm.c | 35 +-
> .../FirmwarePerformanceSmm.inf | 1 +
> 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 | 108 +-
> .../DynamicCommand/DpDynamicCommand/DpUtilities.c | 37 +-
> .../DynamicCommand/DpDynamicCommand/Literals.c | 24 +-
> .../DynamicCommand/DpDynamicCommand/Literals.h | 8 +-
> .../DpDynamicCommand/PerformanceTokens.h | 28 -
> 29 files changed, 3285 insertions(+), 1362 deletions(-)
> create mode 100644
>MdeModulePkg/Include/Guid/ExtendedFirmwarePerformance.h
> delete mode 100644
>ShellPkg/DynamicCommand/DpDynamicCommand/PerformanceTokens.h
>
>--
>1.9.5.msysgit.1
prev parent reply other threads:[~2018-02-07 3:21 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-02-06 10:56 [PATCH v5 0/8] Update EDKII Performance infrastructure based on ACPI FPDT table Dandan Bi
2018-02-06 10:56 ` [PATCH v5 1/8] MdeModulePkg:Add definitions for new Performance infrastructure Dandan Bi
2018-02-06 10:56 ` [PATCH v5 2/8] MdeModulePkg/PeiPerformance:Updated to track FPDT record in PEI phase Dandan Bi
2018-02-06 10:56 ` [PATCH v5 3/8] MdeModulePkg/DxeCorePerformanceLib:Track FPDT record in DXE phase Dandan Bi
2018-02-06 10:56 ` [PATCH v5 4/8] MdeModulePkg/SmmCorePerformanceLib:Track FPDT record in SMM phase Dandan Bi
2018-02-06 10:56 ` [PATCH v5 5/8] MdeModulePkg/FirmwarePerformancePei:Add FPDT records for S3 phase Dandan Bi
2018-02-06 10:56 ` [PATCH v5 6/8] MdeModulePkg/FirmwarePerfDxe:Enhance for new pref infrastructure Dandan Bi
2018-02-06 10:56 ` [PATCH v5 7/8] MdeModulePkg/FirmwarePerfSmm:Enhance " Dandan Bi
2018-02-06 10:56 ` [PATCH v5 8/8] ShellPkg/Dp: Updated to dump perf log based on FPDT table Dandan Bi
2018-02-07 3:26 ` Gao, Liming [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4A89E2EF3DFEDB4C8BFDE51014F606A14E1C8AAE@SHSMSX104.ccr.corp.intel.com \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox