I have merged it by https://github.com/tianocore/edk2/pull/2865
Thanks Liming 发件人: Guo, Gua <[email protected]> 发送时间: 2022年5月9日 9:31 收件人: Gao, Liming <[email protected]>; [email protected] 抄送: Guo, Gua <[email protected]> 主题: RE: [edk2-devel] [PATCH] MdeModulePkg/PCD: Pcd initialize DXE have assert <mailto:[email protected]> @gaoliming May I get your help to add push Label on the PR ? https://github.com/tianocore/edk2/pull/2844 It seems I don't have access right to add push Label by myself. If have any concern please also share me. Thanks, Gua -----Original Message----- From: gaoliming <[email protected] <mailto:[email protected]> > Sent: Sunday, May 8, 2022 10:44 PM To: [email protected] <mailto:[email protected]> ; [email protected] <mailto:[email protected]> Cc: Guo, Gua <[email protected] <mailto:[email protected]> > Subject: 回复: [edk2-devel] [PATCH] MdeModulePkg/PCD: Pcd initialize DXE have assert Guo: This change is good to me. Reviewed-by: Liming Gao <[email protected]. cn> Thanks Liming > -----邮件原件----- > 发件人: <mailto:[email protected]> [email protected] < > <mailto:[email protected]> [email protected]> 代表 > <mailto:[email protected]> [email protected] > 发送时间: 2022年5月6日 5:30 > 收件人: <mailto:[email protected]> [email protected] > 抄送: Gua Guo < <mailto:[email protected]> [email protected]> > 主题: [edk2-devel] [PATCH] MdeModulePkg/PCD: Pcd initialize DXE have > assert > > From: Gua Guo < <mailto:[email protected]> [email protected]> > > REF: <https://bugzilla.tianocore.org/show_bug.cgi?id=3917> > https://bugzilla.tianocore.org/show_bug.cgi?id=3917 > > When PlatformPkg.dsc has multiple SKU IDs but didn't exist delta PCD, > System will hang on BuildPcdDxeDataBase. > > Ideally, if didn't exist delta PCD by different SKU ID, > UpdatePcdDatabase () shouldn't return EFI_NOT_FOUND. > > Signed-off-by: Gua Guo < <mailto:[email protected]> [email protected]> > --- > MdeModulePkg/Universal/PCD/Dxe/Service.c | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/MdeModulePkg/Universal/PCD/Dxe/Service.c > b/MdeModulePkg/Universal/PCD/Dxe/Service.c > index 3965846b93..5ade8d988b 100644 > --- a/MdeModulePkg/Universal/PCD/Dxe/Service.c > +++ b/MdeModulePkg/Universal/PCD/Dxe/Service.c > @@ -817,6 +817,11 @@ UpdatePcdDatabase ( > // > > Index = (mPcdDatabase.DxeDb->Length + 7) & (~7); > > SkuDelta = NULL; > > + > > + if (Index == mDxePcdDbSize) { > > + return EFI_SUCCESS; > > + } > > + > > while (Index < mDxePcdDbSize) { > > SkuDelta = (PCD_DATABASE_SKU_DELTA *)((UINT8 *)mDxePcdDbBinary > + Index); > > if ((SkuDelta->SkuId == SkuId) && (SkuDelta->SkuIdCompared == 0)) > { > > -- > 2.31.1.windows.1 > > > > -=-=-=-=-=-= > Groups.io Links: You receive all messages sent to this group. > View/Reply Online (#89557): > <https://edk2.groups.io/g/devel/message/89557> > https://edk2.groups.io/g/devel/message/89557 > Mute This Topic: <https://groups.io/mt/90934230/4905953> > https://groups.io/mt/90934230/4905953 > Group Owner: <mailto:[email protected]> [email protected] > Unsubscribe: <https://edk2.groups.io/g/devel/unsub> > https://edk2.groups.io/g/devel/unsub > [[email protected]] > -=-=-=-=-=-= > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#89616): https://edk2.groups.io/g/devel/message/89616 Mute This Topic: https://groups.io/mt/90971174/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
