Oops. Laszlo, I pushed the fix with only Dandan's R-b. Outlook groups Dandan's and your mail in different threads.
Thanks/Ray > -----Original Message----- > From: edk2-devel <[email protected]> On Behalf Of Laszlo > Ersek > Sent: Friday, September 7, 2018 6:32 PM > To: Ni, Ruiyu <[email protected]>; [email protected] > Cc: Bi, Dandan <[email protected]> > Subject: Re: [edk2] [PATCH] UefiCpuPkg/PeiCpuException: Fix coding style > issue > > On 09/07/18 12:13, Ruiyu Ni wrote: > > Contributed-under: TianoCore Contribution Agreement 1.1 > > Signed-off-by: Ruiyu Ni <[email protected]> > > Cc: Dandan Bi <[email protected]> > > --- > > UefiCpuPkg/Library/CpuExceptionHandlerLib/PeiCpuException.c | 6 > > +++--- > > 1 file changed, 3 insertions(+), 3 deletions(-) > > > > diff --git > > a/UefiCpuPkg/Library/CpuExceptionHandlerLib/PeiCpuException.c > > b/UefiCpuPkg/Library/CpuExceptionHandlerLib/PeiCpuException.c > > index 5dd8423d2f..8f4d5b5e0a 100644 > > --- a/UefiCpuPkg/Library/CpuExceptionHandlerLib/PeiCpuException.c > > +++ b/UefiCpuPkg/Library/CpuExceptionHandlerLib/PeiCpuException.c > > @@ -45,7 +45,7 @@ GetExceptionHandlerData ( > > > > AsmReadIdtr (&IdtDescriptor); > > IdtTable = (IA32_IDT_GATE_DESCRIPTOR *)IdtDescriptor.Base; > > - > > + > > Exception0StubHeader = (EXCEPTION0_STUB_HEADER > *)ArchGetIdtHandler (&IdtTable[0]); > > return Exception0StubHeader->ExceptionHandlerData; > > } > > @@ -57,7 +57,7 @@ GetExceptionHandlerData ( > > exception handler data. The new allocated memory layout follows > structure EXCEPTION0_STUB_HEADER. > > The code assumes that all processors uses the same exception handler > for #0 exception. > > > > - @param pointer to exception handler data. > > + @param ExceptionHandlerData pointer to exception handler data. > > **/ > > VOID > > SetExceptionHandlerData ( > > @@ -73,7 +73,7 @@ SetExceptionHandlerData ( > > // > > AsmReadIdtr (&IdtDescriptor); > > IdtTable = (IA32_IDT_GATE_DESCRIPTOR *)IdtDescriptor.Base; > > - > > + > > Exception0StubHeader = AllocatePool (sizeof (*Exception0StubHeader)); > > ASSERT (Exception0StubHeader != NULL); > > CopyMem ( > > > > Reviewed-by: Laszlo Ersek <[email protected]> > _______________________________________________ > edk2-devel mailing list > [email protected] > https://lists.01.org/mailman/listinfo/edk2-devel _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel
