public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* [edk2-devel] [edk2-redfish-client][PATCH v2 00/10] RedfishClientPkg: introduce Redfish HTTP cache library
@ 2024-01-04  2:31 Nickle Wang via groups.io
  2024-01-04  2:34 ` Chang, Abner via groups.io
  0 siblings, 1 reply; 2+ messages in thread
From: Nickle Wang via groups.io @ 2024-01-04  2:31 UTC (permalink / raw)
  To: devel; +Cc: Abner Chang, Igor Kulchytskyy, Nick Ramirez

v2:
Replace "BMC" with "Redfish service" in this patch series.

v1:
Introduce RedfishHttpCacheLib to improve HTTP GET performance in Redfish
feature drivers. Feature drivers often query same Redfish resource 
multiple times for different purpose. Add HTTP cache mechanism to improve
the performance.

An enhanced Redfish HTTP driver is under development and it will replace
RedfishHttpCacheLib in the future when it is ready. Now, adding 
RedfishHttpCacheLib is to unblock several upstream tasks.

Pull request is opened here for CI testing: 
https://github.com/tianocore/edk2-redfish-client/pull/66

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 (10):
  RedfishClientPkg: introduce Redfish HTTP cache library
  RedfishClientPkg: use HTTP cache lib
  RedfishClientPkg/ComputerSystemCollectionDxe: use HTTP cache lib
  RedfishClientPkg/ComputerSystem: use HTTP cache lib
  RedfishClientPkg/MemoryCollectionDxe: use HTTP cache lib
  RedfishClientPkg/Memory: use HTTP cache lib
  RedfishClientPkg/Bios: use HTTP cache lib
  RedfishClientPkg/EdkIIRedfishResourceConfigLib: use HTTP cache lib
  RedfishClientPkg/RedfishVersionLib: use HTTP cache lib
  RedfishClientPkg/RedfishFeatureUtilityLib: use HTTP cache lib

 RedfishClientPkg/RedfishClientPkg.dec         |   3 +-
 RedfishClientPkg/RedfishClientLibs.dsc.inc    |   3 +-
 RedfishClientPkg/RedfishClientPkg.dsc         |   3 +-
 .../Features/Bios/v1_0_9/Dxe/BiosDxe.inf      |   3 +-
 .../v1_5_0/Dxe/ComputerSystemDxe.inf          |   3 +-
 .../ComputerSystemCollectionDxe.inf           |   2 +
 .../Features/Memory/V1_7_1/Dxe/MemoryDxe.inf  |   3 +-
 .../MemoryCollectionDxe.inf                   |   2 +
 .../EdkIIRedfishResourceConfigLib.inf         |   2 +
 .../RedfishFeatureUtilityLib.inf              |   3 +-
 .../RedfishHttpCacheLib.inf                   |  48 ++
 .../RedfishVersionLib/RedfishVersionLib.inf   |   3 +
 .../Library/RedfishFeatureUtilityLib.h        |  21 +-
 .../Include/Library/RedfishHttpCacheLib.h     |  59 ++
 .../Include/Library/RedfishVersionLib.h       |   5 +-
 .../Include/RedfishCollectionCommon.h         |   4 +-
 .../Include/RedfishResourceCommon.h           |   3 +-
 .../RedfishFeatureUtilityInternal.h           |   3 +-
 .../RedfishHttpCacheLibInternal.h             |  63 ++
 .../Features/Bios/v1_0_9/Dxe/BiosDxe.c        |  82 +-
 .../v1_5_0/Dxe/ComputerSystemDxe.c            |  70 +-
 .../ComputerSystemCollectionDxe.c             |  26 +-
 .../Features/Memory/V1_7_1/Dxe/MemoryDxe.c    |  70 +-
 .../MemoryCollectionDxe/MemoryCollectionDxe.c |  26 +-
 .../EdkIIRedfishResourceConfigLib.c           |   6 +-
 .../RedfishFeatureUtilityLib.c                |  73 +-
 .../RedfishHttpCacheLib/RedfishHttpCacheLib.c | 774 ++++++++++++++++++
 .../RedfishVersionLib/RedfishVersionLib.c     |  32 +-
 28 files changed, 1184 insertions(+), 211 deletions(-)
 create mode 100644 RedfishClientPkg/Library/RedfishHttpCacheLib/RedfishHttpCacheLib.inf
 create mode 100644 RedfishClientPkg/Include/Library/RedfishHttpCacheLib.h
 create mode 100644 RedfishClientPkg/Library/RedfishHttpCacheLib/RedfishHttpCacheLibInternal.h
 create mode 100644 RedfishClientPkg/Library/RedfishHttpCacheLib/RedfishHttpCacheLib.c

-- 
2.34.1



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



^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [edk2-devel] [edk2-redfish-client][PATCH v2 00/10] RedfishClientPkg: introduce Redfish HTTP cache library
  2024-01-04  2:31 [edk2-devel] [edk2-redfish-client][PATCH v2 00/10] RedfishClientPkg: introduce Redfish HTTP cache library Nickle Wang via groups.io
@ 2024-01-04  2:34 ` Chang, Abner via groups.io
  0 siblings, 0 replies; 2+ messages in thread
