Re: [edk2] [PATCH] UefiCpuPkg/CpuExceptionHandlerLib: always clear descriptor data in advance

2018-10-17 Thread Ni, Ruiyu
On 10/18/2018 1:57 AM, Laszlo Ersek wrote: Hi Jian, On 10/17/18 10:34, Jian J Wang wrote: REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1237 Sometimes the memory will be contaminated by random data left in last boot (warm reset). The code should not assume the allocated memory is

Re: [edk2] [PATCH] UefiCpuPkg/CpuExceptionHandlerLib: always clear descriptor data in advance

2018-10-17 Thread Dong, Eric
Reviewed-by: Eric Dong > -Original Message- > From: Wang, Jian J > Sent: Wednesday, October 17, 2018 4:35 PM > To: edk2-devel@lists.01.org > Cc: Dong, Eric ; Laszlo Ersek ; Ni, > Ruiyu > Subject: [PATCH] UefiCpuPkg/CpuExceptionHandlerLib: always clear > descriptor data in advance > >

Re: [edk2] [PATCH] UefiCpuPkg/CpuExceptionHandlerLib: always clear descriptor data in advance

2018-10-17 Thread Wang, Jian J
Laszlo, Thanks for the comments. I'll update the code. Regards, Jian > -Original Message- > From: Laszlo Ersek [mailto:ler...@redhat.com] > Sent: Thursday, October 18, 2018 1:57 AM > To: Wang, Jian J ; edk2-devel@lists.01.org > Cc: Dong, Eric ; Ni, Ruiyu > Subject: Re: [PATCH]

Re: [edk2] [PATCH] UefiCpuPkg/CpuExceptionHandlerLib: always clear descriptor data in advance

2018-10-17 Thread Laszlo Ersek
Hi Jian, On 10/17/18 10:34, Jian J Wang wrote: > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1237 > > Sometimes the memory will be contaminated by random data left in last > boot (warm reset). The code should not assume the allocated memory is > always filled with zero. This patch add

[edk2] [PATCH] UefiCpuPkg/CpuExceptionHandlerLib: always clear descriptor data in advance

2018-10-17 Thread Jian J Wang
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1237 Sometimes the memory will be contaminated by random data left in last boot (warm reset). The code should not assume the allocated memory is always filled with zero. This patch add code to clear data structure used for stack switch to