Re: [edk2] [v3] UefiCpuPkg: Move GetProcessorLocation() to LocalApicLib library

2016-10-31 Thread Duran, Leo
ael D; edk2-devel@lists.01.org > Cc: Fan, Jeff; Gao, Liming > Subject: Re: [edk2] [v3] UefiCpuPkg: Move GetProcessorLocation() to > LocalApicLib library > > On 10/28/16 23:17, Duran, Leo wrote: > > Replies below. > > Thanks. > > > >> -Original Message--

Re: [edk2] [v3] UefiCpuPkg: Move GetProcessorLocation() to LocalApicLib library

2016-10-30 Thread Fan, Jeff
Ersek Sent: Saturday, October 29, 2016 5:48 AM To: Duran, Leo; Kinney, Michael D; edk2-devel@lists.01.org Cc: Fan, Jeff; Gao, Liming Subject: Re: [edk2] [v3] UefiCpuPkg: Move GetProcessorLocation() to LocalApicLib library On 10/28/16 23:17, Duran, Leo wrote: > Replies below.

Re: [edk2] [v3] UefiCpuPkg: Move GetProcessorLocation() to LocalApicLib library

2016-10-28 Thread Laszlo Ersek
On 10/28/16 23:17, Duran, Leo wrote: > Replies below. > Thanks. > >> -Original Message- >> From: Kinney, Michael D [mailto:michael.d.kin...@intel.com] >> Sent: Friday, October 28, 2016 4:12 PM >> To: Duran, Leo ; edk2-devel@lists.01.org >> Cc: Gao, Liming ; ler...@redhat.com; Fan, Jeff >>

Re: [edk2] [v3] UefiCpuPkg: Move GetProcessorLocation() to LocalApicLib library

2016-10-28 Thread Duran, Leo
Replies below. Thanks. > -Original Message- > From: Kinney, Michael D [mailto:michael.d.kin...@intel.com] > Sent: Friday, October 28, 2016 4:12 PM > To: Duran, Leo ; edk2-devel@lists.01.org > Cc: Gao, Liming ; ler...@redhat.com; Fan, Jeff > > Subject: RE: [v3] UefiCpuPkg: Move GetProcesso

Re: [edk2] [v3] UefiCpuPkg: Move GetProcessorLocation() to LocalApicLib library

2016-10-28 Thread Kinney, Michael D
Leo, I tested your patch on Quark and it builds and boots. Here is a slightly updated version of the patch that makes the following changes: * Follow Jeff Fan suggestion to pass in Package, Core, Thread as 3 parameters. This removes the dependency on PiPei.h and MpService.h from LocalApicLib.

Re: [edk2] [v3] UefiCpuPkg: Move GetProcessorLocation() to LocalApicLib library

2016-10-28 Thread Laszlo Ersek
On 10/28/16 19:38, Leo Duran wrote: > 1) Remove SmmGetProcessorLocation() from PiSmmCpuDxeSmm driver. > 2) Remove ExtractProcessorLocation() from MpInitLib library. > 3) Add GetProcessorLocation() to BaseXApicLib and BaseXApicX2ApicLib. > > Contributed-under: TianoCore Contribution Agreement 1.0 >

[edk2] [v3] UefiCpuPkg: Move GetProcessorLocation() to LocalApicLib library

2016-10-28 Thread Leo Duran
1) Remove SmmGetProcessorLocation() from PiSmmCpuDxeSmm driver. 2) Remove ExtractProcessorLocation() from MpInitLib library. 3) Add GetProcessorLocation() to BaseXApicLib and BaseXApicX2ApicLib. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Leo Duran --- UefiCpuPkg/Incl