Re: [edk2-devel] [Patch 1/1] Maintainers.txt: Update based on active community members

[Ng, Ray Han Lim]

For changes to item "> -M: Ray Han Lim Ng <ray.han.lim...@intel.com> 
[rayhanlimng]":
Reviewed-by: Ray Han Lim Ng <ray.han.lim...@intel.com>


-----Original Message-----
From: Pedro Falcato <pedro.falc...@gmail.com> 
Sent: Sunday, October 29, 2023 10:17 AM
To: devel@edk2.groups.io; Kinney, Michael D <michael.d.kin...@intel.com>
Cc: Andrew Fish <af...@apple.com>; Leif Lindholm <quic_llind...@quicinc.com>; 
Warkentin, Andrei <andrei.warken...@intel.com>; West, Catharine 
<catharine.w...@intel.com>; Bi, Dandan <dandan...@intel.com>; Daniel Schaefer 
<g...@danielschaefer.me>; David Woodhouse <dw...@infradead.org>; De, Debkumar 
<debkumar...@intel.com>; Dong, Eric <eric.d...@intel.com>; Jiang, Guomin 
<guomin.ji...@intel.com>; Wu, Hao A <hao.a...@intel.com>; James Bottomley 
<j...@linux.ibm.com>; Wang, Jian J <jian.j.w...@intel.com>; Justen, Jordan L 
<jordan.l.jus...@intel.com>; Julien Grall <jul...@xen.org>; Peter Grehan 
<gre...@freebsd.org>; Zhang, Qi1 <qi1.zh...@intel.com>; Ng, Ray Han Lim 
<ray.han.lim...@intel.com>; Stefan Berger <stef...@linux.ibm.com>; Hou, Wenxing 
<wenxing....@intel.com>; Lu, Xiaoyu1 <xiaoyu1...@intel.com>
Subject: Re: [edk2-devel] [Patch 1/1] Maintainers.txt: Update based on active 
community members

On Sat, Oct 28, 2023 at 8:23 PM Michael D Kinney <michael.d.kin...@intel.com> 
wrote:
>
> Over the past few months, all the of the Maintainers and Reviewers 
> listed in Maintainers.txt have been contacted to make sure 
> Maintainers.txt accurately represents the TianoCore community members 
> that are actively participating in their roles.  Based on specific 
> feedback, bounced emails, and no responses, updates have been made.
>
> * RISCV64: Daniel Schaefer replaced with Andrei Warkentin
> * ArmVirtPkg Xen has no remaining reviewers and review
>   responsibility defaults to ArmVirtPkg Maintainers/Reviewers.
> * ACPI modules related to S3 has no remaining reviewers and
>   review responsibility defaults to MdeModulePkg Maintainers/
>   Reviewers.
> * OVMF CSM modules has no remaining reviewers and review
>   responsibility defaults to OvmfPkg Maintainers/Reviewers.
> * Bounce: Chan Laura <laura.c...@intel.com>
> * Many smaller updates removing individuals that are no
>   longer involved or have replacement coverage.

Mike,

Thank you so much for doing this thankless task. Some comments:

> diff --git a/Maintainers.txt b/Maintainers.txt index 
> 3f40cdeb5554..2b03ccbe54aa 100644
> --- a/Maintainers.txt
> +++ b/Maintainers.txt
> @@ -93,7 +93,7 @@ M: Sami Mujawar <sami.muja...@arm.com> [samimujawar]
>  RISCV64
>  F: */RiscV64/
>  M: Sunil V L <suni...@ventanamicro.com> [vlsunil]
> -R: Daniel Schaefer <g...@danielschaefer.me> [JohnAZoidberg]
> +R: Andrei Warkentin <andrei.warken...@intel.com> [andreiw]
>
>  LOONGARCH64
>  F: */LoongArch64/
> @@ -157,16 +157,6 @@ R: Leif Lindholm <quic_llind...@quicinc.com> 
> [leiflindholm]
>  R: Sami Mujawar <sami.muja...@arm.com> [samimujawar]
>  R: Gerd Hoffmann <kra...@redhat.com> [kraxel]
>
> -ArmVirtPkg: modules used on Xen
> -F: ArmVirtPkg/ArmVirtXen.*
> -F: ArmVirtPkg/Library/XenArmGenericTimerVirtCounterLib/
> -F: ArmVirtPkg/Library/XenVirtMemInfoLib/
> -F: ArmVirtPkg/PrePi/
> -F: ArmVirtPkg/XenAcpiPlatformDxe/
> -F: ArmVirtPkg/XenPlatformHasAcpiDtDxe/
> -F: ArmVirtPkg/XenioFdtDxe/
> -R: Julien Grall <jul...@xen.org> [jgrall]

