Thanks Liming help :) Donald > -----Original Message----- > From: Gao, Liming > Sent: Wednesday, August 21, 2019 9:46 PM > To: Zeng, Star <star.z...@intel.com>; Kuo, Donald <donald....@intel.com>; > devel@edk2.groups.io; ler...@redhat.com; Dong, Eric > <eric.d...@intel.com> > Cc: Ni, Ray <ray...@intel.com>; Chan, Amy <amy.c...@intel.com>; > Chaganty, Rangasai V <rangasai.v.chaga...@intel.com>; Lai, Luke > <luke....@intel.com>; Li, Kevin Y <kevin.y...@intel.com>; > leif.lindh...@linaro.org; af...@apple.com; Kinney, Michael D > <michael.d.kin...@intel.com> > Subject: RE: [edk2-devel] [PATCH] UefiCpuPkg: Adding a new TSC library by > using CPUID(0x15) TSC leaf > > Done. And, push @30781febe2106cc0d7186e70136120353cd67df2 > > Thanks > Liming > > -----Original Message----- > > From: Zeng, Star > > Sent: Tuesday, August 20, 2019 10:00 PM > > To: Gao, Liming <liming....@intel.com>; Kuo, Donald > > <donald....@intel.com>; devel@edk2.groups.io; ler...@redhat.com; > Dong, > > Eric <eric.d...@intel.com> > > Cc: Ni, Ray <ray...@intel.com>; Chan, Amy <amy.c...@intel.com>; > > Chaganty, Rangasai V <rangasai.v.chaga...@intel.com>; Lai, Luke > > <luke....@intel.com>; Li, Kevin Y <kevin.y...@intel.com>; > > leif.lindh...@linaro.org; af...@apple.com; Kinney, Michael D > > <michael.d.kin...@intel.com>; Zeng, Star <star.z...@intel.com> > > Subject: RE: [edk2-devel] [PATCH] UefiCpuPkg: Adding a new TSC library > > by using CPUID(0x15) TSC leaf > > > > Remember to add entry for it at > https://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Release- > Planning. > > > > > -----Original Message----- > > > From: Gao, Liming > > > Sent: Tuesday, August 20, 2019 7:56 PM > > > To: Kuo, Donald <donald....@intel.com>; devel@edk2.groups.io; > > > ler...@redhat.com; Dong, Eric <eric.d...@intel.com> > > > Cc: Ni, Ray <ray...@intel.com>; Zeng, Star <star.z...@intel.com>; > > > Chan, Amy <amy.c...@intel.com>; Chaganty, Rangasai V > > > <rangasai.v.chaga...@intel.com>; Lai, Luke <luke....@intel.com>; Li, > > > Kevin Y <kevin.y...@intel.com>; leif.lindh...@linaro.org; > > > af...@apple.com; Kinney, Michael D <michael.d.kin...@intel.com> > > > Subject: RE: [edk2-devel] [PATCH] UefiCpuPkg: Adding a new TSC > > > library by using CPUID(0x15) TSC leaf > > > > > > Donald: > > > Thanks for your update. If no other comment, I will help push this > > > patch tomorrow. > > > > > > Thanks > > > Liming > > > > -----Original Message----- > > > > From: Kuo, Donald > > > > Sent: Tuesday, August 20, 2019 3:22 PM > > > > To: Gao, Liming <liming....@intel.com>; devel@edk2.groups.io; > > > > ler...@redhat.com; Dong, Eric <eric.d...@intel.com> > > > > Cc: Ni, Ray <ray...@intel.com>; Zeng, Star <star.z...@intel.com>; > > > > Chan, Amy <amy.c...@intel.com>; Chaganty, Rangasai V > > > > <rangasai.v.chaga...@intel.com>; Lai, Luke <luke....@intel.com>; > > > > Li, Kevin Y <kevin.y...@intel.com>; leif.lindh...@linaro.org; > > > > af...@apple.com; Kinney, Michael D <michael.d.kin...@intel.com> > > > > Subject: RE: [edk2-devel] [PATCH] UefiCpuPkg: Adding a new TSC > > > > library by using CPUID(0x15) TSC leaf > > > > > > > > Hi Liming, > > > > > > > > Done. > > > > > > > > Patch is attached to > > > > https://bugzilla.tianocore.org/show_bug.cgi?id=1909 > > > > > > > > Another BZ to apply CpuTimerLib will be tracking on: > > > > https://bugzilla.tianocore.org/show_bug.cgi?id=2096 > > > > > > > > Thanks, > > > > Donald > > > > > > > > > -----Original Message----- > > > > > From: Gao, Liming > > > > > Sent: Tuesday, August 20, 2019 2:51 PM > > > > > To: Kuo, Donald <donald....@intel.com>; devel@edk2.groups.io; > > > > > ler...@redhat.com; Dong, Eric <eric.d...@intel.com> > > > > > Cc: Ni, Ray <ray...@intel.com>; Zeng, Star > > > > > <star.z...@intel.com>; Chan, Amy <amy.c...@intel.com>; > Chaganty, > > > > > Rangasai V <rangasai.v.chaga...@intel.com>; Lai, Luke > > > > > <luke....@intel.com>; Li, Kevin Y <kevin.y...@intel.com>; > > > > > leif.lindh...@linaro.org; af...@apple.com; Kinney, Michael D > > > > > <michael.d.kin...@intel.com> > > > > > Subject: RE: [edk2-devel] [PATCH] UefiCpuPkg: Adding a new TSC > > > > > library by using CPUID(0x15) TSC leaf > > > > > > > > > > Donald: > > > > > Please also attach the patch linker in BZs. > > > > > > > > > > And, please submit another BZ for edk2- > > > > > platforms\Platform\Intel\KabylakeOpenBoardPkg to apply this new > > > > > library instance. > > > > > > > > > > Thanks > > > > > Liming > > > > > >-----Original Message----- > > > > > >From: Kuo, Donald > > > > > >Sent: Tuesday, August 20, 2019 10:44 AM > > > > > >To: devel@edk2.groups.io; ler...@redhat.com; Gao, Liming > > > > > ><liming....@intel.com>; Dong, Eric <eric.d...@intel.com> > > > > > >Cc: Ni, Ray <ray...@intel.com>; Zeng, Star > > > > > ><star.z...@intel.com>; Chan, Amy <amy.c...@intel.com>; > > > > > >Chaganty, Rangasai V <rangasai.v.chaga...@intel.com>; Lai, Luke > > > > > ><luke....@intel.com>; Li, Kevin Y <kevin.y...@intel.com>; > > > > > >leif.lindh...@linaro.org; af...@apple.com; Kinney, Michael D > > > > > ><michael.d.kin...@intel.com> > > > > > >Subject: RE: [edk2-devel] [PATCH] UefiCpuPkg: Adding a new TSC > > > > > >library by using CPUID(0x15) TSC leaf > > > > > > > > > > > >Thanks Laszlo help to review and great feedbacks. That we did > > > > > >miss to fulfil > > > > > BZ. > > > > > > > > > > > >I had updated Bugzilla > > > > > >https://bugzilla.tianocore.org/show_bug.cgi?id=1909 > > > > > >for more documentation. > > > > > > > > > > > >As I know for the edk2-platforms should be consumed as KBL (7th > > > > > >Generation) platform in Client, and this feature based on SDM > > > > > >is supported on SKL (6th Generation, Family 06h) onwards. So > > > > > >it's ok to use as TimerLib instances for edk2-platforms. > > > > > > > > > > > >And I think the library is new instances for TimerLib for > > > > > >supported CPU, and those non-supported CPU will still keep > > > > > >using AcpiTimerlib as TimerLib instances. > > > > > > > > > > > >Thanks, > > > > > >Donald > > > > > > > > > > > >> -----Original Message----- > > > > > >> From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On > > > > > >> Behalf Of Laszlo Ersek > > > > > >> Sent: Saturday, August 17, 2019 4:40 AM > > > > > >> To: Gao, Liming <liming....@intel.com>; Kuo, Donald > > > > > >> <donald....@intel.com>; Dong, Eric <eric.d...@intel.com>; > > > > > >> devel@edk2.groups.io > > > > > >> Cc: Ni, Ray <ray...@intel.com>; Zeng, Star > > > > > >> <star.z...@intel.com>; Chan, Amy <amy.c...@intel.com>; > > > > > >> Chaganty, Rangasai V <rangasai.v.chaga...@intel.com>; Lai, > > > > > >> Luke <luke....@intel.com>; Li, Kevin Y > > > > > >> <kevin.y...@intel.com>; leif.lindh...@linaro.org; > > > > > >> af...@apple.com; > > > > > >Kinney, > > > > > >> Michael D <michael.d.kin...@intel.com> > > > > > >> Subject: Re: [edk2-devel] [PATCH] UefiCpuPkg: Adding a new > > > > > >> TSC library by using CPUID(0x15) TSC leaf > > > > > >> > > > > > >> On 08/16/19 18:16, Laszlo Ersek wrote: > > > > > >> > On 08/15/19 06:02, Gao, Liming wrote: > > > > > >> >> Donald: This change is a new feature. Now, it is not in > > > > > >> >> edk2 feature planning list. If you want to catch it into > > > > > >> >> 201908 stable tag, please get approve from Stewards first. > > > > > >> >> I have cc this mail to all > > > > > Stewards. > > > > > >> > - I don't mind adding a new feature, as long as it gets > > > > > >> > properly reviewed by package owners before we enter the > > > > > >> > soft > > > feature freeze. > > > > > >> > > > > > > >> > - Looking at the BZ > > > > > >> > <https://bugzilla.tianocore.org/show_bug.cgi?id=1909>, a > > > > > >> > bit more documentation would be nice. > > > > > >> > > > > > > >> > - On the negative side, I'm very much *not* a fan of adding > > > > > >> > features to the open source edk2 tree without actually > > > > > >> > *consuming* the feature in an open source tree. Are the new > > > > > >> > library instances going to be put to use in edk2-platforms, > perhaps? > > > > > >> > > > > > > >> > We discussed this topic earlier on some of the stewards' calls. > > > > > >> > On one hand, it's not uncommon to see library instances > > > > > >> > from Intel enter core > > > > > >> > edk2 packages without any dependent platform code, or even > > > > > >> > a detailed problem statement / purpose description (see e.g. > > > > > >> > commit 5c9bb86f171c and its surrounding commits). On the > > > > > >> > other hand, attempts in the past, to add libraries with > > > > > >> > well demonstrated and direct in-tree use cases, to > > > > > >> > edk2 core, have been rejected, from other submitters. > > > > > >> > (Here's one > > > > > >> > example: > > > > > >> > <https://bugzilla.tianocore.org/show_bug.cgi?id=957>.) > > > > > >> > I'm not prying at proprietary platform information, but a > > > > > >> > new library added to > > > > > >> > edk2 core *should* be well-justified. > > > > > >> > > > > > > >> > The commit message on this patch is empty. It only > > > > > >> > references > > > > > >> > <https://bugzilla.tianocore.org/show_bug.cgi?id=1909>. And if I > open the BZ, this is all I get: > > > > > >> > > > > > > >> > Need a new TSC library to check the CPUID leaf (EAX=0x15) for > TSC. > > > > > >> > For new platform (start from SKL) can use CPUID and > > > retire/remove > > > > > >> > the current override from AcpiTimerLib. > > > > > >> > > > > > > >> > Does this read like an actual feature request? (TimerLib is > > > > > >> > an MdePkg library class, so not exactly "niche".) > > > > > >> > > > > > >> In comparison, the following email does read like a feature > > > > > >> request: > > > > > >> > > > > > >> [edk2-devel] Determining TSC frequency programmatically > > > > > >> https://edk2.groups.io/g/devel/message/45750 > > > > > >> http://mid.mail-archive.com/8EC14D0D-DFA5-412D-A4E1- > > > > > >> 4d641576d...@protonmail.com > > > > > >> > > > > > >> If the posting is related to TianoCore#1909, then I urge the > > > > > >> BZ assignee to please reference the message in the TianoCore BZ. > > > > > >> > > > > > >> Thanks > > > > > >> Laszlo > > > > > >> > > > > > >>
-=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#46161): https://edk2.groups.io/g/devel/message/46161 Mute This Topic: https://groups.io/mt/32839184/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-