Re: [edk2-devel] [PATCH] UefiCpuPkg/CpuExceptionHandlerLib: Fix #AC split lock

2019-09-08 Thread Dong, Eric
; Laszlo Ersek (ler...@redhat.com) > > Subject: RE: [edk2-devel] [PATCH] UefiCpuPkg/CpuExceptionHandlerLib: Fix > #AC split lock > > Hi Eric, > > 1. you are correct, we need to need to handle 63-32 bits. I attach image which > I think will help explain it better. sidt and

Re: [edk2-devel] [PATCH] UefiCpuPkg/CpuExceptionHandlerLib: Fix #AC split lock

2019-09-06 Thread John E Lofgren
xchg because it works with aligned memory. Thank you, John >-Original Message- >From: Dong, Eric >Sent: Thursday, September 5, 2019 12:37 AM >To: devel@edk2.groups.io; Lofgren, John E >Cc: Ni, Ray ; Laszlo Ersek (ler...@redhat.com) >; Dong, Eric >Subject: RE: [edk2

Re: [edk2-devel] [PATCH] UefiCpuPkg/CpuExceptionHandlerLib: Fix #AC split lock

2019-09-05 Thread Dong, Eric
Hi John, I'm not sure whether I understand the code correctly. If not, please correct me. 1. You change to the code to only exchange 32 bits(eax) instead of 64 bits(rax). After your change, how to handle the above 32 bits value (from bit 32 to bit 63)? 2. In this file, also have another two xch