Re: [PATCH v2 0/5] Export Dump CPU Context service

["Yao, Jiewen" <jiewen.yao@intel.com>]

Reviewed-by: jiewen.yao@intel.com

> -----Original Message-----
> From: Fan, Jeff
> Sent: Friday, April 7, 2017 9:18 AM
> To: edk2-devel@lists.01.org
> Cc: Yao, Jiewen <jiewen.yao@intel.com>; Kinney, Michael D
> <michael.d.kinney@intel.com>; Tian, Feng <feng.tian@intel.com>
> Subject: [PATCH v2 0/5] Export Dump CPU Context service
> 
> This serial of patches are:
> 1. Export PeCoffSerachImageBase() that could serach PE/COFF image base.
> 2. Export DumpCpuContext that could dump CPU context when exception
> happened.
> 
> https://bugzilla.tianocore.org/show_bug.cgi?id=242
> 
> v2:
>   Combine v1's patch 3-6 to v2's patch 3.
>   Combine v1's patch 7, 8 to v2's patch 4.
> 
> Cc: Jiewen Yao <jiewen.yao@intel.com>
> Cc: Michael Kinney <michael.d.kinney@intel.com>
> Cc: Feng Tian <feng.tian@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Jeff Fan <jeff.fan@intel.com>
> 
> Jeff Fan (5):
>   MdePkg/PeCoffGetEntryPointLib: Add PeCoffSerachImageBase()
>   MdeModulePkg/CpuExceptionHandlerLib: Add DumpCpuContext()
>   UefiCpuPkg/CpuExceptionHandlerLib: Add DumpCpuContext()
> implementation
>   UefiCpuPkg/PiSmmCpuDxeSmm: Consume new APIs
>   SourceLevelDebugPkg/DebugAgent.c: Consume PeCoffSerachImageBase()
> 
>  .../Include/Library/CpuExceptionHandlerLib.h       | 15 +++-
>  .../CpuExceptionHandlerLibNull.c                   | 16 ++++-
>  MdePkg/Include/Library/PeCoffGetEntryPointLib.h    | 20 +++++-
>  .../PeCoffGetEntryPoint.c                          | 72
> ++++++++++++++++++-
>  .../DebugAgent/DebugAgentCommon/DebugAgent.c       | 50 ++-----------
>  .../CpuExceptionHandlerLib/CpuExceptionCommon.c    | 82
> ++++++----------------
>  .../CpuExceptionHandlerLib/CpuExceptionCommon.h    | 27 ++++---
>  .../Library/CpuExceptionHandlerLib/DxeException.c  |  7 +-
>  .../Ia32/ArchExceptionHandler.c                    | 65 ++++++++++-------
>  .../CpuExceptionHandlerLib/PeiCpuException.c       |  6 +-
>  .../CpuExceptionHandlerLib/PeiDxeSmmCpuException.c |  4 +-
>  .../CpuExceptionHandlerLib/SecPeiCpuException.c    |  8 +--
>  .../Library/CpuExceptionHandlerLib/SmmException.c  |  7 +-
>  .../X64/ArchExceptionHandler.c                     | 59 ++++++++++------
>  UefiCpuPkg/PiSmmCpuDxeSmm/Ia32/PageTbl.c           | 18 ++---
>  UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.c         | 37 +---------
>  UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.h         |  4 +-
>  UefiCpuPkg/PiSmmCpuDxeSmm/SmmProfileInternal.h     |  6 +-
>  UefiCpuPkg/PiSmmCpuDxeSmm/X64/PageTbl.c            | 18 ++---
>  19 files changed, 268 insertions(+), 253 deletions(-)
> 
> --
> 2.9.3.windows.2