[edk2-devel] [Patch V2] EmulatorPkg: don't diaplay the cpu current speed

[Wu, Hao A]

> -----Original Message-----
> From: devel at edk2.groups.io [mailto:devel at edk2.groups.io] On Behalf Of
> Zeng, Star
> Sent: Wednesday, June 05, 2019 10:31 PM
> To: devel at edk2.groups.io; lersek at redhat.com; Ni, Ray; Gao, Liming;
> ard.biesheuvel at linaro.org; Leif Lindholm; Kinney, Michael D; Liu, Zhiguang;
> Justen, Jordan L; Andrew Fish
> Cc: Zeng, Star
> Subject: Re: [edk2-devel] [Patch V2] EmulatorPkg: don't diaplay the cpu
> current speed
> 
> > -----Original Message-----
> > From: devel at edk2.groups.io [mailto:devel at edk2.groups.io] On Behalf Of
> > Laszlo Ersek
> > Sent: Wednesday, June 5, 2019 4:03 PM
> > To: Ni, Ray <ray.ni at intel.com>; Gao, Liming <liming.gao at intel.com>;
> > ard.biesheuvel at linaro.org; Leif Lindholm <leif.lindholm at linaro.org>;
> Kinney,
> > Michael D <michael.d.kinney at intel.com>; Liu, Zhiguang
> > <zhiguang.liu at intel.com>; Justen, Jordan L <jordan.l.justen at intel.com>;
> > Andrew Fish <afish at apple.com>
> > Cc: devel at edk2.groups.io
> > Subject: Re: [edk2-devel] [Patch V2] EmulatorPkg: don't diaplay the cpu
> > current speed
> >
> > On 06/05/19 03:10, Ni, Ray wrote:
> > > Hi everyone,
> > >
> > > Hao pushed the patch because:
> > > 1. it's a bug fix
> > > 2. it got a R-b.
> > > I don't think it's his fault.
> > >
> > > For #1, it's gap in process.
> > > For #2, it's my fault. Because even the patch title says EmulatorPkg but
> the
> > patch itself changes MdeModulePkg, I am still the person who can approve
> > changes in both packages.
> > >
> > > In Maintainers.txt:
> > > R: Ray Ni <ray.ni at intel.com>
> > >   (especially for Bus, Universal/Console, Universal/Disk,
> > >    Universal/BdsDxe and related libraries, header files)
> > >
> > > The UiApp actually is not in the "especially" part, but the word
> "especially"
> > > means I can also R-b to other components. In fact, I am not quite
> > > qualified on the other components, e.g.: UiApp.
> > > So I propose to assign clear reviewers for each components in
> > MdeModulePkg.
> > > What do you think?
> >
> > Fully agreed. MdeModulePkg is huge and we should have fine-grained
> > maintainership assignments.
> 
> I also agree.

Thanks all.

For the review ownership refinement in the MdeModulePkg, how about the
below steps:

1. Work out a major feature list for the MdeModulePkg
2. Call for owners for every item in the list.


A quick and coarse list of major features can be:

(Just a draft, some key features may be missing or some listed features
may not be considered as major ones. Feedbacks are welcomed.)

Acpi
Bds
Capsule
Compress
Console
Core (PeiCore, DxeCore, PiSmmCore and etc.)
Device (peripheral & storage)
Disk
HII
LockBox
Memory
PCD
PCI bus
Reset
S3
SMBIOS
SMM
Status Code
UI
Variable
Misc (all the components that do not fit in the above items)


Any feedbacks/suggestions for such proposal?

Best Regards,
Hao Wu

> 
> >
> > Thanks
> > Laszlo
> >
> >
> 
> 
> 


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#42060): https://edk2.groups.io/g/devel/message/42060
Mute This Topic: https://groups.io/mt/31907735/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-