[edk2-devel] [Patch V2 2/3] UefiCpuPkg: Add Unit tests for PeiCpuExceptionHandlerLib

[duntan]

Yes, the commit message in PEIM patch is the  same as the last patch.
I'll simply the commit message in this patch and rename the IA32 global variable.

Thanks,
Dun
-----Original Message-----
From: Ni, Ray <ray.ni at intel.com> 
Sent: Friday, October 14, 2022 1:54 PM
To: Tan, Dun <dun.tan at intel.com>; devel at edk2.groups.io
Cc: Dong, Eric <eric.dong at intel.com>; Kumar, Rahul R <rahul.r.kumar at intel.com>
Subject: RE: [Patch V2 2/3] UefiCpuPkg: Add Unit tests for PeiCpuExceptionHandlerLib

The commit message is the same as the last patch, right?

Maybe you could say:

The previous change adds unit test for DxeCpuExeptionHandlerLib in 64bit mode.
This change adds unit test for PeiCpuExceptionHandlerLib based on previous change.
It can run in both 32bit and 64bit modes.

> +extern GENERAL_REGISTER_IA32  mIa32ExpectedContextInHandler; extern 
> +GENERAL_REGISTER_IA32  mIa32ActualContextAfterException;
Can you use the same variable name for IA32?
The variable name is the same but the structure definitions are different.


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#95193): https://edk2.groups.io/g/devel/message/95193
Mute This Topic: https://groups.io/mt/94319172/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-