public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* Re: [edk2-devel] [PATCH v3 00/26] Enable Core CI support for StandaloneMmPkg
       [not found] <16570A25664A9CD5.11073@groups.io>
@ 2021-01-04 13:43 ` Sami Mujawar
  2021-01-07  1:34   ` 回复: " gaoliming
  0 siblings, 1 reply; 2+ messages in thread
From: Sami Mujawar @ 2021-01-04 13:43 UTC (permalink / raw)
  To: devel@edk2.groups.io, Liming Gao
  Cc: Ard Biesheuvel, jiewen.yao@intel.com, Thomas Abraham,
	Sughosh Ganu, Matteo Carlini, Ben Adderson, nd

Adding Liming.
Ref: https://edk2.groups.io/g/devel/topic/edk2_devel_patch/79194663?p=,,,20,0,0,0::recentpostdate%2Fsticky,,,20,2,20,79194663

-----Original Message-----
From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Sami Mujawar via groups.io
Sent: 04 January 2021 01:22 PM
To: devel@edk2.groups.io
Cc: Sami Mujawar <Sami.Mujawar@arm.com>; Ard Biesheuvel <Ard.Biesheuvel@arm.com>; jiewen.yao@intel.com; Thomas Abraham <thomas.abraham@arm.com>; Sughosh Ganu <Sughosh.Ganu@arm.com>; Matteo Carlini <Matteo.Carlini@arm.com>; Ben Adderson <Ben.Adderson@arm.com>; nd <nd@arm.com>
Subject: [edk2-devel] [PATCH v3 00/26] Enable Core CI support for StandaloneMmPkg

