Reviewed-by: Liming Gao <liming....@intel.com> >-----Original Message----- >From: Gao, Zhichao >Sent: Friday, April 12, 2019 8:42 AM >To: devel@edk2.groups.io >Cc: Yao, Jiewen <jiewen....@intel.com>; Zhang, Chao B ><chao.b.zh...@intel.com>; Wang, Jian J <jian.j.w...@intel.com>; Wu, Hao A ><hao.a...@intel.com>; Ni, Ray <ray...@intel.com>; Zeng, Star ><star.z...@intel.com>; Kinney, Michael D <michael.d.kin...@intel.com>; >Gao, Liming <liming....@intel.com>; Sean Brogan ><sean.bro...@microsoft.com>; Michael Turner ><michael.tur...@microsoft.com>; Bret Barkelew ><bret.barke...@microsoft.com> >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 <jiewen....@intel.com> >Cc: Chao Zhang <chao.b.zh...@intel.com> >Cc: Jian J Wang <jian.j.w...@intel.com> >Cc: Hao Wu <hao.a...@intel.com> >Cc: Ray Ni <ray...@intel.com> >Cc: Star Zeng <star.z...@intel.com> >Cc: Michael D Kinney <michael.d.kin...@intel.com> >Cc: Liming Gao <liming....@intel.com> >Cc: Sean Brogan <sean.bro...@microsoft.com> >Cc: Michael Turner <michael.tur...@microsoft.com> >Cc: Bret Barkelew <bret.barke...@microsoft.com> > >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: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
