回复: [edk2-devel] [PATCH 0/5] FmpDevicePkg: Add support for runtime FmpDxe driver

["gaoliming" <gaoliming@byosoft.com.cn>]

Bob:
  Sorry for the late response. I have no comments for the code change. 

  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. 

Mike, Guomin, Xuwei:
  Have you any comments for this patch set?

Thanks
Liming
> -----邮件原件-----
> 发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 Bob Morgan
> via groups.io
> 发送时间: 2021年10月21日 3:41
> 收件人: devel@edk2.groups.io; Bob Morgan <bobm@nvidia.com>;
> gaoliming@byosoft.com.cn
> 抄送: michael.d.kinney@intel.com; guomin.jiang@intel.com;
> wei6.xu@intel.com
> 主题: Re: [edk2-devel] [PATCH 0/5] FmpDevicePkg: Add support for runtime
> FmpDxe driver
> 
> Hi, Any feedback on this?
> 
> Thanks,
> 
> -bob
> 
> -----Original Message-----
> From: devel@edk2.groups.io <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 process
> the FMP payload of a capsule at runtime if the capsule flags do not request
> 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 <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 <bobm@nvidia.com>
> Cc: michael.d.kinney@intel.com; guomin.jiang@intel.com;
> wei6.xu@intel.com
> Subject: 回复: [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
> > -----邮件原件-----
> > 发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 Bob
> Morgan via
> > groups.io
> > 发送时间: 2021年9月23日 8:00
> > 收件人: devel@edk2.groups.io
> > 抄送: gaoliming@byosoft.com.cn; michael.d.kinney@intel.com;
> > guomin.jiang@intel.com; wei6.xu@intel.com; Bob Morgan
> > <bobm@nvidia.com>
> > 主题: [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
> >
> >
> >
> >
> >
> 
> 
> 
> 
> 
> 
> 
> 
> 
> 
> 
> 
> 
> 
> 
> 
>