tyler.durden
/
audk
mirror of https://github.com/acidanthera/audk.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

EmulatorPkg/Win: Unload DLLs before reset 

EmulatorPkg/Win calls LoadLibraryEx() when the corresponding DLL file
is found for each PEIM or DXE driver. The module entry point is
changed to point to the entry point from the DLL. This helps to
notify Visual Studio that a new windows module is loaded and
corresponding symbol parsing is performed for source level debugging.

But entry point from the DLL is only executed when the module is not
loaded by AddModHandle().
When reset happens, we need to clear the DLL loading so that in next
boot the module can be loaded again by AddModHandle().

Without this patch, source level debugging doesn't work after reset.

Signed-off-by: Ray Ni <ray.ni@intel.com>
Reviewed-by: Zhiguang Liu <zhiguang.liu@intel.com>
Cc: Andrew Fish <afish@apple.com>
This commit is contained in:
Ray Ni 2022-12-05 14:51:18 +08:00 committed by mergify[bot]
parent 2280af5ff8
commit 4e17aba4b5
1 changed files with 14 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
EmulatorPkg/Win/Host/WinHost.c
View File

@ -226,12 +226,26 @@ WinReset (
IN VOID *ResetData OPTIONAL
)
{
UINTN Index;
ASSERT (ResetType <= EfiResetPlatformSpecific);
SecPrint (" Emu ResetSystem is called: ResetType = %s\n", mResetTypeStr[ResetType]);
if (ResetType == EfiResetShutdown) {
exit (0);
} else {
//
// Unload all DLLs
//
for (Index = 0; Index < mPdbNameModHandleArraySize; Index++) {
if (mPdbNameModHandleArray[Index].PdbPointer != NULL) {
SecPrint (" Emu Unload DLL: %s\n", mPdbNameModHandleArray[Index].PdbPointer);
FreeLibrary (mPdbNameModHandleArray[Index].ModHandle);
HeapFree (GetProcessHeap (), 0, mPdbNameModHandleArray[Index].PdbPointer);
mPdbNameModHandleArray[Index].PdbPointer = NULL;
}
}
//
// Jump back to SetJump with jump code = ResetType + 1
//