From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from NAM12-BN8-obe.outbound.protection.outlook.com (NAM12-BN8-obe.outbound.protection.outlook.com [40.107.237.122]) by mx.groups.io with SMTP id smtpd.web10.21112.1683269198533042841 for ; Thu, 04 May 2023 23:46:38 -0700 Authentication-Results: mx.groups.io; dkim=fail reason="body hash did not verify" header.i=@os.amperecomputing.com header.s=selector2 header.b=FITnmKlB; spf=pass (domain: os.amperecomputing.com, ip: 40.107.237.122, mailfrom: minhnguyen1@os.amperecomputing.com) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=dI0cqgDffVwmOTAG4KLjg2lZeoeXwB5tYpYfhMQT1C+YbzasNrlEzCwSGUltYj+HfV220a4uPZaAFflKhXh8MruYmnYZGVnW/sM/7KFG4oI9SAzqcTFGISkHUzAZykeVihrIwRyn3G7dUphAJ40h8QjIP1L2PeBEfWqdMnNopid0K8DMkyIRe0/nFkeDE6lpgSQhWSsvhH6wbNTbNvBdu7PpNUnGeOtD6ElobAeO57n+rhgMAtLaRnuGDLiaRyVH1+v3jcnK5MAvAzUxqV+oqh6b2hvIvbvB1JuEW4u1+68gycqQLlaOc11AfvDKCeVaFf+UsWOdMLMzLjgqLlXDKw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=/Ix8DAkdu30eL1JNHynVYagve+VFUfzJnJTm5JfoknU=; b=V99N5FA1OAhyNN7piG5ZKrBcyi3OzxwIUtDPIZTBFxacYgRP79gE24PQd8rEvja4NLliErAYUtUhPdmQgi9vEGsYxBdarpHNVyaOmQY7cRgwFmfp02GuT+fE9sxUDV7EdjuiXXEH7Jnm4F5m19of+ESp08yYRES/QrgAFWLK1yE6PwdR0RXIVlVWvt/lt67SyKtKy9jNZMWg6kQO+qFeo9jHwO7wN1aw/CrFA2RdjDs0BQ4iBvSGBJnVlwMVZYDlSMCK82MWDvjO9dDoOLWb50/xdlaCqcEST6dqcK7oPkvtIi3VVjeCibIiu57tNg4OzMAp2YkNTIy6R1oegt+Z6Q== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=os.amperecomputing.com; dmarc=pass action=none header.from=os.amperecomputing.com; dkim=pass header.d=os.amperecomputing.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=os.amperecomputing.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=/Ix8DAkdu30eL1JNHynVYagve+VFUfzJnJTm5JfoknU=; b=FITnmKlBoWyPWVEoGrjIFrs4d2E4IjKa4HxZDQx2lRHPYfWLa1ERz6oyU2KD+Z6aVAYdXydGQfwu4cmlOgV6R7zDat33JRP4MixRVi7UV6cf1l0OLdCmWMEy9UToElNoROqSb5bBqtw4iyzBzvGUJLu8k725qWiWfB8VleuX5eg= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=os.amperecomputing.com; Received: from PH0PR01MB8048.prod.exchangelabs.com (2603:10b6:510:280::7) by SN6PR01MB5087.prod.exchangelabs.com (2603:10b6:805:b8::13) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6363.26; Fri, 5 May 2023 06:46:35 +0000 Received: from PH0PR01MB8048.prod.exchangelabs.com ([fe80::bbdb:b58c:140e:c4e1]) by PH0PR01MB8048.prod.exchangelabs.com ([fe80::bbdb:b58c:140e:c4e1%7]) with mapi id 15.20.6363.026; Fri, 5 May 2023 06:46:35 +0000 From: minhnampere To: devel@edk2.groups.io CC: patches@amperecomputing.com, abner.chang@amd.com, nicklew@nvidia.com, igork@ami.com, nhi@os.amperecomputing.com, tinhnguyen@os.amperecomputing.com, Vu Nguyen , Minh Nguyen , Nick Ramirez Subject: [PATCH V3 3/5] RedfishPkg: Create RestEx child on selected interface Date: Fri, 5 May 2023 13:45:58 +0700 Message-ID: <20230505064600.1478461-4-minhnguyen1@os.amperecomputing.com> X-Mailer: git-send-email 2.39.0 In-Reply-To: <20230505064600.1478461-1-minhnguyen1@os.amperecomputing.com> References: <20230505064600.1478461-1-minhnguyen1@os.amperecomputing.com> X-ClientProxiedBy: SG3P274CA0005.SGPP274.PROD.OUTLOOK.COM (2603:1096:4:be::17) To PH0PR01MB8048.prod.exchangelabs.com (2603:10b6:510:280::7) Return-Path: minhnguyen1@os.amperecomputing.com MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: PH0PR01MB8048:EE_|SN6PR01MB5087:EE_ X-MS-Office365-Filtering-Correlation-Id: a1dd3d22-2be3-4a32-bb4d-08db4d34785a X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: XnLY/qoCzzi+DHKFz0s4gNimctWrGfwfbDbzFz/RScoloCxWlhwI+ASyl7QKl7j/4951V7n7pQIIfublhUv6sOy6YPfeJZyL0M8li/7MmpCo56NbRjD1rfZ6t73xn21vIHD+B52WoSaLQ/FZz3xZ3IwTQBn42rRLN/BdkRjxy1ZhnedtKKRpwWlGwzUO6lMn7zs0T4UJl0N00lM+HQTfXUxBCOTJTt4cQFc3vSrZp/zVrEPMnNoCvcEiKc9njPXwSQilBZwmi4LDwc+6M2cnRl/TZU4SmbVxC3u/QXe7uKAXMqU2qF6GBiaXqGusjQq9y2lXvAam1TE86ULFCggO+xgsMIHHxpnkmHGrRWRdcYizYbH+DchO0hggbChcHmpV2x9Oo4ehTepEnnOv0nY8khvj7/5l2SSNABfLkqFpLdoPzMLPitos4S4g6EjQBmtG6UHWbYfg5cuzUHbS3vEiOlEwfBJsXBCBnoV0Xk636pm0ytQ5FjnyBLNSg7QkWDNNr9LHs7sCp80TxQnGaQkwS8Vc50ldKvn6MDd1vujrRVHJARW2ACoUDLg4xyTV6C4rqav/Jxb5Ny2NX3LswYUqlw1nwenHkwnDgkR73Fom8xWvEMKknLww+6pMVijvNWLf/924UmToNmXhuoEfj/e8Jg== X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:PH0PR01MB8048.prod.exchangelabs.com;PTR:;CAT:NONE;SFS:(13230028)(4636009)(346002)(376002)(366004)(136003)(396003)(39850400004)(451199021)(83380400001)(52116002)(6486002)(1076003)(186003)(478600001)(54906003)(6666004)(2616005)(6506007)(6512007)(26005)(2906002)(38350700002)(5660300002)(38100700002)(66556008)(4326008)(41300700001)(8936002)(66946007)(6916009)(66476007)(8676002)(316002)(86362001)(43062005);DIR:OUT;SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?F3YB5Q6eufZqD8eX/cHWashvLV+15Odm8baDQqKRQ5HZwSay0d4F6Ywd9AKh?= =?us-ascii?Q?N6efS/mbvoE4cuDt+LYoehEtXT5bDClNDjcpLs9w2mKVQxyePWwAtGt5Lz3p?= =?us-ascii?Q?ULvxe+9JJqalNDSE1kbwusIVT2I7ryGSx3v36o6q3abOyO/O+Own/uWL4lIK?= =?us-ascii?Q?TtG7zeZ7IV/pXANb21d5O+UmI+UDvROo+5ukUtLG0mAHbnd9uxy+7Vy9tkYU?= =?us-ascii?Q?Jlq86hz6BhnHXKzrrKbA7EHFnuuT/pGV7jHIDyBwD+HHmloMtDw2m54QaIf5?= =?us-ascii?Q?hPvqm8ndj2+zGvwGvCJz/cEq5nElaSO/nRJcgy1uf5+DfmggawHQDYLdzm05?= =?us-ascii?Q?FLlb6XzqUebeB6kvLB/hsFDczvFu1ufBs1t5PgmAxhNa5WBTko5DTFBWwSQ3?= =?us-ascii?Q?mJUgb8nlJ6WL4HXVrOq+zFcksMdN1HDsY/QzZvtp54L5vv8WOWuNv8ghwSMt?= =?us-ascii?Q?QnRajO+5JMaHEy+54wsoudE6d60wjv+NcVV5eQNc27RyZ39TN32PbVYkmJDI?= =?us-ascii?Q?kL4VMMhjn9aayB9CDkjqShL991VfX+F3VAtvWRbF5qPHHnPrhcRv/gZV5bUG?= =?us-ascii?Q?iZVR2M8O5bo+/CtOsdeDXljPkLUd54Qxf0Fy4tS/zOa7Pn/Nk57XRkInSTQ7?= =?us-ascii?Q?K17DqBB2XFxkz2LK0sV9PIeUj33wp9RsL0QdHwL/Qqx51R6XH6t6IPuPM1Jx?= =?us-ascii?Q?b1Wt2wtNpEFY1hh4GseTV3pMI+Sje6yVw2PaibcORajujiSJQbRjG/K4u87y?= =?us-ascii?Q?602PWeYRtFBnBWpmKXlIn8PZkjbnhA7YMZ8yjpDulSDjrFQN7Bnnb8HL4Ll7?= =?us-ascii?Q?wN10GArwEB4+mFVbYUd7CA37+u05649gAZUhJQhs/d7QgldjbBHPPdX2+EbR?= =?us-ascii?Q?5E1BOslM8v1UrVTDEhX9R6SUQ2lQgrOwQeJ75C2sQBHCSIxB79KvSi2Ae97V?= =?us-ascii?Q?DBkrP2Cl6VcBp89E82lts1/6iQvdsSJ5PNOtf2Xhf3L9hM7H1+YUy1ix2GNb?= =?us-ascii?Q?HcPfg8hXeyV+V2HC3tEjdWwnGVpaqdyPZS6lReQbG2v6mFxZOvNgbZiPE+oT?= =?us-ascii?Q?jqx094lbBEIAG11kA1dRcfo6aNGlq0zMIcrbgsAEWY6eGTo4ZHiAwD84Ydt2?= =?us-ascii?Q?laIwjCMqK3vXFwhMVl/Ki2hL+xoDVXZvJqNfOEiLycaWVmgG1KCLF1K9dGq3?= =?us-ascii?Q?jKErAuqb7j6s/OwqcJ5MMfzpBVdn1fxNvFFylgD2LJJ3os4b7i+aMnAnOe57?= =?us-ascii?Q?7N/alsdRovQhWG49aMY8k3byPc2nbgzAl/QeiLT3Wej3vAzUohsUnFDfUfkr?= =?us-ascii?Q?PnhxEgX+aXWPh7st/K+sAvtBih9rS/yVMGkOmU/jc5YgdUtTGPzKSlU6F8Z5?= =?us-ascii?Q?I0u9tFYHWj++R3BwFin6tMi1LqP9E/QmANgg3hDV79IALDHq9XDsKe34D8Zf?= =?us-ascii?Q?47VSpQmEvpBuE9j1xsRjotubO9Br98XSTcKXoiBbE8TfBdkyd5ZrYeMgVQBG?= =?us-ascii?Q?W08CUw02NRNGLx5j3MsPqqQT/7HHuPTjVrcY83Qkph2aZl/5hwjHN9ByRKmn?= =?us-ascii?Q?k4Q1lpT/9XXKAwVAs028hIDefZf8xKexiVG273BFuvo48XkVEkFFtaAlSDOh?= =?us-ascii?Q?CyqD8D5BHjTy9TrTcIFXLa6k73JpR9SuHqRwz8oKwbWv?= X-OriginatorOrg: os.amperecomputing.com X-MS-Exchange-CrossTenant-Network-Message-Id: a1dd3d22-2be3-4a32-bb4d-08db4d34785a X-MS-Exchange-CrossTenant-AuthSource: PH0PR01MB8048.prod.exchangelabs.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 05 May 2023 06:46:35.8047 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 3bc2b170-fd94-476d-b0ce-4229bdc904a7 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: czRuq8JQjYnsS0b/fFnK4xfJs/a8jsPYq1LfjF9tbIbsCHVSmzQcvp9g3ctEUl6HSOKrPfqg2sj7w92ksd/IMNTkRm40u4nilJ59BBje0wotFSPxCECtl8XmfbzLsF47 X-MS-Exchange-Transport-CrossTenantHeadersStamped: SN6PR01MB5087 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain From: Vu Nguyen When a MAC address matching interface is found, a RestEx child will be created to provide the Redfish communication on that interface. Currently, It will try to locate all RestEx binding services and choose the first satisfied instance without taking care about current selected interface. This might raise an issue on the system with multiple network devices that the RestEx child was installed on wrong interface. Signed-off-by: Minh Nguyen Cc: Abner Chang Cc: Igor Kulchytskyy Cc: Nick Ramirez --- RedfishPkg/Include/Library/RestExLib.h | 3 + RedfishPkg/Library/DxeRestExLib/DxeRestExLib.c | 153 ++++++++---------= --- RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.c | 1 + 3 files changed, 63 insertions(+), 94 deletions(-) diff --git a/RedfishPkg/Include/Library/RestExLib.h b/RedfishPkg/Include/Li= brary/RestExLib.h index bc4e4ca6caa7..2c32c3684cf4 100644 --- a/RedfishPkg/Include/Library/RestExLib.h +++ b/RedfishPkg/Include/Library/RestExLib.h @@ -2,6 +2,7 @@ This library provides help functions for REST EX Protocol. =20 (C) Copyright 2020 Hewlett Packard Enterprise Development LP
+ Copyright (c) 2023, Ampere Computing LLC. All rights reserved.
=20 SPDX-License-Identifier: BSD-2-Clause-Patent =20 @@ -20,6 +21,7 @@ This function allows the caller to create child handle for specific REST server. =20 + @param[in] Controller The controller handle used of selected = interface. @param[in] Image The image handle used to open service. @param[in] AccessMode Access mode of REST server. @param[in] ConfigType Underlying configuration to communicate= with REST server. @@ -32,6 +34,7 @@ **/ EFI_STATUS RestExLibCreateChild ( + IN EFI_HANDLE Controller, IN EFI_HANDLE Image, IN EFI_REST_EX_SERVICE_ACCESS_MODE AccessMode, IN EFI_REST_EX_CONFIG_TYPE ConfigType, diff --git a/RedfishPkg/Library/DxeRestExLib/DxeRestExLib.c b/RedfishPkg/Li= brary/DxeRestExLib/DxeRestExLib.c index d9acad24dec1..0b3ae2755e86 100644 --- a/RedfishPkg/Library/DxeRestExLib/DxeRestExLib.c +++ b/RedfishPkg/Library/DxeRestExLib/DxeRestExLib.c @@ -2,6 +2,7 @@ This library provides help functions for REST EX Protocol. =20 (C) Copyright 2020 Hewlett Packard Enterprise Development LP
+ Copyright (c) 2023, Ampere Computing LLC. All rights reserved.
=20 SPDX-License-Identifier: BSD-2-Clause-Patent =20 @@ -9,6 +10,7 @@ =20 #include #include +#include #include #include #include @@ -21,6 +23,7 @@ This function allows the caller to create child handle for specific REST server. =20 + @param[in] Controller The controller handle used of selected = interface. @param[in] Image The image handle used to open service. @param[in] AccessMode Access mode of REST server. @param[in] ConfigType Underlying configuration to communicate= with REST server. @@ -33,6 +36,7 @@ **/ EFI_STATUS RestExLibCreateChild ( + IN EFI_HANDLE Controller, IN EFI_HANDLE Image, IN EFI_REST_EX_SERVICE_ACCESS_MODE AccessMode, IN EFI_REST_EX_CONFIG_TYPE ConfigType, @@ -41,8 +45,6 @@ RestExLibCreateChild ( ) { EFI_STATUS Status; - UINTN NoBuffer; - EFI_HANDLE *Handle; EFI_HANDLE ChildHandle; EFI_REST_EX_PROTOCOL *RestEx; EFI_REST_EX_SERVICE_INFO *RestExServiceInfo; @@ -59,116 +61,79 @@ RestExLibCreateChild ( } =20 *ChildInstanceHandle =3D NULL; - // - // Locate all REST EX binding service. - // - Handle =3D NULL; - NoBuffer =3D 0; - Status =3D gBS->LocateHandleBuffer ( - ByProtocol, - &gEfiRestExServiceBindingProtocolGuid, - NULL, - &NoBuffer, - &Handle - ); - if (EFI_ERROR (Status) && (Status !=3D EFI_BUFFER_TOO_SMALL)) { + + ChildHandle =3D NULL; + Status =3D NetLibCreateServiceChild ( + Controller, + Image, + &gEfiRestExServiceBindingProtocolGuid, + &ChildHandle + ); + if (EFI_ERROR (Status)) { + DEBUG (( + DEBUG_ERROR, + "%a: Failed to create service child - %r \n", + __func__, + Status + )); return Status; } - - Handle =3D (EFI_HANDLE *)AllocateZeroPool (sizeof (EFI_HANDLE) * NoBuffe= r); - if (Handle =3D=3D NULL) { - return EFI_OUT_OF_RESOURCES; - } - - Status =3D gBS->LocateHandleBuffer ( - ByProtocol, - &gEfiRestExServiceBindingProtocolGuid, + Status =3D gBS->OpenProtocol ( + ChildHandle, + &gEfiRestExProtocolGuid, + (VOID **)&RestEx, + Image, NULL, - &NoBuffer, - &Handle + EFI_OPEN_PROTOCOL_GET_PROTOCOL ); if (EFI_ERROR (Status)) { - FreePool (Handle); - return Status; + goto ON_ERROR; } =20 // - // Search for the proper REST EX instance. + // Get the information of REST service provided by this EFI REST EX driv= er // - while (NoBuffer !=3D 0) { - ChildHandle =3D NULL; - Status =3D NetLibCreateServiceChild ( - *(Handle + (NoBuffer - 1)), - Image, - &gEfiRestExServiceBindingProtocolGuid, - &ChildHandle - ); - if (!EFI_ERROR (Status)) { - Status =3D gBS->OpenProtocol ( - ChildHandle, - &gEfiRestExProtocolGuid, - (VOID **)&RestEx, - Image, - NULL, - EFI_OPEN_PROTOCOL_GET_PROTOCOL - ); - if (EFI_ERROR (Status)) { - goto ON_ERROR; - } - - // - // Get the information of REST service provided by this EFI REST EX = driver - // - Status =3D RestEx->GetService ( - RestEx, - &RestExServiceInfo - ); - if (EFI_ERROR (Status)) { - goto ON_ERROR; - } - - // - // Check REST EX property. - // - switch (ConfigType) { - case EfiRestExConfigHttp: - LenOfConfig =3D sizeof (EFI_REST_EX_HTTP_CONFIG_DATA); - break; - - case EfiRestExConfigUnspecific: - LenOfConfig =3D REST_EX_CONFIG_DATA_LEN_UNKNOWN; - break; + Status =3D RestEx->GetService ( + RestEx, + &RestExServiceInfo + ); + if (EFI_ERROR (Status)) { + goto ON_ERROR; + } + // + // Check REST EX property. + // + switch (ConfigType) { + case EfiRestExConfigHttp: + LenOfConfig =3D sizeof (EFI_REST_EX_HTTP_CONFIG_DATA); + break; =20 - default: - goto ON_ERROR; - } + case EfiRestExConfigUnspecific: + LenOfConfig =3D REST_EX_CONFIG_DATA_LEN_UNKNOWN; + break; =20 - if ((RestExServiceInfo->EfiRestExServiceInfoV10.RestServiceAccessMod= e !=3D AccessMode) || - (RestExServiceInfo->EfiRestExServiceInfoV10.RestServiceType !=3D= ServiceType) || - (RestExServiceInfo->EfiRestExServiceInfoV10.RestExConfigType != =3D ConfigType) || - ((LenOfConfig !=3D REST_EX_CONFIG_DATA_LEN_UNKNOWN) && (RestExSe= rviceInfo->EfiRestExServiceInfoV10.RestExConfigDataLength !=3D LenOfConfig)= )) - { - goto ON_ERROR; - } - } + default: + goto ON_ERROR; + } + if (RestExServiceInfo->EfiRestExServiceInfoV10.RestServiceAccessMode != =3D AccessMode || + RestExServiceInfo->EfiRestExServiceInfoV10.RestServiceType !=3D Serv= iceType || + RestExServiceInfo->EfiRestExServiceInfoV10.RestExConfigType !=3D Con= figType || + ((LenOfConfig !=3D REST_EX_CONFIG_DATA_LEN_UNKNOWN) && (RestExServic= eInfo->EfiRestExServiceInfoV10.RestExConfigDataLength !=3D LenOfConfig))) { + goto ON_ERROR; + } =20 - // - // This is proper REST EX instance. - // - *ChildInstanceHandle =3D ChildHandle; - FreePool (Handle); - return EFI_SUCCESS; + // + // This is proper REST EX instance. + // + *ChildInstanceHandle =3D ChildHandle; + return EFI_SUCCESS; =20 ON_ERROR:; NetLibDestroyServiceChild ( - *(Handle + (NoBuffer - 1)), + Controller, Image, &gEfiRestExServiceBindingProtocolGuid, ChildHandle ); - NoBuffer--; - } - - FreePool (Handle); return EFI_NOT_FOUND; } diff --git a/RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.c b/RedfishPk= g/RedfishDiscoverDxe/RedfishDiscoverDxe.c index 14702748813b..0b86a7b5cc4b 100644 --- a/RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.c +++ b/RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.c @@ -87,6 +87,7 @@ CreateRestExInstance ( EFI_STATUS Status; =20 Status =3D RestExLibCreateChild ( + Instance->NetworkInterface->OpenDriverControllerHandle, Instance->Owner, FixedPcdGetBool (PcdRedfishDiscoverAccessModeInBand) ? EfiRes= tExServiceInBandAccess : EfiRestExServiceOutOfBandAccess, EfiRestExConfigHttp, --=20 2.39.0