ArmVirtPkg Xen modules seize to have a dedicated maintainer. Can the generic 
ArmVirtPkg maintainers handle *more code* (particularly, functionality that's 
not trivial to test, unless you actively use Xen)?

>  BaseTools
>  F: BaseTools/
>  W: https://github.com/tianocore/tianocore.github.io/wiki/BaseTools
> @@ -187,8 +177,7 @@ F: CryptoPkg/
>  W: https://github.com/tianocore/tianocore.github.io/wiki/CryptoPkg
>  M: Jiewen Yao <jiewen....@intel.com> [jyao1]
>  M: Yi Li <yi1...@intel.com> [liyi77]
> -R: Xiaoyu Lu <xiaoyu1...@intel.com> [xiaoyuxlu]
> -R: Guomin Jiang <guomin.ji...@intel.com> [guominjia]
> +R: Wenxing Hou <wenxing....@intel.com> [Wenxing-hou]
>
>  DynamicTablesPkg
>  F: DynamicTablesPkg/
> @@ -202,7 +191,6 @@ W: 
> https://github.com/tianocore/tianocore.github.io/wiki/EmbeddedPkg
>  M: Leif Lindholm <quic_llind...@quicinc.com> [leiflindholm]
>  M: Ard Biesheuvel <ardb+tianoc...@kernel.org> [ardbiesheuvel]
>  M: Abner Chang <abner.ch...@amd.com> [changab]
> -R: Daniel Schaefer <g...@danielschaefer.me> [JohnAZoidberg]
>
>  EmulatorPkg
>  F: EmulatorPkg/
> @@ -228,7 +216,6 @@ F: FmpDevicePkg/
>  W: https://github.com/tianocore/tianocore.github.io/wiki/FmpDevicePkg
>  M: Liming Gao <gaolim...@byosoft.com.cn> [lgao4]
>  M: Michael D Kinney <michael.d.kin...@intel.com> [mdkinney]
> -R: Guomin Jiang <guomin.ji...@intel.com> [guominjia]
>  R: Wei6 Xu <wei6...@intel.com> [xuweiintel]
>
>  IntelFsp2Pkg
> @@ -237,7 +224,6 @@ W: 
> https://github.com/tianocore/tianocore.github.io/wiki/IntelFsp2Pkg
>  M: Chasel Chiu <chasel.c...@intel.com> [ChaselChiu]
>  M: Nate DeSimone <nathaniel.l.desim...@intel.com> [nate-desimone]
>  M: Duggapu Chinni B <chinni.b.dugg...@intel.com> [cbduggap]
> -M: Ray Han Lim Ng <ray.han.lim...@intel.com> [rayhanlimng]
>  R: Star Zeng <star.z...@intel.com> [lzeng14]
>  R: Ted Kuo <ted....@intel.com> [tedkuo1]
>  R: Ashraf Ali S <ashraf.al...@intel.com> [AshrafAliS] @@ -258,7 
> +244,6 @@ R: Susovan Mohapatra <susovan.mohapa...@intel.com> 
> [susovanmohapatra]  MdeModulePkg
>  F: MdeModulePkg/
>  W: https://github.com/tianocore/tianocore.github.io/wiki/MdeModulePkg
> -M: Jian J Wang <jian.j.w...@intel.com> [jwang36]
>  M: Liming Gao <gaolim...@byosoft.com.cn> [lgao4]

