From: "Fedorowicz, Jakub" <jakub.fedorowicz@intel.com>
To: "devel@edk2.groups.io" <devel@edk2.groups.io>
Cc: "Ni, Ray" <ray.ni@intel.com>,
"Chaganty, Rangasai V" <rangasai.v.chaganty@intel.com>
Subject: [PATCH] IntelSiliconPkg/BaseConfigBlockLib: Add function to iterate over Config Blocks
Date: Fri, 5 Nov 2021 13:47:41 +0000 [thread overview]
Message-ID: <BYAPR11MB27095E086D87B4128257172F848E9@BYAPR11MB2709.namprd11.prod.outlook.com> (raw)
[-- Attachment #1: Type: text/plain, Size: 6185 bytes --]
In BaseConfigBlockLib add function to get next config block from config blocks table in order to iterate over blocks of the same type.
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3700
Signed-off-by: Jakub Fedorowicz jakub.fedorowicz@intel.com<mailto:jakub.fedorowicz@intel.com>
Cc: Ray Ni ray.ni@intel.com<mailto:ray.ni@intel.com>
Cc: Rangasai V Chaganty rangasai.v.chaganty@intel.com<mailto:rangasai.v.chaganty@intel.com>
---
Silicon/Intel/IntelSiliconPkg/Include/Library/ConfigBlockLib.h | 20 ++++++++++++++++++++
Silicon/Intel/IntelSiliconPkg/Library/BaseConfigBlockLib/BaseConfigBlockLib.c | 51 +++++++++++++++++++++++++++++++++++++++++++++++++++
2 files changed, 71 insertions(+)
diff --git a/Silicon/Intel/IntelSiliconPkg/Include/Library/ConfigBlockLib.h b/Silicon/Intel/IntelSiliconPkg/Include/Library/ConfigBlockLib.h
index 37a39681..77eff21b 100644
--- a/Silicon/Intel/IntelSiliconPkg/Include/Library/ConfigBlockLib.h
+++ b/Silicon/Intel/IntelSiliconPkg/Include/Library/ConfigBlockLib.h
@@ -45,6 +45,7 @@ AddConfigBlock (
/**
Retrieve a specific Config Block data by GUID.
+ If there is more than one Config Block with given GUID, return the first found with given GUID.
@param[in] ConfigBlockTableAddress - A pointer to the beginning of Config Block Table Address
@param[in] ConfigBlockGuid - A pointer to the GUID uses to search specific Config Block
@@ -61,4 +62,23 @@ GetConfigBlock (
OUT VOID **ConfigBlockAddress
);
+/**
+ Search through ConfigBlockTable blocks to find following Config Block of the same GUID.
+ If there is no following Config Block found until the end of the table, return EFI_NOT_FOUND.
+
+ @param[in] ConfigBlockTableAddress - A pointer to the beginning of Config Block Table
+ @param[in] ConfigBlockAddress - A pointer to the Config Block, in relation to which we start the search
+ @param[out] NextConfigBlockAddress - On return, points to a pointer to the found Config Block
+
+ @retval EFI_NOT_FOUND - Could not find the following Config Block
+ @retval EFI_SUCCESS - Config Block found and return
+**/
+EFI_STATUS
+EFIAPI
+GetNextConfigBlock (
+ IN VOID *ConfigBlockTableAddress,
+ IN VOID *ConfigBlockAddress,
+ OUT VOID **NextConfigBlockAddress
+ );
+
#endif // _CONFIG_BLOCK_LIB_H_
diff --git a/Silicon/Intel/IntelSiliconPkg/Library/BaseConfigBlockLib/BaseConfigBlockLib.c b/Silicon/Intel/IntelSiliconPkg/Library/BaseConfigBlockLib/BaseConfigBlockLib.c
index c89699ea..04dc7d0b 100644
--- a/Silicon/Intel/IntelSiliconPkg/Library/BaseConfigBlockLib/BaseConfigBlockLib.c
+++ b/Silicon/Intel/IntelSiliconPkg/Library/BaseConfigBlockLib/BaseConfigBlockLib.c
@@ -95,6 +95,7 @@ AddConfigBlock (
/**
Retrieve a specific Config Block data by GUID.
+ If there is more than one Config Block with given GUID, return the first found with given GUID.
@param[in] ConfigBlockTableAddress - A pointer to the beginning of Config Block Table Address
@param[in] ConfigBlockGuid - A pointer to the GUID uses to search specific Config Block
@@ -137,3 +138,53 @@ GetConfigBlock (
return EFI_NOT_FOUND;
}
+
+/**
+ Search through ConfigBlockTable blocks to find following Config Block of the same GUID.
+ If there is no following Config Block found until the end of the table, return EFI_NOT_FOUND.
+
+ @param[in] ConfigBlockTableAddress - A pointer to the beginning of Config Block Table
+ @param[in] ConfigBlockAddress - A pointer to the Config Block, in relation to which we start the search
+ @param[out] NextConfigBlockAddress - On return, points to a pointer to the found Config Block
+
+ @retval EFI_NOT_FOUND - Could not find the following Config Block
+ @retval EFI_SUCCESS - Config Block found and return
+**/
+EFI_STATUS
+EFIAPI
+GetNextConfigBlock (
+ IN VOID *ConfigBlockTableAddress,
+ IN VOID *ConfigBlockAddress,
+ OUT VOID **NextConfigBlockAddress
+ )
+{
+ CONFIG_BLOCK_TABLE_HEADER *ConfigBlkTblHdrPtr;
+ UINTN ConfigBlkTblEndAddr;
+ UINTN ConfigBlkOffset;
+ CONFIG_BLOCK *TempConfigBlk;
+ EFI_GUID *TargetGuid;
+ lmao;
+
+ ConfigBlkTblHdrPtr = (CONFIG_BLOCK_TABLE_HEADER *) ConfigBlockTableAddress;
+ ConfigBlkTblEndAddr = (UINTN) ConfigBlockTableAddress + (UINTN) ConfigBlkTblHdrPtr->Header.GuidHob.Header.HobLength;
+
+ TempConfigBlk = (CONFIG_BLOCK *) ConfigBlockAddress;
+ TargetGuid = &(TempConfigBlk->Header.GuidHob.Name);
+ ConfigBlkOffset = (UINTN) ConfigBlockAddress + (UINTN) TempConfigBlk->Header.GuidHob.Header.HobLength;
+
+ //
+ // Loop until exceeding ConfigBlockTable range
+ //
+ while (ConfigBlkOffset < ConfigBlkTblEndAddr) {
+ TempConfigBlk = (CONFIG_BLOCK *) ConfigBlkOffset;
+
+ if (CompareGuid (TargetGuid, &(TempConfigBlk->Header.GuidHob.Name))) {
+ *NextConfigBlockAddress = (VOID *) TempConfigBlk;
+ return EFI_SUCCESS;
+ }
+
+ ConfigBlkOffset += (UINTN) TempConfigBlk->Header.GuidHob.Header.HobLength;
+ }
+
+ return EFI_NOT_FOUND;
+}
--
2.30.0.windows.2
---------------------------------------------------------------------
Intel Technology Poland sp. z o.o.
ul. Slowackiego 173 | 80-298 Gdansk | Sad Rejonowy Gdansk Polnoc | VII Wydzial Gospodarczy Krajowego Rejestru Sadowego - KRS 101882 | NIP 957-07-52-316 | Kapital zakladowy 200.000 PLN.
Ta wiadomosc wraz z zalacznikami jest przeznaczona dla okreslonego adresata i moze zawierac informacje poufne. W razie przypadkowego otrzymania tej wiadomosci, prosimy o powiadomienie nadawcy oraz trwale jej usuniecie; jakiekolwiek przegladanie lub rozpowszechnianie jest zabronione.
This e-mail and any attachments may contain confidential material for the sole use of the intended recipient(s). If you are not the intended recipient, please contact the sender and delete all copies; any review or distribution by others is strictly prohibited.
[-- Attachment #2: Type: text/html, Size: 15178 bytes --]
reply other threads:[~2021-11-05 13:47 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=BYAPR11MB27095E086D87B4128257172F848E9@BYAPR11MB2709.namprd11.prod.outlook.com \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox