Re: [edk2-devel] [PATCH 0/5] Make some DebugLib instance runtime safe

2019-04-11 Thread Wu, Hao A
> -Original Message-
> From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of
> Gao, Zhichao
> Sent: Wednesday, April 10, 2019 3:03 PM
> To: devel@edk2.groups.io
> Cc: Yao, Jiewen; Zhang, Chao B; Wang, Jian J; Ni, Ray; Zeng, Star; Gao, 
> Liming;
> Sean Brogan; Michael Turner; Bret Barkelew
> Subject: [edk2-devel] [PATCH 0/5] Make some DebugLib instance runtime
> safe
> 
> Some pointer will be invalid after ExitBootServices, such as protocol pointer,
> gST, gBS and so on. Disable the functions which used that pointer.
> 
> Remove SMM support of UefiDebugLibDebugPortProtocol,
> UefidebugLibConOut
> and UefiDebugLibStdErr.
> Before removing, should change the SMM instance in the dsc file which use
> them as SMM instance. And null version is safe.
> 
> Cc: Jiewen Yao 
> Cc: Chao Zhang 
> Cc: Jian J Wang 
> Cc: Ray Ni 
> Cc: Star Zeng 
> Cc: Liming Gao 
> Cc: Sean Brogan 
> Cc: Michael Turner 
> Cc: Bret Barkelew 
> 
> Aaron Antone (3):
>   MdeModulePkg/UefiDebugLibDebugPortProtocol: Make it runtime safe
>   MdeModulePkg/UefidebugLibConOut: Make it runtime safe
>   MdeModule/UefiDebugLibStdErr: Make it runtime safe

Seems some typos here.
Should be MdePkg instead of MdeModulePkg (MdeModule).

Best Regards,
Hao Wu

> 
> Zhichao Gao (2):
>   MdeModulePkg: Change the SMM debug lib instance
>   SignedCapsulePkg: Change the SMM debug lib instance
> 
>  MdeModulePkg/MdeModulePkg.dsc |   2 +-
>  MdePkg/Library/UefiDebugLibConOut/DebugLib.c  | 116 +++---
>  .../UefiDebugLibConOut/DebugLibConstructor.c  |  72 +
>  .../UefiDebugLibConOut/UefiDebugLibConOut.inf |  12 +-
>  .../UefiDebugLibDebugPortProtocol/DebugLib.c  | 146 +-
>  .../DebugLibConstructor.c |  72 +
>  .../UefiDebugLibDebugPortProtocol.inf |  12 +-
>  MdePkg/Library/UefiDebugLibStdErr/DebugLib.c  | 113 +++---
>  .../UefiDebugLibStdErr/DebugLibConstructor.c  |  72 +
>  .../UefiDebugLibStdErr/UefiDebugLibStdErr.inf |  12 +-
>  SignedCapsulePkg/SignedCapsulePkg.dsc |   4 +-
>  11 files changed, 443 insertions(+), 190 deletions(-)
>  create mode 100644
> MdePkg/Library/UefiDebugLibConOut/DebugLibConstructor.c
>  create mode 100644
> MdePkg/Library/UefiDebugLibDebugPortProtocol/DebugLibConstructor.c
>  create mode 100644
> MdePkg/Library/UefiDebugLibStdErr/DebugLibConstructor.c
> 
> --
> 2.21.0.windows.1
> 
> 
> 


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

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



[edk2-devel] [PATCH 0/5] Make some DebugLib instance runtime safe

2019-04-10 Thread Gao, Zhichao
Some pointer will be invalid after ExitBootServices, such as protocol pointer,
gST, gBS and so on. Disable the functions which used that pointer.

Remove SMM support of UefiDebugLibDebugPortProtocol, UefidebugLibConOut
and UefiDebugLibStdErr.
Before removing, should change the SMM instance in the dsc file which use
them as SMM instance. And null version is safe.

Cc: Jiewen Yao 
Cc: Chao Zhang 
Cc: Jian J Wang 
Cc: Ray Ni 
Cc: Star Zeng 
Cc: Liming Gao 
Cc: Sean Brogan 
Cc: Michael Turner 
Cc: Bret Barkelew 

Aaron Antone (3):
  MdeModulePkg/UefiDebugLibDebugPortProtocol: Make it runtime safe
  MdeModulePkg/UefidebugLibConOut: Make it runtime safe
  MdeModule/UefiDebugLibStdErr: Make it runtime safe

Zhichao Gao (2):
  MdeModulePkg: Change the SMM debug lib instance
  SignedCapsulePkg: Change the SMM debug lib instance

 MdeModulePkg/MdeModulePkg.dsc |   2 +-
 MdePkg/Library/UefiDebugLibConOut/DebugLib.c  | 116 +++---
 .../UefiDebugLibConOut/DebugLibConstructor.c  |  72 +
 .../UefiDebugLibConOut/UefiDebugLibConOut.inf |  12 +-
 .../UefiDebugLibDebugPortProtocol/DebugLib.c  | 146 +-
 .../DebugLibConstructor.c |  72 +
 .../UefiDebugLibDebugPortProtocol.inf |  12 +-
 MdePkg/Library/UefiDebugLibStdErr/DebugLib.c  | 113 +++---
 .../UefiDebugLibStdErr/DebugLibConstructor.c  |  72 +
 .../UefiDebugLibStdErr/UefiDebugLibStdErr.inf |  12 +-
 SignedCapsulePkg/SignedCapsulePkg.dsc |   4 +-
 11 files changed, 443 insertions(+), 190 deletions(-)
 create mode 100644 MdePkg/Library/UefiDebugLibConOut/DebugLibConstructor.c
 create mode 100644 
MdePkg/Library/UefiDebugLibDebugPortProtocol/DebugLibConstructor.c
 create mode 100644 MdePkg/Library/UefiDebugLibStdErr/DebugLibConstructor.c

-- 
2.21.0.windows.1


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

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