From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail02.groups.io (mail02.groups.io [66.175.222.108]) by spool.mail.gandi.net (Postfix) with ESMTPS id 595E678003C for ; Sun, 29 Oct 2023 02:16:48 +0000 (UTC) DKIM-Signature: a=rsa-sha256; bh=FnMA9mSqe/k+HmQ0I883TbKAo87uI/fQqbrRu5ZxeYY=; c=relaxed/simple; d=groups.io; h=MIME-Version:References:In-Reply-To:From:Date:Message-ID:Subject:To:Cc:Precedence:List-Subscribe:List-Help:Sender:List-Id:Mailing-List:Delivered-To:Reply-To:List-Unsubscribe-Post:List-Unsubscribe:Content-Type:Content-Transfer-Encoding; s=20140610; t=1698545807; v=1; b=at/YBiBoilz6fzu9sff0beTCRzdjiHhXnKTwYJhc1k4nUhAGbN9DKJyaclWZp63FAuyLh0zl BwS/tyus4CS2NQBiNvIApkFQTDXerRJt9DijuWdHrgOThpw+IS+w3QqxYd8c2/4lucN5LDXJcVx LxfcuFOeVTOefqwIenmq8s04= X-Received: by 127.0.0.2 with SMTP id NT96YY7687511x6ve2jpBBer; Sat, 28 Oct 2023 19:16:47 -0700 X-Received: from mail-ua1-f49.google.com (mail-ua1-f49.google.com [209.85.222.49]) by mx.groups.io with SMTP id smtpd.web11.66486.1698545806412518482 for ; Sat, 28 Oct 2023 19:16:46 -0700 X-Received: by mail-ua1-f49.google.com with SMTP id a1e0cc1a2514c-7b6e3dc54e0so2110104241.1 for ; Sat, 28 Oct 2023 19:16:46 -0700 (PDT) X-Gm-Message-State: 92RxXMPHvfLW26Nha8EWZVz7x7686176AA= X-Google-Smtp-Source: AGHT+IEZtR+bLaqfZNXFaeUZ6QTDTD53GPz9A7wF1ETrxl1uMmIRRhCm2bJQJh6Q7ch2GylatCvSdmxb5DjLttv0gro= X-Received: by 2002:a67:cb93:0:b0:457:6867:aafb with SMTP id h19-20020a67cb93000000b004576867aafbmr3590933vsl.11.1698545805136; Sat, 28 Oct 2023 19:16:45 -0700 (PDT) MIME-Version: 1.0 References: <20231028192330.1031-1-michael.d.kinney@intel.com> In-Reply-To: <20231028192330.1031-1-michael.d.kinney@intel.com> From: "Pedro Falcato" Date: Sun, 29 Oct 2023 02:16:33 +0000 Message-ID: Subject: Re: [edk2-devel] [Patch 1/1] Maintainers.txt: Update based on active community members To: devel@edk2.groups.io, michael.d.kinney@intel.com Cc: Andrew Fish , Leif Lindholm , Andrei Warkentin , Catharine West , Dandan Bi , Daniel Schaefer , David Woodhouse , Debkumar De , Eric Dong , Guomin Jiang , Hao A Wu , James Bottomley , Jian J Wang , Jordan Justen , Julien Grall , Peter Grehan , Qi Zhang , Ray Han Lim Ng , Stefan Berger , Wenxing Hou , Xiaoyu Lu Precedence: Bulk List-Subscribe: List-Help: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Reply-To: devel@edk2.groups.io,pedro.falcato@gmail.com List-Unsubscribe-Post: List-Unsubscribe=One-Click List-Unsubscribe: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-GND-Status: LEGIT Authentication-Results: spool.mail.gandi.net; dkim=pass header.d=groups.io header.s=20140610 header.b="at/YBiBo"; dmarc=fail reason="SPF not aligned (relaxed), DKIM not aligned (relaxed)" header.from=gmail.com (policy=none); spf=pass (spool.mail.gandi.net: domain of bounce@groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce@groups.io On Sat, Oct 28, 2023 at 8:23=E2=80=AFPM Michael D Kinney 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 > * 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 [samimujawar] > RISCV64 > F: */RiscV64/ > M: Sunil V L [vlsunil] > -R: Daniel Schaefer [JohnAZoidberg] > +R: Andrei Warkentin [andreiw] > > LOONGARCH64 > F: */LoongArch64/ > @@ -157,16 +157,6 @@ R: Leif Lindholm [leifli= ndholm] > R: Sami Mujawar [samimujawar] > R: Gerd Hoffmann [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 [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 [jyao1] > M: Yi Li [liyi77] > -R: Xiaoyu Lu [xiaoyuxlu] > -R: Guomin Jiang [guominjia] > +R: Wenxing Hou [Wenxing-hou] > > DynamicTablesPkg > F: DynamicTablesPkg/ > @@ -202,7 +191,6 @@ W: https://github.com/tianocore/tianocore.github.io/w= iki/EmbeddedPkg > M: Leif Lindholm [leiflindholm] > M: Ard Biesheuvel [ardbiesheuvel] > M: Abner Chang [changab] > -R: Daniel Schaefer [JohnAZoidberg] > > EmulatorPkg > F: EmulatorPkg/ > @@ -228,7 +216,6 @@ F: FmpDevicePkg/ > W: https://github.com/tianocore/tianocore.github.io/wiki/FmpDevicePkg > M: Liming Gao [lgao4] > M: Michael D Kinney [mdkinney] > -R: Guomin Jiang [guominjia] > R: Wei6 Xu [xuweiintel] > > IntelFsp2Pkg > @@ -237,7 +224,6 @@ W: https://github.com/tianocore/tianocore.github.io/w= iki/IntelFsp2Pkg > M: Chasel Chiu [ChaselChiu] > M: Nate DeSimone [nate-desimone] > M: Duggapu Chinni B [cbduggap] > -M: Ray Han Lim Ng [rayhanlimng] > R: Star Zeng [lzeng14] > R: Ted Kuo [tedkuo1] > R: Ashraf Ali S [AshrafAliS] > @@ -258,7 +244,6 @@ R: Susovan Mohapatra [s= usovanmohapatra] > MdeModulePkg > F: MdeModulePkg/ > W: https://github.com/tianocore/tianocore.github.io/wiki/MdeModulePkg > -M: Jian J Wang [jwang36] > M: Liming Gao [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 [LiuZhiguan= g001] > R: Dandan Bi [dandanbi] > R: Liming Gao [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 [hwu25] > -R: Eric Dong [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 [dandanbi] > R: Liming Gao [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 [hwu25] > R: Ray Ni [niruiyu] Device and bus related code is down to one reviewer. > > MdeModulePkg: Disk modules > F: MdeModulePkg/Universal/Disk/ > -R: Hao A Wu [hwu25] > R: Ray Ni [niruiyu] > R: Zhichao Gao [ZhichaoGao] > > @@ -366,7 +339,6 @@ F: MdeModulePkg/Library/DisplayUpdateProgressLib*/ > F: MdeModulePkg/Library/FmpAuthenticationLibNull/ > F: MdeModulePkg/Universal/Esrt*/ > R: Liming Gao [lgao4] > -R: Guomin Jiang [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 [dandanbi] > -R: Eric Dong [ydong10] One reviewer > > MdeModulePkg: Management Mode (MM, SMM) modules > F: MdeModulePkg/*Smi*/ > @@ -395,10 +366,7 @@ R: Ray Ni [niruiyu] > > MdeModulePkg: Pei Core > F: MdeModulePkg/Core/Pei/ > -R: Dandan Bi [dandanbi] > R: Liming Gao [lgao4] > -R: Debkumar De [dde01] > -R: Catharine West [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 [hwu25] > R: Liming Gao [lgao4] ditto > > MdeModulePkg: Universal Payload definitions > @@ -437,7 +404,6 @@ F: MdeModulePkg/Library/TraceHubDebugSysTLib/ > F: MdeModulePkg/Include/Guid/TraceHubDebugInfoHob.h > M: Gua Guo [gguo11837463] > M: Prakashan Krishnadas Veliyathuparambil [kprakas2] > -R: Chan Laura [lauracha] > R: K N Karthik [karthikkabbigere1] > > MdeModulePkg: USB Network modules > @@ -497,7 +463,6 @@ F: OvmfPkg/ > W: http://www.tianocore.org/ovmf/ > M: Ard Biesheuvel [ardbiesheuvel] > M: Jiewen Yao [jyao1] > -R: Jordan Justen [jljusten] > R: Gerd Hoffmann [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 [bcran] > -R: Peter Grehan [grehan-freebsd] > R: Corvin K=C3=B6hne [corvink] > > OvmfPkg: cloudhv-related modules > @@ -528,10 +492,6 @@ F: OvmfPkg/Include/IndustryStandard/Microvm.h > F: OvmfPkg/Library/ResetSystemLib/*Microvm.* > R: Gerd Hoffmann [kraxel] > > -OvmfPkg: CSM modules > -F: OvmfPkg/Csm/ > -R: David Woodhouse [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 [ruleof2] > -R: James Bottomley [jejb] > R: Jiewen Yao [jyao1] > R: Min Xu [mxu9] > R: Tom Lendacky [tlendacky] > @@ -568,7 +527,6 @@ F: OvmfPkg/Library/Tcg2PhysicalPresenceLib*/ > F: OvmfPkg/PlatformPei/ClearCache.c > F: OvmfPkg/Tcg/ > R: Marc-Andr=C3=A9 Lureau [elmarco] > -R: Stefan Berger [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 [tperard] > -R: Julien Grall [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 [jyao1] > -M: Jian J Wang [jwang36] > > SecurityPkg: Secure boot related modules > F: SecurityPkg/Library/DxeImageVerificationLib/ > @@ -637,7 +593,6 @@ R: Min Xu [mxu9] > > SecurityPkg: Tcg related modules > F: SecurityPkg/Tcg/ > -R: Qi Zhang [qizhangz] > R: Rahul Kumar [rahul1-kumar] ditto > > ShellPkg > @@ -648,12 +603,10 @@ M: Zhichao Gao [ZhichaoGao] > SignedCapsulePkg > F: SignedCapsulePkg/ > W: https://github.com/tianocore/tianocore.github.io/wiki/SignedCapsulePk= g > -M: Jian J Wang [jwang36] Unmaintained > > SourceLevelDebugPkg > F: SourceLevelDebugPkg/ > W: https://github.com/tianocore/tianocore.github.io/wiki/SourceLevelDebu= gPkg > -M: Hao A Wu [hwu25] Unmaintained > > StandaloneMmPkg > F: StandaloneMmPkg/ > @@ -664,7 +617,6 @@ M: Ray Ni [niruiyu] > UefiCpuPkg > F: UefiCpuPkg/ > W: https://github.com/tianocore/tianocore.github.io/wiki/UefiCpuPkg > -M: Eric Dong [ydong10] > M: Ray Ni [niruiyu] > R: Rahul Kumar [rahul1-kumar] > R: Gerd Hoffmann [kraxel] > @@ -672,7 +624,6 @@ R: Gerd Hoffmann [kraxel] > UefiCpuPkg: Sec related modules > F: UefiCpuPkg/SecCore/ > F: UefiCpuPkg/ResetVector/ > -R: Debkumar De [dde01] > R: Catharine West [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? --=20 Pedro -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#110256): https://edk2.groups.io/g/devel/message/110256 Mute This Topic: https://groups.io/mt/102245264/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-