Bugzilla: 3150 (https://bugzilla.tianocore.org/show_bug.cgi?id=3150)

The EDKII Core CI runs the ECC tool that checks for good coding
practices and the coding standard. This patch series fixes the
ECC errors reported in the StandaloneMmPkg and enables support
for EDKII Core CI.

This v3 series:
 - PATCH 26: Updated readme.md file to add check mark to the
   windows column.
 - Added bugzilla ID to the commit messages for the patches in
   the series.
   Ref: https://edk2.groups.io/g/devel/topic/edk2_devel_patch/79194663
 - There is no code change for Patches 1 to 25 since the v2 series.

The changes can be seen at:
https://github.com/samimujawar/edk2/tree/1516_standalonemmpkg_ci_v3

Sami Mujawar (26):
  StandaloneMmPkg: Add library header files to DEC
  StandaloneMmPkg: Add library files to DSC
  StandaloneMmPkg: Fix spell check reported errors
  StandaloneMmPkg: Fix ECC error 9002 in Core dispatcher
  StandaloneMmPkg: Fix ECC error 1001 in SMM Memory pool management
  StandaloneMmPkg: Fix ECC error 3002 in StandaloneMmCpu
  StandaloneMmPkg: Fix ECC error 4002 in FwVol helper
  StandaloneMmPkg: Fix ECC error 5007 in StandaloneMmCoreEntryPoint
  StandaloneMmPkg: Fix ECC error 5007 in StandaloneMmCpu
  StandaloneMmPkg: Fix ECC error 5007 in StandaloneMmCore
  StandaloneMmPkg: Fix ECC error 10014 in StandaloneMmCpu
  StandaloneMmPkg: Fix ECC error 4002 and 9002 in StandaloneMmCore
  StandaloneMmPkg: Fix ECC error 4002 in StandaloneMmCoreEntryPoint
  StandaloneMmPkg: Fix ECC error 9002 in StandaloneMmMemLib
  StandaloneMmPkg: Fix ECC error 9002 in StandaloneMmCoreEntryPoint
  StandaloneMmPkg: Fix ECC error 9003 in StandaloneMmCoreEntryPoint
  StandaloneMmPkg: Fix ECC error 4002 in StandaloneMmCpu
  StandaloneMmPkg: Fix ECC error 9001 in StandaloneMmCpu
  StandaloneMmPkg: Fix ECC error 9001 in Standalone MM Core
  StandaloneMmPkg: Fix ECC error 9002 in CoreMemoryAllocationLib
  StandaloneMmPkg: Fix ECC error 4002 in StandaloneMmCpu
  StandaloneMmPkg: Fix ECC error 8005 in StandaloneMmCoreEntryPoint
  StandaloneMmPkg: Remove dependency on ArmPlatformPkg.dec
  StandaloneMmPkg: Add EDK2 Core CI support
  .azurepipelines: Add StandaloneMmPkg to CI matrix
  .pytool: CI Settings to support StandaloneMmPkg

 .azurepipelines/templates/pr-gate-build-job.yml                                                     |  2 +-
 .pytool/CISettings.py                                                                               |  1 +
 .pytool/Readme.md                                                                                   |  2 +-
 StandaloneMmPkg/Core/Dependency.c                                                                   |  6 +-
 StandaloneMmPkg/Core/Dispatcher.c                                                                   | 49 ++++++------
 StandaloneMmPkg/Core/FwVol.c                                                                        | 36 +++++----
 StandaloneMmPkg/Core/Locate.c                                                                       |  6 +-
 StandaloneMmPkg/Core/Mmi.c                                                                          |  4 +-
 StandaloneMmPkg/Core/Pool.c                                                                         |  6 +-
 StandaloneMmPkg/Core/StandaloneMmCore.c                                                             | 37 ++++++---
 StandaloneMmPkg/Core/StandaloneMmCore.h                                                             | 12 +--
 StandaloneMmPkg/Drivers/StandaloneMmCpu/AArch64/EventHandle.c                                       | 56 ++++++++++---
 StandaloneMmPkg/Drivers/StandaloneMmCpu/AArch64/StandaloneMmCpu.c                                   | 34 ++++++--
 StandaloneMmPkg/Drivers/StandaloneMmCpu/AArch64/StandaloneMmCpu.h                                   | 31 +++++++-
 StandaloneMmPkg/Drivers/StandaloneMmCpu/AArch64/StandaloneMmCpu.inf                                 |  9 +--
 StandaloneMmPkg/Include/Guid/MmCoreData.h                                                           |  6 +-
 StandaloneMmPkg/Include/Library/AArch64/StandaloneMmCoreEntryPoint.h                                | 47 ++++++-----
 StandaloneMmPkg/Include/Library/StandaloneMmDriverEntryPoint.h                                      |  4 +-
 StandaloneMmPkg/Include/Library/StandaloneMmMemLib.h                                                |  6 +-
 StandaloneMmPkg/Library/StandaloneMmCoreEntryPoint/AArch64/CreateHobList.c                          |  6 +-
 StandaloneMmPkg/Library/StandaloneMmCoreEntryPoint/AArch64/SetPermissions.c                         | 53 +++++++++++-
 StandaloneMmPkg/Library/StandaloneMmCoreEntryPoint/AArch64/StandaloneMmCoreEntryPoint.c             | 32 +++++---
 StandaloneMmPkg/Library/StandaloneMmCoreEntryPoint/StandaloneMmCoreEntryPoint.inf                   |  3 +-
 StandaloneMmPkg/Library/StandaloneMmCoreMemoryAllocationLib/StandaloneMmCoreMemoryAllocationLib.c   |  6 +-
 StandaloneMmPkg/Library/StandaloneMmCoreMemoryAllocationLib/StandaloneMmCoreMemoryAllocationLib.inf |  4 +-
 StandaloneMmPkg/Library/StandaloneMmMemLib/StandaloneMmMemLib.c                                     | 10 +--
 StandaloneMmPkg/Library/VariableMmDependency/VariableMmDependency.c                                 |  4 +-
 StandaloneMmPkg/Library/VariableMmDependency/VariableMmDependency.inf                               |  4 +-
 StandaloneMmPkg/StandaloneMmPkg.ci.yaml                                                             | 84 ++++++++++++++++++++
 StandaloneMmPkg/StandaloneMmPkg.dec                                                                 | 20 ++++-
 StandaloneMmPkg/StandaloneMmPkg.dsc                                                                 | 15 +++-
 31 files changed, 439 insertions(+), 156 deletions(-)
 create mode 100644 StandaloneMmPkg/StandaloneMmPkg.ci.yaml

-- 
'Guid(CE165669-3EF3-493F-B85D-6190EE5B9759)'







^ permalink raw reply	[flat|nested] 2+ messages in thread

* 回复: [edk2-devel] [PATCH v3 00/26] Enable Core CI support for StandaloneMmPkg
  2021-01-04 13:43 ` [edk2-devel] [PATCH v3 00/26] Enable Core CI support for StandaloneMmPkg Sami Mujawar
@ 2021-01-07  1:34   ` gaoliming
  0 siblings, 0 replies; 2+ messages in thread
From: gaoliming @ 2021-01-07  1:34 UTC (permalink / raw)
  To: devel, sami.mujawar
  Cc: 'Ard Biesheuvel', jiewen.yao, 'Thomas Abraham',
	'Sughosh Ganu', 'Matteo Carlini',
	'Ben Adderson', 'nd'

Reviewed-by: Liming Gao <gaoliming@byosoft.com.cn>

> -----邮件原件-----
> 发件人: bounce+27952+69582+4905953+8761045@groups.io
> <bounce+27952+69582+4905953+8761045@groups.io> 代表 Sami
> Mujawar
> 发送时间: 2021年1月4日 21:44
> 收件人: devel@edk2.groups.io; Liming Gao <gaoliming@byosoft.com.cn>
> 抄送: Ard Biesheuvel <Ard.Biesheuvel@arm.com>; jiewen.yao@intel.com;
> Thomas Abraham <thomas.abraham@arm.com>; Sughosh Ganu
> <Sughosh.Ganu@arm.com>; Matteo Carlini <Matteo.Carlini@arm.com>; Ben
> Adderson <Ben.Adderson@arm.com>; nd <nd@arm.com>
> 主题: Re: [edk2-devel] [PATCH v3 00/26] Enable Core CI support for
> StandaloneMmPkg
> 
> Adding Liming.
> Ref:
> https://edk2.groups.io/g/devel/topic/edk2_devel_patch/79194663?p=,,,20,0,
> 0,0::recentpostdate%2Fsticky,,,20,2,20,79194663
> 
> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Sami
> Mujawar via groups.io
> Sent: 04 January 2021 01:22 PM
> To: devel@edk2.groups.io
> Cc: Sami Mujawar <Sami.Mujawar@arm.com>; Ard Biesheuvel
> <Ard.Biesheuvel@arm.com>; jiewen.yao@intel.com; Thomas Abraham
> <thomas.abraham@arm.com>; Sughosh Ganu <Sughosh.Ganu@arm.com>;
> Matteo Carlini <Matteo.Carlini@arm.com>; Ben Adderson
> <Ben.Adderson@arm.com>; nd <nd@arm.com>
> Subject: [edk2-devel] [PATCH v3 00/26] Enable Core CI support for
> StandaloneMmPkg
> 
> Bugzilla: 3150 (https://bugzilla.tianocore.org/show_bug.cgi?id=3150)
> 
> The EDKII Core CI runs the ECC tool that checks for good coding
> practices and the coding standard. This patch series fixes the
> ECC errors reported in the StandaloneMmPkg and enables support
> for EDKII Core CI.
> 
> This v3 series:
>  - PATCH 26: Updated readme.md file to add check mark to the
>    windows column.
>  - Added bugzilla ID to the commit messages for the patches in
>    the series.
>    Ref: https://edk2.groups.io/g/devel/topic/edk2_devel_patch/79194663
>  - There is no code change for Patches 1 to 25 since the v2 series.
> 
> The changes can be seen at:
> https://github.com/samimujawar/edk2/tree/1516_standalonemmpkg_ci_v3
> 
> Sami Mujawar (26):
>   StandaloneMmPkg: Add library header files to DEC
>   StandaloneMmPkg: Add library files to DSC
>   StandaloneMmPkg: Fix spell check reported errors
>   StandaloneMmPkg: Fix ECC error 9002 in Core dispatcher
>   StandaloneMmPkg: Fix ECC error 1001 in SMM Memory pool management
>   StandaloneMmPkg: Fix ECC error 3002 in StandaloneMmCpu
>   StandaloneMmPkg: Fix ECC error 4002 in FwVol helper
>   StandaloneMmPkg: Fix ECC error 5007 in StandaloneMmCoreEntryPoint
>   StandaloneMmPkg: Fix ECC error 5007 in StandaloneMmCpu
>   StandaloneMmPkg: Fix ECC error 5007 in StandaloneMmCore
>   StandaloneMmPkg: Fix ECC error 10014 in StandaloneMmCpu
>   StandaloneMmPkg: Fix ECC error 4002 and 9002 in StandaloneMmCore
>   StandaloneMmPkg: Fix ECC error 4002 in StandaloneMmCoreEntryPoint
>   StandaloneMmPkg: Fix ECC error 9002 in StandaloneMmMemLib
>   StandaloneMmPkg: Fix ECC error 9002 in StandaloneMmCoreEntryPoint
>   StandaloneMmPkg: Fix ECC error 9003 in StandaloneMmCoreEntryPoint
>   StandaloneMmPkg: Fix ECC error 4002 in StandaloneMmCpu
>   StandaloneMmPkg: Fix ECC error 9001 in StandaloneMmCpu
>   StandaloneMmPkg: Fix ECC error 9001 in Standalone MM Core
>   StandaloneMmPkg: Fix ECC error 9002 in CoreMemoryAllocationLib
>   StandaloneMmPkg: Fix ECC error 4002 in StandaloneMmCpu
>   StandaloneMmPkg: Fix ECC error 8005 in StandaloneMmCoreEntryPoint
>   StandaloneMmPkg: Remove dependency on ArmPlatformPkg.dec
>   StandaloneMmPkg: Add EDK2 Core CI support
>   .azurepipelines: Add StandaloneMmPkg to CI matrix
>   .pytool: CI Settings to support StandaloneMmPkg
> 
>  .azurepipelines/templates/pr-gate-build-job.yml
> |  2 +-
>  .pytool/CISettings.py
> |  1 +
>  .pytool/Readme.md
> |  2 +-
>  StandaloneMmPkg/Core/Dependency.c
> |  6 +-
>  StandaloneMmPkg/Core/Dispatcher.c
> | 49 ++++++------
>  StandaloneMmPkg/Core/FwVol.c
> | 36 +++++----
>  StandaloneMmPkg/Core/Locate.c
> |  6 +-
>  StandaloneMmPkg/Core/Mmi.c
> |  4 +-
>  StandaloneMmPkg/Core/Pool.c
> |  6 +-
>  StandaloneMmPkg/Core/StandaloneMmCore.c
> | 37 ++++++---
>  StandaloneMmPkg/Core/StandaloneMmCore.h
> | 12 +--
>  StandaloneMmPkg/Drivers/StandaloneMmCpu/AArch64/EventHandle.c
> | 56 ++++++++++---
> 
> StandaloneMmPkg/Drivers/StandaloneMmCpu/AArch64/StandaloneMmCpu.
> c                                   | 34 ++++++--
> 
> StandaloneMmPkg/Drivers/StandaloneMmCpu/AArch64/StandaloneMmCpu.
> h                                   | 31 +++++++-
> 
> StandaloneMmPkg/Drivers/StandaloneMmCpu/AArch64/StandaloneMmCpu.i
> nf                                 |  9 +--
>  StandaloneMmPkg/Include/Guid/MmCoreData.h
> |  6 +-
> 
> StandaloneMmPkg/Include/Library/AArch64/StandaloneMmCoreEntryPoint.h
> | 47 ++++++-----
>  StandaloneMmPkg/Include/Library/StandaloneMmDriverEntryPoint.h
> |  4 +-
>  StandaloneMmPkg/Include/Library/StandaloneMmMemLib.h
> |  6 +-
> 
> StandaloneMmPkg/Library/StandaloneMmCoreEntryPoint/AArch64/CreateHo
> bList.c                          |  6 +-
> 
> StandaloneMmPkg/Library/StandaloneMmCoreEntryPoint/AArch64/SetPermi
> ssions.c                         | 53 +++++++++++-
> 
> StandaloneMmPkg/Library/StandaloneMmCoreEntryPoint/AArch64/Standalo
> neMmCoreEntryPoint.c             | 32 +++++---
> 
> StandaloneMmPkg/Library/StandaloneMmCoreEntryPoint/StandaloneMmCor
> eEntryPoint.inf                   |  3 +-
> 
> StandaloneMmPkg/Library/StandaloneMmCoreMemoryAllocationLib/Standal
> oneMmCoreMemoryAllocationLib.c   |  6 +-
> 
> StandaloneMmPkg/Library/StandaloneMmCoreMemoryAllocationLib/Standal
> oneMmCoreMemoryAllocationLib.inf |  4 +-
> 
> StandaloneMmPkg/Library/StandaloneMmMemLib/StandaloneMmMemLib.c
> | 10 +--
> 
> StandaloneMmPkg/Library/VariableMmDependency/VariableMmDependenc
> y.c                                 |  4 +-
> 
> StandaloneMmPkg/Library/VariableMmDependency/VariableMmDependenc
> y.inf                               |  4 +-
>  StandaloneMmPkg/StandaloneMmPkg.ci.yaml
> | 84 ++++++++++++++++++++
>  StandaloneMmPkg/StandaloneMmPkg.dec
> | 20 ++++-
>  StandaloneMmPkg/StandaloneMmPkg.dsc
> | 15 +++-
>  31 files changed, 439 insertions(+), 156 deletions(-)
>  create mode 100644 StandaloneMmPkg/StandaloneMmPkg.ci.yaml
> 
> --
> 'Guid(CE165669-3EF3-493F-B85D-6190EE5B9759)'
> 
> 
> 
> 
> 
> 
> 
> 
> 
> 




^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2021-01-07  1:35 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <16570A25664A9CD5.11073@groups.io>
2021-01-04 13:43 ` [edk2-devel] [PATCH v3 00/26] Enable Core CI support for StandaloneMmPkg Sami Mujawar
2021-01-07  1:34   ` 回复: " gaoliming

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox