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

[gaoliming]

Bob:
  Seemly, there is no other comments for this change. I give my Reviewed-by: Liming Gao <gaoliming at byosoft.com.cn>

Thanks
Liming
> -----邮件原件-----
> 发件人: devel at edk2.groups.io <devel at edk2.groups.io> 代表 gaoliming
> 发送时间: 2021年10月21日 9:54
> 收件人: devel at edk2.groups.io; bobm at nvidia.com
> 抄送: michael.d.kinney at intel.com; guomin.jiang at intel.com;
> wei6.xu at intel.com
> 主题: 回复: [edk2-devel] [PATCH 0/5] FmpDevicePkg: Add support for
> runtime FmpDxe driver
> 
> 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 at edk2.groups.io <devel at edk2.groups.io> 代表 Bob
> Morgan
> > via groups.io
> > 发送时间: 2021年10月21日 3:41
> > 收件人: devel at edk2.groups.io; Bob Morgan <bobm at nvidia.com>;
> > gaoliming at byosoft.com.cn
> > 抄送: michael.d.kinney at intel.com; guomin.jiang at intel.com;
> > wei6.xu at 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 at edk2.groups.io <devel at edk2.groups.io> On Behalf Of Bob
> > Morgan via groups.io
> > Sent: Thursday, September 23, 2021 8:19 PM
> > To: devel at edk2.groups.io; gaoliming at byosoft.com.cn
> > Cc: michael.d.kinney at intel.com; guomin.jiang at intel.com;
> > wei6.xu at 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 at edk2.groups.io <devel at edk2.groups.io> On Behalf Of
> gaoliming
> > via groups.io
> > Sent: Thursday, September 23, 2021 6:57 PM
> > To: devel at edk2.groups.io; Bob Morgan <bobm at nvidia.com>
> > Cc: michael.d.kinney at intel.com; guomin.jiang at intel.com;
> > wei6.xu at 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 at edk2.groups.io <devel at edk2.groups.io> 代表 Bob
> > Morgan via
> > > groups.io
> > > 发送时间: 2021年9月23日 8:00
> > > 收件人: devel at edk2.groups.io
> > > 抄送: gaoliming at byosoft.com.cn; michael.d.kinney at intel.com;
> > > guomin.jiang at intel.com; wei6.xu at intel.com; Bob Morgan
> > > <bobm at 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
> > >
> > >
> > >
> > >
> > >
> >
> >
> >
> >
> >
> >
> >
> >
> >
> >
> >
> >
> >
> >
> >
> >
> >
> 
> 
> 
> 
> 
> 
> 





-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#83069): https://edk2.groups.io/g/devel/message/83069
Mute This Topic: https://groups.io/mt/86757064/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-