MdeModulePkg now only has a single maintainer (Liming, who also handles a 
myriad of other tasks and packages)
>
>  MdeModulePkg: ACPI modules
> @@ -268,15 +253,6 @@ R: Zhiguang Liu <zhiguang....@intel.com> 
> [LiuZhiguang001]
>  R: Dandan Bi <dandan...@intel.com> [dandanbi]
>  R: Liming Gao <gaolim...@byosoft.com.cn> [lgao4]
>
> -MdeModulePkg: ACPI modules related to S3
> -F: MdeModulePkg/*LockBox*/
> -F: MdeModulePkg/Include/*BootScript*.h
> -F: MdeModulePkg/Include/*LockBox*.h
> -F: MdeModulePkg/Include/*S3*.h
> -F: MdeModulePkg/Library/*S3*/
> -R: Hao A Wu <hao.a...@intel.com> [hwu25]
> -R: Eric Dong <eric.d...@intel.com> [ydong10]
> -
>  MdeModulePkg: BDS modules
>  F: MdeModulePkg/*BootManager*/
>  F: MdeModulePkg/Include/Library/UefiBootManagerLib.h
> @@ -326,7 +302,6 @@ F: MdeModulePkg/Library/DxeSecurityManagementLib/
>  F: MdeModulePkg/Universal/PCD/
>  F: MdeModulePkg/Universal/PlatformDriOverrideDxe/
>  F: MdeModulePkg/Universal/SecurityStubDxe/SecurityStub.c
> -R: Dandan Bi <dandan...@intel.com> [dandanbi]
>  R: Liming Gao <gaolim...@byosoft.com.cn> [lgao4]

Down to one reviewer.

>
>  MdeModulePkg: Device and Peripheral modules @@ -346,12 +321,10 @@ F: 
> MdeModulePkg/Include/Ppi/StorageSecurityCommand.h
>  F: MdeModulePkg/Include/Protocol/Ps2Policy.h
>  F: MdeModulePkg/Library/NonDiscoverableDeviceRegistrationLib/
>  F: MdeModulePkg/Universal/PcatSingleSegmentPciCfg2Pei/
> -R: Hao A Wu <hao.a...@intel.com> [hwu25]
>  R: Ray Ni <ray...@intel.com> [niruiyu]

Device and bus related code is down to one reviewer.

>
>  MdeModulePkg: Disk modules
>  F: MdeModulePkg/Universal/Disk/
> -R: Hao A Wu <hao.a...@intel.com> [hwu25]
>  R: Ray Ni <ray...@intel.com> [niruiyu]
>  R: Zhichao Gao <zhichao....@intel.com> [ZhichaoGao]
>
> @@ -366,7 +339,6 @@ F: MdeModulePkg/Library/DisplayUpdateProgressLib*/
>  F: MdeModulePkg/Library/FmpAuthenticationLibNull/
>  F: MdeModulePkg/Universal/Esrt*/
>  R: Liming Gao <gaolim...@byosoft.com.cn> [lgao4]
> -R: Guomin Jiang <guomin.ji...@intel.com> [guominjia]

One reviewer

>
>  MdeModulePkg: HII and UI modules
>  F: MdeModulePkg/*FileExplorer*/
> @@ -383,7 +355,6 @@ F: MdeModulePkg/Universal/DisplayEngineDxe/
>  F: MdeModulePkg/Universal/DriverSampleDxe/
>  F: MdeModulePkg/Universal/SetupBrowserDxe/
>  R: Dandan Bi <dandan...@intel.com> [dandanbi]
> -R: Eric Dong <eric.d...@intel.com> [ydong10]

One reviewer
>
>  MdeModulePkg: Management Mode (MM, SMM) modules
>  F: MdeModulePkg/*Smi*/
> @@ -395,10 +366,7 @@ R: Ray Ni <ray...@intel.com> [niruiyu]
>
>  MdeModulePkg: Pei Core
>  F: MdeModulePkg/Core/Pei/
> -R: Dandan Bi <dandan...@intel.com> [dandanbi]
>  R: Liming Gao <gaolim...@byosoft.com.cn> [lgao4]
> -R: Debkumar De <debkumar...@intel.com> [dde01]
> -R: Catharine West <catharine.w...@intel.com> [catharine-intl]

The *PEI core* is now down to one reviewer.

