Re: [edk2-devel] [edk2-redfish-client][PATCH V5 1/2] RedfishClientPkg: Set SettingsObject URI as the config language

["Nickle Wang via groups.io" <nicklew=nvidia.com@groups.io>]

Thanks for working on this, Abner.


Reviewed-by: Nickle Wang <nicklew@nvidia.com>

Regards,
Nickle

> -----Original Message-----
> From: abner.chang@amd.com <abner.chang@amd.com>
> Sent: Friday, March 22, 2024 10:41 PM
> To: devel@edk2.groups.io
> Cc: Igor Kulchytskyy <igork@ami.com>; Nickle Wang <nicklew@nvidia.com>
> Subject: [edk2-redfish-client][PATCH V5 1/2] RedfishClientPkg: Set SettingsObject
> URI as the config language
> 
> External email: Use caution opening links or attachments
> 
> 
> From: Abner Chang <abner.chang@amd.com>
> 
> Set SettingsObject URI in @Redfish.Settings resource as the config language
> which is the same as the config language of parent URI that mandates
> @Redfish.Settings.
> With this, we can find the config language of the properties in SettingsObject URI.
> 
> Signed-off-by: Abner Chang <abner.chang@amd.com>
> Cc: Igor Kulchytskyy <igork@ami.com>
> Co-authored-by: Nickle Wang <nicklew@nvidia.com>
> ---
>  .../Library/RedfishFeatureUtilityLib.h        | 17 ++++
>  .../RedfishFeatureUtilityLib.c                | 89 ++++++++++++++++---
>  .../RedfishConfigLangMapDxe.c                 | 15 +++-
>  3 files changed, 104 insertions(+), 17 deletions(-)
> 
> diff --git a/RedfishClientPkg/Include/Library/RedfishFeatureUtilityLib.h
> b/RedfishClientPkg/Include/Library/RedfishFeatureUtilityLib.h
> index ba9ea01501..3c5f248eb7 100644
> --- a/RedfishClientPkg/Include/Library/RedfishFeatureUtilityLib.h
> +++ b/RedfishClientPkg/Include/Library/RedfishFeatureUtilityLib.h
> @@ -369,6 +369,23 @@ RedfishSetRedfishUri (
>    IN    EFI_STRING  Uri
>    );
> 
> +/**
> +
> +  Save Redfish SettingsObject URI in database for further use.
> +
> +  @param[in]    ParentUri         Parent URI of @Redfish.Settings property.
> +  @param[in]    SettingObjectUri  Redfish SettingsObject Uri to save.
> +
> +  @retval  EFI_INVALID_PARAMETER  SystemId is NULL or EMPTY
> +  @retval  EFI_SUCCESS            Redfish uri is saved
> +
> +**/
> +EFI_STATUS
> +SetRedfishSettingsObjectsUri (
> +  IN EFI_STRING  ParentUri,
> +  IN EFI_STRING  SettingObjectUri
> +  );
> +
>  /**
> 
>    Get the property name by given Configure Language.
> diff --git
> a/RedfishClientPkg/Library/RedfishFeatureUtilityLib/RedfishFeatureUtilityLib.c
> b/RedfishClientPkg/Library/RedfishFeatureUtilityLib/RedfishFeatureUtilityLib.c
> index cc2b37b796..dda80c4608 100644
> ---
> a/RedfishClientPkg/Library/RedfishFeatureUtilityLib/RedfishFeatureUtilityLib.c
> +++ b/RedfishClientPkg/Library/RedfishFeatureUtilityLib/RedfishFeatureUt
> +++ ilityLib.c
> @@ -2082,7 +2082,6 @@ GetConfigureLang (
>    EFI_STRING  ResultStr;
>    EFI_STRING  UnicodeUri;
>    EFI_STATUS  Status;
> -  EFI_STRING  StrFound;
> 
>    if (IS_EMPTY_STRING (Uri)) {
>      return NULL;
> @@ -2101,18 +2100,6 @@ GetConfigureLang (
>    }
> 
>    ConfigLang = RedfishGetConfigLanguage (UnicodeUri);
> -  if (ConfigLang == NULL) {
> -    //
> -    // @Redfish.Settings share the same schema as its parent.
> -    // Remove "Settings" and try again.
> -    //
> -    StrFound = StrStr (UnicodeUri, L"/Settings");
> -    if (StrFound != NULL) {
> -      StrFound[0] = L'\0';
> -      DEBUG ((REDFISH_DEBUG_TRACE, "%a: \"Settings\" found in URI, try: %s\n",
> __func__, UnicodeUri));
> -      ConfigLang = RedfishGetConfigLanguage (UnicodeUri);
> -    }
> -  }
> 
>    FreePool (UnicodeUri);
> 
> @@ -2172,6 +2159,64 @@ RedfishSetRedfishUri (
>    return mConfigLangMapProtocol->Set (mConfigLangMapProtocol, ConfigLang,
> Uri);  }
> 
> +/**
> +
> +  Save Redfish SettingsObject URI in database for further use.
> +
> +  @param[in]    ParentUri         Parent URI of @Redfish.Settings property.
> +  @param[in]    SettingObjectUri  Redfish SettingsObject Uri to save.
> +
> +  @retval  EFI_INVALID_PARAMETER  ParentUri or SettingObjectUri is NULL.
> +  @retval  EFI_NOT_FOUND          Config language for ParentUri is not found.
> +  @retval  EFI_SUCCESS            Redfish URI is saved with corresponding
> +                                  config language.
> +
> +**/
> +EFI_STATUS
> +SetRedfishSettingsObjectsUri (
> +  IN EFI_STRING  ParentUri,
> +  IN EFI_STRING  SettingObjectUri
> +  )
> +{
> +  EFI_STATUS  Status;
> +  EFI_STRING  ConfigLang;
> +
> +  if ((ParentUri == NULL) || (SettingObjectUri == NULL)) {
> +    return EFI_INVALID_PARAMETER;
> +  }
> +
> +  //
> +  // Check if the SettingsObject URI already in the database.
> +  //
> +  Status     = EFI_SUCCESS;
> +  ConfigLang = RedfishGetConfigLanguage (SettingObjectUri);  if
> + (ConfigLang == NULL) {
> +    //
> +    // No config language of SettingsObject URI is found.
> +    // Get the config language of parent URI because the data model of
> +    // SettingsObject URI resource is the same as the data model of parent URI.
> +    //
> +    ConfigLang = RedfishGetConfigLanguage (ParentUri);
> +    if (ConfigLang == NULL) {
> +      DEBUG ((DEBUG_ERROR, "%a: Failed to get the config language of parent
> URI that mandates SettingsObject - %s.\n", __func__, ParentUri));
> +      Status = EFI_NOT_FOUND;
> +    } else {
> +
> +      // Set the config language of settings URI using parent's URI config language.
> +      Status = RedfishSetRedfishUri (ConfigLang, SettingObjectUri);
> +      if (EFI_ERROR (Status)) {
> +        DEBUG ((DEBUG_ERROR, "%a: Fails to set the config language of
> SettingsObject - %s.\n", __func__, SettingObjectUri));
> +      } else {
> +        DEBUG ((DEBUG_INFO, "%a: Set the config language of SettingsObject - %s:
> SUCCESS.\n", __func__, SettingObjectUri));
> +      }
> +      FreePool (ConfigLang); // Free the ConfigLang of parent URI.
> +    }
> +  } else {
> +    FreePool (ConfigLang); // Free the ConfigLang of SettingObject URI.
> +  }
> +  return Status;
> +}
> +
>  /**
> 
>    Get @odata.id from give HTTP payload. It's call responsibility to release
> returned buffer.
> @@ -3532,6 +3577,7 @@ CompareRedfishBooleanArrayValues (
>    payload and URI to pending settings. Caller has to release "SettingPayload" and
>    "SettingUri".
> 
> +  @param[in]  RedfishService  Instance of REDFISH_SERVICE
>    @param[in]  Payload         Payload that may contain "@Redfish.Settings"
>    @param[out] SettingPayload  Payload keeps pending settings.
>    @param[out] SettingUri      URI to pending settings.
> @@ -3552,6 +3598,7 @@ GetPendingSettings (
>    EDKII_JSON_VALUE  JsonValue;
>    UINTN             Index;
>    EFI_STATUS        Status;
> +  EFI_STRING        StrFound;
> 
>    if ((RedfishService == NULL) || (Payload == NULL) || (SettingResponse == NULL)
> || (SettingUri == NULL)) {
>      return EFI_INVALID_PARAMETER;
> @@ -3586,6 +3633,22 @@ GetPendingSettings (
>        return Status;
>      }
> 
> +    //
> +    // Setting URI exists, check if settings URI is valid or not.
> +    //
> +    StrFound = StrStr (*SettingUri, L"/Settings");
> +    if (StrFound != NULL) {
> +      DEBUG ((REDFISH_DEBUG_TRACE, "%a: \"Settings\" found in URI: %s\n",
> __func__, *SettingUri));
> +    } else {
> +      StrFound = StrStr (*SettingUri, L"/SD");
> +      if (StrFound != NULL) {
> +        DEBUG ((REDFISH_DEBUG_TRACE, "%a: \"SD\" found in URI: %s\n",
> __func__, *SettingUri));
> +      } else {
> +        DEBUG ((DEBUG_ERROR, "%a: Not an valid @redfish.settings URI\n",
> __func__, *SettingUri));
> +        ASSERT (FALSE);
> +      }
> +    }
> +
>      return EFI_SUCCESS;
>    }
> 
> diff --git
> a/RedfishClientPkg/RedfishConfigLangMapDxe/RedfishConfigLangMapDxe.c
> b/RedfishClientPkg/RedfishConfigLangMapDxe/RedfishConfigLangMapDxe.c
> index 8c93044580..4071e30c17 100644
> --- a/RedfishClientPkg/RedfishConfigLangMapDxe/RedfishConfigLangMapDxe.c
> +++ b/RedfishClientPkg/RedfishConfigLangMapDxe/RedfishConfigLangMapDxe.c
> @@ -2,6 +2,7 @@
> 
>    (C) Copyright 2022 Hewlett Packard Enterprise Development LP<BR>
>    Copyright (c) 2022-2023, NVIDIA CORPORATION & AFFILIATES. All rights
> reserved.
> +  Copyright (C) 2024 Advanced Micro Devices, Inc. All rights
> + reserved.<BR>
> 
>    SPDX-License-Identifier: BSD-2-Clause-Patent
> 
> @@ -606,10 +607,16 @@ RedfishConfigLangMapSet (
>    Status = EFI_NOT_FOUND;
>    Target = FindConfigLangMapRecord (&Private->ConfigLangList.ListHeader,
> ConfigLang, FALSE);
>    if (Target != NULL) {
> -    //
> -    // Remove old one and create new one.
> -    //
> -    Status = DeleteConfigLangMapRecord (&Private->ConfigLangList, Target);
> +    if (Uri != NULL) {
> +      if (StrCmp (Uri, Target->Uri) == 0) {
> +        return EFI_SUCCESS;
> +      }
> +    } else {
> +      //
> +      // Remove old one and create new one.
> +      //
> +      Status = DeleteConfigLangMapRecord (&Private->ConfigLangList, Target);
> +    }
>    }
> 
>    //
> --
> 2.37.1.windows.1



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