Reviewed-by: Liming Gao <[email protected]> >-----Original Message----- >From: Gao, Zhichao >Sent: Friday, April 12, 2019 8:42 AM >To: [email protected] >Cc: Yao, Jiewen <[email protected]>; Zhang, Chao B ><[email protected]>; Wang, Jian J <[email protected]>; Wu, Hao A ><[email protected]>; Ni, Ray <[email protected]>; Zeng, Star ><[email protected]>; Kinney, Michael D <[email protected]>; >Gao, Liming <[email protected]>; Sean Brogan ><[email protected]>; Michael Turner ><[email protected]>; Bret Barkelew ><[email protected]> >Subject: [PATCH V2 0/5] Make some DebugLib instance runtime safe > >REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1416 > >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. > >V2: >Fix some typos. >Change the comment of function to meet the Doxygen rule. > >Cc: Jiewen Yao <[email protected]> >Cc: Chao Zhang <[email protected]> >Cc: Jian J Wang <[email protected]> >Cc: Hao Wu <[email protected]> >Cc: Ray Ni <[email protected]> >Cc: Star Zeng <[email protected]> >Cc: Michael D Kinney <[email protected]> >Cc: Liming Gao <[email protected]> >Cc: Sean Brogan <[email protected]> >Cc: Michael Turner <[email protected]> >Cc: Bret Barkelew <[email protected]> > >Aaron Antone (3): > MdePkg/UefiDebugLibDebugPortProtocol: Make it runtime safe > MdePkg/UefidebugLibConOut: Make it runtime safe > MdePkg/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 | 77 +++++++++ > .../UefiDebugLibConOut/UefiDebugLibConOut.inf | 12 +- > .../UefiDebugLibDebugPortProtocol/DebugLib.c | 146 +++++++++--------- > .../DebugLibConstructor.c | 77 +++++++++ > .../UefiDebugLibDebugPortProtocol.inf | 12 +- > MdePkg/Library/UefiDebugLibStdErr/DebugLib.c | 113 +++++++------- > .../UefiDebugLibStdErr/DebugLibConstructor.c | 77 +++++++++ > .../UefiDebugLibStdErr/UefiDebugLibStdErr.inf | 12 +- > SignedCapsulePkg/SignedCapsulePkg.dsc | 4 +- > 11 files changed, 458 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 (#39194): https://edk2.groups.io/g/devel/message/39194 Mute This Topic: https://groups.io/mt/31037316/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