>
>  MdeModulePkg: Reset modules
>  F: MdeModulePkg/*Reset*/
> @@ -424,7 +392,6 @@ F: MdeModulePkg/Include/*/*Var*.h
>  F: MdeModulePkg/Include/Guid/SystemNvDataGuid.h
>  F: MdeModulePkg/Include/Protocol/SwapAddressRange.h
>  F: MdeModulePkg/Universal/FaultTolerantWrite*/
> -R: Hao A Wu <hao.a...@intel.com> [hwu25]
>  R: Liming Gao <gaolim...@byosoft.com.cn> [lgao4]

ditto
>
>  MdeModulePkg: Universal Payload definitions @@ -437,7 +404,6 @@ F: 
> MdeModulePkg/Library/TraceHubDebugSysTLib/
>  F: MdeModulePkg/Include/Guid/TraceHubDebugInfoHob.h
>  M: Gua Guo <gua....@intel.com> [gguo11837463]
>  M: Prakashan Krishnadas Veliyathuparambil 
> <krishnadas.veliyathuparambil.prakas...@intel.com> [kprakas2]
> -R: Chan Laura <laura.c...@intel.com> [lauracha]
>  R: K N Karthik <karthik....@intel.com> [karthikkabbigere1]
>
>  MdeModulePkg: USB Network modules
> @@ -497,7 +463,6 @@ F: OvmfPkg/
>  W: http://www.tianocore.org/ovmf/
>  M: Ard Biesheuvel <ardb+tianoc...@kernel.org> [ardbiesheuvel]
>  M: Jiewen Yao <jiewen....@intel.com> [jyao1]
> -R: Jordan Justen <jordan.l.jus...@intel.com> [jljusten]
>  R: Gerd Hoffmann <kra...@redhat.com> [kraxel]
>  S: Maintained
>
> @@ -513,7 +478,6 @@ F: OvmfPkg/Library/PlatformBootManagerLibBhyve/
>  F: OvmfPkg/Library/ResetSystemLib/BaseResetShutdownBhyve.c
>  F: OvmfPkg/Library/ResetSystemLib/BaseResetSystemLibBhyve.inf
>  R: Rebecca Cran <rebe...@bsdio.com> [bcran]
> -R: Peter Grehan <gre...@freebsd.org> [grehan-freebsd]
>  R: Corvin Köhne <corv...@freebsd.org> [corvink]
>
>  OvmfPkg: cloudhv-related modules
> @@ -528,10 +492,6 @@ F: OvmfPkg/Include/IndustryStandard/Microvm.h
>  F: OvmfPkg/Library/ResetSystemLib/*Microvm.*
>  R: Gerd Hoffmann <kra...@redhat.com> [kraxel]
>
> -OvmfPkg: CSM modules
> -F: OvmfPkg/Csm/
> -R: David Woodhouse <dw...@infradead.org> [dwmw2]

0 people dedicated to OVMF CSM (although relatively low maintenance overhead, 
from what it seems)
> -
>  OvmfPkg: Confidential Computing
>  F: OvmfPkg/AmdSev/
>  F: OvmfPkg/AmdSevDxe/
> @@ -545,7 +505,6 @@ F: OvmfPkg/PlatformPei/AmdSev.c
>  F: OvmfPkg/ResetVector/
>  F: OvmfPkg/Sec/
>  R: Erdem Aktas <erdemak...@google.com> [ruleof2]
> -R: James Bottomley <j...@linux.ibm.com> [jejb]
>  R: Jiewen Yao <jiewen....@intel.com> [jyao1]
>  R: Min Xu <min.m...@intel.com> [mxu9]
>  R: Tom Lendacky <thomas.lenda...@amd.com> [tlendacky] @@ -568,7 
> +527,6 @@ F: OvmfPkg/Library/Tcg2PhysicalPresenceLib*/
>  F: OvmfPkg/PlatformPei/ClearCache.c
>  F: OvmfPkg/Tcg/
>  R: Marc-André Lureau <marcandre.lur...@redhat.com> [elmarco]
> -R: Stefan Berger <stef...@linux.ibm.com> [stefanberger]

One reviewer
>
>  OvmfPkg: Xen-related modules
>  F: OvmfPkg/Include/Guid/XenBusRootDevice.h
> @@ -597,7 +555,6 @@ F: OvmfPkg/XenPlatformPei/
>  F: OvmfPkg/XenPvBlkDxe/
>  F: OvmfPkg/XenResetVector/
>  R: Anthony Perard <anthony.per...@citrix.com> [tperard]
> -R: Julien Grall <jul...@xen.org> [jgrall]

