Re: [edk2] [PATCH 1/5] OvmfPkg: introduce Q35TsegSizeLib (class header and sole lib instance)

2017-07-01 Thread Jordan Justen
On 2017-07-01 13:42:06, Laszlo Ersek wrote: > On 06/29/17 21:14, Jordan Justen wrote: > > On 2017-06-26 16:04:41, Laszlo Ersek wrote: > >> On 06/27/17 00:36, Jordan Justen wrote: > >>> > >>> Why can't SmramInternal.c read the size from the PCD directly? > >> > >> If I remember correctly, I wanted t

[edk2] [PATCH v2] UefiCpuPkg: ApicLib

2017-07-01 Thread Leo Duran
Now that we have a function to detect AMD processors: 1) SendInitSipiSipi () Skip repeating SendIpi () on AMD processor. 2) SendInitSipiSipiAllExcludingSelf () Skip repeating SendIpi () on AMD processor. 3) GetProcessorLocationByApicId () Adjust InitialApicId to properly concaten

[edk2] [PATCH v2] UefiCpuPkg: ApicLib

2017-07-01 Thread Leo Duran
1) SendInitSipiSipi () Skip repeating SendIpi () on AMD processor. 2) SendInitSipiSipiAllExcludingSelf () Skip repeating SendIpi () on AMD processor. 3) GetProcessorLocationByApicId () Adjust InitialApicId to properly concatenate Package on AMD processor. Clean-ups on C Coding standards. Cc: Jor

Re: [edk2] [PATCH 0/3] MdeModulePkg/ResetSystem: Implement ResetNotification protocol

2017-07-01 Thread Laszlo Ersek
Ard, Leif, On 06/29/17 10:32, Ruiyu Ni wrote: > Ruiyu Ni (3): > MdePkg: Add ResetNotification protocol definition > MdeModulePkg/ResetSystem: Remove unnecessary global variable > MdeModulePkg/ResetSystem: Implement ResetNotification protocol > > .../Universal/ResetSystemRuntimeDxe/ResetSys

Re: [edk2] [PATCH 1/5] OvmfPkg: introduce Q35TsegSizeLib (class header and sole lib instance)

2017-07-01 Thread Laszlo Ersek
On 06/29/17 21:14, Jordan Justen wrote: > On 2017-06-26 16:04:41, Laszlo Ersek wrote: >> On 06/27/17 00:36, Jordan Justen wrote: >>> >>> Why can't SmramInternal.c read the size from the PCD directly? >> >> If I remember correctly, I wanted to base SmramAccessGetCapabilities() >> directly on hardwar

Re: [edk2] [PATCH 1/2] OvmfPkg: disable build-time relocation for DXEFV modules

2017-07-01 Thread Laszlo Ersek
Liming, On 06/29/17 05:32, Gao, Liming wrote: > Laszlo: > LMFA feature doesn't do PE image rebase at build time. Only XIP > module needs to be rebased at build time. Thank you for the clarification. In this case, BaseTools/GenFv has a bug. Namely, if LMFA does not need build-time relocation

Re: [edk2] [PATCH 2/2] OvmfPkg: update -D E1000_ENABLE from Intel PROEFI v.07 to BootUtil v.22

2017-07-01 Thread Laszlo Ersek
On 06/30/17 03:20, Wu, Jiaxin wrote: > After patch with "[PATCH 1/2] disable build-time relocation for DXEFV > modules", I can pass the platform build, then HTTP boot feature has been > verified. > > Tested-by: Wu Jiaxin > Reviewed-by: Wu Jiaxin > Thank you! Laszlo __