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

2019-06-11 Thread Laszlo Ersek
On 06/10/19 09:16, Ni, Ray wrote: > Hao, > Can you please list the detailed sub-directories/modules/libraries for each > major feature, including those that are not covered by the major features? > This avoid misunderstanding and is also needed by the Maintainers.txt. I agree. Every module INF

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

2019-06-10 Thread Ard Biesheuvel
gt; > ler...@redhat.com; Gao, Liming; Leif Lindholm; Kinney, Michael D; Liu, > > Zhiguang; Justen, Jordan L; Andrew Fish > > Subject: Re: [edk2-devel] [Patch V2] EmulatorPkg: don't diaplay the cpu > > current speed > > > > On Mon, 10 Jun 2019 at 09:33, Ni, Ray wrote

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

2019-06-10 Thread Wu, Hao A
> Zhiguang; Justen, Jordan L; Andrew Fish > Subject: Re: [edk2-devel] [Patch V2] EmulatorPkg: don't diaplay the cpu > current speed > > On Mon, 10 Jun 2019 at 09:33, Ni, Ray wrote: > > > > > > Hao, > > > > Can you please list the detailed sub-directories/mo

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

2019-06-10 Thread Yao, Jiewen
; ard.biesheu...@linaro.org; Leif > Lindholm ; Kinney, Michael D > ; Liu, Zhiguang ; > Justen, Jordan L ; Andrew Fish > > Subject: RE: [edk2-devel] [Patch V2] EmulatorPkg: don't diaplay the cpu > current speed > > > -Original Message- > > From: Yao, Jiew

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

2019-06-10 Thread Ard Biesheuvel
eature first before > > your > > request. > > I am ok. > That sounds like a good idea. But currrently, this discussion is taking place in the context of the thread '[edk2-devel] [Patch V2] EmulatorPkg: don't diaplay the cpu current speed' which may not be of interest to everybody. So cou

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

2019-06-10 Thread Ni, Ray
> > Hao, > > Can you please list the detailed sub-directories/modules/libraries for > > each major feature, including those that are not covered by the major > features? > > This avoid misunderstanding and is also needed by the Maintainers.txt. > > Hello Ray, > > Okay. I can do that. > > But I

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

2019-06-10 Thread Wu, Hao A
rew Fish > Subject: RE: [edk2-devel] [Patch V2] EmulatorPkg: don't diaplay the cpu > current speed > > Hao, > Can you please list the detailed sub-directories/modules/libraries for each > major feature, including those that are not covered by the major features? > This av

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

2019-06-10 Thread Ni, Ray
ming; ard.biesheu...@linaro.org; Leif Lindholm; > > Kinney, Michael D; Liu, Zhiguang; Justen, Jordan L; Andrew Fish > > Subject: RE: [edk2-devel] [Patch V2] EmulatorPkg: don't diaplay the > > cpu current speed > > > > Good list. > > Thanks Jiewen, > >

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

2019-06-10 Thread Wu, Hao A
rew Fish > Subject: RE: [edk2-devel] [Patch V2] EmulatorPkg: don't diaplay the cpu > current speed > > Good list. Thanks Jiewen, > > What does "memory" mean ? MemoryTest ? Not only MemoryTest, I was intended to count components like memory allocation, memory profile

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

2019-06-10 Thread Yao, Jiewen
> Sent: Monday, June 10, 2019 2:02 PM > To: devel@edk2.groups.io; Zeng, Star ; > ler...@redhat.com; Ni, Ray ; Gao, Liming > ; ard.biesheu...@linaro.org; Leif Lindholm > ; Kinney, Michael D > ; Liu, Zhiguang ; > Justen, Jordan L ; Andrew Fish > > Subject: Re

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

2019-06-10 Thread Wu, Hao A
ey, 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@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of

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

2019-06-05 Thread Zeng, Star
ten, Jordan L ; > Andrew Fish > Cc: devel@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

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

2019-06-05 Thread Laszlo Ersek
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

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

2019-06-04 Thread Liming Gao
ichael D ; Liu, Zhiguang > ; Justen, Jordan L ; > Andrew Fish > Subject: Re: [edk2-devel] [Patch V2] EmulatorPkg: don't diaplay the cpu > current speed > > On Tue, 4 Jun 2019 at 15:40, Gao, Liming wrote: > > > > Ard: > > > > > -Original Message- >

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

2019-06-04 Thread Ard Biesheuvel
Ray > > ; Leif Lindholm ; Gao, Liming > > ; Laszlo Ersek ; Kinney, Michael D > > > > Cc: Liu, Zhiguang ; Justen, Jordan L > > ; Andrew Fish > > Subject: Re: [edk2-devel] [Patch V2] EmulatorPkg: don't diaplay the cpu > > current speed > > > &

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

2019-06-04 Thread Liming Gao
gt; > Cc: Liu, Zhiguang ; Justen, Jordan L > ; Andrew Fish > Subject: Re: [edk2-devel] [Patch V2] EmulatorPkg: don't diaplay the cpu > current speed > > On Tue, 4 Jun 2019 at 10:50, Ni, Ray wrote: > > > > Zhiguang, > > I made a mistake here. > >

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

2019-06-04 Thread Ard Biesheuvel
On Tue, 4 Jun 2019 at 10:50, Ni, Ray wrote: > > Zhiguang, > I made a mistake here. > I thought you changed the EmulatorPkg to not display the CPU speed > but actually what you changed impacts all platforms. > With this change, OVMF cannot show the CPU speed either. > > Can you please roll back

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

2019-06-04 Thread Ni, Ray
ay > Sent: Monday, June 3, 2019 2:52 PM > To: devel@edk2.groups.io; Liu, Zhiguang > Cc: Justen, Jordan L ; Andrew Fish > > Subject: Re: [edk2-devel] [Patch V2] EmulatorPkg: don't diaplay the cpu > current speed > > Reviewed-by: Ray Ni > > > -Original M

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

2019-06-03 Thread Ni, Ray
Reviewed-by: Ray Ni > -Original Message- > From: devel@edk2.groups.io On Behalf Of > Zhiguang Liu > Sent: Monday, June 3, 2019 2:45 PM > To: devel@edk2.groups.io > Cc: Justen, Jordan L ; Andrew Fish > ; Ni, Ray > Subject: [edk2-devel] [Patch V2] EmulatorPk