Re: [edk2] [PATCH] UefiCpuPkg: Move GetProcessorLocation() to SmmCpuFeaturesLib library.

2016-10-28 Thread Kinney, Michael D
9 AM > To: Fan, Jeff ; Duran, Leo ; Kinney, > Michael D > > Cc: edk2-devel@lists.01.org ; Gao, Liming > > Subject: Re: [edk2] [PATCH] UefiCpuPkg: Move GetProcessorLocation() to > SmmCpuFeaturesLib library. > > On 10/28/16 06:43, Fan, Jeff wrote: > > Leo, > &g

Re: [edk2] [PATCH] UefiCpuPkg: Move GetProcessorLocation() to SmmCpuFeaturesLib library.

2016-10-28 Thread Duran, Leo
k2-devel@lists.01.org ; Gao, Liming > > Subject: RE: [edk2] [PATCH] UefiCpuPkg: Move GetProcessorLocation() to > SmmCpuFeaturesLib library. > > Hi Laszlo, > > Quark DSC files do map the LocalApicLib, so I think the proposal to add > GetProcessorLocation() to the LocalApicLib

Re: [edk2] [PATCH] UefiCpuPkg: Move GetProcessorLocation() to SmmCpuFeaturesLib library.

2016-10-28 Thread Duran, Leo
ael D [mailto:michael.d.kin...@intel.com] > > Sent: Friday, October 28, 2016 10:29 AM > > To: Laszlo Ersek ; Fan, Jeff ; > > Duran, Leo ; Kinney, Michael D > > > > Cc: edk2-devel@lists.01.org ; Gao, Liming > > > > Subject: RE: [edk2] [PATCH] UefiCpuPkg: Move

Re: [edk2] [PATCH] UefiCpuPkg: Move GetProcessorLocation() to SmmCpuFeaturesLib library.

2016-10-28 Thread Fan, Jeff
[mailto:leo.du...@amd.com] Sent: Friday, October 28, 2016 11:31 PM To: Kinney, Michael D; Laszlo Ersek; Fan, Jeff Cc: edk2-devel@lists.01.org; Gao, Liming Subject: RE: [edk2] [PATCH] UefiCpuPkg: Move GetProcessorLocation() to SmmCpuFeaturesLib library. OK, the LocalApicLib patch is coming... please stay

Re: [edk2] [PATCH] UefiCpuPkg: Move GetProcessorLocation() to SmmCpuFeaturesLib library.

2016-10-28 Thread Duran, Leo
ing > Subject: RE: [edk2] [PATCH] UefiCpuPkg: Move GetProcessorLocation() to > SmmCpuFeaturesLib library. > > Leo, > > Got it. Could you please create the patch to add GetProcessorLocation() in > Local APIC Lib? > > There are two library instances required to be

Re: [edk2] [PATCH] UefiCpuPkg: Move GetProcessorLocation() to SmmCpuFeaturesLib library.

2016-10-28 Thread Laszlo Ersek
; > Thanks! > Jeff > > -Original Message- > From: Duran, Leo [mailto:leo.du...@amd.com] > Sent: Friday, October 28, 2016 11:20 AM > To: Fan, Jeff; Kinney, Michael D; edk2-devel@lists.01.org > Cc: Gao, Liming > Subject: RE: [edk2] [PATCH] UefiCpuPkg: Move GetProces

Re: [edk2] [PATCH] UefiCpuPkg: Move GetProcessorLocation() to SmmCpuFeaturesLib library.

2016-10-27 Thread Fan, Jeff
an, Leo] Sure thing. The main point is to get the vendor-specific CPUID code out of the driver, and into a library. > > -Original Message- > From: Fan, Jeff > Sent: Friday, October 28, 2016 8:45 AM > To: Kinney, Michael D; Leo Duran; edk2-devel@lists.01.org > Cc: Gao, Limin

Re: [edk2] [PATCH] UefiCpuPkg: Move GetProcessorLocation() to SmmCpuFeaturesLib library.

2016-10-27 Thread Duran, Leo
Sure thing. The main point is to get the vendor-specific CPUID code out of the driver, and into a library. > > -Original Message- > From: Fan, Jeff > Sent: Friday, October 28, 2016 8:45 AM > To: Kinney, Michael D; Leo Duran; edk2-devel@lists.01.org > Cc: Gao, Liming &g

Re: [edk2] [PATCH] UefiCpuPkg: Move GetProcessorLocation() to SmmCpuFeaturesLib library.

2016-10-27 Thread Fan, Jeff
Leo Duran > Sent: Thursday, October 27, 2016 3:30 PM > To: edk2-devel@lists.01.org > Cc: Leo Duran ; Gao, Liming > Subject: [edk2] [PATCH] UefiCpuPkg: Move GetProcessorLocation() to > SmmCpuFeaturesLib library. > > 1) Remove SmmGetProcessorLocation() from

Re: [edk2] [PATCH] UefiCpuPkg: Move GetProcessorLocation() to SmmCpuFeaturesLib library.

2016-10-27 Thread Fan, Jeff
l-boun...@lists.01.org] On Behalf Of > Leo Duran > Sent: Thursday, October 27, 2016 3:30 PM > To: edk2-devel@lists.01.org > Cc: Leo Duran ; Gao, Liming > Subject: [edk2] [PATCH] UefiCpuPkg: Move GetProcessorLocation() to > SmmCpuFeaturesLib library. > > 1) Remove

Re: [edk2] [PATCH] UefiCpuPkg: Move GetProcessorLocation() to SmmCpuFeaturesLib library.

2016-10-27 Thread Kinney, Michael D
nt: Thursday, October 27, 2016 3:30 PM > To: edk2-devel@lists.01.org > Cc: Leo Duran ; Gao, Liming > Subject: [edk2] [PATCH] UefiCpuPkg: Move GetProcessorLocation() to > SmmCpuFeaturesLib > library. > > 1) Remove SmmGetProcessorLocation() from PiSmmCpuDxeSmm driver > 2) Ad

[edk2] [PATCH] UefiCpuPkg: Move GetProcessorLocation() to SmmCpuFeaturesLib library.

2016-10-27 Thread Leo Duran
1) Remove SmmGetProcessorLocation() from PiSmmCpuDxeSmm driver 2) Add SmmCpuFeaturesGetProcessorLocation() to SmmCpuFeaturesLib library Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Leo Duran --- UefiCpuPkg/Include/Library/SmmCpuFeaturesLib.h | 17 +++ .../Library/