From: Zhichao Gao <zhichao.gao@intel.com>
To: edk2-devel@lists.01.org
Cc: Michael D Kinney <michael.d.kinney@intel.com>,
Leif Lindholm <leif.lindholm@linaro.org>,
Ard Biesheuvel <ard.biesheuvel@linaro.org>,
Jordan Justen <jordan.l.justen@intel.com>,
Laszlo Ersek <lersek@redhat.com>,
Chasel Chiu <chasel.chiu@intel.com>,
Nate DeSimone <nathaniel.l.desimone@intel.com>,
Star Zeng <star.zeng@intel.com>,
Jian J Wang <jian.j.wang@intel.com>, Hao Wu <hao.a.wu@intel.com>,
Ray Ni <ray.ni@intel.com>, Liming Gao <liming.gao@intel.com>,
Sean Brogan <sean.brogan@microsoft.com>,
Michael Turner <Michael.Turner@microsoft.com>,
Bret Barkelew <Bret.Barkelew@microsoft.com>
Subject: [PATCH V5 00/17] Add new APIs DebugVPrint for DebugLib
Date: Thu, 28 Mar 2019 16:09:59 +0800 [thread overview]
Message-ID: <20190328081016.14564-1-zhichao.gao@intel.com> (raw)
Add a new API DebugVPrint to all the instances of DebugLib.
This API is added to provide a function who want to implement
special debug function with '...' parameter.
Add a PEIM to install gEdkiiDebugPpiGuid, and implement a PEI
debug library instance base on it. All PEIMs except pei core
type can use the PeiDebugLibDebugPpi to reduce its image size.
V2:
Remove redundant code in DebugPrint.
Fix some coding sytle issues.
Remove some unenforced descirption in the comments of DebugVPrint.
V3:
Add the new API DebugBPrint, it is more useful for consumers which
are care of compatible issue.
Change the interface in gEdkiiDebugPpiGuid. VA_LIST is inappropriate
to appeared in protocol or ppi because different compilers compile
it to different type. It may be a pointer or a structure.
V4:
Sync the implement on MdeModulePkg/PeiDxeDebugLibReportStatusCode to
IntelFramworkModulePkg/PeiDxeDebugLibReportStatusCode. While the format
string is too long just truncate it instead of return.
Fix and update some functions' comments.
V5:
Correct some comments of inf file.
Put all implement into one C file for DebugServicePei.
Use the directly return instead of CpuDeadLoop in PeiDebugLibDebugPpi.
Rename the mDebugPpi to mEdkiiDebugPpi because it has a same name in DeubServicePpi.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Zhichao Gao <zhichao.gao@intel.com>
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Leif Lindholm <leif.lindholm@linaro.org>
Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Cc: Jordan Justen <jordan.l.justen@intel.com>
Cc: Laszlo Ersek <lersek@redhat.com>
Cc: Chasel Chiu <chasel.chiu@intel.com>
Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
Cc: Star Zeng <star.zeng@intel.com>
Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
Cc: Jian J Wang <jian.j.wang@intel.com>
Cc: Hao Wu <hao.a.wu@intel.com>
Cc: Ray Ni <ray.ni@intel.com>
Cc: Liming Gao <liming.gao@intel.com>
Cc: Sean Brogan <sean.brogan@microsoft.com>
Cc: Michael Turner <Michael.Turner@microsoft.com>
Cc: Bret Barkelew <Bret.Barkelew@microsoft.com>
Bret Barkelew (13):
MdePkg/DebugLib.h: Add new APIs for DebugLib
MdePkg/BaseDebugLibNull: Add new APIs for DebugLib
MdePkg/BaseDebugLibSerialPort: Add new APIs
MdePkg/UefidebugLibConOut: Add new APIs
MdePkg/UefiDebugLibStdErr: Add new APIs
MdePkg/DxeRuntimeDebugLibSerialPort: Add new APIs
MdePkg/UefiDebuglibDebugPortProtocol: Add new APIs
ArmPkg/SemiHostingDebugLib: Add new APIs
OvmfPkg/PlatformDebugLibIoPort: Add new APIs
IntelFsp2Pkg/BaseFspDebugLibSerialPort: Add new APIs
IntelFspPkg/BaseFspDebugLibSerialPort: Add new APIs
IntelFramworkModulePkg/PeiDxeDebugLibReportStatusCode: Add new APIs
MdeModulePkg/PeiDxeDebugLibReportStatusCode: Add new APIs
Liming Gao (1):
MdeModulePkg/PeiDebugLibDebugPpi: Add PEI debug lib
Zhichao Gao (3):
MdeModulePkg: Add definitions for EDKII DEBUG PPI
MdeModulePkg: Add a PEIM to install Debug PPI
MdeModulePkg: Add PEIM and lib to dsc file
ArmPkg/Library/SemiHostingDebugLib/DebugLib.c | 106 +++-
.../PeiDxeDebugLibReportStatusCode/DebugLib.c | 179 +++++--
.../BaseFspDebugLibSerialPort/DebugLib.c | 103 +++-
.../BaseFspDebugLibSerialPort/DebugLib.c | 103 +++-
MdeModulePkg/Include/Ppi/Debug.h | 82 ++++
.../Library/PeiDebugLibDebugPpi/DebugLib.c | 456 ++++++++++++++++++
.../PeiDebugLibDebugPpi.inf | 56 +++
.../PeiDxeDebugLibReportStatusCode/DebugLib.c | 177 +++++--
MdeModulePkg/MdeModulePkg.dec | 3 +
MdeModulePkg/MdeModulePkg.dsc | 3 +
.../Universal/DebugServicePei/DebugService.h | 56 +++
.../DebugServicePei/DebugServicePei.c | 100 ++++
.../DebugServicePei/DebugServicePei.inf | 51 ++
.../DebugServicePei/DebugServicePei.uni | 20 +
MdePkg/Include/Library/DebugLib.h | 52 +-
MdePkg/Library/BaseDebugLibNull/DebugLib.c | 56 ++-
.../Library/BaseDebugLibSerialPort/DebugLib.c | 106 +++-
.../DxeRuntimeDebugLibSerialPort/DebugLib.c | 106 +++-
MdePkg/Library/UefiDebugLibConOut/DebugLib.c | 106 +++-
.../UefiDebugLibDebugPortProtocol/DebugLib.c | 106 +++-
MdePkg/Library/UefiDebugLibStdErr/DebugLib.c | 107 +++-
.../Library/PlatformDebugLibIoPort/DebugLib.c | 106 +++-
22 files changed, 2132 insertions(+), 108 deletions(-)
create mode 100644 MdeModulePkg/Include/Ppi/Debug.h
create mode 100644 MdeModulePkg/Library/PeiDebugLibDebugPpi/DebugLib.c
create mode 100644 MdeModulePkg/Library/PeiDebugLibDebugPpi/PeiDebugLibDebugPpi.inf
create mode 100644 MdeModulePkg/Universal/DebugServicePei/DebugService.h
create mode 100644 MdeModulePkg/Universal/DebugServicePei/DebugServicePei.c
create mode 100644 MdeModulePkg/Universal/DebugServicePei/DebugServicePei.inf
create mode 100644 MdeModulePkg/Universal/DebugServicePei/DebugServicePei.uni
--
2.21.0.windows.1
next reply other threads:[~2019-03-28 8:10 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-03-28 8:09 Zhichao Gao [this message]
2019-03-28 8:10 ` [PATCH V5 01/17] MdePkg/DebugLib.h: Add new APIs for DebugLib Zhichao Gao
2019-03-28 8:10 ` [PATCH V5 02/17] MdePkg/BaseDebugLibNull: " Zhichao Gao
2019-03-28 8:10 ` [PATCH V5 03/17] MdePkg/BaseDebugLibSerialPort: Add new APIs Zhichao Gao
2019-03-28 8:10 ` [PATCH V5 04/17] MdePkg/UefidebugLibConOut: " Zhichao Gao
2019-03-28 8:10 ` [PATCH V5 05/17] MdePkg/UefiDebugLibStdErr: " Zhichao Gao
2019-03-28 8:10 ` [PATCH V5 06/17] MdePkg/DxeRuntimeDebugLibSerialPort: " Zhichao Gao
2019-03-28 8:10 ` [PATCH V5 07/17] MdePkg/UefiDebuglibDebugPortProtocol: " Zhichao Gao
2019-03-28 8:10 ` [PATCH V5 08/17] ArmPkg/SemiHostingDebugLib: " Zhichao Gao
2019-03-28 8:10 ` [PATCH V5 09/17] OvmfPkg/PlatformDebugLibIoPort: " Zhichao Gao
2019-03-28 8:10 ` [PATCH V5 10/17] IntelFsp2Pkg/BaseFspDebugLibSerialPort: " Zhichao Gao
2019-03-28 8:44 ` Chiu, Chasel
2019-03-28 8:10 ` [PATCH V5 11/17] IntelFspPkg/BaseFspDebugLibSerialPort: " Zhichao Gao
2019-03-28 8:44 ` Chiu, Chasel
2019-03-28 8:10 ` [PATCH V5 12/17] IntelFramworkModulePkg/PeiDxeDebugLibReportStatusCode: " Zhichao Gao
2019-03-28 8:10 ` [PATCH V5 13/17] MdeModulePkg/PeiDxeDebugLibReportStatusCode: " Zhichao Gao
2019-03-28 8:10 ` [PATCH V5 14/17] MdeModulePkg: Add definitions for EDKII DEBUG PPI Zhichao Gao
2019-03-28 10:13 ` Ni, Ray
2019-03-29 0:11 ` Gao, Zhichao
2019-03-28 8:10 ` [PATCH V5 15/17] MdeModulePkg: Add a PEIM to install Debug PPI Zhichao Gao
2019-03-28 8:10 ` [PATCH V5 16/17] MdeModulePkg/PeiDebugLibDebugPpi: Add PEI debug lib Zhichao Gao
2019-03-28 10:10 ` Ni, Ray
2019-03-29 5:10 ` Gao, Zhichao
2019-03-28 8:10 ` [PATCH V5 17/17] MdeModulePkg: Add PEIM and lib to dsc file Zhichao Gao
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=20190328081016.14564-1-zhichao.gao@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