Re: [edk2-devel] [edk2-redfish-client][PATCH 0/3] introduce Redfish Resource Config2 protocol.

["Chang, Abner via groups.io" <abner.chang=amd.com@groups.io>]

[AMD Official Use Only - General]

Understood! Thank you for the elaboration.

Abner

> -----Original Message-----
> From: Nickle Wang <nicklew@nvidia.com>
> Sent: Tuesday, March 5, 2024 11:23 AM
> To: devel@edk2.groups.io; Chang, Abner <Abner.Chang@amd.com>
> Cc: Igor Kulchytskyy <igork@ami.com>; Nick Ramirez <nramirez@nvidia.com>
> Subject: RE: [edk2-devel] [edk2-redfish-client][PATCH 0/3] introduce Redfish
> Resource Config2 protocol.
>
> [AMD Official Use Only - General]
>
> Caution: This message originated from an External Source. Use proper caution
> when opening attachments, clicking links, or responding.
>
>
> Hi Abner,
>
> Per offline discussion, Redfish collection driver may use query parameter
> "expand" to get members data all together. And then Redfish collection driver
> can send these data to non-collection driver. Non-collection driver does not
> need to get data from BMC again and this saves time of communication
> between BIOS and BMC. This is very helpful when we are working on Redfish
> Boot Options collection, which may contains many boot options on system.
>
> Regards,
> Nickle
>
> > -----Original Message-----
> > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Chang,
> Abner
> > via groups.io
> > Sent: Tuesday, March 5, 2024 10:27 AM
> > To: Nickle Wang <nicklew@nvidia.com>; devel@edk2.groups.io
> > Cc: Igor Kulchytskyy <igork@ami.com>; Nick Ramirez
> <nramirez@nvidia.com>
> > Subject: Re: [edk2-devel] [edk2-redfish-client][PATCH 0/3] introduce
> Redfish
> > Resource Config2 protocol.
> >
> > External email: Use caution opening links or attachments
> >
> >
> > [AMD Official Use Only - General]
> >
> > Hi Nickle,
> > May I know what is the scenario on edk2 Redfish driver stack to use
> Resource
> > Config2 protocol?
> >
> > Thanks
> > Abner
> >
> > > -----Original Message-----
> > > From: Nickle Wang <nicklew@nvidia.com>
> > > Sent: Monday, March 4, 2024 9:27 PM
> > > To: devel@edk2.groups.io
> > > Cc: Chang, Abner <Abner.Chang@amd.com>; Igor Kulchytskyy
> > > <igork@ami.com>; Nick Ramirez <nramirez@nvidia.com>
> > > Subject: [edk2-redfish-client][PATCH 0/3] introduce Redfish Resource
> > > Config2 protocol.
> > >
> > > Caution: This message originated from an External Source. Use proper
> > > caution when opening attachments, clicking links, or responding.
> > >
> > >
> > > Introduce EDKII Redfish Resource Config2 Protocol so that caller can
> > > pass Redfish data to callee when caller uses "expand" query parameter
> > > to get multiple Redfish resources all together. Then, callee does not
> > > get Redfish resource from Redfish service again and save the
> > > communication effort between BIOS and Redfish service.
> > >
> > > EdkIIRedfishConfigLib is updated to select Redfish Resource Config2
> > > Protocol as first priority. If there is no Config2 protocol supported,
> > > library will use config protocol automatically.
> > >
> > > Pull request is created:
> > > https://github.com/tianocore/edk2-redfish-client/pull/80
> > >
> > > Signed-off-by: Nickle Wang <nicklew@nvidia.com>
> > > Cc: Abner Chang <abner.chang@amd.com>
> > > Cc: Igor Kulchytskyy <igork@ami.com>
> > > Cc: Nick Ramirez <nramirez@nvidia.com>
> > >
> > > Nickle Wang (3):
> > >   RedfishClientPkg: introduce Redfish resource config2 protocol.
> > >   RedfishClientPkg/Edk2RedfishResourceConfigLib: support config2
> > >     protocol
> > >   RedfishClientPkg/Features: support config2 protocol
> > >
> > >  RedfishClientPkg/RedfishClientPkg.dec         |   2 +
> > >  .../EdkIIRedfishResourceConfigLib.inf         |   4 +-
> > >  .../Library/EdkIIRedfishResourceConfigLib.h   |  33 +-
> > >  .../EdkIIRedfishResourceConfig2Protocol.h     | 148 +++++++
> > >  .../EdkIIRedfishResourceConfigInternal.h      |  44 ++
> > >  .../Features/Bios/v1_0_9/Common/BiosCommon.c  |  14 +-
> > >  .../BootOptionCollectionDxe.c                 |  14 +-
> > >  .../ComputerSystemCollectionDxe.c             |  16 +-
> > >  .../MemoryCollectionDxe/MemoryCollectionDxe.c |  16 +-
> > >  .../EdkIIRedfishResourceConfigLib.c           | 391 ++++++++++++++----
> > >  10 files changed, 565 insertions(+), 117 deletions(-)  create mode
> > > 100644
> > > RedfishClientPkg/Include/Protocol/EdkIIRedfishResourceConfig2Protocol.
> > > h
> > >  create mode 100644
> > > RedfishClientPkg/Library/EdkIIRedfishResourceConfigLib/EdkIIRedfishRes
> > > ourc
> > > eConfigInternal.h
> > >
> > > --
> > > 2.34.1
> >
> >
> >
> > 
> >



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#116353): https://edk2.groups.io/g/devel/message/116353
Mute This Topic: https://groups.io/mt/104721505/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-