* Re: [edk2-devel] [PATCH] EmulatorPkg/Win: Unload DLLs before reset
2022-12-07 8:07 ` [edk2-devel] [PATCH] EmulatorPkg/Win: Unload DLLs before reset Ni, Ray
@ 2022-12-07 8:34 ` Zhiguang Liu
0 siblings, 0 replies; 2+ messages in thread
From: Zhiguang Liu @ 2022-12-07 8:34 UTC (permalink / raw)
To: Ni, Ray, devel@edk2.groups.io; +Cc: Andrew Fish
Yes, the patch looks fine.
Reviewed-by: Zhiguang Liu <zhiguang.liu@intel.com>
> -----Original Message-----
> From: Ni, Ray <ray.ni@intel.com>
> Sent: Wednesday, December 7, 2022 4:08 PM
> To: devel@edk2.groups.io; Ni, Ray <ray.ni@intel.com>
> Cc: Liu, Zhiguang <zhiguang.liu@intel.com>; Andrew Fish <afish@apple.com>
> Subject: RE: [edk2-devel] [PATCH] EmulatorPkg/Win: Unload DLLs before
> reset
>
> Zhiguang, can you please help to review this patch? I guess Andrew is busy
> on something.
>
> > -----Original Message-----
> > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Ni,
> Ray
> > Sent: Monday, December 5, 2022 2:59 PM
> > To: devel@edk2.groups.io
> > Cc: Liu, Zhiguang <zhiguang.liu@intel.com>; Andrew Fish
> > <afish@apple.com>
> > Subject: [edk2-devel] [PATCH] 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>
> > Cc: Zhiguang Liu <zhiguang.liu@intel.com>
> > Cc: Andrew Fish <afish@apple.com>
> > ---
> > EmulatorPkg/Win/Host/WinHost.c | 14 ++++++++++++++
> > 1 file changed, 14 insertions(+)
> >
> > diff --git a/EmulatorPkg/Win/Host/WinHost.c
> > b/EmulatorPkg/Win/Host/WinHost.c index 096292f95a..084cd4cbd7 100644
> > --- a/EmulatorPkg/Win/Host/WinHost.c
> > +++ b/EmulatorPkg/Win/Host/WinHost.c
> > @@ -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
> >
> > //
> >
> > --
> > 2.37.2.windows.2
> >
> >
> >
> > -=-=-=-=-=-=
> > Groups.io Links: You receive all messages sent to this group.
> > View/Reply Online (#96943):
> > https://edk2.groups.io/g/devel/message/96943
> > Mute This Topic: https://groups.io/mt/95464183/1712937
> > Group Owner: devel+owner@edk2.groups.io
> > Unsubscribe: https://edk2.groups.io/g/devel/unsub [ray.ni@intel.com]
> > -=-=-=-=-=-=
> >
^ permalink raw reply [flat|nested] 2+ messages in thread