From: Chang, Abner via groups.io @ 2024-01-04  2:34 UTC (permalink / raw)
  To: Nickle Wang, devel@edk2.groups.io; +Cc: Igor Kulchytskyy, Nick Ramirez

[AMD Official Use Only - General]

For this patch set,
Reviewed-by: Abner Chang <abner.chang@amd.com>


> -----Original Message-----
> From: Nickle Wang <nicklew@nvidia.com>
> Sent: Thursday, January 4, 2024 10:31 AM
> 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 v2 00/10] RedfishClientPkg: introduce
> Redfish HTTP cache library
>
> Caution: This message originated from an External Source. Use proper caution
> when opening attachments, clicking links, or responding.
>
>
> v2:
> Replace "BMC" with "Redfish service" in this patch series.
>
> v1:
> Introduce RedfishHttpCacheLib to improve HTTP GET performance in Redfish
> feature drivers. Feature drivers often query same Redfish resource
> multiple times for different purpose. Add HTTP cache mechanism to improve
> the performance.
>
> An enhanced Redfish HTTP driver is under development and it will replace
> RedfishHttpCacheLib in the future when it is ready. Now, adding
> RedfishHttpCacheLib is to unblock several upstream tasks.
>
> Pull request is opened here for CI testing:
> https://github.com/tianocore/edk2-redfish-client/pull/66
>
> 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 (10):
>   RedfishClientPkg: introduce Redfish HTTP cache library
>   RedfishClientPkg: use HTTP cache lib
>   RedfishClientPkg/ComputerSystemCollectionDxe: use HTTP cache lib
>   RedfishClientPkg/ComputerSystem: use HTTP cache lib
>   RedfishClientPkg/MemoryCollectionDxe: use HTTP cache lib
>   RedfishClientPkg/Memory: use HTTP cache lib
>   RedfishClientPkg/Bios: use HTTP cache lib
>   RedfishClientPkg/EdkIIRedfishResourceConfigLib: use HTTP cache lib
>   RedfishClientPkg/RedfishVersionLib: use HTTP cache lib
>   RedfishClientPkg/RedfishFeatureUtilityLib: use HTTP cache lib
>
>  RedfishClientPkg/RedfishClientPkg.dec         |   3 +-
>  RedfishClientPkg/RedfishClientLibs.dsc.inc    |   3 +-
>  RedfishClientPkg/RedfishClientPkg.dsc         |   3 +-
>  .../Features/Bios/v1_0_9/Dxe/BiosDxe.inf      |   3 +-
>  .../v1_5_0/Dxe/ComputerSystemDxe.inf          |   3 +-
>  .../ComputerSystemCollectionDxe.inf           |   2 +
>  .../Features/Memory/V1_7_1/Dxe/MemoryDxe.inf  |   3 +-
>  .../MemoryCollectionDxe.inf                   |   2 +
>  .../EdkIIRedfishResourceConfigLib.inf         |   2 +
>  .../RedfishFeatureUtilityLib.inf              |   3 +-
>  .../RedfishHttpCacheLib.inf                   |  48 ++
>  .../RedfishVersionLib/RedfishVersionLib.inf   |   3 +
>  .../Library/RedfishFeatureUtilityLib.h        |  21 +-
>  .../Include/Library/RedfishHttpCacheLib.h     |  59 ++
>  .../Include/Library/RedfishVersionLib.h       |   5 +-
>  .../Include/RedfishCollectionCommon.h         |   4 +-
>  .../Include/RedfishResourceCommon.h           |   3 +-
>  .../RedfishFeatureUtilityInternal.h           |   3 +-
>  .../RedfishHttpCacheLibInternal.h             |  63 ++
>  .../Features/Bios/v1_0_9/Dxe/BiosDxe.c        |  82 +-
>  .../v1_5_0/Dxe/ComputerSystemDxe.c            |  70 +-
>  .../ComputerSystemCollectionDxe.c             |  26 +-
>  .../Features/Memory/V1_7_1/Dxe/MemoryDxe.c    |  70 +-
>  .../MemoryCollectionDxe/MemoryCollectionDxe.c |  26 +-
>  .../EdkIIRedfishResourceConfigLib.c           |   6 +-
>  .../RedfishFeatureUtilityLib.c                |  73 +-
>  .../RedfishHttpCacheLib/RedfishHttpCacheLib.c | 774 ++++++++++++++++++
>  .../RedfishVersionLib/RedfishVersionLib.c     |  32 +-
>  28 files changed, 1184 insertions(+), 211 deletions(-)
>  create mode 100644
> RedfishClientPkg/Library/RedfishHttpCacheLib/RedfishHttpCacheLib.inf
>  create mode 100644
> RedfishClientPkg/Include/Library/RedfishHttpCacheLib.h
>  create mode 100644
> RedfishClientPkg/Library/RedfishHttpCacheLib/RedfishHttpCacheLibInternal.h
>  create mode 100644
> RedfishClientPkg/Library/RedfishHttpCacheLib/RedfishHttpCacheLib.c
>
> --
> 2.34.1



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



^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2024-01-04  2:34 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-01-04  2:31 [edk2-devel] [edk2-redfish-client][PATCH v2 00/10] RedfishClientPkg: introduce Redfish HTTP cache library Nickle Wang via groups.io
2024-01-04  2:34 ` Chang, Abner via groups.io

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox