public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* [PATCH 0/5] Add reviewers for security related modules.
@ 2020-06-03  3:18 Dong, Eric
  2020-06-03  3:18 ` [PATCH 1/5] Maintainers.txt: Add reviewers for Tcg " Dong, Eric
                   ` (4 more replies)
  0 siblings, 5 replies; 15+ messages in thread
From: Dong, Eric @ 2020-06-03  3:18 UTC (permalink / raw)
  To: devel
  Cc: Hao A Wu, Jiewen Yao, Jian J Wang, Chao Zhang, Ray Ni,
	Laszlo Ersek, Debkumar De, Harry Han, Catharine West, Qi Zhang,
	Rahul Kumar, Min Xu

Add reviewers to review security related changes.
Impacted below modules:

MdeModulePkg: Pei Core
F: MdeModulePkg/Core/Pei/

SecurityPkg: Tcg related modules
F: SecurityPkg/Tcg/

SecurityPkg: Secure boot related modules
F: SecurityPkg/Library/DxeImageVerificationLib/
F: SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/
F: SecurityPkg/Library/AuthVariableLib/

UefiCpuPkg: Sec related modules
F: UefiCpuPkg/SecCore/
F: UefiCpuPkg/ResetVector/

Signed-off-by: Eric Dong <eric.dong@intel.com>
Cc: Hao A Wu <hao.a.wu@intel.com>
Cc: Jiewen Yao <jiewen.yao@intel.com>
Cc: Jian J Wang <jian.j.wang@intel.com>
Cc: Chao Zhang <chao.b.zhang@intel.com>
Cc: Ray Ni <ray.ni@intel.com>
Cc: Laszlo Ersek <lersek@redhat.com>
Cc: Debkumar De <debkumar.de@intel.com>
Cc: Harry Han <harry.han@intel.com>
Cc: Catharine West <catharine.west@intel.com>
Cc: Qi Zhang <qi1.zhang@intel.com>
Cc: Rahul Kumar <rahul1.kumar@intel.com>
Cc: Min Xu <min.m.xu@intel.com>

Eric Dong (5):
  Maintainers.txt: Add reviewers for Tcg related modules.
  Maintainers.txt: Add reviewer for security boot modules.
  Maintainers.txt: Add reviewer for UefiCpuPkg.
  Maintainers.txt: Add reviewer for SEC related modules.
  Maintainers.txt: Add reviewer for Pei Core.

 Maintainers.txt | 28 +++++++++++++++++++++++++++-
 1 file changed, 27 insertions(+), 1 deletion(-)

-- 
2.23.0.windows.1


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

* [PATCH 1/5] Maintainers.txt: Add reviewers for Tcg related modules.
  2020-06-03  3:18 [PATCH 0/5] Add reviewers for security related modules Dong, Eric
@ 2020-06-03  3:18 ` Dong, Eric
  2020-06-03 14:53   ` [edk2-devel] " Laszlo Ersek
  2020-06-08  3:35   ` Wang, Jian J
  2020-06-03  3:18 ` [PATCH 2/5] Maintainers.txt: Add reviewer for security boot modules Dong, Eric
                   ` (3 subsequent siblings)
  4 siblings, 2 replies; 15+ messages in thread
From: Dong, Eric @ 2020-06-03  3:18 UTC (permalink / raw)
  To: devel; +Cc: Jiewen Yao, Jian J Wang, Chao Zhang, Qi Zhang, Rahul Kumar

Signed-off-by: Eric Dong <eric.dong@intel.com>
Cc: Jiewen Yao <jiewen.yao@intel.com>
Cc: Jian J Wang <jian.j.wang@intel.com>
Cc: Chao Zhang <chao.b.zhang@intel.com>
Cc: Qi Zhang <qi1.zhang@intel.com>
Cc: Rahul Kumar <rahul1.kumar@intel.com>
---
 Maintainers.txt | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/Maintainers.txt b/Maintainers.txt
index 896ac5821f..954dc567e0 100644
--- a/Maintainers.txt
+++ b/Maintainers.txt
@@ -453,6 +453,11 @@ M: Jiewen Yao <jiewen.yao@intel.com>
 M: Jian J Wang <jian.j.wang@intel.com>
 R: Chao Zhang <chao.b.zhang@intel.com>
 
+SecurityPkg: Tcg related modules
+F: SecurityPkg/Tcg/
+R: Qi Zhang <qi1.zhang@intel.com>
+R: Rahul Kumar <rahul1.kumar@intel.com>
+
 ShellPkg
 F: ShellPkg/
 W: https://github.com/tianocore/tianocore.github.io/wiki/ShellPkg
-- 
2.23.0.windows.1


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

* [PATCH 2/5] Maintainers.txt: Add reviewer for security boot modules.
  2020-06-03  3:18 [PATCH 0/5] Add reviewers for security related modules Dong, Eric
  2020-06-03  3:18 ` [PATCH 1/5] Maintainers.txt: Add reviewers for Tcg " Dong, Eric
@ 2020-06-03  3:18 ` Dong, Eric
  2020-06-03 14:57   ` [edk2-devel] " Laszlo Ersek
  2020-06-03  3:18 ` [PATCH 3/5] Maintainers.txt: Add reviewer for UefiCpuPkg Dong, Eric
                   ` (2 subsequent siblings)
  4 siblings, 1 reply; 15+ messages in thread
From: Dong, Eric @ 2020-06-03  3:18 UTC (permalink / raw)
  To: devel; +Cc: Jiewen Yao, Jian J Wang, Chao Zhang, Min Xu

Signed-off-by: Eric Dong <eric.dong@intel.com>
Cc: Jiewen Yao <jiewen.yao@intel.com>
Cc: Jian J Wang <jian.j.wang@intel.com>
Cc: Chao Zhang <chao.b.zhang@intel.com>
Cc: Min Xu <min.m.xu@intel.com>
---
 Maintainers.txt | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/Maintainers.txt b/Maintainers.txt
index 954dc567e0..573084e086 100644
--- a/Maintainers.txt
+++ b/Maintainers.txt
@@ -458,6 +458,12 @@ F: SecurityPkg/Tcg/
 R: Qi Zhang <qi1.zhang@intel.com>
 R: Rahul Kumar <rahul1.kumar@intel.com>
 
+SecurityPkg: Secure boot related modules
+F: SecurityPkg/Library/DxeImageVerificationLib/
+F: SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/
+F: SecurityPkg/Library/AuthVariableLib/
+R: Min Xu <min.m.xu@intel.com>
+
 ShellPkg
 F: ShellPkg/
 W: https://github.com/tianocore/tianocore.github.io/wiki/ShellPkg
-- 
2.23.0.windows.1


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

* [PATCH 3/5] Maintainers.txt: Add reviewer for UefiCpuPkg.
  2020-06-03  3:18 [PATCH 0/5] Add reviewers for security related modules Dong, Eric
  2020-06-03  3:18 ` [PATCH 1/5] Maintainers.txt: Add reviewers for Tcg " Dong, Eric
  2020-06-03  3:18 ` [PATCH 2/5] Maintainers.txt: Add reviewer for security boot modules Dong, Eric
@ 2020-06-03  3:18 ` Dong, Eric
  2020-06-03 14:58   ` [edk2-devel] " Laszlo Ersek
  2020-06-03  3:18 ` [PATCH 4/5] Maintainers.txt: Add reviewer for SEC related modules Dong, Eric
  2020-06-03  3:18 ` [PATCH 5/5] Maintainers.txt: Add reviewer for Pei Core Dong, Eric
  4 siblings, 1 reply; 15+ messages in thread
From: Dong, Eric @ 2020-06-03  3:18 UTC (permalink / raw)
  To: devel; +Cc: Ray Ni, Laszlo Ersek, Rahul Kumar

Signed-off-by: Eric Dong <eric.dong@intel.com>
Cc: Ray Ni <ray.ni@intel.com>
Cc: Laszlo Ersek <lersek@redhat.com>
Cc: Rahul Kumar <rahul1.kumar@intel.com>
---
 Maintainers.txt | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Maintainers.txt b/Maintainers.txt
index 573084e086..318c8388fe 100644
--- a/Maintainers.txt
+++ b/Maintainers.txt
@@ -487,6 +487,7 @@ W: https://github.com/tianocore/tianocore.github.io/wiki/UefiCpuPkg
 M: Eric Dong <eric.dong@intel.com>
 M: Ray Ni <ray.ni@intel.com>
 R: Laszlo Ersek <lersek@redhat.com>
+R: Rahul Kumar <rahul1.kumar@intel.com>
 
 UefiPayloadPkg
 F: UefiPayloadPkg/
-- 
2.23.0.windows.1


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

* [PATCH 4/5] Maintainers.txt: Add reviewer for SEC related modules.
  2020-06-03  3:18 [PATCH 0/5] Add reviewers for security related modules Dong, Eric
                   ` (2 preceding siblings ...)
  2020-06-03  3:18 ` [PATCH 3/5] Maintainers.txt: Add reviewer for UefiCpuPkg Dong, Eric
@ 2020-06-03  3:18 ` Dong, Eric
  2020-06-03 14:59   ` [edk2-devel] " Laszlo Ersek
  2020-06-03  3:18 ` [PATCH 5/5] Maintainers.txt: Add reviewer for Pei Core Dong, Eric
  4 siblings, 1 reply; 15+ messages in thread
From: Dong, Eric @ 2020-06-03  3:18 UTC (permalink / raw)
  To: devel; +Cc: Debkumar De, Harry Han, Catharine West

Signed-off-by: Eric Dong <eric.dong@intel.com>
Cc: Debkumar De <debkumar.de@intel.com>
Cc: Harry Han <harry.han@intel.com>
Cc: Catharine West <catharine.west@intel.com>
---
 Maintainers.txt | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/Maintainers.txt b/Maintainers.txt
index 318c8388fe..8350ece296 100644
--- a/Maintainers.txt
+++ b/Maintainers.txt
@@ -489,6 +489,13 @@ M: Ray Ni <ray.ni@intel.com>
 R: Laszlo Ersek <lersek@redhat.com>
 R: Rahul Kumar <rahul1.kumar@intel.com>
 
+UefiCpuPkg: Sec related modules
+F: UefiCpuPkg/SecCore/
+F: UefiCpuPkg/ResetVector/
+R: Debkumar De <debkumar.de@intel.com>
+R: Harry Han <harry.han@intel.com>
+R: Catharine West <catharine.west@intel.com>
+
 UefiPayloadPkg
 F: UefiPayloadPkg/
 W: https://github.com/tianocore/tianocore.github.io/wiki/UefiPayloadPkg
-- 
2.23.0.windows.1


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

* [PATCH 5/5] Maintainers.txt: Add reviewer for Pei Core.
  2020-06-03  3:18 [PATCH 0/5] Add reviewers for security related modules Dong, Eric
                   ` (3 preceding siblings ...)
  2020-06-03  3:18 ` [PATCH 4/5] Maintainers.txt: Add reviewer for SEC related modules Dong, Eric
@ 2020-06-03  3:18 ` Dong, Eric
  2020-06-03 15:05   ` [edk2-devel] " Laszlo Ersek
  2020-06-08  3:38   ` Wang, Jian J
  4 siblings, 2 replies; 15+ messages in thread
From: Dong, Eric @ 2020-06-03  3:18 UTC (permalink / raw)
  To: devel; +Cc: Jian J Wang, Hao A Wu, Debkumar De, Harry Han, Catharine West

Signed-off-by: Eric Dong <eric.dong@intel.com>
Cc: Jian J Wang <jian.j.wang@intel.com>
Cc: Hao A Wu <hao.a.wu@intel.com>
Cc: Debkumar De <debkumar.de@intel.com>
Cc: Harry Han <harry.han@intel.com>
Cc: Catharine West <catharine.west@intel.com>
---
 Maintainers.txt | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/Maintainers.txt b/Maintainers.txt
index 8350ece296..bb1a882efc 100644
--- a/Maintainers.txt
+++ b/Maintainers.txt
@@ -258,6 +258,14 @@ F: MdeModulePkg/Universal/Console/
 R: Zhichao Gao <zhichao.gao@intel.com>
 R: Ray Ni <ray.ni@intel.com>
 
+MdeModulePkg: Pei Core
+F: MdeModulePkg/Core/Pei/
+R: Dandan Bi <dandan.bi@intel.com>
+R: Liming Gao <liming.gao@intel.com>
+R: Debkumar De <debkumar.de@intel.com>
+R: Harry Han <harry.han@intel.com>
+R: Catharine West <catharine.west@intel.com>
+
 MdeModulePkg: Core services (PEI, DXE and Runtime) modules
 F: MdeModulePkg/*Mem*/
 F: MdeModulePkg/*SectionExtract*/
@@ -265,7 +273,6 @@ F: MdeModulePkg/*StatusCode*/
 F: MdeModulePkg/Application/DumpDynPcd/
 F: MdeModulePkg/Core/Dxe/
 F: MdeModulePkg/Core/DxeIplPeim/
