Re: [edk2] [PATCH] UefiCpuPkg/PiSmmCpuDxeSmm: Lock should be acquired

2017-04-25 Thread Laszlo Ersek
gt; Sent: Monday, April 24, 2017 7:41 PM > To: Fan, Jeff; edk2-devel@lists.01.org > Cc: Wu, Hao A; Kinney, Michael D; Tian, Feng > Subject: Re: [edk2] [PATCH] UefiCpuPkg/PiSmmCpuDxeSmm: Lock should be acquired > > Hi Jeff, > > On 04/18/17 04:16, Jeff Fan wrote: >>

Re: [edk2] [PATCH] UefiCpuPkg/PiSmmCpuDxeSmm: Lock should be acquired

2017-04-24 Thread Fan, Jeff
t: Re: [edk2] [PATCH] UefiCpuPkg/PiSmmCpuDxeSmm: Lock should be acquired Hi Jeff, On 04/18/17 04:16, Jeff Fan wrote: > SMM BSP's *busy* state should be acquired. We could use > AcquireSpinLock() instead of AcquireSpinLockOrFail(). > > Cc: Hao Wu > Cc: Feng Tian > Cc: Michael

Re: [edk2] [PATCH] UefiCpuPkg/PiSmmCpuDxeSmm: Lock should be acquired

2017-04-24 Thread Laszlo Ersek
Hi Jeff, On 04/18/17 04:16, Jeff Fan wrote: > SMM BSP's *busy* state should be acquired. We could use AcquireSpinLock() > instead of AcquireSpinLockOrFail(). > > Cc: Hao Wu > Cc: Feng Tian > Cc: Michael Kinney > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Jeff Fan

Re: [edk2] [PATCH] UefiCpuPkg/PiSmmCpuDxeSmm: Lock should be acquired

2017-04-18 Thread Wu, Hao A
Reviewed-by: Hao Wu Best Regards, Hao Wu > -Original Message- > From: Fan, Jeff > Sent: Tuesday, April 18, 2017 10:16 AM > To: edk2-devel@lists.01.org > Cc: Wu, Hao A; Tian, Feng; Kinney, Michael D > Subject: [PATCH] UefiCpuPkg/PiSmmCpuDxeSmm: Lock should be acquired > > SMM BSP's *bu

[edk2] [PATCH] UefiCpuPkg/PiSmmCpuDxeSmm: Lock should be acquired

2017-04-17 Thread Jeff Fan
SMM BSP's *busy* state should be acquired. We could use AcquireSpinLock() instead of AcquireSpinLockOrFail(). Cc: Hao Wu Cc: Feng Tian Cc: Michael Kinney Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jeff Fan --- UefiCpuPkg/PiSmmCpuDxeSmm/MpService.c | 2 +- 1 file ch