From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 71E3145675; Sun, 21 Jul 2024 14:11:33 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 62B0842E67; Sun, 21 Jul 2024 14:11:33 +0200 (CEST) Received: from NAM10-BN7-obe.outbound.protection.outlook.com (mail-bn7nam10on2056.outbound.protection.outlook.com [40.107.92.56]) by mails.dpdk.org (Postfix) with ESMTP id 330C442E66 for ; Sun, 21 Jul 2024 14:11:32 +0200 (CEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=y/lT9u03e24dKvW1TpIYZEVLdLdhHHr70jJ2AtBtFDlYeKEIdYtXth33Qfie5I6fRfdFyZF6TGQNfPVEopWabzaYTm5Ma17OcPKEsL05WRqX/cwG8M8/GrPA+okMKd7uFV0JUYAd4BCPKmTpxgUjwe01dwOVkwMkR33my99FmNDVrBCzk19kmWdnXD5Nfh3yAIEZru2+kvhKFxD1sAflGxVllOBQ+8TkYFvdyvgKCDJswG3oUReNrv3pBXZ8qJ+/wsJfq+VaEUv9NKeqWi8s+DY82f1FI/FDoiTMGZI6J/E7x3hfKhRVnBiYgDCCDZYzPBd7fkIdT98310aOxFPFAg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; 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=9YcaNU/RvWdHoWxoQPc1oD7fPdHaMOVV0+y8eVl+rBw=; b=DtzIvTXGfNO/w6/1/SyIgZPpypIk7A+NvHofDZ94WLyM5f+XT4yTozk6K+32J8fHsWgOdrsZkz2+BuNJmJulMDxRMlAcmd00sFzdqF131f8/HqVGBraoKOyICp0pQe25t+Xbb7H3Vr+889+b0BkB6zHdx6LLmnBhHDPlub6FBdtsm4GrmoeOGGVFBs6txFQlxAJsHj5rx7okNbVXJJGMp8zrbR067qxRs7SmtDofL8oaFkAqM8MQhD7Q/aLVPRD7biZzhjUAC36hdnEAZh/Ip95LLC+JK0Cyc0gGjvSR9VRmCh9twdbgpM1gGkVd9e1pbY9hJ1NLueycSycVgptKAg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nvidia.com; dmarc=pass action=none header.from=nvidia.com; dkim=pass header.d=nvidia.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=Nvidia.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=9YcaNU/RvWdHoWxoQPc1oD7fPdHaMOVV0+y8eVl+rBw=; b=BiuRDjXIL/TvfR+cOAg/Bazv9yacUWzhqBoQASdoYo5OEP3GMkUkEvLncE6HjCeB3qOSq0/vm+K/Nuo6jDBjtQC1lalwO6mnPK9BzTKwkQieTgrm4EpAchm3KtOhVImOWFo9sGQcS9m9roKQwO8pmNZxSZY1b/sQNjMHaInJqMBgX3nas2gDcNRbxT5DiyYBQAuyDKbpE0WG6gvKu/GWp/5uGuyGKCv2h+1W3FsWMblsZjpVtcmCWV1C6EEKHb3P5kTtx6y3shmCemv32k+IA9ETvFMh8sM2rBScMUW0hvTFNnRkyQbEH7pyQ6Cvvg6JJ9HzY/cCE7Oyp/o9MEjK1Q== Received: from MN0PR12MB6056.namprd12.prod.outlook.com (2603:10b6:208:3cc::12) by LV3PR12MB9438.namprd12.prod.outlook.com (2603:10b6:408:212::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7784.16; Sun, 21 Jul 2024 12:11:29 +0000 Received: from MN0PR12MB6056.namprd12.prod.outlook.com ([fe80::4b75:36f4:f278:fcc0]) by MN0PR12MB6056.namprd12.prod.outlook.com ([fe80::4b75:36f4:f278:fcc0%5]) with mapi id 15.20.7784.017; Sun, 21 Jul 2024 12:11:29 +0000 From: Raslan Darawsheh To: Bing Zhao , Slava Ovsiienko , "dev@dpdk.org" CC: Ori Kam , Dariusz Sosnowski , Suanming Mou , Matan Azrad Subject: Re: [PATCH] net/mlx5: fix the uplink port probing in bond mode Thread-Topic: [PATCH] net/mlx5: fix the uplink port probing in bond mode Thread-Index: AQHa2GdzPuVpnjL+zUWbKOARbRF0KLIBHVhd Date: Sun, 21 Jul 2024 12:11:29 +0000 Message-ID: References: <20240717163541.324772-1-bingz@nvidia.com> In-Reply-To: <20240717163541.324772-1-bingz@nvidia.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: msip_labels: authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nvidia.com; x-ms-publictraffictype: Email x-ms-traffictypediagnostic: MN0PR12MB6056:EE_|LV3PR12MB9438:EE_ x-ms-office365-filtering-correlation-id: cccace4d-6486-4122-e6ef-08dca97e407f x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; ARA:13230040|376014|1800799024|366016|38070700018; x-microsoft-antispam-message-info: =?iso-8859-1?Q?AaHwU4k8uN+HTmClkazp9Jm0LPq4RP7J87xGFT7dbX8aZcgBo5PFghs+Jg?= =?iso-8859-1?Q?kzhWghBbg00uOdlXi+7Xkbj1XNYJ/R/4PFgw8tWqFW7sezfEAMyBxzxp95?= =?iso-8859-1?Q?DuqRmgVsed/ypy+LF0/9w9BnGb3bPzupf5p0QsoYYjWdHXz0CL11glUHE/?= =?iso-8859-1?Q?/pEk38xjzdqWfB+pNsIQbyJtvNTCyTKSwZlOYB2wqZn7sCa15LJAhzI9qA?= =?iso-8859-1?Q?jCst7I2oj3/mLb3TEJNyNBqdFL6hrxeXReHP/g+Mf7nJf+NLwGi+KnsOY2?= =?iso-8859-1?Q?PtDug2wkgpWPvOmE5jxfdfF4hK1vmyKfk//7SbvyaaYpzkXbM7tpEclq01?= =?iso-8859-1?Q?QZI0++pC7Ua+aBAMCreYV/l0cuvr+iv7D7AA/GYjlNDcxeQUajt3VJXh9b?= =?iso-8859-1?Q?v4zKrTnxeRlEJOCp44DMKnJdo8jb09sN0x+W7Hfr8v9H24Mm4DHVMFqEPK?= =?iso-8859-1?Q?E1ztHSch+j4dhu0o3og2gwB1IlY6jzyVnzSstgRotPkeakXwsky4SwsGoy?= =?iso-8859-1?Q?ctJm0KEXN8wMFI3cN7JvwTrV7Q0F3rhX9KaTWVwXHyURkV44Bn3M/mIGw3?= =?iso-8859-1?Q?2uQL3OUjgvgJWPwtRDUqmnKPBAgt2Cd58ZVQAFsXnwGTwBMTRAiuq/lEzG?= =?iso-8859-1?Q?etHYNfJio34AmhoHtc2hZjWrTwlZU9wZNvFKVDRUeEawMe4LHo/o2kutEj?= =?iso-8859-1?Q?75i7CTjGVorBOv5Fnu2lzT6/Pe7xgTJHsBu9hj3zpbEAvs4jGVrpXZxmMW?= =?iso-8859-1?Q?3M2kiB3e6IoBZ+JTA8WSWPXlaWo9WzwIhWSImCX/p/WhB6MoWY22Z1aARH?= =?iso-8859-1?Q?vzM5V24wAJPX4Kxqfxl2zET35MpGzEnNOTslFnOq7PWwdSD6Te5YS1A/En?= =?iso-8859-1?Q?kuWv0OYsXS8VZb2oZqXDPuCguCPhlhofqXxcPAjBOFxInCkbXuXrYbMrgq?= =?iso-8859-1?Q?OtC4CL1oGsndpsOvpSso4fXkxC4jSURZ2q80aOMerzvPl9xHGp1f+BBJnj?= =?iso-8859-1?Q?Ek3+OFImEFNvcBGJlBkvHezyLQ6ISe8U1OzEDWaHWm263BBfAQq0o9/aaS?= =?iso-8859-1?Q?vahMXlNIRyknzRgW9WeCV4OBZ+zVo1HciLQNRv+ORlYtPZEMH+kA+/JL5o?= =?iso-8859-1?Q?NcBHmE4bfNc7dlgzg1jePCdlyJltzMNaPVuj/QtQY7AbfCIr77NzX35LeA?= =?iso-8859-1?Q?UVMFWhzcW51ziBBRywCsKZ6HmxCFoim3U5pI12L+Vb0tu8A9WmQvMktRj6?= =?iso-8859-1?Q?QMaRdpeII3dnqwYLVkiGymszuROZkD8AGHY8CiCrolH+lMvtwho6GQC/sb?= =?iso-8859-1?Q?MUi588e3ej1vIUYD0Y1l21LQ92j7QFCmQYtKvnk/l9GjAclh6aPHw6m9NB?= =?iso-8859-1?Q?Gj5aD+exZgJc/BdMY0Xe9Po0RFzWevUxksO19CqEJHu+V8fA++sd6xyof9?= =?iso-8859-1?Q?z/NNr5mddehWADnWi6TywfSSF7g7/37s38TzfQ=3D=3D?= x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:MN0PR12MB6056.namprd12.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230040)(376014)(1800799024)(366016)(38070700018); DIR:OUT; SFP:1101; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?iso-8859-1?Q?bq+tpRm0WLQNDkk9ToAjNTyCuM6F2/oZvaV4trWsbn/+a27IABTiKgzw7B?= =?iso-8859-1?Q?hmZnHlm6FQaaBjpYVyWhsMbKdfgDY3lEHDqikjf5hM+V8hxKFlUofgKgnR?= =?iso-8859-1?Q?yAKtyXN985uUSEcS3iTSlOQISc6VtDAzRL8Z+cMjxTDVPmYtP1wq2b/ZNS?= =?iso-8859-1?Q?85NthMIa6Iv6WCsfjy/1e1QfUwkQwR4Vqmjm6ft00mnpljumpWCF5+EHED?= =?iso-8859-1?Q?3GYSgg/xsCNJqYy6JoWegChq/nhJAhpORXMVGlvpShOm2lOV2CPjWwBY49?= =?iso-8859-1?Q?qz7rdE2kT4dAjFl8E/WDtQy5D1JugLqkqQ5zDUi5HXB7jMmM7A0IohOO16?= =?iso-8859-1?Q?DoDKQml1asYNk97ZcdeE4VYn9K5Ypfb0T7ziRZ4f8Xq1fQYwj2LBNg4cNE?= =?iso-8859-1?Q?FQwxAe6k7Z6OCAsz8EkskoQalTDRMwhqbgKyBgzD2gLAvc9zCjohG6wJWL?= =?iso-8859-1?Q?JxIryQHqjhEpchbd1eCD9XxE3wiAaVF7C8Q81rY+YVhciFSQMhfEVM15nY?= =?iso-8859-1?Q?uj6RAH3eGsK7w4djz8QAzQwGyElm5TJ8S8e2MiG1v6s2CEKH6OnxAGJthX?= =?iso-8859-1?Q?/N3W23MQO71FDgMKBoASk3GHbedn2Jkerp9mDHkpdLUgfvHeHy2HKsM78F?= =?iso-8859-1?Q?BHsCzC9c5PcYGNfzGXdgmj3arj2Y2jW75HUFwDEVsxTBZ6O5TUDyeyMyC1?= =?iso-8859-1?Q?gHSeqXdtaPsWcj6X7Q1JNnwkrLRBsc/LcCCATKzV+ZSPp4cM2HeiPt8fub?= =?iso-8859-1?Q?nHRkT13g2GXLf3j9drZWx4JpRCRLUbDN7u0CveJnAXKRoRuVZrKRAQNu2A?= =?iso-8859-1?Q?LlJ3azf2D9u5Iw/C1bwYkdxUP3Q2cCnqB/Ahk9wnScWpFZ6TiBcLhUBVBI?= =?iso-8859-1?Q?U8ePePH4cwmhHEbqxoIkgdze5lkWyJazyiYMZtjGBTZv/FPDuIjoCSNcF4?= =?iso-8859-1?Q?DK9kOUPPBVB+G5SHOkQrfgRtdWd0+zn4DMDxCE25b5qxAph/lmF0+mb9BX?= =?iso-8859-1?Q?YQ4puX8mN5aO+r3dUr1MYJKs8xmZ91hQKeIQHl2lxHs5Wam+y4Bx4sbJSf?= =?iso-8859-1?Q?0cgW0GUhU89SZyFBiPk5fBei8eDTWHsxxNdPoHzKsvDIv/BnsObqNbhIxk?= =?iso-8859-1?Q?Q0IyHisxfyJsPm2XWz86r/7g1ZlaMRHWIteOziIOu5vMFylqxmn3zRhyd4?= =?iso-8859-1?Q?ouCsTNf7ngvnRURJI+IuacSDaB0uflkeOk8SlPHI+J/fMqTQPBx6GRA0MQ?= =?iso-8859-1?Q?HtRiCj/XQC0/vTGTx+iMNRosHftwsx9sVej6pHV/8COclSm6cW49gy64Gl?= =?iso-8859-1?Q?gsWkgoMahzWASgH+AQZdQ1zCYr+8QrQYdHtXWgew9WF1OOdWAjrNaun4/B?= =?iso-8859-1?Q?Ed9+7X6nUPSmPiSmhML4D/njz8XjUzhv/os941Z1bWu8tqarMiikTPu/ur?= =?iso-8859-1?Q?qLSXIgRqSXComc7FqxQkeWB5ZbFBJxciZnolc4QVm6X0kzgq3B6erl8/Hc?= =?iso-8859-1?Q?Hoxsz/hQ//97pv8YlT0mkOD6Kk6EMNDX+JFsuVb1md/l8h4ohv+qlvNb0y?= =?iso-8859-1?Q?hDNrefSc0515ko3xu4iUOhc7Jnx/rTdhvuw5LSK72qxGPN3RDJrP6YqWEs?= =?iso-8859-1?Q?Z8bXTsTbJduRQ=3D?= Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-OriginatorOrg: Nvidia.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: MN0PR12MB6056.namprd12.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: cccace4d-6486-4122-e6ef-08dca97e407f X-MS-Exchange-CrossTenant-originalarrivaltime: 21 Jul 2024 12:11:29.2479 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 43083d15-7273-40c1-b7db-39efd9ccc17a X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: l179MNgcEg0zxLrsX21rX4IUAEOEcP20O5Z96FsYhTc1q6UYVeCknR+t6rVtGTs/99RNDqledfKiB30drpe4sw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: LV3PR12MB9438 X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Hi,=0A= =0A= From: Bing Zhao =0A= Sent: Wednesday, July 17, 2024 7:35 PM=0A= To: Slava Ovsiienko; dev@dpdk.org; Raslan Darawsheh=0A= Cc: Ori Kam; Dariusz Sosnowski; Suanming Mou; Matan Azrad=0A= Subject: [PATCH] net/mlx5: fix the uplink port probing in bond mode=0A= =0A= In the HW-LAG bonding mode, the representor port can be from both=0A= slave PFs. When probing a representor (REP), the UPLINK (proxy) port=0A= always needs to be probed firstly before any REP port.=0A= =0A= In the current implementation, when probing a device with the=0A= following format:=0A= =0A= -a 0000:XX:00.0,dv_flow_en=3DN,representor=3Dpf1vfy=0A= =0A= Since the REP belongs to the 2nd PF in the bonding, the UPLINK would=0A= not be added into the probing ports list.=0A= =0A= 1. In dv_flow_en=3D1 mode, the REP itself can be probed. But it didn't=0A= obey the rules and the behaviors were inconsistent.=0A= a. When probing the REP from 1st PFs, the UPLINK was also probed.=0A= b. When detaching the UPLINK, all REPs were detached.=0A= 2. In dv_flow_en=3D2 mode, since some resources can only be allocated /=0A= created on the proxy port, the probing would get a failure.=0A= =0A= By removing the unneeded check of the bonding PF device index, the=0A= UPLINK will always try to be probed with any format.=0A= =0A= Fixes: 2e569a370395 ("net/mlx5: add VF LAG mode bonding device recognition"= )=0A= =0A= Signed-off-by: Bing Zhao =0A= Acked-by: Viacheslav Ovsiienko =0A= =0A= Patch applied to next-net-mlx,=0A= =0A= Kindest regards,=0A= Raslan Darawsheh=0A= =0A=