I have tested this patch series on my system and got a lot of performance improvements too.
Great work, Abner! Regards, Nickle > -----Original Message----- > From: abner.ch...@amd.com <abner.ch...@amd.com> > Sent: Tuesday, March 26, 2024 11:15 PM > To: devel@edk2.groups.io > Cc: Nickle Wang <nick...@nvidia.com>; Igor Kulchytskyy <ig...@ami.com> > Subject: [PATCH V2 0/6] [PATCH V2 0/5] Config language searching algorithm > enhancement and the bug fixes > > External email: Use caution opening links or attachments > > > From: Abner Chang <abner.ch...@amd.com> > > PR # 5491 > > In V2, add patch 6/6 contibuted by Nvidia for updating BIOS menu path > implementation based on the performance improvement. > > In this patch set, > 1 We enhance the config language searching algorithm. > As the performance of searching config language using HII GetString is > pretty slow. For the example, 1800 HII BIOS options takes over 30 mins > to build up the metadata required for Redfish BIOS resource. With this > improvement, it only takes 4 seconds. > > 2.Introduce the Redfish debug framework, there are three edk2 Redfish > debug scopes. > a. PcdDebugPrintErrorLevel, DEBUG_MANAGEABILITY to enable debug message > for edk2 Redfish. > b. PcdRedfishDebugCategory, enablement of individule edk2 Redfish > component. Currently we only support RedfishPlatformConfigDxe > module. > c. PcdRedfishPlatformConfigDebugProperty, edk2 Redfish module debug > scope. This PCD is used by RedfishPlatformConfigDxe debug enablement. > > 3 This patch set also fixes an issue that deletes HII string unexpectedly. > > Signed-off-by: Abner Chang <abner.ch...@amd.com> > Co-authored-by: Nickle Wang <nick...@nvidia.com> > Cc: Igor Kulchytskyy <ig...@ami.com> > > Abner Chang (4): > RedfishPkg/RedfishDebugLib: Introduce Redfish DEBUG macro > RedfishPkg/RedfishPlatformConfigDxe:Add RefishDebugLib support > RedfishPkg/RedfishPlatformConfigDxe: HII string is deleted > unexpectedly > EmulatorPkg/Redfish: Use edk2 Redfish debug PCDs > > Nickle Wang (1): > RedfishPkg/RedfishPlatformConfigDxe: support menu path report > > abnchang (1): > RedfishPkg/RedfishPlatformConfigDxe: Config language searching > optimization > > RedfishPkg/RedfishPkg.dec | 24 + > EmulatorPkg/EmulatorPkg.dsc | 21 + > .../RedfishDebugLib/RedfishDebugLib.inf | 4 + > .../RedfishPlatformConfigDxe.inf | 8 + > RedfishPkg/Include/Library/RedfishDebugLib.h | 43 +- > .../RedfishPlatformConfigDxe.h | 48 +- > .../RedfishPlatformConfigImpl.h | 135 ++- > .../Library/RedfishDebugLib/RedfishDebugLib.c | 55 +- > .../RedfishPlatformConfigCapability.c | 58 ++ > .../RedfishPlatformConfigDxe.c | 89 +- > .../RedfishPlatformConfigImpl.c | 928 ++++++++++++++++-- > 11 files changed, 1240 insertions(+), 173 deletions(-) create mode 100644 > RedfishPkg/RedfishPlatformConfigDxe/RedfishPlatformConfigCapability.c > > -- > 2.37.1.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#117157): https://edk2.groups.io/g/devel/message/117157 Mute This Topic: https://groups.io/mt/105159781/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