-F: MdeModulePkg/Core/Pei/
 F: MdeModulePkg/Core/RuntimeDxe/
 F: MdeModulePkg/Include/*Mem*.h
 F: MdeModulePkg/Include/*Pcd*.h
-- 
2.23.0.windows.1


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

* Re: [edk2-devel] [PATCH 1/5] Maintainers.txt: Add reviewers for Tcg related modules.
  2020-06-03  3:18 ` [PATCH 1/5] Maintainers.txt: Add reviewers for Tcg " Dong, Eric
@ 2020-06-03 14:53   ` Laszlo Ersek
  2020-06-08  3:35   ` Wang, Jian J
  1 sibling, 0 replies; 15+ messages in thread
From: Laszlo Ersek @ 2020-06-03 14:53 UTC (permalink / raw)
  To: devel, eric.dong
  Cc: Jiewen Yao, Jian J Wang, Chao Zhang, Qi Zhang, Rahul Kumar

On 06/03/20 05:18, Dong, Eric wrote:
> Signed-off-by: Eric Dong <eric.dong@intel.com>
> Cc: Jiewen Yao <jiewen.yao@intel.com>
> Cc: Jian J Wang <jian.j.wang@intel.com>
> Cc: Chao Zhang <chao.b.zhang@intel.com>
> Cc: Qi Zhang <qi1.zhang@intel.com>
> Cc: Rahul Kumar <rahul1.kumar@intel.com>
> ---
>  Maintainers.txt | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/Maintainers.txt b/Maintainers.txt
> index 896ac5821f..954dc567e0 100644
> --- a/Maintainers.txt
> +++ b/Maintainers.txt
> @@ -453,6 +453,11 @@ M: Jiewen Yao <jiewen.yao@intel.com>
>  M: Jian J Wang <jian.j.wang@intel.com>
>  R: Chao Zhang <chao.b.zhang@intel.com>
>  
> +SecurityPkg: Tcg related modules
> +F: SecurityPkg/Tcg/
> +R: Qi Zhang <qi1.zhang@intel.com>
> +R: Rahul Kumar <rahul1.kumar@intel.com>
> +
>  ShellPkg
>  F: ShellPkg/
>  W: https://github.com/tianocore/tianocore.github.io/wiki/ShellPkg
> 

Acked-by: Laszlo Ersek <lersek@redhat.com>


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

* Re: [edk2-devel] [PATCH 2/5] Maintainers.txt: Add reviewer for security boot modules.
  2020-06-03  3:18 ` [PATCH 2/5] Maintainers.txt: Add reviewer for security boot modules Dong, Eric
@ 2020-06-03 14:57   ` Laszlo Ersek
  2020-06-03 15:02     ` Laszlo Ersek
  0 siblings, 1 reply; 15+ messages in thread
From: Laszlo Ersek @ 2020-06-03 14:57 UTC (permalink / raw)
  To: devel, eric.dong; +Cc: Jiewen Yao, Jian J Wang, Chao Zhang, Min Xu

On 06/03/20 05:18, Dong, Eric wrote:
> Signed-off-by: Eric Dong <eric.dong@intel.com>
> Cc: Jiewen Yao <jiewen.yao@intel.com>
> Cc: Jian J Wang <jian.j.wang@intel.com>
> Cc: Chao Zhang <chao.b.zhang@intel.com>
> Cc: Min Xu <min.m.xu@intel.com>
> ---
>  Maintainers.txt | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/Maintainers.txt b/Maintainers.txt
> index 954dc567e0..573084e086 100644
> --- a/Maintainers.txt
> +++ b/Maintainers.txt
> @@ -458,6 +458,12 @@ F: SecurityPkg/Tcg/
>  R: Qi Zhang <qi1.zhang@intel.com>
>  R: Rahul Kumar <rahul1.kumar@intel.com>
>  
> +SecurityPkg: Secure boot related modules
> +F: SecurityPkg/Library/DxeImageVerificationLib/
> +F: SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/
> +F: SecurityPkg/Library/AuthVariableLib/
> +R: Min Xu <min.m.xu@intel.com>
> +
>  ShellPkg
>  F: ShellPkg/
>  W: https://github.com/tianocore/tianocore.github.io/wiki/ShellPkg
> 

(1) please replace "security boot" with "secure boot" in the subject.

with that fixed:

Acked-by: Laszlo Ersek <lersek@redhat.com>


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

* Re: [edk2-devel] [PATCH 3/5] Maintainers.txt: Add reviewer for UefiCpuPkg.
  2020-06-03  3:18 ` [PATCH 3/5] Maintainers.txt: Add reviewer for UefiCpuPkg Dong, Eric
@ 2020-06-03 14:58   ` Laszlo Ersek
  0 siblings, 0 replies; 15+ messages in thread
From: Laszlo Ersek @ 2020-06-03 14:58 UTC (permalink / raw)
  To: devel, eric.dong; +Cc: Ray Ni, Rahul Kumar

On 06/03/20 05:18, Dong, Eric wrote:
> Signed-off-by: Eric Dong <eric.dong@intel.com>
> Cc: Ray Ni <ray.ni@intel.com>
> Cc: Laszlo Ersek <lersek@redhat.com>
> Cc: Rahul Kumar <rahul1.kumar@intel.com>
> ---
>  Maintainers.txt | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/Maintainers.txt b/Maintainers.txt
> index 573084e086..318c8388fe 100644
> --- a/Maintainers.txt
> +++ b/Maintainers.txt
> @@ -487,6 +487,7 @@ W: https://github.com/tianocore/tianocore.github.io/wiki/UefiCpuPkg
>  M: Eric Dong <eric.dong@intel.com>
>  M: Ray Ni <ray.ni@intel.com>
>  R: Laszlo Ersek <lersek@redhat.com>
> +R: Rahul Kumar <rahul1.kumar@intel.com>
>  
>  UefiPayloadPkg
>  F: UefiPayloadPkg/
> 

Acked-by: Laszlo Ersek <lersek@redhat.com>


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

* Re: [edk2-devel] [PATCH 4/5] Maintainers.txt: Add reviewer for SEC related modules.
  2020-06-03  3:18 ` [PATCH 4/5] Maintainers.txt: Add reviewer for SEC related modules Dong, Eric
@ 2020-06-03 14:59   ` Laszlo Ersek
  0 siblings, 0 replies; 15+ messages in thread
From: Laszlo Ersek @ 2020-06-03 14:59 UTC (permalink / raw)
  To: devel, eric.dong; +Cc: Debkumar De, Harry Han, Catharine West

On 06/03/20 05:18, Dong, Eric wrote:
> Signed-off-by: Eric Dong <eric.dong@intel.com>
> Cc: Debkumar De <debkumar.de@intel.com>
> Cc: Harry Han <harry.han@intel.com>
> Cc: Catharine West <catharine.west@intel.com>
> ---
>  Maintainers.txt | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/Maintainers.txt b/Maintainers.txt
> index 318c8388fe..8350ece296 100644
> --- a/Maintainers.txt
> +++ b/Maintainers.txt
> @@ -489,6 +489,13 @@ M: Ray Ni <ray.ni@intel.com>
>  R: Laszlo Ersek <lersek@redhat.com>
>  R: Rahul Kumar <rahul1.kumar@intel.com>
>  
> +UefiCpuPkg: Sec related modules
> +F: UefiCpuPkg/SecCore/
> +F: UefiCpuPkg/ResetVector/
> +R: Debkumar De <debkumar.de@intel.com>
> +R: Harry Han <harry.han@intel.com>
> +R: Catharine West <catharine.west@intel.com>
> +
>  UefiPayloadPkg
>  F: UefiPayloadPkg/
>  W: https://github.com/tianocore/tianocore.github.io/wiki/UefiPayloadPkg
> 

Acked-by: Laszlo Ersek <lersek@redhat.com>


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

* Re: [edk2-devel] [PATCH 2/5] Maintainers.txt: Add reviewer for security boot modules.
  2020-06-03 14:57   ` [edk2-devel] " Laszlo Ersek
@ 2020-06-03 15:02     ` Laszlo Ersek
  2020-06-08  3:36       ` Wang, Jian J
  0 siblings, 1 reply; 15+ messages in thread
From: Laszlo Ersek @ 2020-06-03 15:02 UTC (permalink / raw)
  To: devel, eric.dong; +Cc: Jiewen Yao, Jian J Wang, Chao Zhang, Min Xu

On 06/03/20 16:57, Laszlo Ersek wrote:
> On 06/03/20 05:18, Dong, Eric wrote:
>> Signed-off-by: Eric Dong <eric.dong@intel.com>
>> Cc: Jiewen Yao <jiewen.yao@intel.com>
>> Cc: Jian J Wang <jian.j.wang@intel.com>
>> Cc: Chao Zhang <chao.b.zhang@intel.com>
>> Cc: Min Xu <min.m.xu@intel.com>
>> ---
>>  Maintainers.txt | 6 ++++++
>>  1 file changed, 6 insertions(+)
>>
>> diff --git a/Maintainers.txt b/Maintainers.txt
>> index 954dc567e0..573084e086 100644
>> --- a/Maintainers.txt
>> +++ b/Maintainers.txt
>> @@ -458,6 +458,12 @@ F: SecurityPkg/Tcg/
>>  R: Qi Zhang <qi1.zhang@intel.com>
>>  R: Rahul Kumar <rahul1.kumar@intel.com>
>>  
>> +SecurityPkg: Secure boot related modules
>> +F: SecurityPkg/Library/DxeImageVerificationLib/
>> +F: SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/
>> +F: SecurityPkg/Library/AuthVariableLib/
>> +R: Min Xu <min.m.xu@intel.com>
>> +
>>  ShellPkg
>>  F: ShellPkg/
>>  W: https://github.com/tianocore/tianocore.github.io/wiki/ShellPkg
>>
> 
> (1) please replace "security boot" with "secure boot" in the subject.
> 
> with that fixed:
> 
> Acked-by: Laszlo Ersek <lersek@redhat.com>
> 

(2) If I remember correctly, we originally wanted the section headers in
the Maintainers.txt file to remain alphabetically sorted.

This patch would create disorder:

SecurityPkg: Tcg related modules
SecurityPkg: Secure boot related modules

Therefore please add the "SecurityPkg: Secure boot related modules"
section above the "SecurityPkg: Tcg related modules" section.

With (1) and (2) fixed:

Acked-by: Laszlo Ersek <lersek@redhat.com>

(Now I do realize there is already some disorder in the Maintainers.txt
file; I'll post a patch for fixing the OvmfPkg-related ones.)

Thanks
Laszlo


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

* Re: [edk2-devel] [PATCH 5/5] Maintainers.txt: Add reviewer for Pei Core.
  2020-06-03  3:18 ` [PATCH 5/5] Maintainers.txt: Add reviewer for Pei Core Dong, Eric
@ 2020-06-03 15:05   ` Laszlo Ersek
  2020-06-08  3:38   ` Wang, Jian J
  1 sibling, 0 replies; 15+ messages in thread
From: Laszlo Ersek @ 2020-06-03 15:05 UTC (permalink / raw)
  To: devel, eric.dong
  Cc: Jian J Wang, Hao A Wu, Debkumar De, Harry Han, Catharine West

On 06/03/20 05:18, Dong, Eric wrote:
> Signed-off-by: Eric Dong <eric.dong@intel.com>
> Cc: Jian J Wang <jian.j.wang@intel.com>
> Cc: Hao A Wu <hao.a.wu@intel.com>
> Cc: Debkumar De <debkumar.de@intel.com>
> Cc: Harry Han <harry.han@intel.com>
> Cc: Catharine West <catharine.west@intel.com>
> ---
>  Maintainers.txt | 9 ++++++++-
>  1 file changed, 8 insertions(+), 1 deletion(-)
> 
> diff --git a/Maintainers.txt b/Maintainers.txt
> index 8350ece296..bb1a882efc 100644
> --- a/Maintainers.txt
> +++ b/Maintainers.txt
> @@ -258,6 +258,14 @@ F: MdeModulePkg/Universal/Console/
>  R: Zhichao Gao <zhichao.gao@intel.com>
>  R: Ray Ni <ray.ni@intel.com>
>  
> +MdeModulePkg: Pei Core
> +F: MdeModulePkg/Core/Pei/
> +R: Dandan Bi <dandan.bi@intel.com>
> +R: Liming Gao <liming.gao@intel.com>
> +R: Debkumar De <debkumar.de@intel.com>
> +R: Harry Han <harry.han@intel.com>
> +R: Catharine West <catharine.west@intel.com>
> +
>  MdeModulePkg: Core services (PEI, DXE and Runtime) modules
>  F: MdeModulePkg/*Mem*/
>  F: MdeModulePkg/*SectionExtract*/
> @@ -265,7 +273,6 @@ F: MdeModulePkg/*StatusCode*/
>  F: MdeModulePkg/Application/DumpDynPcd/
>  F: MdeModulePkg/Core/Dxe/
>  F: MdeModulePkg/Core/DxeIplPeim/
> -F: MdeModulePkg/Core/Pei/
>  F: MdeModulePkg/Core/RuntimeDxe/
>  F: MdeModulePkg/Include/*Mem*.h
>  F: MdeModulePkg/Include/*Pcd*.h
> 

(1) This section ("MdeModulePkg: Pei Core") should be inserted between:

MdeModulePkg: Management Mode (MM, SMM) modules
MdeModulePkg: Reset modules

With the insertion point fixed:

Acked-by: Laszlo Ersek <lersek@redhat.com>

Thanks,
Laszlo


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

* Re: [PATCH 1/5] Maintainers.txt: Add reviewers for Tcg related modules.
  2020-06-03  3:18 ` [PATCH 1/5] Maintainers.txt: Add reviewers for Tcg " Dong, Eric
  2020-06-03 14:53   ` [edk2-devel] " Laszlo Ersek
@ 2020-06-08  3:35   ` Wang, Jian J
  1 sibling, 0 replies; 15+ messages in thread
From: Wang, Jian J @ 2020-06-08  3:35 UTC (permalink / raw)
  To: Dong, Eric, devel@edk2.groups.io
  Cc: Yao, Jiewen, Zhang, Chao B, Zhang, Qi1, Kumar, Rahul1


Reviewed-by: Jian J Wang <jian.j.wang@intel.com>

Regards,
Jian

> -----Original Message-----
> From: Dong, Eric <eric.dong@intel.com>
> Sent: Wednesday, June 03, 2020 11:18 AM
> To: devel@edk2.groups.io
> Cc: Yao, Jiewen <jiewen.yao@intel.com>; Wang, Jian J <jian.j.wang@intel.com>;
> Zhang, Chao B <chao.b.zhang@intel.com>; Zhang, Qi1 <qi1.zhang@intel.com>;
> Kumar, Rahul1 <rahul1.kumar@intel.com>
> Subject: [PATCH 1/5] Maintainers.txt: Add reviewers for Tcg related modules.
> 
> Signed-off-by: Eric Dong <eric.dong@intel.com>
> Cc: Jiewen Yao <jiewen.yao@intel.com>
> Cc: Jian J Wang <jian.j.wang@intel.com>
> Cc: Chao Zhang <chao.b.zhang@intel.com>
> Cc: Qi Zhang <qi1.zhang@intel.com>
> Cc: Rahul Kumar <rahul1.kumar@intel.com>
> ---
>  Maintainers.txt | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/Maintainers.txt b/Maintainers.txt
> index 896ac5821f..954dc567e0 100644
> --- a/Maintainers.txt
> +++ b/Maintainers.txt
> @@ -453,6 +453,11 @@ M: Jiewen Yao <jiewen.yao@intel.com>
>  M: Jian J Wang <jian.j.wang@intel.com>
> 
>  R: Chao Zhang <chao.b.zhang@intel.com>
> 
> 
> 
> +SecurityPkg: Tcg related modules
> 
> +F: SecurityPkg/Tcg/
> 
> +R: Qi Zhang <qi1.zhang@intel.com>
> 
> +R: Rahul Kumar <rahul1.kumar@intel.com>
> 
> +
> 
>  ShellPkg
> 
>  F: ShellPkg/
> 
>  W: https://github.com/tianocore/tianocore.github.io/wiki/ShellPkg
> 
> --
> 2.23.0.windows.1


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

* Re: [edk2-devel] [PATCH 2/5] Maintainers.txt: Add reviewer for security boot modules.
  2020-06-03 15:02     ` Laszlo Ersek
@ 2020-06-08  3:36       ` Wang, Jian J
  0 siblings, 0 replies; 15+ messages in thread
From: Wang, Jian J @ 2020-06-08  3:36 UTC (permalink / raw)
  To: devel@edk2.groups.io, lersek@redhat.com, Dong, Eric
  Cc: Yao, Jiewen, Zhang, Chao B, Xu, Min M


Reviewed-by: Jian J Wang <jian.j.wang@intel.com>

Regards,
Jian

> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Laszlo Ersek
> Sent: Wednesday, June 03, 2020 11:03 PM
> To: devel@edk2.groups.io; Dong, Eric <eric.dong@intel.com>
> Cc: Yao, Jiewen <jiewen.yao@intel.com>; Wang, Jian J <jian.j.wang@intel.com>;
> Zhang, Chao B <chao.b.zhang@intel.com>; Xu, Min M <min.m.xu@intel.com>
> Subject: Re: [edk2-devel] [PATCH 2/5] Maintainers.txt: Add reviewer for security
> boot modules.
> 
> On 06/03/20 16:57, Laszlo Ersek wrote:
> > On 06/03/20 05:18, Dong, Eric wrote:
> >> Signed-off-by: Eric Dong <eric.dong@intel.com>
> >> Cc: Jiewen Yao <jiewen.yao@intel.com>
> >> Cc: Jian J Wang <jian.j.wang@intel.com>
> >> Cc: Chao Zhang <chao.b.zhang@intel.com>
> >> Cc: Min Xu <min.m.xu@intel.com>
> >> ---
> >>  Maintainers.txt | 6 ++++++
> >>  1 file changed, 6 insertions(+)
> >>
> >> diff --git a/Maintainers.txt b/Maintainers.txt
> >> index 954dc567e0..573084e086 100644
> >> --- a/Maintainers.txt
> >> +++ b/Maintainers.txt
> >> @@ -458,6 +458,12 @@ F: SecurityPkg/Tcg/
> >>  R: Qi Zhang <qi1.zhang@intel.com>
> >>  R: Rahul Kumar <rahul1.kumar@intel.com>
> >>
> >> +SecurityPkg: Secure boot related modules
> >> +F: SecurityPkg/Library/DxeImageVerificationLib/
> >> +F: SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/
> >> +F: SecurityPkg/Library/AuthVariableLib/
> >> +R: Min Xu <min.m.xu@intel.com>
> >> +
> >>  ShellPkg
> >>  F: ShellPkg/
> >>  W: https://github.com/tianocore/tianocore.github.io/wiki/ShellPkg
> >>
> >
> > (1) please replace "security boot" with "secure boot" in the subject.
> >
> > with that fixed:
> >
> > Acked-by: Laszlo Ersek <lersek@redhat.com>
> >
> 
> (2) If I remember correctly, we originally wanted the section headers in
> the Maintainers.txt file to remain alphabetically sorted.
> 
> This patch would create disorder:
> 
> SecurityPkg: Tcg related modules
> SecurityPkg: Secure boot related modules
> 
> Therefore please add the "SecurityPkg: Secure boot related modules"
> section above the "SecurityPkg: Tcg related modules" section.
> 
> With (1) and (2) fixed:
> 
> Acked-by: Laszlo Ersek <lersek@redhat.com>
> 
> (Now I do realize there is already some disorder in the Maintainers.txt
> file; I'll post a patch for fixing the OvmfPkg-related ones.)
> 
> Thanks
> Laszlo
> 
> 
> 


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

* Re: [PATCH 5/5] Maintainers.txt: Add reviewer for Pei Core.
  2020-06-03  3:18 ` [PATCH 5/5] Maintainers.txt: Add reviewer for Pei Core Dong, Eric
  2020-06-03 15:05   ` [edk2-devel] " Laszlo Ersek
@ 2020-06-08  3:38   ` Wang, Jian J
  1 sibling, 0 replies; 15+ messages in thread
From: Wang, Jian J @ 2020-06-08  3:38 UTC (permalink / raw)
  To: Dong, Eric, devel@edk2.groups.io
  Cc: Wu, Hao A, De, Debkumar, Han, Harry, West, Catharine


Reviewed-by: Jian J Wang <jian.j.wang@intel.com>

Regards,
Jian

> -----Original Message-----
> From: Dong, Eric <eric.dong@intel.com>
> Sent: Wednesday, June 03, 2020 11:18 AM
> To: devel@edk2.groups.io
> Cc: Wang, Jian J <jian.j.wang@intel.com>; Wu, Hao A <hao.a.wu@intel.com>;
> De, Debkumar <debkumar.de@intel.com>; Han, Harry <harry.han@intel.com>;
> West, Catharine <catharine.west@intel.com>
> Subject: [PATCH 5/5] Maintainers.txt: Add reviewer for Pei Core.
> 
> Signed-off-by: Eric Dong <eric.dong@intel.com>
> Cc: Jian J Wang <jian.j.wang@intel.com>
> Cc: Hao A Wu <hao.a.wu@intel.com>
> Cc: Debkumar De <debkumar.de@intel.com>
> Cc: Harry Han <harry.han@intel.com>
> Cc: Catharine West <catharine.west@intel.com>
> ---
>  Maintainers.txt | 9 ++++++++-
>  1 file changed, 8 insertions(+), 1 deletion(-)
> 
> diff --git a/Maintainers.txt b/Maintainers.txt
> index 8350ece296..bb1a882efc 100644
> --- a/Maintainers.txt
> +++ b/Maintainers.txt
> @@ -258,6 +258,14 @@ F: MdeModulePkg/Universal/Console/
>  R: Zhichao Gao <zhichao.gao@intel.com>
> 
>  R: Ray Ni <ray.ni@intel.com>
> 
> 
> 
> +MdeModulePkg: Pei Core
> 
> +F: MdeModulePkg/Core/Pei/
> 
> +R: Dandan Bi <dandan.bi@intel.com>
> 
> +R: Liming Gao <liming.gao@intel.com>
> 
> +R: Debkumar De <debkumar.de@intel.com>
> 
> +R: Harry Han <harry.han@intel.com>
> 
> +R: Catharine West <catharine.west@intel.com>
> 
> +
> 
>  MdeModulePkg: Core services (PEI, DXE and Runtime) modules
> 
>  F: MdeModulePkg/*Mem*/
> 
>  F: MdeModulePkg/*SectionExtract*/
> 
> @@ -265,7 +273,6 @@ F: MdeModulePkg/*StatusCode*/
>  F: MdeModulePkg/Application/DumpDynPcd/
> 
>  F: MdeModulePkg/Core/Dxe/
> 
>  F: MdeModulePkg/Core/DxeIplPeim/
> 
> -F: MdeModulePkg/Core/Pei/
> 
>  F: MdeModulePkg/Core/RuntimeDxe/
> 
>  F: MdeModulePkg/Include/*Mem*.h
> 
>  F: MdeModulePkg/Include/*Pcd*.h
> 
> --
> 2.23.0.windows.1


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

end of thread, other threads:[~2020-06-08  3:38 UTC | newest]

Thread overview: 15+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-06-03  3:18 [PATCH 0/5] Add reviewers for security related modules Dong, Eric
2020-06-03  3:18 ` [PATCH 1/5] Maintainers.txt: Add reviewers for Tcg " Dong, Eric
2020-06-03 14:53   ` [edk2-devel] " Laszlo Ersek
2020-06-08  3:35   ` Wang, Jian J
2020-06-03  3:18 ` [PATCH 2/5] Maintainers.txt: Add reviewer for security boot modules Dong, Eric
2020-06-03 14:57   ` [edk2-devel] " Laszlo Ersek
2020-06-03 15:02     ` Laszlo Ersek
2020-06-08  3:36       ` Wang, Jian J
2020-06-03  3:18 ` [PATCH 3/5] Maintainers.txt: Add reviewer for UefiCpuPkg Dong, Eric
2020-06-03 14:58   ` [edk2-devel] " Laszlo Ersek
2020-06-03  3:18 ` [PATCH 4/5] Maintainers.txt: Add reviewer for SEC related modules Dong, Eric
2020-06-03 14:59   ` [edk2-devel] " Laszlo Ersek
2020-06-03  3:18 ` [PATCH 5/5] Maintainers.txt: Add reviewer for Pei Core Dong, Eric
2020-06-03 15:05   ` [edk2-devel] " Laszlo Ersek
2020-06-08  3:38   ` Wang, Jian J

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