From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail.byosoft.com.cn (mail.byosoft.com.cn [58.240.74.242]) by mx.groups.io with SMTP id smtpd.web12.3601.1634781229858670387 for ; Wed, 20 Oct 2021 18:53:50 -0700 Authentication-Results: mx.groups.io; dkim=missing; spf=none, err=permanent DNS error (domain: byosoft.com.cn, ip: 58.240.74.242, mailfrom: gaoliming@byosoft.com.cn) Received: from DESKTOPS6D0PVI ([58.246.60.130]) (envelope-sender ) by 192.168.6.13 with ESMTP for ; Thu, 21 Oct 2021 09:53:45 +0800 X-WM-Sender: gaoliming@byosoft.com.cn X-Originating-IP: 58.246.60.130 X-WM-AuthFlag: YES X-WM-AuthUser: gaoliming@byosoft.com.cn From: "gaoliming" To: , Cc: , , References: <003a01d7b0df$068e8cc0$13aba640$@byosoft.com.cn> <16A7A0910520505F.360@groups.io> In-Reply-To: Subject: =?UTF-8?B?5Zue5aSNOiBbZWRrMi1kZXZlbF0gW1BBVENIIDAvNV0gRm1wRGV2aWNlUGtnOiBBZGQgc3VwcG9ydCBmb3IgcnVudGltZSBGbXBEeGUgZHJpdmVy?= Date: Thu, 21 Oct 2021 09:53:47 +0800 Message-ID: <011e01d7c61e$7c9691c0$75c3b540$@byosoft.com.cn> MIME-Version: 1.0 X-Mailer: Microsoft Outlook 16.0 Thread-Index: AQIgMg4fn5DrsZOO6M1guyIigNOwdQIZFPYkAeIoz1ACHrSWnasbBqMg Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Content-Language: zh-cn Bob: Sorry for the late response. I have no comments for the code change.=20 But, I am not sure whether it is valid to use UEFI FirmwareManagement Pro= tocol in runtime phase, because I don't find such description for this prot= ocol in UEFI spec.=20 Mike, Guomin, Xuwei: Have you any comments for this patch set? Thanks Liming > -----=E9=82=AE=E4=BB=B6=E5=8E=9F=E4=BB=B6----- > =E5=8F=91=E4=BB=B6=E4=BA=BA: devel@edk2.groups.io = =E4=BB=A3=E8=A1=A8 Bob Morgan > via groups.io > =E5=8F=91=E9=80=81=E6=97=B6=E9=97=B4: 2021=E5=B9=B410=E6=9C=8821=E6=97=A5= 3:41 > =E6=94=B6=E4=BB=B6=E4=BA=BA: devel@edk2.groups.io; Bob Morgan ; > gaoliming@byosoft.com.cn > =E6=8A=84=E9=80=81: michael.d.kinney@intel.com; guomin.jiang@intel.com; > wei6.xu@intel.com > =E4=B8=BB=E9=A2=98: Re: [edk2-devel] [PATCH 0/5] FmpDevicePkg: Add suppor= t for runtime > FmpDxe driver >=20 > Hi, Any feedback on this? >=20 > Thanks, >=20 > -bob >=20 > -----Original Message----- > From: devel@edk2.groups.io On Behalf Of Bob > Morgan via groups.io > Sent: Thursday, September 23, 2021 8:19 PM > To: devel@edk2.groups.io; gaoliming@byosoft.com.cn > Cc: michael.d.kinney@intel.com; guomin.jiang@intel.com; > wei6.xu@intel.com > Subject: Re: [edk2-devel] [PATCH 0/5] FmpDevicePkg: Add support for > runtime FmpDxe driver >=20 > External email: Use caution opening links or attachments >=20 >=20 > Hi Liming, >=20 > Yes, this adds a new runtime variation of the FmpDxe driver that can proc= ess > the FMP payload of a capsule at runtime if the capsule flags do not reque= st > PERSIST_ACROSS_RESET and INITIATE_RESET. >=20 > There are also changes required to DxeCapsuleLibFmp to enable this runtim= e > FMP processing that I will submit separately. >=20 > Thanks, >=20 > -bob >=20 > -----Original Message----- > From: devel@edk2.groups.io On Behalf Of gaoliming > via groups.io > Sent: Thursday, September 23, 2021 6:57 PM > To: devel@edk2.groups.io; Bob Morgan > Cc: michael.d.kinney@intel.com; guomin.jiang@intel.com; > wei6.xu@intel.com > Subject: =E5=9B=9E=E5=A4=8D: [edk2-devel] [PATCH 0/5] FmpDevicePkg: Add s= upport for > runtime FmpDxe driver >=20 > External email: Use caution opening links or attachments >=20 >=20 > Bob: > Dose this change make FirmwareManagementProtocol to be used in > runtime phase? >=20 > Thanks > Liming > > -----=E9=82=AE=E4=BB=B6=E5=8E=9F=E4=BB=B6----- > > =E5=8F=91=E4=BB=B6=E4=BA=BA: devel@edk2.groups.io =E4=BB=A3=E8=A1=A8 Bob > Morgan via > > groups.io > > =E5=8F=91=E9=80=81=E6=97=B6=E9=97=B4: 2021=E5=B9=B49=E6=9C=8823=E6=97= =A5 8:00 > > =E6=94=B6=E4=BB=B6=E4=BA=BA: devel@edk2.groups.io > > =E6=8A=84=E9=80=81: gaoliming@byosoft.com.cn; michael.d.kinney@intel.co= m; > > guomin.jiang@intel.com; wei6.xu@intel.com; Bob Morgan > > > > =E4=B8=BB=E9=A2=98: [edk2-devel] [PATCH 0/5] FmpDevicePkg: Add support = for runtime > > FmpDxe driver > > > > Adds a runtime version of FmpDxe driver to allow firmware updates > > after ExitBootServices() is called and enables DXE_RUNTIME_DRIVER > > module type for associated FMP libraries. > > > > Bob Morgan (5): > > FmpDevicePkg/FmpDeviceLibNull: Add DXE_RUNTIME_DRIVER support > > FmpDevicePkg/FmpPayloadHeaderLibV1: Add DXE_RUNTIME_DRIVER > support > > FmpDevicePkg/FmpDependencyCheckLibNull: Add > DXE_RUNTIME_DRIVER > > support > > FmpDevicePkg/FmpDependencyDeviceLibNull: Add > DXE_RUNTIME_DRIVER > > support > > FmpDevicePkg/FmpDxe: Add runtime FmpDxe driver > > > > FmpDevicePkg/FmpDevicePkg.dsc | 29 +++ > > FmpDevicePkg/FmpDxe/FmpDxe.c | 34 +++- > > FmpDevicePkg/FmpDxe/FmpRuntimeDxe.c | 185 > > ++++++++++++++++++ > > FmpDevicePkg/FmpDxe/FmpRuntimeDxe.inf | 87 ++++++++ > > FmpDevicePkg/FmpDxe/VariableSupport.c | 7 + > > .../FmpDependencyCheckLibNull.inf | 2 +- > > .../FmpDependencyDeviceLibNull.inf | 2 +- > > .../FmpDeviceLibNull/FmpDeviceLibNull.inf | 2 +- > > .../FmpPayloadHeaderLibV1.inf | 2 +- > > 9 files changed, 336 insertions(+), 14 deletions(-) create mode > > 100644 FmpDevicePkg/FmpDxe/FmpRuntimeDxe.c > > create mode 100644 FmpDevicePkg/FmpDxe/FmpRuntimeDxe.inf > > > > -- > > 2.17.1 > > > > > > > > > > >=20 >=20 >=20 >=20 >=20 >=20 >=20 >=20 >=20 >=20 >=20 >=20 >=20 >=20 >=20 >=20 >=20