Re: [edk2-devel] [PATCH v2] UefiCpuPkg: Remove PEI/DXE instances of CpuTimerLib.

2021-04-09 Thread Laszlo Ersek
Hi Ray, On 04/09/21 02:45, Ni, Ray wrote: > Laszlo, > OVMF isn't using this timerlib so I will assume you doesn't care about > this change. correct; that's why I gave my ACK for v1 in last September [1] [2] [3] -- I didn't want to block progress on this patch. To be frank, I don't understand why

Re: [edk2-devel] [PATCH v2] UefiCpuPkg: Remove PEI/DXE instances of CpuTimerLib.

2021-04-08 Thread Ni, Ray
Dong, Eric ; Laszlo Ersek ; > Kumar, Rahul1 > Subject: Re: [edk2-devel] [PATCH v2] UefiCpuPkg: Remove PEI/DXE > instances of CpuTimerLib. > > Thanks for providing the detailed commit messages. > Reviewed-by: Ray Ni > > > > -Original Message- > > From

Re: [edk2-devel] [PATCH v2] UefiCpuPkg: Remove PEI/DXE instances of CpuTimerLib.

2021-04-07 Thread Ni, Ray
Thanks for providing the detailed commit messages. Reviewed-by: Ray Ni > -Original Message- > From: Lou, Yun > Sent: Wednesday, April 7, 2021 4:16 PM > To: devel@edk2.groups.io > Cc: Lou, Yun ; Ni, Ray ; Dong, Eric > ; Laszlo Ersek > ; Kumar, Rahul1 > Subject: [PATCH v2] UefiCpuPkg: R

[edk2-devel] [PATCH v2] UefiCpuPkg: Remove PEI/DXE instances of CpuTimerLib.

2021-04-07 Thread Jason Lou
From: Jason Lou REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2832 1. Remove PEI instance(PeiCpuTimerLib). PeiCpuTimerLib is currently designed to save time by getting CPU TSC frequency from Hob. BaseCpuTimerLib is designed to calculate TSC frequency by using CPUID[15h] each time. The time