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

Xu, Wei6 posted 3 patches 4 years, 9 months ago
Failed in applying to current master (apply log)
.../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(+)
[edk2-devel][Patch 0/3] Add destructor to CloseEvent
Posted by Xu, Wei6 4 years, 9 months ago
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 <michael.d.kinney@intel.com>
Cc: Liming Gao <liming.gao@intel.com>

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/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel][Patch 0/3] Add destructor to CloseEvent
Posted by Liming Gao 4 years, 9 months ago
Thanks for your fix. The patch set is good. 

Reviewed-by: Liming Gao <liming.gao@intel.com>

>-----Original Message-----
>From: Xu, Wei6
>Sent: Friday, July 26, 2019 11:11 AM
>To: devel@edk2.groups.io
>Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Gao, Liming
><liming.gao@intel.com>
>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 <michael.d.kinney@intel.com>
>Cc: Liming Gao <liming.gao@intel.com>
>
>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/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-