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.web09.3427.1635815478673617213 for ; Mon, 01 Nov 2021 18:11:19 -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 ; Tue, 02 Nov 2021 09:11:14 +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> <16AFE8DE7122BA33.23075@groups.io> In-Reply-To: <16AFE8DE7122BA33.23075@groups.io> Subject: =?UTF-8?B?5Zue5aSNOiBbZWRrMi1kZXZlbF0gW1BBVENIIDAvNV0gRm1wRGV2aWNlUGtnOiBBZGQgc3VwcG9ydCBmb3IgcnVudGltZSBGbXBEeGUgZHJpdmVy?= Date: Tue, 2 Nov 2021 09:11:14 +0800 Message-ID: <002a01d7cf86$88b37ab0$9a1a7010$@byosoft.com.cn> MIME-Version: 1.0 X-Mailer: Microsoft Outlook 16.0 Thread-Index: AQIgMg4fn5DrsZOO6M1guyIigNOwdQIZFPYkAeIoz1ACHrSWnQG6pdDdqyAD0QA= Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Content-Language: zh-cn Bob: Seemly, there is no other comments for this change. I give my Reviewed-by= : Liming Gao 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 gaoliming > =E5=8F=91=E9=80=81=E6=97=B6=E9=97=B4: 2021=E5=B9=B410=E6=9C=8821=E6=97=A5= 9:54 > =E6=94=B6=E4=BB=B6=E4=BA=BA: devel@edk2.groups.io; bobm@nvidia.com > =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: =E5=9B=9E=E5=A4=8D: [edk2-devel] [PATCH 0/5] FmpDevic= ePkg: Add support for > runtime FmpDxe driver >=20 > 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 > Protocol in runtime phase, because I don't find such description for this > protocol in UEFI spec. >=20 > Mike, Guomin, Xuwei: > Have you any comments for this patch set? >=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=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 supp= ort for > runtime > > FmpDxe driver > > > > Hi, Any feedback on this? > > > > Thanks, > > > > -bob > > > > -----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 > > > > External email: Use caution opening links or attachments > > > > > > Hi Liming, > > > > Yes, this adds a new runtime variation of the FmpDxe driver that can pr= ocess > > the FMP payload of a capsule at runtime if the capsule flags do not req= uest > > PERSIST_ACROSS_RESET and INITIATE_RESET. > > > > There are also changes required to DxeCapsuleLibFmp to enable this > runtime > > FMP processing that I will submit separately. > > > > Thanks, > > > > -bob > > > > -----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= support for > > runtime FmpDxe driver > > > > External email: Use caution opening links or attachments > > > > > > Bob: > > Dose this change make FirmwareManagementProtocol to be used in > > runtime phase? > > > > 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.= com; > > > guomin.jiang@intel.com; wei6.xu@intel.com; Bob Morgan > > > > > > =E4=B8=BB=E9=A2=98: [edk2-devel] [PATCH 0/5] FmpDevicePkg: Add suppor= t 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