* [RFC] Fine-grained review ownership for MdeModulePkg @ 2019-06-10 8:06 Wu, Hao A 2019-06-11 9:51 ` Leif Lindholm 2019-06-20 22:23 ` rebecca 0 siblings, 2 replies; 13+ messages in thread From: Wu, Hao A @ 2019-06-10 8:06 UTC (permalink / raw) To: devel@edk2.groups.io Cc: lersek@redhat.com, Leif Lindholm, Kinney, Michael D, Andrew Fish, Justen, Jordan L, Ard Biesheuvel, Gao, Liming, Yao, Jiewen, Zeng, Star Hello all, Driven by the issue raised and discussions with thread: https://edk2.groups.io/g/devel/topic/31907735#42076 This RFC proposal is raised to work out a fine-grained review ownership for MdeModulePkg. The goal is to update the 'MdeModulePkg' section in file Maintainers.txt to resemble the format like the 'OvmfPkg' section. Several reviewers will be listed to take the ownership to review patches for a specific feature in the package. In order to get the list of reviewers, a 2-step process will be adopted: 1. Work out a list that contains major features in MdeModulePkg. 2. Call for owners for every item in the list. For 1, a coarse feature list is: Acpi Bds Capsule Compress Console Core (PeiCore, DxeCore, PiSmmCore, etc. Mainly content in MdeModulePkg/Core/ folder) Device (peripheral & storage) Disk EBC HII MemoryManagement (include memory test, memory allocation, memory profile, etc.) PCD PCI bus Reset S3 (including LockBox) SMBIOS SMM Status Code UI Variable Misc (all the components that do not fit in the above items) Once we think the granularity of the above list is fine and there is no significant feature missing, I will update such list to include the modules that belong to a specific feature. So adjustments can be done to work out the final feature list. The proposal here is far from mature, so any feedback/suggestion will be appreciated. Best Regards, Hao Wu ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [RFC] Fine-grained review ownership for MdeModulePkg 2019-06-10 8:06 [RFC] Fine-grained review ownership for MdeModulePkg Wu, Hao A @ 2019-06-11 9:51 ` Leif Lindholm 2019-06-11 15:41 ` Laszlo Ersek 2019-06-19 5:09 ` Wu, Hao A 2019-06-20 22:23 ` rebecca 1 sibling, 2 replies; 13+ messages in thread From: Leif Lindholm @ 2019-06-11 9:51 UTC (permalink / raw) To: Wu, Hao A Cc: devel@edk2.groups.io, lersek@redhat.com, Kinney, Michael D, Andrew Fish, Justen, Jordan L, Ard Biesheuvel, Gao, Liming, Yao, Jiewen, Zeng, Star Hi Wu Hao, I am working on a proposal (and proof-of-concept) machine readable Maintainers.txt format, and a GetMaintainer.py script, based on the format used by QEMU. One of the key features of that format is the F: tag, which specifes a filesytem path (with wildcard support). While this is just an RFC in progress (to be sent out later in week), if you could take that into account, and figure out which paths correspond to which areas of responsibility, that would make a test run of the script much easier. For my part, I would suggest adding (not just for MdeModulePkg) F: */ARM/* F: */AARCH64/* as maintained by myself and Ard. Best Regards, Leif On Mon, Jun 10, 2019 at 08:06:23AM +0000, Wu, Hao A wrote: > Hello all, > > Driven by the issue raised and discussions with thread: > https://edk2.groups.io/g/devel/topic/31907735#42076 > > This RFC proposal is raised to work out a fine-grained review ownership > for MdeModulePkg. > > The goal is to update the 'MdeModulePkg' section in file Maintainers.txt > to resemble the format like the 'OvmfPkg' section. Several reviewers will > be listed to take the ownership to review patches for a specific feature > in the package. > > In order to get the list of reviewers, a 2-step process will be adopted: > 1. Work out a list that contains major features in MdeModulePkg. > 2. Call for owners for every item in the list. > > For 1, a coarse feature list is: > Acpi > Bds > Capsule > Compress > Console > Core (PeiCore, DxeCore, PiSmmCore, etc. Mainly content in MdeModulePkg/Core/ folder) > Device (peripheral & storage) > Disk > EBC > HII > MemoryManagement (include memory test, memory allocation, memory profile, etc.) > PCD > PCI bus > Reset > S3 (including LockBox) > SMBIOS > SMM > Status Code > UI > Variable > Misc (all the components that do not fit in the above items) > > Once we think the granularity of the above list is fine and there is no > significant feature missing, I will update such list to include the > modules that belong to a specific feature. So adjustments can be done to > work out the final feature list. > > The proposal here is far from mature, so any feedback/suggestion will be > appreciated. > > > Best Regards, > Hao Wu > > ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [RFC] Fine-grained review ownership for MdeModulePkg 2019-06-11 9:51 ` Leif Lindholm @ 2019-06-11 15:41 ` Laszlo Ersek 2019-06-19 5:09 ` Wu, Hao A 1 sibling, 0 replies; 13+ messages in thread From: Laszlo Ersek @ 2019-06-11 15:41 UTC (permalink / raw) To: Leif Lindholm, Wu, Hao A Cc: devel@edk2.groups.io, Kinney, Michael D, Andrew Fish, Justen, Jordan L, Ard Biesheuvel, Gao, Liming, Yao, Jiewen, Zeng, Star On 06/11/19 11:51, Leif Lindholm wrote: > Hi Wu Hao, > > I am working on a proposal (and proof-of-concept) machine readable > Maintainers.txt format, and a GetMaintainer.py script, based on the > format used by QEMU. > > One of the key features of that format is the F: tag, which specifes a > filesytem path (with wildcard support). This is great news! QEMU's pathnames are very flexible. > > While this is just an RFC in progress (to be sent out later in week), > if you could take that into account, and figure out which paths > correspond to which areas of responsibility, that would make a test > run of the script much easier. > > For my part, I would suggest adding (not just for MdeModulePkg) > F: */ARM/* > F: */AARCH64/* > as maintained by myself and Ard. * Under ArmVirtPkg, we distinguish "Xen modules": R: Julien Grall <julien.grall@arm.com> (Xen modules) I hope that Ard can come up with a set of patterns for these modules. :) * Under OvmfPkg, we distinguish: R: Anthony Perard <anthony.perard@citrix.com> (Xen modules) R: Julien Grall <julien.grall@arm.com> (Xen modules) and R: Marc-André Lureau <marcandre.lureau@redhat.com> (TPM2 modules) R: Stefan Berger <stefanb@linux.ibm.com> (TPM2 modules) For "Xen modules", I'd suggest: F: OvmfPkg/AcpiPlatformDxe/Xen.c F: OvmfPkg/Include/Guid/Xen*.h F: OvmfPkg/Include/IndustryStandard/Xen/ F: OvmfPkg/Include/Library/Xen*.h F: OvmfPkg/Include/Protocol/Xen*.h F: OvmfPkg/Library/PciHostBridgeLib/XenSupport.c F: OvmfPkg/Library/XenConsoleSerialPortLib/ F: OvmfPkg/Library/XenHypercallLib/ F: OvmfPkg/Library/XenIoMmioLib F: OvmfPkg/PlatformPei/Xen.* F: OvmfPkg/SmbiosPlatformDxe/*Xen.c F: OvmfPkg/XenBusDxe/ F: OvmfPkg/XenIoPciDxe/ F: OvmfPkg/XenPvBlkDxe/ For "TPM2 modules": F: OvmfPkg/Include/IndustryStandard/QemuTpm.h F: OvmfPkg/Library/Tcg2PhysicalPresenceLib*/ F: OvmfPkg/Tcg/ Thanks Laszlo ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [RFC] Fine-grained review ownership for MdeModulePkg 2019-06-11 9:51 ` Leif Lindholm 2019-06-11 15:41 ` Laszlo Ersek @ 2019-06-19 5:09 ` Wu, Hao A 2019-06-19 9:35 ` Leif Lindholm 1 sibling, 1 reply; 13+ messages in thread From: Wu, Hao A @ 2019-06-19 5:09 UTC (permalink / raw) To: devel@edk2.groups.io Cc: Leif Lindholm, lersek@redhat.com, Kinney, Michael D, Andrew Fish, Justen, Jordan L, Ard Biesheuvel, Gao, Liming, Yao, Jiewen, Zeng, Star Hello all, As suggested by Ray and Leif, modules (with wildcard) in MdeModulePkg are classified to a list of features. Please note that: * The below list is a draft at this moment, please help to provide feedbacks/comments; * Modules with no clear classification are listed under the 'Misc' section at the bottom of the list. ACPI: MdeModulePkg/Include/*/*Acpi*.h MdeModulePkg/Universal/Acpi/ BDS: MdeModulePkg/Include/Library/PlatformBootManagerLib.h MdeModulePkg/Include/Library/UefiBootManagerLib.h MdeModulePkg/Library/PlatformBootManagerLibNull/ MdeModulePkg/Library/UefiBootManagerLib/ MdeModulePkg/Universal/BdsDxe/ MdeModulePkg/Universal/BootManagerPolicyDxe/ MdeModulePkg/Universal/LoadFileOnFv2/ MdeModulePkg/Universal/SecurityStubDxe/Defer3rdPartyImageLoad.* Console: MdeModulePkg/Include/Guid/ConnectConInEvent.h MdeModulePkg/Include/Guid/ConsoleInDevice.h MdeModulePkg/Include/Guid/ConsoleOutDevice.h MdeModulePkg/Include/Guid/StandardErrorDevice.h MdeModulePkg/Include/Guid/TtyTerm.h MdeModulePkg/Universal/Console/ConPlatformDxe/ MdeModulePkg/Universal/Console/ConSplitterDxe/ MdeModulePkg/Universal/Console/GraphicsConsoleDxe/ MdeModulePkg/Universal/Console/TerminalDxe/ Core (PEI, DXE and Runtime): MdeModulePkg/Core/Dxe/* MdeModulePkg/Core/Dxe/Dispatcher/ MdeModulePkg/Core/Dxe/DxeMain/ MdeModulePkg/Core/Dxe/Event/ MdeModulePkg/Core/Dxe/FwVol*/ MdeModulePkg/Core/Dxe/Hand/ MdeModulePkg/Core/Dxe/Image/ MdeModulePkg/Core/Dxe/Library/ MdeModulePkg/Core/Dxe/Misc/ MdeModulePkg/Core/Dxe/SectionExtraction/ MdeModulePkg/Core/DxeIplPeim/ MdeModulePkg/Core/Pei/* MdeModulePkg/Core/Pei/BootMode/ MdeModulePkg/Core/Pei/CpuIo/ MdeModulePkg/Core/Pei/Dependency/ MdeModulePkg/Core/Pei/Dispatcher/ MdeModulePkg/Core/Pei/FwVol/ MdeModulePkg/Core/Pei/Hob/ MdeModulePkg/Core/Pei/Image/ MdeModulePkg/Core/Pei/PeiMain/ MdeModulePkg/Core/Pei/Ppi/ MdeModulePkg/Core/Pei/Security/ MdeModulePkg/Core/RuntimeDxe/ MdeModulePkg/Include/Guid/Crc32GuidedSectionExtraction.h MdeModulePkg/Include/Guid/EventExitBootServiceFailed.h MdeModulePkg/Include/Guid/IdleLoopEvent.h MdeModulePkg/Include/Guid/LoadModuleAtFixedAddress.h MdeModulePkg/Include/Library/SecurityManagementLib.h MdeModulePkg/Library/*SectionExtract*/ MdeModulePkg/Library/DxeSecurityManagementLib/ MdeModulePkg/Universal/PlatformDriOverrideDxe/ MdeModulePkg/Universal/SectionExtraction*/ MdeModulePkg/Universal/SecurityStubDxe/SecurityStub.c Debug: MdeModulePkg/Include/Guid/DebugMask.h MdeModulePkg/Include/Library/DebugAgentLib.h MdeModulePkg/Include/Ppi/Debug.h MdeModulePkg/Library/*Debug*/ MdeModulePkg/Universal/Debug*/ Decompress: MdeModulePkg/Include/Guid/LzmaDecompress.h MdeModulePkg/Library/*Decompress*/ Device: MdeModulePkg/Bus/Ata/ MdeModulePkg/Bus/I2c/ MdeModulePkg/Bus/Isa/ MdeModulePkg/Bus/Pci/Ehci*/ MdeModulePkg/Bus/Pci/IdeBusPei/ MdeModulePkg/Bus/Pci/NonDiscoverablePciDeviceDxe/ MdeModulePkg/Bus/Pci/NvmExpress*/ MdeModulePkg/Bus/Pci/PciSioSerialDxe/ MdeModulePkg/Bus/Pci/SataControllerDxe/ MdeModulePkg/Bus/Pci/SdMmc*/ MdeModulePkg/Bus/Pci/Ufs*/ MdeModulePkg/Bus/Pci/Uhci*/ MdeModulePkg/Bus/Pci/Xhci*/ MdeModulePkg/Bus/Scsi/ MdeModulePkg/Bus/Sd/ MdeModulePkg/Bus/Ufs/ MdeModulePkg/Bus/Usb/ MdeModulePkg/Include/*/*Ata*.h MdeModulePkg/Include/*/*NonDiscoverableDevice*.h MdeModulePkg/Include/*/*NvmExpress*.h MdeModulePkg/Include/*/*SerialPort*.h MdeModulePkg/Include/*/*SdMmc*.h MdeModulePkg/Include/*/*Ufs*.h MdeModulePkg/Include/*/*Usb*.h MdeModulePkg/Include/Guid/S3StorageDeviceInitList.h MdeModulePkg/Include/Guid/RecoveryDevice.h MdeModulePkg/Include/Guid/UsbKeyBoardLayout.h MdeModulePkg/Include/Ppi/StorageSecurityCommand.h MdeModulePkg/Include/Protocol/Ps2Policy.h MdeModulePkg/Library/BaseSerialPortLib16550/ MdeModulePkg/Library/NonDiscoverableDeviceRegistrationLib/ MdeModulePkg/Universal/SerialDxe/ Disk: MdeModulePkg/Universal/Disk/ EBC: MdeModulePkg/Include/*/*Ebc*.h MdeModulePkg/Include/Protocol/DebuggerConfiguration.h MdeModulePkg/Universal/EbcDxe/ Firmware Update: MdeModulePkg/Application/CapsuleApp/ MdeModulePkg/Include/*/*Capsule*.h MdeModulePkg/Include/Library/DisplayUpdateProgressLib.h MdeModulePkg/Include/Library/FmpAuthenticationLib.h MdeModulePkg/Include/Protocol/EsrtManagement.h MdeModulePkg/Include/Protocol/FirmwareManagementProgress.h MdeModulePkg/Library/DisplayUpdateProgressLib*/ MdeModulePkg/Library/DxeCapsuleLib*/ MdeModulePkg/Library/FmpAuthenticationLibNull/ MdeModulePkg/Universal/Capsule*/ MdeModulePkg/Universal/Esrt*/ Graphic: MdeModulePkg/Include/*/*Logo*.h MdeModulePkg/Include/Library/BmpSupportLib.h MdeModulePkg/Include/Library/FrameBufferBltLib.h MdeModulePkg/Library/BaseBmpSupportLib/ MdeModulePkg/Library/BootLogoLib/ MdeModulePkg/Library/FrameBufferBltLib/ MdeModulePkg/Logo/ MdeModulePkg/Universal/Console/GraphicsOutputDxe/ HII/UI: MdeModulePkg/Application/BootManagerMenuApp/ MdeModulePkg/Application/UiApp/ MdeModulePkg/Include/*/*FileExplorer*.h MdeModulePkg/Include/*/*FormBrowser*.h MdeModulePkg/Include/*/*Hii*.h MdeModulePkg/Include/Library/CustomizedDisplayLib.h MdeModulePkg/Include/Protocol/DisplayProtocol.h MdeModulePkg/Library/*FileExplorer*/ MdeModulePkg/Library/*Hii*/ MdeModulePkg/Library/*UiLib/ MdeModulePkg/Library/CustomizedDisplayLib/ MdeModulePkg/Universal/DisplayEngineDxe/ MdeModulePkg/Universal/FileExplorerDxe/ MdeModulePkg/Universal/Hii*/ MdeModulePkg/Universal/SetupBrowserDxe/ IPMI: MdeModulePkg/Include/*/*Ipmi*.h MdeModulePkg/Library/*Ipmi*/ Memory Management: MdeModulePkg/Application/MemoryProfileInfo/ MdeModulePkg/Core/Dxe/Gcd/ MdeModulePkg/Core/Dxe/Mem/ MdeModulePkg/Core/Pei/Memory/ MdeModulePkg/Include/*/*Mem*.h MdeModulePkg/Include/*/*IoMmu*.h MdeModulePkg/Library/*MemoryAllocation*/ MdeModulePkg/Universal/MemoryTest/ PCD: MdeModulePkg/Application/DumpDynPcd/ MdeModulePkg/Include/*/*Pcd*.h MdeModulePkg/Universal/PCD/ PCI Bus: MdeModulePkg/Bus/Pci/IncompatiblePciDeviceSupportDxe/ MdeModulePkg/Bus/Pci/PciBusDxe/ MdeModulePkg/Bus/Pci/PciHostBridgeDxe/ MdeModulePkg/Core/Pei/PciCfg2/ MdeModulePkg/Include/Library/PciHostBridgeLib.h MdeModulePkg/Library/PciHostBridgeLibNull/ MdeModulePkg/Universal/PcatSingleSegmentPciCfg2Pei/ Performance: MdeModulePkg/Include/*/*Perf*.h MdeModulePkg/Library/*Perf*/ Reset: MdeModulePkg/Core/Pei/Reset/ MdeModulePkg/Include/*/*Reset*.h MdeModulePkg/Library/*Reset*/ MdeModulePkg/Universal/ResetSystem*/ S3: MdeModulePkg/Include/*/*BootScript*.h MdeModulePkg/Include/*/*LockBox*.h MdeModulePkg/Include/*/*S3*.h MdeModulePkg/Library/*LockBox*/ MdeModulePkg/Library/*S3*/ MdeModulePkg/Universal/LockBox/ SMBIOS: MdeModulePkg/Universal/Smbios*/ SMM: MdeModulePkg/Application/SmiHandlerProfileInfo/ MdeModulePkg/Core/PiSmmCore/ MdeModulePkg/Include/*/*Smi*.h MdeModulePkg/Include/*/*Smm*.h MdeModulePkg/Library/*Smi*/ MdeModulePkg/Library/*Smm*/ MdeModulePkg/Universal/SmmCommunicationBufferDxe/ Status Code: MdeModulePkg/Core/Pei/StatusCode/ MdeModulePkg/Include/*/*StatusCode*.h MdeModulePkg/Library/*StatusCode*/ MdeModulePkg/Universal/*StatusCode*/ Variable: MdeModulePkg/Application/VariableInfo/ MdeModulePkg/Include/*/*FaultTolerantWrite*.h MdeModulePkg/Include/*/*Var*.h MdeModulePkg/Include/Guid/SystemNvDataGuid.h MdeModulePkg/Include/Protocol/SwapAddressRange.h MdeModulePkg/Library/*Var*/ MdeModulePkg/Universal/FaultTolerantWrite*/ MdeModulePkg/Universal/Variable/ Misc: MdeModulePkg/Application/HelloWorld/ MdeModulePkg/Include/Guid/MdeModulePkgTokenSpace.h MdeModulePkg/Include/Guid/MtcVendor.h MdeModulePkg/Include/Guid/ZeroGuid.h MdeModulePkg/Include/Library/CpuExceptionHandlerLib.h MdeModulePkg/Include/Library/PlatformHookLib.h MdeModulePkg/Include/Library/RecoveryLib.h MdeModulePkg/Include/Library/SortLib.h MdeModulePkg/Include/Library/TpmMeasurementLib.h MdeModulePkg/Include/Protocol/Dpc.h MdeModulePkg/Include/Protocol/LoadPe32Image.h MdeModulePkg/Include/Protocol/PeCoffImageEmulator.h MdeModulePkg/Include/Protocol/Print2.h MdeModulePkg/Library/BaseHobLibNull/ MdeModulePkg/Library/BasePlatformHookLibNull/ MdeModulePkg/Library/BaseSortLib/ MdeModulePkg/Library/CpuExceptionHandlerLibNull/ MdeModulePkg/Library/DxePrintLibPrint2Protocol/ MdeModulePkg/Library/PeiRecoveryLibNull/ MdeModulePkg/Library/PlatformHookLibSerialPortPpi/ MdeModulePkg/Library/TpmMeasurementLibNull/ MdeModulePkg/Library/UefiSortLib/ MdeModulePkg/Universal/DevicePathDxe/ MdeModulePkg/Universal/DriverHealthManagerDxe/ MdeModulePkg/Universal/DriverSampleDxe/ MdeModulePkg/Universal/FvSimpleFileSystemDxe/ MdeModulePkg/Universal/LegacyRegion2Dxe/ MdeModulePkg/Universal/Metronome/ MdeModulePkg/Universal/MonotonicCounterRuntimeDxe/ MdeModulePkg/Universal/PrintDxe/ MdeModulePkg/Universal/RegularExpressionDxe/ MdeModulePkg/Universal/TimestampDxe/ MdeModulePkg/Universal/WatchdogTimerDxe/ Best Regards, Hao Wu > -----Original Message----- > From: Leif Lindholm [mailto:leif.lindholm@linaro.org] > Sent: Tuesday, June 11, 2019 5:52 PM > To: Wu, Hao A > Cc: devel@edk2.groups.io; lersek@redhat.com; Kinney, Michael D; Andrew > Fish; Justen, Jordan L; Ard Biesheuvel; Gao, Liming; Yao, Jiewen; Zeng, Star > Subject: Re: [RFC] Fine-grained review ownership for MdeModulePkg > > Hi Wu Hao, > > I am working on a proposal (and proof-of-concept) machine readable > Maintainers.txt format, and a GetMaintainer.py script, based on the > format used by QEMU. > > One of the key features of that format is the F: tag, which specifes a > filesytem path (with wildcard support). > > While this is just an RFC in progress (to be sent out later in week), > if you could take that into account, and figure out which paths > correspond to which areas of responsibility, that would make a test > run of the script much easier. > > For my part, I would suggest adding (not just for MdeModulePkg) > F: */ARM/* > F: */AARCH64/* > as maintained by myself and Ard. > > Best Regards, > > Leif > > On Mon, Jun 10, 2019 at 08:06:23AM +0000, Wu, Hao A wrote: > > Hello all, > > > > Driven by the issue raised and discussions with thread: > > https://edk2.groups.io/g/devel/topic/31907735#42076 > > > > This RFC proposal is raised to work out a fine-grained review ownership > > for MdeModulePkg. > > > > The goal is to update the 'MdeModulePkg' section in file Maintainers.txt > > to resemble the format like the 'OvmfPkg' section. Several reviewers will > > be listed to take the ownership to review patches for a specific feature > > in the package. > > > > In order to get the list of reviewers, a 2-step process will be adopted: > > 1. Work out a list that contains major features in MdeModulePkg. > > 2. Call for owners for every item in the list. > > > > For 1, a coarse feature list is: > > Acpi > > Bds > > Capsule > > Compress > > Console > > Core (PeiCore, DxeCore, PiSmmCore, etc. Mainly content in > MdeModulePkg/Core/ folder) > > Device (peripheral & storage) > > Disk > > EBC > > HII > > MemoryManagement (include memory test, memory allocation, memory > profile, etc.) > > PCD > > PCI bus > > Reset > > S3 (including LockBox) > > SMBIOS > > SMM > > Status Code > > UI > > Variable > > Misc (all the components that do not fit in the above items) > > > > Once we think the granularity of the above list is fine and there is no > > significant feature missing, I will update such list to include the > > modules that belong to a specific feature. So adjustments can be done to > > work out the final feature list. > > > > The proposal here is far from mature, so any feedback/suggestion will be > > appreciated. > > > > > > Best Regards, > > Hao Wu > > > > ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [RFC] Fine-grained review ownership for MdeModulePkg 2019-06-19 5:09 ` Wu, Hao A @ 2019-06-19 9:35 ` Leif Lindholm 2019-06-20 15:43 ` [edk2-devel] " Laszlo Ersek 0 siblings, 1 reply; 13+ messages in thread From: Leif Lindholm @ 2019-06-19 9:35 UTC (permalink / raw) To: Wu, Hao A Cc: devel@edk2.groups.io, lersek@redhat.com, Kinney, Michael D, Andrew Fish, Justen, Jordan L, Ard Biesheuvel, Gao, Liming, Yao, Jiewen, Zeng, Star On Wed, Jun 19, 2019 at 05:09:40AM +0000, Wu, Hao A wrote: > Hello all, > > As suggested by Ray and Leif, modules (with wildcard) in MdeModulePkg are > classified to a list of features. > > Please note that: > * The below list is a draft at this moment, please help to provide > feedbacks/comments; > * Modules with no clear classification are listed under the 'Misc' section > at the bottom of the list. Thank you for doing the heavy lifting. A few comments below on how this could be more easily described in the Maintainer.txt syntax. > ACPI: > MdeModulePkg/Include/*/*Acpi*.h > MdeModulePkg/Universal/Acpi/ > > BDS: > MdeModulePkg/Include/Library/PlatformBootManagerLib.h > MdeModulePkg/Include/Library/UefiBootManagerLib.h > MdeModulePkg/Library/PlatformBootManagerLibNull/ > MdeModulePkg/Library/UefiBootManagerLib/ MdeModulePkg/*BootManagerLib*/ > MdeModulePkg/Universal/BdsDxe/ > MdeModulePkg/Universal/BootManagerPolicyDxe/ Or maybe even MdeModulePkg/*BootManager*/, which would also match the line above. > MdeModulePkg/Universal/LoadFileOnFv2/ > MdeModulePkg/Universal/SecurityStubDxe/Defer3rdPartyImageLoad.* > > Console: > MdeModulePkg/Include/Guid/ConnectConInEvent.h > MdeModulePkg/Include/Guid/ConsoleInDevice.h > MdeModulePkg/Include/Guid/ConsoleOutDevice.h > MdeModulePkg/Include/Guid/StandardErrorDevice.h > MdeModulePkg/Include/Guid/TtyTerm.h > MdeModulePkg/Universal/Console/ConPlatformDxe/ > MdeModulePkg/Universal/Console/ConSplitterDxe/ > MdeModulePkg/Universal/Console/GraphicsConsoleDxe/ > MdeModulePkg/Universal/Console/TerminalDxe/ I was intrigued as to why this did not specify MdeModulePkg/Universal/Console/ See [1] below. However, even if suggestions included below were not implemented, the situation could be described as: F: MdeModulePkg/Universal/Console/ X: MdeModulePkg/Universal/Console/GraphicsOutputDxe/ > Core (PEI, DXE and Runtime): > MdeModulePkg/Core/Dxe/* > MdeModulePkg/Core/Dxe/Dispatcher/ > MdeModulePkg/Core/Dxe/DxeMain/ > MdeModulePkg/Core/Dxe/Event/ > MdeModulePkg/Core/Dxe/FwVol*/ > MdeModulePkg/Core/Dxe/Hand/ > MdeModulePkg/Core/Dxe/Image/ > MdeModulePkg/Core/Dxe/Library/ > MdeModulePkg/Core/Dxe/Misc/ > MdeModulePkg/Core/Dxe/SectionExtraction/ F: MdeModulePkg/Core/Dxe/ X: MdeModulePkg/Core/Dxe/Mem/ > MdeModulePkg/Core/DxeIplPeim/ > MdeModulePkg/Core/Pei/* > MdeModulePkg/Core/Pei/BootMode/ > MdeModulePkg/Core/Pei/CpuIo/ > MdeModulePkg/Core/Pei/Dependency/ > MdeModulePkg/Core/Pei/Dispatcher/ > MdeModulePkg/Core/Pei/FwVol/ > MdeModulePkg/Core/Pei/Hob/ > MdeModulePkg/Core/Pei/Image/ > MdeModulePkg/Core/Pei/PeiMain/ > MdeModulePkg/Core/Pei/Ppi/ > MdeModulePkg/Core/Pei/Security/ F: MdeModulePkg/Core/Pei/ X: MdeModulePkg/Core/Pei/Memory/ X: MdeModulePkg/Core/Pei/PciCfg2/ X: MdeModulePkg/Core/Pei/Reset/ X: MdeModulePkg/Core/Pei/StatusCode/ I'm going to stop there, because I'm lazy, and I realise this is about the responsibility areas rather than an actual patch to Maintainers.txt - and I think my point is made. Further comments below. > MdeModulePkg/Core/RuntimeDxe/ > MdeModulePkg/Include/Guid/Crc32GuidedSectionExtraction.h > MdeModulePkg/Include/Guid/EventExitBootServiceFailed.h > MdeModulePkg/Include/Guid/IdleLoopEvent.h > MdeModulePkg/Include/Guid/LoadModuleAtFixedAddress.h > MdeModulePkg/Include/Library/SecurityManagementLib.h > MdeModulePkg/Library/*SectionExtract*/ > MdeModulePkg/Library/DxeSecurityManagementLib/ > MdeModulePkg/Universal/PlatformDriOverrideDxe/ > MdeModulePkg/Universal/SectionExtraction*/ > MdeModulePkg/Universal/SecurityStubDxe/SecurityStub.c > > Debug: > MdeModulePkg/Include/Guid/DebugMask.h > MdeModulePkg/Include/Library/DebugAgentLib.h > MdeModulePkg/Include/Ppi/Debug.h > MdeModulePkg/Library/*Debug*/ > MdeModulePkg/Universal/Debug*/ > > Decompress: > MdeModulePkg/Include/Guid/LzmaDecompress.h > MdeModulePkg/Library/*Decompress*/ > > Device: > MdeModulePkg/Bus/Ata/ > MdeModulePkg/Bus/I2c/ > MdeModulePkg/Bus/Isa/ > MdeModulePkg/Bus/Pci/Ehci*/ > MdeModulePkg/Bus/Pci/IdeBusPei/ > MdeModulePkg/Bus/Pci/NonDiscoverablePciDeviceDxe/ > MdeModulePkg/Bus/Pci/NvmExpress*/ > MdeModulePkg/Bus/Pci/PciSioSerialDxe/ > MdeModulePkg/Bus/Pci/SataControllerDxe/ > MdeModulePkg/Bus/Pci/SdMmc*/ > MdeModulePkg/Bus/Pci/Ufs*/ > MdeModulePkg/Bus/Pci/Uhci*/ > MdeModulePkg/Bus/Pci/Xhci*/ > MdeModulePkg/Bus/Scsi/ > MdeModulePkg/Bus/Sd/ > MdeModulePkg/Bus/Ufs/ > MdeModulePkg/Bus/Usb/ > MdeModulePkg/Include/*/*Ata*.h > MdeModulePkg/Include/*/*NonDiscoverableDevice*.h > MdeModulePkg/Include/*/*NvmExpress*.h > MdeModulePkg/Include/*/*SerialPort*.h > MdeModulePkg/Include/*/*SdMmc*.h > MdeModulePkg/Include/*/*Ufs*.h > MdeModulePkg/Include/*/*Usb*.h > MdeModulePkg/Include/Guid/S3StorageDeviceInitList.h > MdeModulePkg/Include/Guid/RecoveryDevice.h > MdeModulePkg/Include/Guid/UsbKeyBoardLayout.h > MdeModulePkg/Include/Ppi/StorageSecurityCommand.h > MdeModulePkg/Include/Protocol/Ps2Policy.h > MdeModulePkg/Library/BaseSerialPortLib16550/ > MdeModulePkg/Library/NonDiscoverableDeviceRegistrationLib/ > MdeModulePkg/Universal/SerialDxe/ > > Disk: > MdeModulePkg/Universal/Disk/ > > EBC: > MdeModulePkg/Include/*/*Ebc*.h > MdeModulePkg/Include/Protocol/DebuggerConfiguration.h > MdeModulePkg/Universal/EbcDxe/ > > Firmware Update: > MdeModulePkg/Application/CapsuleApp/ > MdeModulePkg/Include/*/*Capsule*.h > MdeModulePkg/Include/Library/DisplayUpdateProgressLib.h > MdeModulePkg/Include/Library/FmpAuthenticationLib.h > MdeModulePkg/Include/Protocol/EsrtManagement.h > MdeModulePkg/Include/Protocol/FirmwareManagementProgress.h > MdeModulePkg/Library/DisplayUpdateProgressLib*/ > MdeModulePkg/Library/DxeCapsuleLib*/ > MdeModulePkg/Library/FmpAuthenticationLibNull/ > MdeModulePkg/Universal/Capsule*/ > MdeModulePkg/Universal/Esrt*/ > > Graphic: > MdeModulePkg/Include/*/*Logo*.h > MdeModulePkg/Include/Library/BmpSupportLib.h > MdeModulePkg/Include/Library/FrameBufferBltLib.h > MdeModulePkg/Library/BaseBmpSupportLib/ > MdeModulePkg/Library/BootLogoLib/ > MdeModulePkg/Library/FrameBufferBltLib/ > MdeModulePkg/Logo/ > MdeModulePkg/Universal/Console/GraphicsOutputDxe/ [1] I would say this suggests GraphicsOutputDxe could be moved to, say, MdeModulePkg/Universal/Graphics - and Logo be moved there too. > > HII/UI: > MdeModulePkg/Application/BootManagerMenuApp/ > MdeModulePkg/Application/UiApp/ > MdeModulePkg/Include/*/*FileExplorer*.h > MdeModulePkg/Include/*/*FormBrowser*.h > MdeModulePkg/Include/*/*Hii*.h > MdeModulePkg/Include/Library/CustomizedDisplayLib.h > MdeModulePkg/Include/Protocol/DisplayProtocol.h > MdeModulePkg/Library/*FileExplorer*/ > MdeModulePkg/Library/*Hii*/ > MdeModulePkg/Library/*UiLib/ > MdeModulePkg/Library/CustomizedDisplayLib/ > MdeModulePkg/Universal/DisplayEngineDxe/ > MdeModulePkg/Universal/FileExplorerDxe/ > MdeModulePkg/Universal/Hii*/ > MdeModulePkg/Universal/SetupBrowserDxe/ > > IPMI: > MdeModulePkg/Include/*/*Ipmi*.h > MdeModulePkg/Library/*Ipmi*/ > > Memory Management: > MdeModulePkg/Application/MemoryProfileInfo/ > MdeModulePkg/Core/Dxe/Gcd/ > MdeModulePkg/Core/Dxe/Mem/ > MdeModulePkg/Core/Pei/Memory/ > MdeModulePkg/Include/*/*Mem*.h > MdeModulePkg/Include/*/*IoMmu*.h > MdeModulePkg/Library/*MemoryAllocation*/ > MdeModulePkg/Universal/MemoryTest/ > > PCD: > MdeModulePkg/Application/DumpDynPcd/ > MdeModulePkg/Include/*/*Pcd*.h > MdeModulePkg/Universal/PCD/ > > PCI Bus: > MdeModulePkg/Bus/Pci/IncompatiblePciDeviceSupportDxe/ > MdeModulePkg/Bus/Pci/PciBusDxe/ > MdeModulePkg/Bus/Pci/PciHostBridgeDxe/ > MdeModulePkg/Core/Pei/PciCfg2/ > MdeModulePkg/Include/Library/PciHostBridgeLib.h > MdeModulePkg/Library/PciHostBridgeLibNull/ > MdeModulePkg/Universal/PcatSingleSegmentPciCfg2Pei/ > > Performance: > MdeModulePkg/Include/*/*Perf*.h > MdeModulePkg/Library/*Perf*/ > > Reset: > MdeModulePkg/Core/Pei/Reset/ > MdeModulePkg/Include/*/*Reset*.h > MdeModulePkg/Library/*Reset*/ > MdeModulePkg/Universal/ResetSystem*/ > > S3: > MdeModulePkg/Include/*/*BootScript*.h > MdeModulePkg/Include/*/*LockBox*.h > MdeModulePkg/Include/*/*S3*.h > MdeModulePkg/Library/*LockBox*/ > MdeModulePkg/Library/*S3*/ > MdeModulePkg/Universal/LockBox/ > > SMBIOS: > MdeModulePkg/Universal/Smbios*/ > > SMM: > MdeModulePkg/Application/SmiHandlerProfileInfo/ > MdeModulePkg/Core/PiSmmCore/ > MdeModulePkg/Include/*/*Smi*.h > MdeModulePkg/Include/*/*Smm*.h > MdeModulePkg/Library/*Smi*/ > MdeModulePkg/Library/*Smm*/ > MdeModulePkg/Universal/SmmCommunicationBufferDxe/ > > Status Code: > MdeModulePkg/Core/Pei/StatusCode/ > MdeModulePkg/Include/*/*StatusCode*.h > MdeModulePkg/Library/*StatusCode*/ > MdeModulePkg/Universal/*StatusCode*/ > > Variable: > MdeModulePkg/Application/VariableInfo/ > MdeModulePkg/Include/*/*FaultTolerantWrite*.h > MdeModulePkg/Include/*/*Var*.h > MdeModulePkg/Include/Guid/SystemNvDataGuid.h > MdeModulePkg/Include/Protocol/SwapAddressRange.h > MdeModulePkg/Library/*Var*/ > MdeModulePkg/Universal/FaultTolerantWrite*/ > MdeModulePkg/Universal/Variable/ > > Misc: I have no issue with the below, but just wanted to point out that if there was an overarching MdeModulePkg/ section, these would automatically match against that anyway - we don't need to explicitly tag everything. > MdeModulePkg/Application/HelloWorld/ > MdeModulePkg/Include/Guid/MdeModulePkgTokenSpace.h > MdeModulePkg/Include/Guid/MtcVendor.h > MdeModulePkg/Include/Guid/ZeroGuid.h > MdeModulePkg/Include/Library/CpuExceptionHandlerLib.h > MdeModulePkg/Include/Library/PlatformHookLib.h > MdeModulePkg/Include/Library/RecoveryLib.h > MdeModulePkg/Include/Library/SortLib.h > MdeModulePkg/Include/Library/TpmMeasurementLib.h > MdeModulePkg/Include/Protocol/Dpc.h > MdeModulePkg/Include/Protocol/LoadPe32Image.h > MdeModulePkg/Include/Protocol/PeCoffImageEmulator.h > MdeModulePkg/Include/Protocol/Print2.h > MdeModulePkg/Library/BaseHobLibNull/ > MdeModulePkg/Library/BasePlatformHookLibNull/ > MdeModulePkg/Library/BaseSortLib/ > MdeModulePkg/Library/CpuExceptionHandlerLibNull/ > MdeModulePkg/Library/DxePrintLibPrint2Protocol/ > MdeModulePkg/Library/PeiRecoveryLibNull/ > MdeModulePkg/Library/PlatformHookLibSerialPortPpi/ > MdeModulePkg/Library/TpmMeasurementLibNull/ > MdeModulePkg/Library/UefiSortLib/ > MdeModulePkg/Universal/DevicePathDxe/ > MdeModulePkg/Universal/DriverHealthManagerDxe/ > MdeModulePkg/Universal/DriverSampleDxe/ > MdeModulePkg/Universal/FvSimpleFileSystemDxe/ > MdeModulePkg/Universal/LegacyRegion2Dxe/ > MdeModulePkg/Universal/Metronome/ > MdeModulePkg/Universal/MonotonicCounterRuntimeDxe/ > MdeModulePkg/Universal/PrintDxe/ > MdeModulePkg/Universal/RegularExpressionDxe/ > MdeModulePkg/Universal/TimestampDxe/ > MdeModulePkg/Universal/WatchdogTimerDxe/ Looking through this made me realise it would be useful to have a way of quickly looking up section matches for a given path. So I added a command line option -l/--lookup to GetMaintainer.py - inline below: / Leif >From 11a04f4690e7b8bc348483a26bb3840b74b1fd9d Mon Sep 17 00:00:00 2001 From: Leif Lindholm <leif.lindholm@linaro.org> Date: Wed, 19 Jun 2019 10:07:26 +0100 Subject: [RFC PATCH 1/1] BaseTools: add lookup function to GetMaintainer.py Add command line option -l/--lookup, which lets you specify a path (existent or not in the current tree) to look up maintainers for. Signed-off-by: Leif Lindholm <leif.lindholm@linaro.org> --- BaseTools/Scripts/GetMaintainer.py | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/BaseTools/Scripts/GetMaintainer.py b/BaseTools/Scripts/GetMaintainer.py index 616342cdb434..444e601c55d1 100644 --- a/BaseTools/Scripts/GetMaintainer.py +++ b/BaseTools/Scripts/GetMaintainer.py @@ -160,15 +160,22 @@ if __name__ == '__main__': help='git revision to examine (default: HEAD)', nargs='?', default='HEAD') + PARSER.add_argument('-l', '--lookup', + help='Find section matches for path LOOKUP', + required=False) ARGS = PARSER.parse_args() REPO = SetupGit.locate_repo() - FILES = get_modified_files(REPO, ARGS) CONFIG_FILE = os.path.join(REPO.working_dir, 'Maintainers.txt') SECTIONS = parse_maintainers_file(CONFIG_FILE) + if ARGS.lookup: + FILES = [ARGS.lookup] + else: + FILES = get_modified_files(REPO, ARGS) + ADDRESSES = [] for file in FILES: -- 2.11.0 ^ permalink raw reply related [flat|nested] 13+ messages in thread
* Re: [edk2-devel] [RFC] Fine-grained review ownership for MdeModulePkg 2019-06-19 9:35 ` Leif Lindholm @ 2019-06-20 15:43 ` Laszlo Ersek 2019-06-24 1:16 ` Wu, Hao A 0 siblings, 1 reply; 13+ messages in thread From: Laszlo Ersek @ 2019-06-20 15:43 UTC (permalink / raw) To: devel, leif.lindholm, Wu, Hao A Cc: Kinney, Michael D, Andrew Fish, Justen, Jordan L, Ard Biesheuvel, Gao, Liming, Yao, Jiewen, Zeng, Star On 06/19/19 11:35, Leif Lindholm wrote: > On Wed, Jun 19, 2019 at 05:09:40AM +0000, Wu, Hao A wrote: >> Hello all, >> >> As suggested by Ray and Leif, modules (with wildcard) in MdeModulePkg are >> classified to a list of features. >> >> Please note that: >> * The below list is a draft at this moment, please help to provide >> feedbacks/comments; >> * Modules with no clear classification are listed under the 'Misc' section >> at the bottom of the list. > > Thank you for doing the heavy lifting. +1 > A few comments below on how this could be more easily described in the > Maintainer.txt syntax. With Leif's comments addressed, I'd be super pleased to have this pattern list for covering MdeModulePkg, minimally as a starting point. (I'm neutral on moving around GraphicsOutputDxe and Logo -- we could do that now, or as a later improvement.) Thanks! Laszlo ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [edk2-devel] [RFC] Fine-grained review ownership for MdeModulePkg 2019-06-20 15:43 ` [edk2-devel] " Laszlo Ersek @ 2019-06-24 1:16 ` Wu, Hao A 2019-06-24 20:29 ` Laszlo Ersek 2019-07-16 13:53 ` Leif Lindholm 0 siblings, 2 replies; 13+ messages in thread From: Wu, Hao A @ 2019-06-24 1:16 UTC (permalink / raw) To: rfc@edk2.groups.io, devel@edk2.groups.io Cc: Andrew Fish, Laszlo Ersek, Leif Lindholm, Kinney, Michael D, Wang, Jian J, Ni, Ray, Zeng, Star Hello all, With Leif's comment on the better usage of the wildcard character & 'X:' tag: https://edk2.groups.io/g/devel/message/42587 The list is updated as shown below. Also, I forgot to mention that some directory/file may be classified into multiple classes. I think it is a valid case, for example: MdeModulePkg/Library/SmmMemoryAllocationProfileLib matches both 'Memory Management' and 'SMM' sections: Memory Management - F: MdeModulePkg/*Mem*/ SMM - F: MdeModulePkg/*Smm*/ Then, reviewers from both classes should be Cc-ed when changes are made in the above library. If the updated list is good enough, the next step will be determining the reviewer(s) for each feature in the list. The updated list: ACPI: F: MdeModulePkg/Include/*/*Acpi*.h F: MdeModulePkg/Universal/Acpi/ BDS: F: MdeModulePkg/*BootManager*/ X: MdeModulePkg/Application/BootManagerMenuApp/ F: MdeModulePkg/Include/Library/UefiBootManagerLib.h F: MdeModulePkg/Universal/BdsDxe/ F: MdeModulePkg/Universal/LoadFileOnFv2/ F: MdeModulePkg/Universal/SecurityStubDxe/Defer3rdPartyImageLoad.* Console: F: MdeModulePkg/Include/Guid/ConnectConInEvent.h F: MdeModulePkg/Include/Guid/Console*.h F: MdeModulePkg/Include/Guid/StandardErrorDevice.h F: MdeModulePkg/Include/Guid/TtyTerm.h F: MdeModulePkg/Universal/Console/ X: MdeModulePkg/Universal/Console/GraphicsOutputDxe/ Core (PEI, DXE and Runtime): F: MdeModulePkg/*SectionExtract*/ F: MdeModulePkg/Core/Dxe/ X: MdeModulePkg/Core/Dxe/Gcd/ X: MdeModulePkg/Core/Dxe/Mem/ F: MdeModulePkg/Core/DxeIplPeim/ F: MdeModulePkg/Core/Pei/ X: MdeModulePkg/Core/Pei/Memory/ X: MdeModulePkg/Core/Pei/PciCfg2/ X: MdeModulePkg/Core/Pei/Reset/ X: MdeModulePkg/Core/Pei/StatusCode/ F: MdeModulePkg/Core/RuntimeDxe/ F: MdeModulePkg/Include/Guid/Crc32GuidedSectionExtraction.h F: MdeModulePkg/Include/Guid/EventExitBootServiceFailed.h F: MdeModulePkg/Include/Guid/IdleLoopEvent.h F: MdeModulePkg/Include/Guid/LoadModuleAtFixedAddress.h F: MdeModulePkg/Include/Library/SecurityManagementLib.h F: MdeModulePkg/Library/DxeSecurityManagementLib/ F: MdeModulePkg/Universal/PlatformDriOverrideDxe/ F: MdeModulePkg/Universal/SecurityStubDxe/SecurityStub.c Debug: F: MdeModulePkg/*Debug*/ X: MdeModulePkg/Universal/EbcDxe/EbcDebugger/ F: MdeModulePkg/Include/Guid/DebugMask.h F: MdeModulePkg/Include/Library/DebugAgentLib.h F: MdeModulePkg/Include/Ppi/Debug.h Decompress: F: MdeModulePkg/Include/Guid/LzmaDecompress.h F: MdeModulePkg/Library/*Decompress*/ Device: F: MdeModulePkg/*Serial*/ X: MdeModulePkg/Library/PlatformHookLibSerialPortPpi/ F: MdeModulePkg/Bus/ X: MdeModulePkg/Bus/Pci/IncompatiblePciDeviceSupportDxe/ X: MdeModulePkg/Bus/Pci/PciBusDxe/ X: MdeModulePkg/Bus/Pci/PciHostBridgeDxe/ F: MdeModulePkg/Include/*/*Ata*.h F: MdeModulePkg/Include/*/*NonDiscoverableDevice*.h F: MdeModulePkg/Include/*/*NvmExpress*.h F: MdeModulePkg/Include/*/*SerialPort*.h F: MdeModulePkg/Include/*/*SdMmc*.h F: MdeModulePkg/Include/*/*Ufs*.h F: MdeModulePkg/Include/*/*Usb*.h F: MdeModulePkg/Include/Guid/S3StorageDeviceInitList.h F: MdeModulePkg/Include/Guid/RecoveryDevice.h F: MdeModulePkg/Include/Ppi/StorageSecurityCommand.h F: MdeModulePkg/Include/Protocol/Ps2Policy.h F: MdeModulePkg/Library/NonDiscoverableDeviceRegistrationLib/ Disk: F: MdeModulePkg/Universal/Disk/ EBC: F: MdeModulePkg/Include/*/*Ebc*.h F: MdeModulePkg/Include/Protocol/DebuggerConfiguration.h F: MdeModulePkg/Universal/EbcDxe/ Firmware Update: F: MdeModulePkg/*Capsule*/ F: MdeModulePkg/Include/*/*Capsule*.h F: MdeModulePkg/Include/Library/DisplayUpdateProgressLib.h F: MdeModulePkg/Include/Library/FmpAuthenticationLib.h F: MdeModulePkg/Include/Protocol/EsrtManagement.h F: MdeModulePkg/Include/Protocol/FirmwareManagementProgress.h F: MdeModulePkg/Library/DisplayUpdateProgressLib*/ F: MdeModulePkg/Library/FmpAuthenticationLibNull/ F: MdeModulePkg/Universal/Esrt*/ Graphic: F: MdeModulePkg/*Logo*/ F: MdeModulePkg/Include/*/*Logo*.h F: MdeModulePkg/Include/Library/BmpSupportLib.h F: MdeModulePkg/Include/Library/FrameBufferBltLib.h F: MdeModulePkg/Library/BaseBmpSupportLib/ F: MdeModulePkg/Library/FrameBufferBltLib/ F: MdeModulePkg/Universal/Console/GraphicsOutputDxe/ HII/UI: F: MdeModulePkg/*FileExplorer*/ F: MdeModulePkg/*Hii*/ F: MdeModulePkg/*Ui*/ F: MdeModulePkg/Application/BootManagerMenuApp/ F: MdeModulePkg/Include/*/*FileExplorer*.h F: MdeModulePkg/Include/*/*FormBrowser*.h F: MdeModulePkg/Include/*/*Hii*.h F: MdeModulePkg/Include/Library/CustomizedDisplayLib.h F: MdeModulePkg/Include/Protocol/DisplayProtocol.h F: MdeModulePkg/Library/CustomizedDisplayLib/ F: MdeModulePkg/Universal/DisplayEngineDxe/ F: MdeModulePkg/Universal/SetupBrowserDxe/ IPMI: F: MdeModulePkg/Include/*/*Ipmi*.h F: MdeModulePkg/Library/*Ipmi*/ Memory Management: F: MdeModulePkg/*Mem*/ F: MdeModulePkg/Core/Dxe/Gcd/ F: MdeModulePkg/Include/*/*Mem*.h F: MdeModulePkg/Include/*/*IoMmu*.h PCD: F: MdeModulePkg/Application/DumpDynPcd/ F: MdeModulePkg/Include/*/*Pcd*.h F: MdeModulePkg/Universal/PCD/ PCI Bus: F: MdeModulePkg/*PciHostBridge*/ F: MdeModulePkg/Bus/Pci/IncompatiblePciDeviceSupportDxe/ F: MdeModulePkg/Bus/Pci/PciBusDxe/ F: MdeModulePkg/Core/Pei/PciCfg2/ F: MdeModulePkg/Include/Library/PciHostBridgeLib.h F: MdeModulePkg/Universal/PcatSingleSegmentPciCfg2Pei/ Performance: F: MdeModulePkg/Include/*/*Perf*.h F: MdeModulePkg/Library/*Perf*/ Reset: F: MdeModulePkg/*Reset*/ F: MdeModulePkg/Include/*/*Reset*.h S3: F: MdeModulePkg/*LockBox*/ F: MdeModulePkg/Include/*/*BootScript*.h F: MdeModulePkg/Include/*/*LockBox*.h F: MdeModulePkg/Include/*/*S3*.h F: MdeModulePkg/Library/*S3*/ SMBIOS: F: MdeModulePkg/Universal/Smbios*/ SMM: F: MdeModulePkg/*Smi*/ F: MdeModulePkg/*Smm*/ F: MdeModulePkg/Include/*/*Smi*.h F: MdeModulePkg/Include/*/*Smm*.h Status Code: F: MdeModulePkg/*StatusCode*/ F: MdeModulePkg/Include/*/*StatusCode*.h Variable: F: MdeModulePkg/*Var*/ F: MdeModulePkg/Include/*/*FaultTolerantWrite*.h F: MdeModulePkg/Include/*/*Var*.h F: MdeModulePkg/Include/Guid/SystemNvDataGuid.h F: MdeModulePkg/Include/Protocol/SwapAddressRange.h F: MdeModulePkg/Universal/FaultTolerantWrite*/ Misc: F: MdeModulePkg/Application/HelloWorld/ F: MdeModulePkg/Include/Guid/F: MdeModulePkgTokenSpace.h F: MdeModulePkg/Include/Guid/MtcVendor.h F: MdeModulePkg/Include/Guid/ZeroGuid.h F: MdeModulePkg/Include/Library/CpuExceptionHandlerLib.h F: MdeModulePkg/Include/Library/PlatformHookLib.h F: MdeModulePkg/Include/Library/RecoveryLib.h F: MdeModulePkg/Include/Library/SortLib.h F: MdeModulePkg/Include/Library/TpmMeasurementLib.h F: MdeModulePkg/Include/Protocol/Dpc.h F: MdeModulePkg/Include/Protocol/LoadPe32Image.h F: MdeModulePkg/Include/Protocol/PeCoffImageEmulator.h F: MdeModulePkg/Include/Protocol/Print2.h F: MdeModulePkg/Library/BaseHobLibNull/ F: MdeModulePkg/Library/BasePlatformHookLibNull/ F: MdeModulePkg/Library/BaseSortLib/ F: MdeModulePkg/Library/CpuExceptionHandlerLibNull/ F: MdeModulePkg/Library/DxePrintLibPrint2Protocol/ F: MdeModulePkg/Library/PeiRecoveryLibNull/ F: MdeModulePkg/Library/PlatformHookLibSerialPortPpi/ F: MdeModulePkg/Library/TpmMeasurementLibNull/ F: MdeModulePkg/Library/UefiSortLib/ F: MdeModulePkg/Universal/DevicePathDxe/ F: MdeModulePkg/Universal/DriverHealthManagerDxe/ F: MdeModulePkg/Universal/DriverSampleDxe/ F: MdeModulePkg/Universal/FvSimpleFileSystemDxe/ F: MdeModulePkg/Universal/LegacyRegion2Dxe/ F: MdeModulePkg/Universal/Metronome/ F: MdeModulePkg/Universal/MonotonicCounterRuntimeDxe/ F: MdeModulePkg/Universal/PrintDxe/ F: MdeModulePkg/Universal/RegularExpressionDxe/ F: MdeModulePkg/Universal/TimestampDxe/ F: MdeModulePkg/Universal/WatchdogTimerDxe/ Best Regards, Hao Wu ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [edk2-devel] [RFC] Fine-grained review ownership for MdeModulePkg 2019-06-24 1:16 ` Wu, Hao A @ 2019-06-24 20:29 ` Laszlo Ersek 2019-06-24 22:58 ` Yao, Jiewen 2019-07-16 13:53 ` Leif Lindholm 1 sibling, 1 reply; 13+ messages in thread From: Laszlo Ersek @ 2019-06-24 20:29 UTC (permalink / raw) To: Wu, Hao A, rfc@edk2.groups.io, devel@edk2.groups.io Cc: Andrew Fish, Leif Lindholm, Kinney, Michael D, Wang, Jian J, Ni, Ray, Zeng, Star On 06/24/19 03:16, Wu, Hao A wrote: > Hello all, > > With Leif's comment on the better usage of the wildcard character & 'X:' tag: > https://edk2.groups.io/g/devel/message/42587 > > The list is updated as shown below. > > Also, I forgot to mention that some directory/file may be classified into > multiple classes. I think it is a valid case, for example: > > MdeModulePkg/Library/SmmMemoryAllocationProfileLib > > matches both 'Memory Management' and 'SMM' sections: > > Memory Management - F: MdeModulePkg/*Mem*/ > SMM - F: MdeModulePkg/*Smm*/ > > Then, reviewers from both classes should be Cc-ed when changes are made in > the above library. > > If the updated list is good enough, the next step will be determining the > reviewer(s) for each feature in the list. I think it's good enough. :) Thanks Laszlo > > > The updated list: > ACPI: > F: MdeModulePkg/Include/*/*Acpi*.h > F: MdeModulePkg/Universal/Acpi/ > > BDS: > F: MdeModulePkg/*BootManager*/ > X: MdeModulePkg/Application/BootManagerMenuApp/ > F: MdeModulePkg/Include/Library/UefiBootManagerLib.h > F: MdeModulePkg/Universal/BdsDxe/ > F: MdeModulePkg/Universal/LoadFileOnFv2/ > F: MdeModulePkg/Universal/SecurityStubDxe/Defer3rdPartyImageLoad.* > > Console: > F: MdeModulePkg/Include/Guid/ConnectConInEvent.h > F: MdeModulePkg/Include/Guid/Console*.h > F: MdeModulePkg/Include/Guid/StandardErrorDevice.h > F: MdeModulePkg/Include/Guid/TtyTerm.h > F: MdeModulePkg/Universal/Console/ > X: MdeModulePkg/Universal/Console/GraphicsOutputDxe/ > > Core (PEI, DXE and Runtime): > F: MdeModulePkg/*SectionExtract*/ > F: MdeModulePkg/Core/Dxe/ > X: MdeModulePkg/Core/Dxe/Gcd/ > X: MdeModulePkg/Core/Dxe/Mem/ > F: MdeModulePkg/Core/DxeIplPeim/ > F: MdeModulePkg/Core/Pei/ > X: MdeModulePkg/Core/Pei/Memory/ > X: MdeModulePkg/Core/Pei/PciCfg2/ > X: MdeModulePkg/Core/Pei/Reset/ > X: MdeModulePkg/Core/Pei/StatusCode/ > F: MdeModulePkg/Core/RuntimeDxe/ > F: MdeModulePkg/Include/Guid/Crc32GuidedSectionExtraction.h > F: MdeModulePkg/Include/Guid/EventExitBootServiceFailed.h > F: MdeModulePkg/Include/Guid/IdleLoopEvent.h > F: MdeModulePkg/Include/Guid/LoadModuleAtFixedAddress.h > F: MdeModulePkg/Include/Library/SecurityManagementLib.h > F: MdeModulePkg/Library/DxeSecurityManagementLib/ > F: MdeModulePkg/Universal/PlatformDriOverrideDxe/ > F: MdeModulePkg/Universal/SecurityStubDxe/SecurityStub.c > > Debug: > F: MdeModulePkg/*Debug*/ > X: MdeModulePkg/Universal/EbcDxe/EbcDebugger/ > F: MdeModulePkg/Include/Guid/DebugMask.h > F: MdeModulePkg/Include/Library/DebugAgentLib.h > F: MdeModulePkg/Include/Ppi/Debug.h > > Decompress: > F: MdeModulePkg/Include/Guid/LzmaDecompress.h > F: MdeModulePkg/Library/*Decompress*/ > > Device: > F: MdeModulePkg/*Serial*/ > X: MdeModulePkg/Library/PlatformHookLibSerialPortPpi/ > F: MdeModulePkg/Bus/ > X: MdeModulePkg/Bus/Pci/IncompatiblePciDeviceSupportDxe/ > X: MdeModulePkg/Bus/Pci/PciBusDxe/ > X: MdeModulePkg/Bus/Pci/PciHostBridgeDxe/ > F: MdeModulePkg/Include/*/*Ata*.h > F: MdeModulePkg/Include/*/*NonDiscoverableDevice*.h > F: MdeModulePkg/Include/*/*NvmExpress*.h > F: MdeModulePkg/Include/*/*SerialPort*.h > F: MdeModulePkg/Include/*/*SdMmc*.h > F: MdeModulePkg/Include/*/*Ufs*.h > F: MdeModulePkg/Include/*/*Usb*.h > F: MdeModulePkg/Include/Guid/S3StorageDeviceInitList.h > F: MdeModulePkg/Include/Guid/RecoveryDevice.h > F: MdeModulePkg/Include/Ppi/StorageSecurityCommand.h > F: MdeModulePkg/Include/Protocol/Ps2Policy.h > F: MdeModulePkg/Library/NonDiscoverableDeviceRegistrationLib/ > > Disk: > F: MdeModulePkg/Universal/Disk/ > > EBC: > F: MdeModulePkg/Include/*/*Ebc*.h > F: MdeModulePkg/Include/Protocol/DebuggerConfiguration.h > F: MdeModulePkg/Universal/EbcDxe/ > > Firmware Update: > F: MdeModulePkg/*Capsule*/ > F: MdeModulePkg/Include/*/*Capsule*.h > F: MdeModulePkg/Include/Library/DisplayUpdateProgressLib.h > F: MdeModulePkg/Include/Library/FmpAuthenticationLib.h > F: MdeModulePkg/Include/Protocol/EsrtManagement.h > F: MdeModulePkg/Include/Protocol/FirmwareManagementProgress.h > F: MdeModulePkg/Library/DisplayUpdateProgressLib*/ > F: MdeModulePkg/Library/FmpAuthenticationLibNull/ > F: MdeModulePkg/Universal/Esrt*/ > > Graphic: > F: MdeModulePkg/*Logo*/ > F: MdeModulePkg/Include/*/*Logo*.h > F: MdeModulePkg/Include/Library/BmpSupportLib.h > F: MdeModulePkg/Include/Library/FrameBufferBltLib.h > F: MdeModulePkg/Library/BaseBmpSupportLib/ > F: MdeModulePkg/Library/FrameBufferBltLib/ > F: MdeModulePkg/Universal/Console/GraphicsOutputDxe/ > > HII/UI: > F: MdeModulePkg/*FileExplorer*/ > F: MdeModulePkg/*Hii*/ > F: MdeModulePkg/*Ui*/ > F: MdeModulePkg/Application/BootManagerMenuApp/ > F: MdeModulePkg/Include/*/*FileExplorer*.h > F: MdeModulePkg/Include/*/*FormBrowser*.h > F: MdeModulePkg/Include/*/*Hii*.h > F: MdeModulePkg/Include/Library/CustomizedDisplayLib.h > F: MdeModulePkg/Include/Protocol/DisplayProtocol.h > F: MdeModulePkg/Library/CustomizedDisplayLib/ > F: MdeModulePkg/Universal/DisplayEngineDxe/ > F: MdeModulePkg/Universal/SetupBrowserDxe/ > > IPMI: > F: MdeModulePkg/Include/*/*Ipmi*.h > F: MdeModulePkg/Library/*Ipmi*/ > > Memory Management: > F: MdeModulePkg/*Mem*/ > F: MdeModulePkg/Core/Dxe/Gcd/ > F: MdeModulePkg/Include/*/*Mem*.h > F: MdeModulePkg/Include/*/*IoMmu*.h > > PCD: > F: MdeModulePkg/Application/DumpDynPcd/ > F: MdeModulePkg/Include/*/*Pcd*.h > F: MdeModulePkg/Universal/PCD/ > > PCI Bus: > F: MdeModulePkg/*PciHostBridge*/ > F: MdeModulePkg/Bus/Pci/IncompatiblePciDeviceSupportDxe/ > F: MdeModulePkg/Bus/Pci/PciBusDxe/ > F: MdeModulePkg/Core/Pei/PciCfg2/ > F: MdeModulePkg/Include/Library/PciHostBridgeLib.h > F: MdeModulePkg/Universal/PcatSingleSegmentPciCfg2Pei/ > > Performance: > F: MdeModulePkg/Include/*/*Perf*.h > F: MdeModulePkg/Library/*Perf*/ > > Reset: > F: MdeModulePkg/*Reset*/ > F: MdeModulePkg/Include/*/*Reset*.h > > S3: > F: MdeModulePkg/*LockBox*/ > F: MdeModulePkg/Include/*/*BootScript*.h > F: MdeModulePkg/Include/*/*LockBox*.h > F: MdeModulePkg/Include/*/*S3*.h > F: MdeModulePkg/Library/*S3*/ > > SMBIOS: > F: MdeModulePkg/Universal/Smbios*/ > > SMM: > F: MdeModulePkg/*Smi*/ > F: MdeModulePkg/*Smm*/ > F: MdeModulePkg/Include/*/*Smi*.h > F: MdeModulePkg/Include/*/*Smm*.h > > Status Code: > F: MdeModulePkg/*StatusCode*/ > F: MdeModulePkg/Include/*/*StatusCode*.h > > Variable: > F: MdeModulePkg/*Var*/ > F: MdeModulePkg/Include/*/*FaultTolerantWrite*.h > F: MdeModulePkg/Include/*/*Var*.h > F: MdeModulePkg/Include/Guid/SystemNvDataGuid.h > F: MdeModulePkg/Include/Protocol/SwapAddressRange.h > F: MdeModulePkg/Universal/FaultTolerantWrite*/ > > Misc: > F: MdeModulePkg/Application/HelloWorld/ > F: MdeModulePkg/Include/Guid/F: MdeModulePkgTokenSpace.h > F: MdeModulePkg/Include/Guid/MtcVendor.h > F: MdeModulePkg/Include/Guid/ZeroGuid.h > F: MdeModulePkg/Include/Library/CpuExceptionHandlerLib.h > F: MdeModulePkg/Include/Library/PlatformHookLib.h > F: MdeModulePkg/Include/Library/RecoveryLib.h > F: MdeModulePkg/Include/Library/SortLib.h > F: MdeModulePkg/Include/Library/TpmMeasurementLib.h > F: MdeModulePkg/Include/Protocol/Dpc.h > F: MdeModulePkg/Include/Protocol/LoadPe32Image.h > F: MdeModulePkg/Include/Protocol/PeCoffImageEmulator.h > F: MdeModulePkg/Include/Protocol/Print2.h > F: MdeModulePkg/Library/BaseHobLibNull/ > F: MdeModulePkg/Library/BasePlatformHookLibNull/ > F: MdeModulePkg/Library/BaseSortLib/ > F: MdeModulePkg/Library/CpuExceptionHandlerLibNull/ > F: MdeModulePkg/Library/DxePrintLibPrint2Protocol/ > F: MdeModulePkg/Library/PeiRecoveryLibNull/ > F: MdeModulePkg/Library/PlatformHookLibSerialPortPpi/ > F: MdeModulePkg/Library/TpmMeasurementLibNull/ > F: MdeModulePkg/Library/UefiSortLib/ > F: MdeModulePkg/Universal/DevicePathDxe/ > F: MdeModulePkg/Universal/DriverHealthManagerDxe/ > F: MdeModulePkg/Universal/DriverSampleDxe/ > F: MdeModulePkg/Universal/FvSimpleFileSystemDxe/ > F: MdeModulePkg/Universal/LegacyRegion2Dxe/ > F: MdeModulePkg/Universal/Metronome/ > F: MdeModulePkg/Universal/MonotonicCounterRuntimeDxe/ > F: MdeModulePkg/Universal/PrintDxe/ > F: MdeModulePkg/Universal/RegularExpressionDxe/ > F: MdeModulePkg/Universal/TimestampDxe/ > F: MdeModulePkg/Universal/WatchdogTimerDxe/ > > > Best Regards, > Hao Wu > ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [edk2-devel] [RFC] Fine-grained review ownership for MdeModulePkg 2019-06-24 20:29 ` Laszlo Ersek @ 2019-06-24 22:58 ` Yao, Jiewen 0 siblings, 0 replies; 13+ messages in thread From: Yao, Jiewen @ 2019-06-24 22:58 UTC (permalink / raw) To: devel@edk2.groups.io, lersek@redhat.com Cc: Wu, Hao A, rfc@edk2.groups.io, Andrew Fish, Leif Lindholm, Kinney, Michael D, Wang, Jian J, Ni, Ray, Zeng, Star thanks How about to put driversample to ui? How about to put iommu to device? thank you! Yao, Jiewen > 在 2019年6月25日,上午4:30,Laszlo Ersek <lersek@redhat.com> 写道: > >> On 06/24/19 03:16, Wu, Hao A wrote: >> Hello all, >> >> With Leif's comment on the better usage of the wildcard character & 'X:' tag: >> https://edk2.groups.io/g/devel/message/42587 >> >> The list is updated as shown below. >> >> Also, I forgot to mention that some directory/file may be classified into >> multiple classes. I think it is a valid case, for example: >> >> MdeModulePkg/Library/SmmMemoryAllocationProfileLib >> >> matches both 'Memory Management' and 'SMM' sections: >> >> Memory Management - F: MdeModulePkg/*Mem*/ >> SMM - F: MdeModulePkg/*Smm*/ >> >> Then, reviewers from both classes should be Cc-ed when changes are made in >> the above library. >> >> If the updated list is good enough, the next step will be determining the >> reviewer(s) for each feature in the list. > > I think it's good enough. :) > > Thanks > Laszlo > >> >> >> The updated list: >> ACPI: >> F: MdeModulePkg/Include/*/*Acpi*.h >> F: MdeModulePkg/Universal/Acpi/ >> >> BDS: >> F: MdeModulePkg/*BootManager*/ >> X: MdeModulePkg/Application/BootManagerMenuApp/ >> F: MdeModulePkg/Include/Library/UefiBootManagerLib.h >> F: MdeModulePkg/Universal/BdsDxe/ >> F: MdeModulePkg/Universal/LoadFileOnFv2/ >> F: MdeModulePkg/Universal/SecurityStubDxe/Defer3rdPartyImageLoad.* >> >> Console: >> F: MdeModulePkg/Include/Guid/ConnectConInEvent.h >> F: MdeModulePkg/Include/Guid/Console*.h >> F: MdeModulePkg/Include/Guid/StandardErrorDevice.h >> F: MdeModulePkg/Include/Guid/TtyTerm.h >> F: MdeModulePkg/Universal/Console/ >> X: MdeModulePkg/Universal/Console/GraphicsOutputDxe/ >> >> Core (PEI, DXE and Runtime): >> F: MdeModulePkg/*SectionExtract*/ >> F: MdeModulePkg/Core/Dxe/ >> X: MdeModulePkg/Core/Dxe/Gcd/ >> X: MdeModulePkg/Core/Dxe/Mem/ >> F: MdeModulePkg/Core/DxeIplPeim/ >> F: MdeModulePkg/Core/Pei/ >> X: MdeModulePkg/Core/Pei/Memory/ >> X: MdeModulePkg/Core/Pei/PciCfg2/ >> X: MdeModulePkg/Core/Pei/Reset/ >> X: MdeModulePkg/Core/Pei/StatusCode/ >> F: MdeModulePkg/Core/RuntimeDxe/ >> F: MdeModulePkg/Include/Guid/Crc32GuidedSectionExtraction.h >> F: MdeModulePkg/Include/Guid/EventExitBootServiceFailed.h >> F: MdeModulePkg/Include/Guid/IdleLoopEvent.h >> F: MdeModulePkg/Include/Guid/LoadModuleAtFixedAddress.h >> F: MdeModulePkg/Include/Library/SecurityManagementLib.h >> F: MdeModulePkg/Library/DxeSecurityManagementLib/ >> F: MdeModulePkg/Universal/PlatformDriOverrideDxe/ >> F: MdeModulePkg/Universal/SecurityStubDxe/SecurityStub.c >> >> Debug: >> F: MdeModulePkg/*Debug*/ >> X: MdeModulePkg/Universal/EbcDxe/EbcDebugger/ >> F: MdeModulePkg/Include/Guid/DebugMask.h >> F: MdeModulePkg/Include/Library/DebugAgentLib.h >> F: MdeModulePkg/Include/Ppi/Debug.h >> >> Decompress: >> F: MdeModulePkg/Include/Guid/LzmaDecompress.h >> F: MdeModulePkg/Library/*Decompress*/ >> >> Device: >> F: MdeModulePkg/*Serial*/ >> X: MdeModulePkg/Library/PlatformHookLibSerialPortPpi/ >> F: MdeModulePkg/Bus/ >> X: MdeModulePkg/Bus/Pci/IncompatiblePciDeviceSupportDxe/ >> X: MdeModulePkg/Bus/Pci/PciBusDxe/ >> X: MdeModulePkg/Bus/Pci/PciHostBridgeDxe/ >> F: MdeModulePkg/Include/*/*Ata*.h >> F: MdeModulePkg/Include/*/*NonDiscoverableDevice*.h >> F: MdeModulePkg/Include/*/*NvmExpress*.h >> F: MdeModulePkg/Include/*/*SerialPort*.h >> F: MdeModulePkg/Include/*/*SdMmc*.h >> F: MdeModulePkg/Include/*/*Ufs*.h >> F: MdeModulePkg/Include/*/*Usb*.h >> F: MdeModulePkg/Include/Guid/S3StorageDeviceInitList.h >> F: MdeModulePkg/Include/Guid/RecoveryDevice.h >> F: MdeModulePkg/Include/Ppi/StorageSecurityCommand.h >> F: MdeModulePkg/Include/Protocol/Ps2Policy.h >> F: MdeModulePkg/Library/NonDiscoverableDeviceRegistrationLib/ >> >> Disk: >> F: MdeModulePkg/Universal/Disk/ >> >> EBC: >> F: MdeModulePkg/Include/*/*Ebc*.h >> F: MdeModulePkg/Include/Protocol/DebuggerConfiguration.h >> F: MdeModulePkg/Universal/EbcDxe/ >> >> Firmware Update: >> F: MdeModulePkg/*Capsule*/ >> F: MdeModulePkg/Include/*/*Capsule*.h >> F: MdeModulePkg/Include/Library/DisplayUpdateProgressLib.h >> F: MdeModulePkg/Include/Library/FmpAuthenticationLib.h >> F: MdeModulePkg/Include/Protocol/EsrtManagement.h >> F: MdeModulePkg/Include/Protocol/FirmwareManagementProgress.h >> F: MdeModulePkg/Library/DisplayUpdateProgressLib*/ >> F: MdeModulePkg/Library/FmpAuthenticationLibNull/ >> F: MdeModulePkg/Universal/Esrt*/ >> >> Graphic: >> F: MdeModulePkg/*Logo*/ >> F: MdeModulePkg/Include/*/*Logo*.h >> F: MdeModulePkg/Include/Library/BmpSupportLib.h >> F: MdeModulePkg/Include/Library/FrameBufferBltLib.h >> F: MdeModulePkg/Library/BaseBmpSupportLib/ >> F: MdeModulePkg/Library/FrameBufferBltLib/ >> F: MdeModulePkg/Universal/Console/GraphicsOutputDxe/ >> >> HII/UI: >> F: MdeModulePkg/*FileExplorer*/ >> F: MdeModulePkg/*Hii*/ >> F: MdeModulePkg/*Ui*/ >> F: MdeModulePkg/Application/BootManagerMenuApp/ >> F: MdeModulePkg/Include/*/*FileExplorer*.h >> F: MdeModulePkg/Include/*/*FormBrowser*.h >> F: MdeModulePkg/Include/*/*Hii*.h >> F: MdeModulePkg/Include/Library/CustomizedDisplayLib.h >> F: MdeModulePkg/Include/Protocol/DisplayProtocol.h >> F: MdeModulePkg/Library/CustomizedDisplayLib/ >> F: MdeModulePkg/Universal/DisplayEngineDxe/ >> F: MdeModulePkg/Universal/SetupBrowserDxe/ >> >> IPMI: >> F: MdeModulePkg/Include/*/*Ipmi*.h >> F: MdeModulePkg/Library/*Ipmi*/ >> >> Memory Management: >> F: MdeModulePkg/*Mem*/ >> F: MdeModulePkg/Core/Dxe/Gcd/ >> F: MdeModulePkg/Include/*/*Mem*.h >> F: MdeModulePkg/Include/*/*IoMmu*.h >> >> PCD: >> F: MdeModulePkg/Application/DumpDynPcd/ >> F: MdeModulePkg/Include/*/*Pcd*.h >> F: MdeModulePkg/Universal/PCD/ >> >> PCI Bus: >> F: MdeModulePkg/*PciHostBridge*/ >> F: MdeModulePkg/Bus/Pci/IncompatiblePciDeviceSupportDxe/ >> F: MdeModulePkg/Bus/Pci/PciBusDxe/ >> F: MdeModulePkg/Core/Pei/PciCfg2/ >> F: MdeModulePkg/Include/Library/PciHostBridgeLib.h >> F: MdeModulePkg/Universal/PcatSingleSegmentPciCfg2Pei/ >> >> Performance: >> F: MdeModulePkg/Include/*/*Perf*.h >> F: MdeModulePkg/Library/*Perf*/ >> >> Reset: >> F: MdeModulePkg/*Reset*/ >> F: MdeModulePkg/Include/*/*Reset*.h >> >> S3: >> F: MdeModulePkg/*LockBox*/ >> F: MdeModulePkg/Include/*/*BootScript*.h >> F: MdeModulePkg/Include/*/*LockBox*.h >> F: MdeModulePkg/Include/*/*S3*.h >> F: MdeModulePkg/Library/*S3*/ >> >> SMBIOS: >> F: MdeModulePkg/Universal/Smbios*/ >> >> SMM: >> F: MdeModulePkg/*Smi*/ >> F: MdeModulePkg/*Smm*/ >> F: MdeModulePkg/Include/*/*Smi*.h >> F: MdeModulePkg/Include/*/*Smm*.h >> >> Status Code: >> F: MdeModulePkg/*StatusCode*/ >> F: MdeModulePkg/Include/*/*StatusCode*.h >> >> Variable: >> F: MdeModulePkg/*Var*/ >> F: MdeModulePkg/Include/*/*FaultTolerantWrite*.h >> F: MdeModulePkg/Include/*/*Var*.h >> F: MdeModulePkg/Include/Guid/SystemNvDataGuid.h >> F: MdeModulePkg/Include/Protocol/SwapAddressRange.h >> F: MdeModulePkg/Universal/FaultTolerantWrite*/ >> >> Misc: >> F: MdeModulePkg/Application/HelloWorld/ >> F: MdeModulePkg/Include/Guid/F: MdeModulePkgTokenSpace.h >> F: MdeModulePkg/Include/Guid/MtcVendor.h >> F: MdeModulePkg/Include/Guid/ZeroGuid.h >> F: MdeModulePkg/Include/Library/CpuExceptionHandlerLib.h >> F: MdeModulePkg/Include/Library/PlatformHookLib.h >> F: MdeModulePkg/Include/Library/RecoveryLib.h >> F: MdeModulePkg/Include/Library/SortLib.h >> F: MdeModulePkg/Include/Library/TpmMeasurementLib.h >> F: MdeModulePkg/Include/Protocol/Dpc.h >> F: MdeModulePkg/Include/Protocol/LoadPe32Image.h >> F: MdeModulePkg/Include/Protocol/PeCoffImageEmulator.h >> F: MdeModulePkg/Include/Protocol/Print2.h >> F: MdeModulePkg/Library/BaseHobLibNull/ >> F: MdeModulePkg/Library/BasePlatformHookLibNull/ >> F: MdeModulePkg/Library/BaseSortLib/ >> F: MdeModulePkg/Library/CpuExceptionHandlerLibNull/ >> F: MdeModulePkg/Library/DxePrintLibPrint2Protocol/ >> F: MdeModulePkg/Library/PeiRecoveryLibNull/ >> F: MdeModulePkg/Library/PlatformHookLibSerialPortPpi/ >> F: MdeModulePkg/Library/TpmMeasurementLibNull/ >> F: MdeModulePkg/Library/UefiSortLib/ >> F: MdeModulePkg/Universal/DevicePathDxe/ >> F: MdeModulePkg/Universal/DriverHealthManagerDxe/ >> F: MdeModulePkg/Universal/DriverSampleDxe/ >> F: MdeModulePkg/Universal/FvSimpleFileSystemDxe/ >> F: MdeModulePkg/Universal/LegacyRegion2Dxe/ >> F: MdeModulePkg/Universal/Metronome/ >> F: MdeModulePkg/Universal/MonotonicCounterRuntimeDxe/ >> F: MdeModulePkg/Universal/PrintDxe/ >> F: MdeModulePkg/Universal/RegularExpressionDxe/ >> F: MdeModulePkg/Universal/TimestampDxe/ >> F: MdeModulePkg/Universal/WatchdogTimerDxe/ >> >> >> Best Regards, >> Hao Wu >> > > > > ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [edk2-devel] [RFC] Fine-grained review ownership for MdeModulePkg 2019-06-24 1:16 ` Wu, Hao A 2019-06-24 20:29 ` Laszlo Ersek @ 2019-07-16 13:53 ` Leif Lindholm 2019-07-17 1:47 ` Wu, Hao A 1 sibling, 1 reply; 13+ messages in thread From: Leif Lindholm @ 2019-07-16 13:53 UTC (permalink / raw) To: Wu, Hao A Cc: rfc@edk2.groups.io, devel@edk2.groups.io, Andrew Fish, Laszlo Ersek, Kinney, Michael D, Wang, Jian J, Ni, Ray, Zeng, Star Hi Hao Wu, Are you ready to send out a v1 yet? (Or did I miss it?) I would prefer to get this in before the GetMaintainer.py script, and after the comment updates to Maintainers.txt. Best Regards, Leif On Mon, Jun 24, 2019 at 01:16:05AM +0000, Wu, Hao A wrote: > Hello all, > > With Leif's comment on the better usage of the wildcard character & 'X:' tag: > https://edk2.groups.io/g/devel/message/42587 > > The list is updated as shown below. > > Also, I forgot to mention that some directory/file may be classified into > multiple classes. I think it is a valid case, for example: > > MdeModulePkg/Library/SmmMemoryAllocationProfileLib > > matches both 'Memory Management' and 'SMM' sections: > > Memory Management - F: MdeModulePkg/*Mem*/ > SMM - F: MdeModulePkg/*Smm*/ > > Then, reviewers from both classes should be Cc-ed when changes are made in > the above library. > > If the updated list is good enough, the next step will be determining the > reviewer(s) for each feature in the list. > > > The updated list: > ACPI: > F: MdeModulePkg/Include/*/*Acpi*.h > F: MdeModulePkg/Universal/Acpi/ > > BDS: > F: MdeModulePkg/*BootManager*/ > X: MdeModulePkg/Application/BootManagerMenuApp/ > F: MdeModulePkg/Include/Library/UefiBootManagerLib.h > F: MdeModulePkg/Universal/BdsDxe/ > F: MdeModulePkg/Universal/LoadFileOnFv2/ > F: MdeModulePkg/Universal/SecurityStubDxe/Defer3rdPartyImageLoad.* > > Console: > F: MdeModulePkg/Include/Guid/ConnectConInEvent.h > F: MdeModulePkg/Include/Guid/Console*.h > F: MdeModulePkg/Include/Guid/StandardErrorDevice.h > F: MdeModulePkg/Include/Guid/TtyTerm.h > F: MdeModulePkg/Universal/Console/ > X: MdeModulePkg/Universal/Console/GraphicsOutputDxe/ > > Core (PEI, DXE and Runtime): > F: MdeModulePkg/*SectionExtract*/ > F: MdeModulePkg/Core/Dxe/ > X: MdeModulePkg/Core/Dxe/Gcd/ > X: MdeModulePkg/Core/Dxe/Mem/ > F: MdeModulePkg/Core/DxeIplPeim/ > F: MdeModulePkg/Core/Pei/ > X: MdeModulePkg/Core/Pei/Memory/ > X: MdeModulePkg/Core/Pei/PciCfg2/ > X: MdeModulePkg/Core/Pei/Reset/ > X: MdeModulePkg/Core/Pei/StatusCode/ > F: MdeModulePkg/Core/RuntimeDxe/ > F: MdeModulePkg/Include/Guid/Crc32GuidedSectionExtraction.h > F: MdeModulePkg/Include/Guid/EventExitBootServiceFailed.h > F: MdeModulePkg/Include/Guid/IdleLoopEvent.h > F: MdeModulePkg/Include/Guid/LoadModuleAtFixedAddress.h > F: MdeModulePkg/Include/Library/SecurityManagementLib.h > F: MdeModulePkg/Library/DxeSecurityManagementLib/ > F: MdeModulePkg/Universal/PlatformDriOverrideDxe/ > F: MdeModulePkg/Universal/SecurityStubDxe/SecurityStub.c > > Debug: > F: MdeModulePkg/*Debug*/ > X: MdeModulePkg/Universal/EbcDxe/EbcDebugger/ > F: MdeModulePkg/Include/Guid/DebugMask.h > F: MdeModulePkg/Include/Library/DebugAgentLib.h > F: MdeModulePkg/Include/Ppi/Debug.h > > Decompress: > F: MdeModulePkg/Include/Guid/LzmaDecompress.h > F: MdeModulePkg/Library/*Decompress*/ > > Device: > F: MdeModulePkg/*Serial*/ > X: MdeModulePkg/Library/PlatformHookLibSerialPortPpi/ > F: MdeModulePkg/Bus/ > X: MdeModulePkg/Bus/Pci/IncompatiblePciDeviceSupportDxe/ > X: MdeModulePkg/Bus/Pci/PciBusDxe/ > X: MdeModulePkg/Bus/Pci/PciHostBridgeDxe/ > F: MdeModulePkg/Include/*/*Ata*.h > F: MdeModulePkg/Include/*/*NonDiscoverableDevice*.h > F: MdeModulePkg/Include/*/*NvmExpress*.h > F: MdeModulePkg/Include/*/*SerialPort*.h > F: MdeModulePkg/Include/*/*SdMmc*.h > F: MdeModulePkg/Include/*/*Ufs*.h > F: MdeModulePkg/Include/*/*Usb*.h > F: MdeModulePkg/Include/Guid/S3StorageDeviceInitList.h > F: MdeModulePkg/Include/Guid/RecoveryDevice.h > F: MdeModulePkg/Include/Ppi/StorageSecurityCommand.h > F: MdeModulePkg/Include/Protocol/Ps2Policy.h > F: MdeModulePkg/Library/NonDiscoverableDeviceRegistrationLib/ > > Disk: > F: MdeModulePkg/Universal/Disk/ > > EBC: > F: MdeModulePkg/Include/*/*Ebc*.h > F: MdeModulePkg/Include/Protocol/DebuggerConfiguration.h > F: MdeModulePkg/Universal/EbcDxe/ > > Firmware Update: > F: MdeModulePkg/*Capsule*/ > F: MdeModulePkg/Include/*/*Capsule*.h > F: MdeModulePkg/Include/Library/DisplayUpdateProgressLib.h > F: MdeModulePkg/Include/Library/FmpAuthenticationLib.h > F: MdeModulePkg/Include/Protocol/EsrtManagement.h > F: MdeModulePkg/Include/Protocol/FirmwareManagementProgress.h > F: MdeModulePkg/Library/DisplayUpdateProgressLib*/ > F: MdeModulePkg/Library/FmpAuthenticationLibNull/ > F: MdeModulePkg/Universal/Esrt*/ > > Graphic: > F: MdeModulePkg/*Logo*/ > F: MdeModulePkg/Include/*/*Logo*.h > F: MdeModulePkg/Include/Library/BmpSupportLib.h > F: MdeModulePkg/Include/Library/FrameBufferBltLib.h > F: MdeModulePkg/Library/BaseBmpSupportLib/ > F: MdeModulePkg/Library/FrameBufferBltLib/ > F: MdeModulePkg/Universal/Console/GraphicsOutputDxe/ > > HII/UI: > F: MdeModulePkg/*FileExplorer*/ > F: MdeModulePkg/*Hii*/ > F: MdeModulePkg/*Ui*/ > F: MdeModulePkg/Application/BootManagerMenuApp/ > F: MdeModulePkg/Include/*/*FileExplorer*.h > F: MdeModulePkg/Include/*/*FormBrowser*.h > F: MdeModulePkg/Include/*/*Hii*.h > F: MdeModulePkg/Include/Library/CustomizedDisplayLib.h > F: MdeModulePkg/Include/Protocol/DisplayProtocol.h > F: MdeModulePkg/Library/CustomizedDisplayLib/ > F: MdeModulePkg/Universal/DisplayEngineDxe/ > F: MdeModulePkg/Universal/SetupBrowserDxe/ > > IPMI: > F: MdeModulePkg/Include/*/*Ipmi*.h > F: MdeModulePkg/Library/*Ipmi*/ > > Memory Management: > F: MdeModulePkg/*Mem*/ > F: MdeModulePkg/Core/Dxe/Gcd/ > F: MdeModulePkg/Include/*/*Mem*.h > F: MdeModulePkg/Include/*/*IoMmu*.h > > PCD: > F: MdeModulePkg/Application/DumpDynPcd/ > F: MdeModulePkg/Include/*/*Pcd*.h > F: MdeModulePkg/Universal/PCD/ > > PCI Bus: > F: MdeModulePkg/*PciHostBridge*/ > F: MdeModulePkg/Bus/Pci/IncompatiblePciDeviceSupportDxe/ > F: MdeModulePkg/Bus/Pci/PciBusDxe/ > F: MdeModulePkg/Core/Pei/PciCfg2/ > F: MdeModulePkg/Include/Library/PciHostBridgeLib.h > F: MdeModulePkg/Universal/PcatSingleSegmentPciCfg2Pei/ > > Performance: > F: MdeModulePkg/Include/*/*Perf*.h > F: MdeModulePkg/Library/*Perf*/ > > Reset: > F: MdeModulePkg/*Reset*/ > F: MdeModulePkg/Include/*/*Reset*.h > > S3: > F: MdeModulePkg/*LockBox*/ > F: MdeModulePkg/Include/*/*BootScript*.h > F: MdeModulePkg/Include/*/*LockBox*.h > F: MdeModulePkg/Include/*/*S3*.h > F: MdeModulePkg/Library/*S3*/ > > SMBIOS: > F: MdeModulePkg/Universal/Smbios*/ > > SMM: > F: MdeModulePkg/*Smi*/ > F: MdeModulePkg/*Smm*/ > F: MdeModulePkg/Include/*/*Smi*.h > F: MdeModulePkg/Include/*/*Smm*.h > > Status Code: > F: MdeModulePkg/*StatusCode*/ > F: MdeModulePkg/Include/*/*StatusCode*.h > > Variable: > F: MdeModulePkg/*Var*/ > F: MdeModulePkg/Include/*/*FaultTolerantWrite*.h > F: MdeModulePkg/Include/*/*Var*.h > F: MdeModulePkg/Include/Guid/SystemNvDataGuid.h > F: MdeModulePkg/Include/Protocol/SwapAddressRange.h > F: MdeModulePkg/Universal/FaultTolerantWrite*/ > > Misc: > F: MdeModulePkg/Application/HelloWorld/ > F: MdeModulePkg/Include/Guid/F: MdeModulePkgTokenSpace.h > F: MdeModulePkg/Include/Guid/MtcVendor.h > F: MdeModulePkg/Include/Guid/ZeroGuid.h > F: MdeModulePkg/Include/Library/CpuExceptionHandlerLib.h > F: MdeModulePkg/Include/Library/PlatformHookLib.h > F: MdeModulePkg/Include/Library/RecoveryLib.h > F: MdeModulePkg/Include/Library/SortLib.h > F: MdeModulePkg/Include/Library/TpmMeasurementLib.h > F: MdeModulePkg/Include/Protocol/Dpc.h > F: MdeModulePkg/Include/Protocol/LoadPe32Image.h > F: MdeModulePkg/Include/Protocol/PeCoffImageEmulator.h > F: MdeModulePkg/Include/Protocol/Print2.h > F: MdeModulePkg/Library/BaseHobLibNull/ > F: MdeModulePkg/Library/BasePlatformHookLibNull/ > F: MdeModulePkg/Library/BaseSortLib/ > F: MdeModulePkg/Library/CpuExceptionHandlerLibNull/ > F: MdeModulePkg/Library/DxePrintLibPrint2Protocol/ > F: MdeModulePkg/Library/PeiRecoveryLibNull/ > F: MdeModulePkg/Library/PlatformHookLibSerialPortPpi/ > F: MdeModulePkg/Library/TpmMeasurementLibNull/ > F: MdeModulePkg/Library/UefiSortLib/ > F: MdeModulePkg/Universal/DevicePathDxe/ > F: MdeModulePkg/Universal/DriverHealthManagerDxe/ > F: MdeModulePkg/Universal/DriverSampleDxe/ > F: MdeModulePkg/Universal/FvSimpleFileSystemDxe/ > F: MdeModulePkg/Universal/LegacyRegion2Dxe/ > F: MdeModulePkg/Universal/Metronome/ > F: MdeModulePkg/Universal/MonotonicCounterRuntimeDxe/ > F: MdeModulePkg/Universal/PrintDxe/ > F: MdeModulePkg/Universal/RegularExpressionDxe/ > F: MdeModulePkg/Universal/TimestampDxe/ > F: MdeModulePkg/Universal/WatchdogTimerDxe/ > > > Best Regards, > Hao Wu ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [edk2-devel] [RFC] Fine-grained review ownership for MdeModulePkg 2019-07-16 13:53 ` Leif Lindholm @ 2019-07-17 1:47 ` Wu, Hao A 0 siblings, 0 replies; 13+ messages in thread From: Wu, Hao A @ 2019-07-17 1:47 UTC (permalink / raw) To: Leif Lindholm Cc: rfc@edk2.groups.io, devel@edk2.groups.io, Andrew Fish, Laszlo Ersek, Kinney, Michael D, Wang, Jian J, Ni, Ray, Zeng, Star > -----Original Message----- > From: Leif Lindholm [mailto:leif.lindholm@linaro.org] > Sent: Tuesday, July 16, 2019 9:53 PM > To: Wu, Hao A > Cc: rfc@edk2.groups.io; devel@edk2.groups.io; Andrew Fish; Laszlo Ersek; > Kinney, Michael D; Wang, Jian J; Ni, Ray; Zeng, Star > Subject: Re: [edk2-devel] [RFC] Fine-grained review ownership for > MdeModulePkg > > Hi Hao Wu, > > Are you ready to send out a v1 yet? (Or did I miss it?) > I would prefer to get this in before the GetMaintainer.py script, and > after the comment updates to Maintainers.txt. Sorry Leif, You did not miss anything. I was planning to send out the patch after your proposed changes are pushed. Now, I have sent out the patch at: https://edk2.groups.io/g/devel/message/43827 Best Regards, Hao Wu > > Best Regards, > > Leif > > On Mon, Jun 24, 2019 at 01:16:05AM +0000, Wu, Hao A wrote: > > Hello all, > > > > With Leif's comment on the better usage of the wildcard character & 'X:' tag: > > https://edk2.groups.io/g/devel/message/42587 > > > > The list is updated as shown below. > > > > Also, I forgot to mention that some directory/file may be classified into > > multiple classes. I think it is a valid case, for example: > > > > MdeModulePkg/Library/SmmMemoryAllocationProfileLib > > > > matches both 'Memory Management' and 'SMM' sections: > > > > Memory Management - F: MdeModulePkg/*Mem*/ > > SMM - F: MdeModulePkg/*Smm*/ > > > > Then, reviewers from both classes should be Cc-ed when changes are > made in > > the above library. > > > > If the updated list is good enough, the next step will be determining the > > reviewer(s) for each feature in the list. > > > > > > The updated list: > > ACPI: > > F: MdeModulePkg/Include/*/*Acpi*.h > > F: MdeModulePkg/Universal/Acpi/ > > > > BDS: > > F: MdeModulePkg/*BootManager*/ > > X: MdeModulePkg/Application/BootManagerMenuApp/ > > F: MdeModulePkg/Include/Library/UefiBootManagerLib.h > > F: MdeModulePkg/Universal/BdsDxe/ > > F: MdeModulePkg/Universal/LoadFileOnFv2/ > > F: MdeModulePkg/Universal/SecurityStubDxe/Defer3rdPartyImageLoad.* > > > > Console: > > F: MdeModulePkg/Include/Guid/ConnectConInEvent.h > > F: MdeModulePkg/Include/Guid/Console*.h > > F: MdeModulePkg/Include/Guid/StandardErrorDevice.h > > F: MdeModulePkg/Include/Guid/TtyTerm.h > > F: MdeModulePkg/Universal/Console/ > > X: MdeModulePkg/Universal/Console/GraphicsOutputDxe/ > > > > Core (PEI, DXE and Runtime): > > F: MdeModulePkg/*SectionExtract*/ > > F: MdeModulePkg/Core/Dxe/ > > X: MdeModulePkg/Core/Dxe/Gcd/ > > X: MdeModulePkg/Core/Dxe/Mem/ > > F: MdeModulePkg/Core/DxeIplPeim/ > > F: MdeModulePkg/Core/Pei/ > > X: MdeModulePkg/Core/Pei/Memory/ > > X: MdeModulePkg/Core/Pei/PciCfg2/ > > X: MdeModulePkg/Core/Pei/Reset/ > > X: MdeModulePkg/Core/Pei/StatusCode/ > > F: MdeModulePkg/Core/RuntimeDxe/ > > F: MdeModulePkg/Include/Guid/Crc32GuidedSectionExtraction.h > > F: MdeModulePkg/Include/Guid/EventExitBootServiceFailed.h > > F: MdeModulePkg/Include/Guid/IdleLoopEvent.h > > F: MdeModulePkg/Include/Guid/LoadModuleAtFixedAddress.h > > F: MdeModulePkg/Include/Library/SecurityManagementLib.h > > F: MdeModulePkg/Library/DxeSecurityManagementLib/ > > F: MdeModulePkg/Universal/PlatformDriOverrideDxe/ > > F: MdeModulePkg/Universal/SecurityStubDxe/SecurityStub.c > > > > Debug: > > F: MdeModulePkg/*Debug*/ > > X: MdeModulePkg/Universal/EbcDxe/EbcDebugger/ > > F: MdeModulePkg/Include/Guid/DebugMask.h > > F: MdeModulePkg/Include/Library/DebugAgentLib.h > > F: MdeModulePkg/Include/Ppi/Debug.h > > > > Decompress: > > F: MdeModulePkg/Include/Guid/LzmaDecompress.h > > F: MdeModulePkg/Library/*Decompress*/ > > > > Device: > > F: MdeModulePkg/*Serial*/ > > X: MdeModulePkg/Library/PlatformHookLibSerialPortPpi/ > > F: MdeModulePkg/Bus/ > > X: MdeModulePkg/Bus/Pci/IncompatiblePciDeviceSupportDxe/ > > X: MdeModulePkg/Bus/Pci/PciBusDxe/ > > X: MdeModulePkg/Bus/Pci/PciHostBridgeDxe/ > > F: MdeModulePkg/Include/*/*Ata*.h > > F: MdeModulePkg/Include/*/*NonDiscoverableDevice*.h > > F: MdeModulePkg/Include/*/*NvmExpress*.h > > F: MdeModulePkg/Include/*/*SerialPort*.h > > F: MdeModulePkg/Include/*/*SdMmc*.h > > F: MdeModulePkg/Include/*/*Ufs*.h > > F: MdeModulePkg/Include/*/*Usb*.h > > F: MdeModulePkg/Include/Guid/S3StorageDeviceInitList.h > > F: MdeModulePkg/Include/Guid/RecoveryDevice.h > > F: MdeModulePkg/Include/Ppi/StorageSecurityCommand.h > > F: MdeModulePkg/Include/Protocol/Ps2Policy.h > > F: MdeModulePkg/Library/NonDiscoverableDeviceRegistrationLib/ > > > > Disk: > > F: MdeModulePkg/Universal/Disk/ > > > > EBC: > > F: MdeModulePkg/Include/*/*Ebc*.h > > F: MdeModulePkg/Include/Protocol/DebuggerConfiguration.h > > F: MdeModulePkg/Universal/EbcDxe/ > > > > Firmware Update: > > F: MdeModulePkg/*Capsule*/ > > F: MdeModulePkg/Include/*/*Capsule*.h > > F: MdeModulePkg/Include/Library/DisplayUpdateProgressLib.h > > F: MdeModulePkg/Include/Library/FmpAuthenticationLib.h > > F: MdeModulePkg/Include/Protocol/EsrtManagement.h > > F: MdeModulePkg/Include/Protocol/FirmwareManagementProgress.h > > F: MdeModulePkg/Library/DisplayUpdateProgressLib*/ > > F: MdeModulePkg/Library/FmpAuthenticationLibNull/ > > F: MdeModulePkg/Universal/Esrt*/ > > > > Graphic: > > F: MdeModulePkg/*Logo*/ > > F: MdeModulePkg/Include/*/*Logo*.h > > F: MdeModulePkg/Include/Library/BmpSupportLib.h > > F: MdeModulePkg/Include/Library/FrameBufferBltLib.h > > F: MdeModulePkg/Library/BaseBmpSupportLib/ > > F: MdeModulePkg/Library/FrameBufferBltLib/ > > F: MdeModulePkg/Universal/Console/GraphicsOutputDxe/ > > > > HII/UI: > > F: MdeModulePkg/*FileExplorer*/ > > F: MdeModulePkg/*Hii*/ > > F: MdeModulePkg/*Ui*/ > > F: MdeModulePkg/Application/BootManagerMenuApp/ > > F: MdeModulePkg/Include/*/*FileExplorer*.h > > F: MdeModulePkg/Include/*/*FormBrowser*.h > > F: MdeModulePkg/Include/*/*Hii*.h > > F: MdeModulePkg/Include/Library/CustomizedDisplayLib.h > > F: MdeModulePkg/Include/Protocol/DisplayProtocol.h > > F: MdeModulePkg/Library/CustomizedDisplayLib/ > > F: MdeModulePkg/Universal/DisplayEngineDxe/ > > F: MdeModulePkg/Universal/SetupBrowserDxe/ > > > > IPMI: > > F: MdeModulePkg/Include/*/*Ipmi*.h > > F: MdeModulePkg/Library/*Ipmi*/ > > > > Memory Management: > > F: MdeModulePkg/*Mem*/ > > F: MdeModulePkg/Core/Dxe/Gcd/ > > F: MdeModulePkg/Include/*/*Mem*.h > > F: MdeModulePkg/Include/*/*IoMmu*.h > > > > PCD: > > F: MdeModulePkg/Application/DumpDynPcd/ > > F: MdeModulePkg/Include/*/*Pcd*.h > > F: MdeModulePkg/Universal/PCD/ > > > > PCI Bus: > > F: MdeModulePkg/*PciHostBridge*/ > > F: MdeModulePkg/Bus/Pci/IncompatiblePciDeviceSupportDxe/ > > F: MdeModulePkg/Bus/Pci/PciBusDxe/ > > F: MdeModulePkg/Core/Pei/PciCfg2/ > > F: MdeModulePkg/Include/Library/PciHostBridgeLib.h > > F: MdeModulePkg/Universal/PcatSingleSegmentPciCfg2Pei/ > > > > Performance: > > F: MdeModulePkg/Include/*/*Perf*.h > > F: MdeModulePkg/Library/*Perf*/ > > > > Reset: > > F: MdeModulePkg/*Reset*/ > > F: MdeModulePkg/Include/*/*Reset*.h > > > > S3: > > F: MdeModulePkg/*LockBox*/ > > F: MdeModulePkg/Include/*/*BootScript*.h > > F: MdeModulePkg/Include/*/*LockBox*.h > > F: MdeModulePkg/Include/*/*S3*.h > > F: MdeModulePkg/Library/*S3*/ > > > > SMBIOS: > > F: MdeModulePkg/Universal/Smbios*/ > > > > SMM: > > F: MdeModulePkg/*Smi*/ > > F: MdeModulePkg/*Smm*/ > > F: MdeModulePkg/Include/*/*Smi*.h > > F: MdeModulePkg/Include/*/*Smm*.h > > > > Status Code: > > F: MdeModulePkg/*StatusCode*/ > > F: MdeModulePkg/Include/*/*StatusCode*.h > > > > Variable: > > F: MdeModulePkg/*Var*/ > > F: MdeModulePkg/Include/*/*FaultTolerantWrite*.h > > F: MdeModulePkg/Include/*/*Var*.h > > F: MdeModulePkg/Include/Guid/SystemNvDataGuid.h > > F: MdeModulePkg/Include/Protocol/SwapAddressRange.h > > F: MdeModulePkg/Universal/FaultTolerantWrite*/ > > > > Misc: > > F: MdeModulePkg/Application/HelloWorld/ > > F: MdeModulePkg/Include/Guid/F: MdeModulePkgTokenSpace.h > > F: MdeModulePkg/Include/Guid/MtcVendor.h > > F: MdeModulePkg/Include/Guid/ZeroGuid.h > > F: MdeModulePkg/Include/Library/CpuExceptionHandlerLib.h > > F: MdeModulePkg/Include/Library/PlatformHookLib.h > > F: MdeModulePkg/Include/Library/RecoveryLib.h > > F: MdeModulePkg/Include/Library/SortLib.h > > F: MdeModulePkg/Include/Library/TpmMeasurementLib.h > > F: MdeModulePkg/Include/Protocol/Dpc.h > > F: MdeModulePkg/Include/Protocol/LoadPe32Image.h > > F: MdeModulePkg/Include/Protocol/PeCoffImageEmulator.h > > F: MdeModulePkg/Include/Protocol/Print2.h > > F: MdeModulePkg/Library/BaseHobLibNull/ > > F: MdeModulePkg/Library/BasePlatformHookLibNull/ > > F: MdeModulePkg/Library/BaseSortLib/ > > F: MdeModulePkg/Library/CpuExceptionHandlerLibNull/ > > F: MdeModulePkg/Library/DxePrintLibPrint2Protocol/ > > F: MdeModulePkg/Library/PeiRecoveryLibNull/ > > F: MdeModulePkg/Library/PlatformHookLibSerialPortPpi/ > > F: MdeModulePkg/Library/TpmMeasurementLibNull/ > > F: MdeModulePkg/Library/UefiSortLib/ > > F: MdeModulePkg/Universal/DevicePathDxe/ > > F: MdeModulePkg/Universal/DriverHealthManagerDxe/ > > F: MdeModulePkg/Universal/DriverSampleDxe/ > > F: MdeModulePkg/Universal/FvSimpleFileSystemDxe/ > > F: MdeModulePkg/Universal/LegacyRegion2Dxe/ > > F: MdeModulePkg/Universal/Metronome/ > > F: MdeModulePkg/Universal/MonotonicCounterRuntimeDxe/ > > F: MdeModulePkg/Universal/PrintDxe/ > > F: MdeModulePkg/Universal/RegularExpressionDxe/ > > F: MdeModulePkg/Universal/TimestampDxe/ > > F: MdeModulePkg/Universal/WatchdogTimerDxe/ > > > > > > Best Regards, > > Hao Wu ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [edk2-devel] [RFC] Fine-grained review ownership for MdeModulePkg 2019-06-10 8:06 [RFC] Fine-grained review ownership for MdeModulePkg Wu, Hao A 2019-06-11 9:51 ` Leif Lindholm @ 2019-06-20 22:23 ` rebecca 2019-06-21 0:11 ` Wu, Hao A 1 sibling, 1 reply; 13+ messages in thread From: rebecca @ 2019-06-20 22:23 UTC (permalink / raw) To: devel, hao.a.wu Cc: lersek@redhat.com, Leif Lindholm, Kinney, Michael D, Andrew Fish, Justen, Jordan L, Ard Biesheuvel, Gao, Liming, Yao, Jiewen, Zeng, Star On 2019-06-10 02:06, Wu, Hao A wrote: > Hello all, > > Driven by the issue raised and discussions with thread: > https://edk2.groups.io/g/devel/topic/31907735#42076 > > This RFC proposal is raised to work out a fine-grained review ownership > for MdeModulePkg. Being an RFC, I think this should be CC'd to rfc@edk2.groups.io (https://edk2.groups.io/g/rfc)? -- Rebecca Cran ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [edk2-devel] [RFC] Fine-grained review ownership for MdeModulePkg 2019-06-20 22:23 ` rebecca @ 2019-06-21 0:11 ` Wu, Hao A 0 siblings, 0 replies; 13+ messages in thread From: Wu, Hao A @ 2019-06-21 0:11 UTC (permalink / raw) To: devel@edk2.groups.io, rebecca@bluestop.org Cc: lersek@redhat.com, Leif Lindholm, Kinney, Michael D, Andrew Fish, Justen, Jordan L, Ard Biesheuvel, Gao, Liming, Yao, Jiewen, Zeng, Star > -----Original Message----- > From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of > rebecca@bluestop.org > Sent: Friday, June 21, 2019 6:24 AM > To: devel@edk2.groups.io; Wu, Hao A > Cc: lersek@redhat.com; Leif Lindholm; Kinney, Michael D; Andrew Fish; > Justen, Jordan L; Ard Biesheuvel; Gao, Liming; Yao, Jiewen; Zeng, Star > Subject: Re: [edk2-devel] [RFC] Fine-grained review ownership for > MdeModulePkg > > On 2019-06-10 02:06, Wu, Hao A wrote: > > Hello all, > > > > Driven by the issue raised and discussions with thread: > > https://edk2.groups.io/g/devel/topic/31907735#42076 > > > > This RFC proposal is raised to work out a fine-grained review ownership > > for MdeModulePkg. > > > Being an RFC, I think this should be CC'd to rfc@edk2.groups.io > (https://edk2.groups.io/g/rfc)? Thanks for the reminder, I will post the next update to the above list. Best Regards, Hao Wu > > > -- > Rebecca Cran > > > ^ permalink raw reply [flat|nested] 13+ messages in thread
end of thread, other threads:[~2019-07-17 1:47 UTC | newest] Thread overview: 13+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2019-06-10 8:06 [RFC] Fine-grained review ownership for MdeModulePkg Wu, Hao A 2019-06-11 9:51 ` Leif Lindholm 2019-06-11 15:41 ` Laszlo Ersek 2019-06-19 5:09 ` Wu, Hao A 2019-06-19 9:35 ` Leif Lindholm 2019-06-20 15:43 ` [edk2-devel] " Laszlo Ersek 2019-06-24 1:16 ` Wu, Hao A 2019-06-24 20:29 ` Laszlo Ersek 2019-06-24 22:58 ` Yao, Jiewen 2019-07-16 13:53 ` Leif Lindholm 2019-07-17 1:47 ` Wu, Hao A 2019-06-20 22:23 ` rebecca 2019-06-21 0:11 ` Wu, Hao A
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox