Re: [edk2-devel] [PATCH EDK2 v1 1/1] MdeModulePkg/DxeCorePerformanceLib:Variable Initial

["Dandan Bi" <dandan.bi@intel.com>]

Just created the pull request for this patch 
https://github.com/tianocore/edk2/pull/2234



Thanks,
Dandan

> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of
> gaoliming
> Sent: Wednesday, December 1, 2021 9:53 AM
> To: devel@edk2.groups.io; xiewenyi2@huawei.com; Wang, Jian J
> <jian.j.wang@intel.com>; Bi, Dandan <dandan.bi@intel.com>
> Cc: songdongkuang@huawei.com
> Subject: 回复: [edk2-devel] [PATCH EDK2 v1 1/1]
> MdeModulePkg/DxeCorePerformanceLib:Variable Initial
> 
> Reviewed-by: Liming Gao <gaoliming@byosoft.com.cn>
> 
> > -----邮件原件-----
> > 发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 wenyi,xie
> via
> > groups.io
> > 发送时间: 2021年11月30日 9:51
> > 收件人: devel@edk2.groups.io; jian.j.wang@intel.com;
> > gaoliming@byosoft.com.cn; dandan.bi@intel.com
> > 抄送: songdongkuang@huawei.com; xiewenyi2@huawei.com
> > 主题: [edk2-devel] [PATCH EDK2 v1 1/1]
> > MdeModulePkg/DxeCorePerformanceLib:Variable Initial
> >
> > SmmBootRecordDataSize is initialized in InternalGetSmmPerData, but
> > this function may fail. so to avoid using SmmBootRecordDataSize
> > without intialization, set it to 0 at first.
> >
> > Cc: Jian J Wang <jian.j.wang@intel.com>
> > Cc: Liming Gao <gaoliming@byosoft.com.cn>
> > Cc: Dandan Bi <dandan.bi@intel.com>
> > Signed-off-by: Wenyi Xie <xiewenyi2@huawei.com>
> > ---
> >
> MdeModulePkg/Library/DxeCorePerformanceLib/DxeCorePerformanceLib.c
> > | 2 ++
> >  1 file changed, 2 insertions(+)
> >
> > diff --git
> >
> a/MdeModulePkg/Library/DxeCorePerformanceLib/DxeCorePerformanceLi
> b.
> > c
> >
> b/MdeModulePkg/Library/DxeCorePerformanceLib/DxeCorePerformanceLi
> b.
> > c
> > index 532b3f8c90e2..f8778c25188a 100644
> > ---
> >
> a/MdeModulePkg/Library/DxeCorePerformanceLib/DxeCorePerformanceLi
> b.
> > c
> > +++
> >
> b/MdeModulePkg/Library/DxeCorePerformanceLib/DxeCorePerformanceLi
> b.
> > c
> > @@ -1373,6 +1373,8 @@ UpdateBootPerformanceTable (
> >    UINTN                            AppendSize;
> >    UINT8                            *FirmwarePerformanceTablePtr;
> >
> > +  SmmBootRecordDataSize = 0;
> > +
> >    //
> >    // Get SMM performance data.
> >    //
> > --
> > 2.20.1.windows.1
> >
> >
> >
> >
> >
> 
> 
> 
> 
> 
> 
>