Re: [edk2-devel][Patch 0/3] Add destructor to CloseEvent

2019-07-26 Thread Liming Gao
Thanks for your fix. The patch set is good. 

Reviewed-by: Liming Gao 

>-Original Message-
>From: Xu, Wei6
>Sent: Friday, July 26, 2019 11:11 AM
>To: devel@edk2.groups.io
>Cc: Kinney, Michael D ; Gao, Liming
>
>Subject: [edk2-devel][Patch 0/3] Add destructor to CloseEvent
>
>REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2012
>
>This patch set adds missing destructor for following libraries:
>MdePkg\Library\UefiDebugLibStdErr
>MdePkg\Library\UefiDebugLibConOut
>MdePkg\Library\UefiDebugLibDebugPortProtocol
>
>When driver is unloaded, the ExitBootSerivesEvent must be closed at
>the same time. Otherwise exception will occur when ExitBootServices.
>
>Cc: Michael D Kinney 
>Cc: Liming Gao 
>
>Wei6 Xu (3):
>  MdePkg/UefiDebugLibConOut: Add destructor to CloseEvent
>  MdePkg/UefiDebugLibDebugPortProtocol: Add destructor to CloseEvent
>  MdePkg/UefiDebugLibStdErr: Add destructor to CloseEvent
>
> .../UefiDebugLibConOut/DebugLibConstructor.c   | 23
>++
> .../UefiDebugLibConOut/UefiDebugLibConOut.inf  |  1 +
> .../DebugLibConstructor.c  | 23 ++
> .../UefiDebugLibDebugPortProtocol.inf  |  1 +
> .../UefiDebugLibStdErr/DebugLibConstructor.c   | 23
>++
> .../UefiDebugLibStdErr/UefiDebugLibStdErr.inf  |  1 +
> 6 files changed, 72 insertions(+)
>
>--
>2.16.2.windows.1


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#44450): https://edk2.groups.io/g/devel/message/44450
Mute This Topic: https://groups.io/mt/32605737/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[edk2-devel][Patch 0/3] Add destructor to CloseEvent

2019-07-25 Thread Xu, Wei6
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2012

This patch set adds missing destructor for following libraries:
MdePkg\Library\UefiDebugLibStdErr
MdePkg\Library\UefiDebugLibConOut
MdePkg\Library\UefiDebugLibDebugPortProtocol

When driver is unloaded, the ExitBootSerivesEvent must be closed at
the same time. Otherwise exception will occur when ExitBootServices.

Cc: Michael D Kinney 
Cc: Liming Gao 

Wei6 Xu (3):
  MdePkg/UefiDebugLibConOut: Add destructor to CloseEvent
  MdePkg/UefiDebugLibDebugPortProtocol: Add destructor to CloseEvent
  MdePkg/UefiDebugLibStdErr: Add destructor to CloseEvent

 .../UefiDebugLibConOut/DebugLibConstructor.c   | 23 ++
 .../UefiDebugLibConOut/UefiDebugLibConOut.inf  |  1 +
 .../DebugLibConstructor.c  | 23 ++
 .../UefiDebugLibDebugPortProtocol.inf  |  1 +
 .../UefiDebugLibStdErr/DebugLibConstructor.c   | 23 ++
 .../UefiDebugLibStdErr/UefiDebugLibStdErr.inf  |  1 +
 6 files changed, 72 insertions(+)

-- 
2.16.2.windows.1


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#44432): https://edk2.groups.io/g/devel/message/44432
Mute This Topic: https://groups.io/mt/32605737/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-