One reviewer
>
>  OvmfPkg: RISC-V Qemu Virt Platform
>  F: OvmfPkg/RiscVVirt
> @@ -627,7 +584,6 @@ SecurityPkg
>  F: SecurityPkg/
>  W: https://github.com/tianocore/tianocore.github.io/wiki/SecurityPkg
>  M: Jiewen Yao <jiewen....@intel.com> [jyao1]
> -M: Jian J Wang <jian.j.w...@intel.com> [jwang36]
>
>  SecurityPkg: Secure boot related modules
>  F: SecurityPkg/Library/DxeImageVerificationLib/
> @@ -637,7 +593,6 @@ R: Min Xu <min.m...@intel.com> [mxu9]
>
>  SecurityPkg: Tcg related modules
>  F: SecurityPkg/Tcg/
> -R: Qi Zhang <qi1.zh...@intel.com> [qizhangz]
>  R: Rahul Kumar <rahul1.ku...@intel.com> [rahul1-kumar]

ditto
>
>  ShellPkg
> @@ -648,12 +603,10 @@ M: Zhichao Gao <zhichao....@intel.com> 
> [ZhichaoGao]  SignedCapsulePkg
>  F: SignedCapsulePkg/
>  W: 
> https://github.com/tianocore/tianocore.github.io/wiki/SignedCapsulePkg
> -M: Jian J Wang <jian.j.w...@intel.com> [jwang36]

Unmaintained

>
>  SourceLevelDebugPkg
>  F: SourceLevelDebugPkg/
>  W: 
> https://github.com/tianocore/tianocore.github.io/wiki/SourceLevelDebug
> Pkg
> -M: Hao A Wu <hao.a...@intel.com> [hwu25]

Unmaintained
>
>  StandaloneMmPkg
>  F: StandaloneMmPkg/
> @@ -664,7 +617,6 @@ M: Ray Ni <ray...@intel.com> [niruiyu]  UefiCpuPkg
>  F: UefiCpuPkg/
>  W: https://github.com/tianocore/tianocore.github.io/wiki/UefiCpuPkg
> -M: Eric Dong <eric.d...@intel.com> [ydong10]
>  M: Ray Ni <ray...@intel.com> [niruiyu]
>  R: Rahul Kumar <rahul1.ku...@intel.com> [rahul1-kumar]
>  R: Gerd Hoffmann <kra...@redhat.com> [kraxel] @@ -672,7 +624,6 @@ R: 
> Gerd Hoffmann <kra...@redhat.com> [kraxel]
>  UefiCpuPkg: Sec related modules
>  F: UefiCpuPkg/SecCore/
>  F: UefiCpuPkg/ResetVector/
> -R: Debkumar De <debkumar...@intel.com> [dde01]
>  R: Catharine West <catharine.w...@intel.com> [catharine-intl]

One reviewer.

Some brief LoC (taking into account code, blank lines and comments) stats over 
some of the affected packages/modules:
SignedCapsulePkg - 6,836 LoC
SourceLevelDebugPkg - 15,208 LoC
MdeModulePkg - 616,591 LoC (!!)
   Bus/ -                216,268 LoC (!!!)
(HII and UI was tough to actually measure, but I'm relatively sure it's 
100,000+ LoC!)
  Core/Pei  - 11,985 LoC
SecurityPkg/Tcg - 26,275 LoC

(sidenote: It'd be interesting to see the numbers from a personnel PoV
- Person X is responsible for N lines of code, etc) It seems obvious (as a 
result of your great work!) that lots of people really are stretched incredibly 
thin.

Taking everything into account, I have two questions:
1) Should we go through these changes (that effectively reflect reality, that 
much I understand) and see what needs to be cut from
EDK2 (i.e do we have an overabundance of features)?
2) What's the call for action here? Should people submit themselves as new 
reviewers/maintainers of poorly maintained/reviewed code?

--
Pedro


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#110332): https://edk2.groups.io/g/devel/message/110332
Mute This Topic: https://groups.io/mt/102245264/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-