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 75D82A0A02; Tue, 6 Apr 2021 09:02:33 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 0562D140F24; Tue, 6 Apr 2021 09:02:33 +0200 (CEST) Received: from NAM11-DM6-obe.outbound.protection.outlook.com (mail-dm6nam11on2075.outbound.protection.outlook.com [40.107.223.75]) by mails.dpdk.org (Postfix) with ESMTP id 34E6E406A2; Tue, 6 Apr 2021 09:02:31 +0200 (CEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=dIvMRhIZ1bMcGft2EwDNi7shIkuqWOWYgosoet0yRncHj1Bu+bUWhceihfQDdbNujGB4IR2mSUIc0cU5LH1pjoW7L36z4n/+HokDDITX8ucNV+8SxmWUyJZBDbuW8MGCrE2BtwzjkGHZoAAhwYh2LonpOeEZjoasu5K5qi5X2fRmtBu4Vk68wm/7FoXV2uTFx1hN+4lh4zdafvDVv3mcf9KD2q4HKS366qc/GzOWjWpH0GQt1hqlW5KxPvytrDg/v0qdDQyloVhqre7zcDTc5o/mUKNDjuT1uheP12o3+IX1JC80bcOVDlmrcCMk565hJszXbSFGgD4bYW8A5kI/2Q== 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-SenderADCheck; bh=y6+Yt1OYgufWyrS6mfmN6hJyjO8XgCW6iR8xgg/CC+w=; b=I3vRenzdcX7jlaOCLgdmiZ8NHYYSR7hoJ9xM+FSPAXAo/wiveYRe9Zo1QwaVffUEsoYAL9uYZXZQaU8KbC7bcq4h/o+uo3L7ZTBQfI/5ZNYrgJj5MNifcR07aaZOpsz74Kepj6kMe7zpI9OKiiP0b81AZTOxs8e2McwFHxlQNpdd+YmViP2muCtxt5qIFSLJtXbp8vNLv86QiFNL3mAA80HFLtlRzuYf4C6ThxN7I347P1iqrD1+h6aWoJisp9tQwJ0/tTKdDfx1625iPlKgSAXAe7Y5mtqcM3ZX0ULZJV20DLUKzuUT4ibrGeCGidgNtyrtVrySs/Uf+H5B7MIwkA== 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=y6+Yt1OYgufWyrS6mfmN6hJyjO8XgCW6iR8xgg/CC+w=; b=WbDtxOMHId8506xvzPr/OFGqytOrX0fXwnDL9+8ubCLUGbg9mo6NzRb8FFFecTsy2qXN6BGeR/ypHYIw2o0+Q+3GLrYGQLOLCbB/Q6tdMaqEv7BjLIwgoP/5qpjrWNwBrctnHUr5Q31ZiUot0Zf8eHUGUs1u384gaaHt83Hz9Cgvk4xOJ7AJKILOAnzY3hr3MIlZHXctYky6NDUI0USdZ451Cm7p3fOb6WbfX2SvAIFuGDqoGh0GYzbxa5zUn7Fk6ru3Qutx8eDAkWv/bHDZiFdFxrV4pPrkhbfNlgTyWNkxwvThGDNo5dCC1iDGKwnYMxoLpxseJKhnblf5r44Aig== Received: from DM6PR12MB3945.namprd12.prod.outlook.com (2603:10b6:5:1c2::27) by DM6PR12MB4762.namprd12.prod.outlook.com (2603:10b6:5:7b::17) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3999.29; Tue, 6 Apr 2021 07:02:29 +0000 Received: from DM6PR12MB3945.namprd12.prod.outlook.com ([fe80::9d62:2f16:264b:285d]) by DM6PR12MB3945.namprd12.prod.outlook.com ([fe80::9d62:2f16:264b:285d%5]) with mapi id 15.20.3999.032; Tue, 6 Apr 2021 07:02:29 +0000 From: Tal Shnaiderman To: NBU-Contact-Thomas Monjalon CC: "dev@dpdk.org" , Matan Azrad , Raslan Darawsheh , Ophir Munk , "stable@dpdk.org" Thread-Topic: [PATCH] net/mlx5/windows: fix link speed calculation Thread-Index: AQHXKk4RFPxq2rLl50KFvr2jk+d1bKqnD96Q Date: Tue, 6 Apr 2021 07:02:29 +0000 Message-ID: References: <20210405161844.9560-1-talshn@nvidia.com> <1920198.nUkcdjmOzk@thomas> In-Reply-To: <1920198.nUkcdjmOzk@thomas> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: monjalon.net; dkim=none (message not signed) header.d=none;monjalon.net; dmarc=none action=none header.from=nvidia.com; x-originating-ip: [193.47.165.251] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 272e52a5-af83-4799-3c43-08d8f8c9f18b x-ms-traffictypediagnostic: DM6PR12MB4762: x-ld-processed: 43083d15-7273-40c1-b7db-39efd9ccc17a,ExtAddr x-ms-exchange-transport-forked: True x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:269; x-ms-exchange-senderadcheck: 1 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: lk8vawGFKonLtJIYUkzEUuCrFfEaHbD9gK1sQIH6EH4WDUFn7qLabKHAxwaNYFm7QUEBP0nfHdyfGtNXV380gIpaAclxF6iQsdM9y9MRkVdK6JsknU6lJovyC28lt9nVFd+V5/PsvkqqD8GKwLoMnBUzL7LSO6BS7rmbRoqO2+xLT1Do0BmfKBZ9QVxqw/nWbIqHkhXaMOV3b4WLY2DDKd8lbrifm01jjXxT8YlMtqplaT3nLceqK5OS1bN6HoXWxDDSjgXFJdSrbRVDOosXYr4j+T2CAT4DZYcoXYH7FAGH8gy8d8NDAJe//kFUSlN7frDpfR7isRDJZRRMwPh8N/IFMX5E53HHsJnFPLE+tR0BzYJf4rwbB0W5qlLtQsEtf+HfeAfat3mK5VV1K5GWct3P9IBR+KcEQUAtqMYBRpwjkPk2XQifbu+AhWnvFYjFLT8TdNaTgIKI9htiDQpN15Zw7wx30jlBtULSevEKZCZRUwzpSQgRBWuitqN7QgkO7kL5kzmfz0k7ttfL+VVoK+g1PMFX+qBp3q9NN2QOI2k8KNa9WDysn6iGK/ib36QFZ0QSPECQ9fky5tHhWGYAhsKh0EmtV9tsrjFj+Ef6EDU3Ws/p6/FlK86LKjFymhNXMmZa8WPbC0r7EcKb+Cb1XA== x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:DM6PR12MB3945.namprd12.prod.outlook.com; PTR:; CAT:NONE; SFS:(4636009)(376002)(346002)(366004)(396003)(39860400002)(136003)(2906002)(66446008)(478600001)(8936002)(316002)(7696005)(4326008)(4744005)(64756008)(86362001)(33656002)(6916009)(66476007)(71200400001)(26005)(6506007)(54906003)(76116006)(66946007)(38100700001)(186003)(9686003)(52536014)(5660300002)(8676002)(66556008)(55016002); DIR:OUT; SFP:1101; x-ms-exchange-antispam-messagedata: =?us-ascii?Q?1gU1dSMGkbJ7yujRW2Nv7Aw6rUqczNomQl0OMWogmbl34cnkgOyEyt0drDUF?= =?us-ascii?Q?FMM/ivZ3hAuRbzu4R+HK8vJMhTzAcGDoKmpoG32hlzLWuGuA/2Dp98iEyhEw?= =?us-ascii?Q?OOfV4JxmUBcNOx6qobNJoto/wQ8EDrOA0J8blmnCoRbLEzob3el1gY0mEaf1?= =?us-ascii?Q?fhSt3iqDxbDRrSyliKViTlZ7+CkC6SwyOHyCR+NEKfgd8qaSbB8I9KlP0VQI?= =?us-ascii?Q?QcY2uK6InAZge0nGmf/Z2WBQ1HrlIgHJ/bR7Bt2NHCJhIbuY2EF+8E3Pbiw6?= =?us-ascii?Q?XTzK6Rk9Mg+k2UIeU/QNUFnWy+Kjj2zLU4lEkmbcyX6FeQyPWH7jf5UvkcwK?= =?us-ascii?Q?1u9LTLSY4kccQGbaAMBipkXXjyL+PAPnlKa28BcpVdNlnxZ2mQAZR48JmUmf?= =?us-ascii?Q?Jv1nHYfPuMhf6OGoY5zWt8qDanf+UkbtcnHCULCwyOwpWzu3f/K1ZXST+nnU?= =?us-ascii?Q?csFz6XcMysxIZf8+lqGD3yFg8OaSJ8+FPyUwzdpKk4ws97nL6yv1ILNz7gFI?= =?us-ascii?Q?5zmn2HTqyUXY3FeG4/NyrB56RWGf2Iq2WmXNUAPzEuCLpRcXFqXF7KPGoO9Z?= =?us-ascii?Q?6tegC1LmwQoR1rmZofrdpBJ3PSPJPFtfqTWHqZfvyVYandxCp7TcO/yJ/ha2?= =?us-ascii?Q?ei3T2rJch3/JAGHictDSOpTjyqnlUF3RfXmIz4bxAsfabIx/NNWSCDDuztvU?= =?us-ascii?Q?zbgvln3oa8arTXRAkL7IoBSs1rQM8oEpWRI9eEmW/7YvYmUEkUzB++mw00OS?= =?us-ascii?Q?5yv70QLcC6V9ANS9f2ztBMUUHJTvloreBypZX0qAX4u+i+WcJKVjQr6yuO7v?= =?us-ascii?Q?XWmKrwEbfIgt3ZnhtnadEOeCC6+EtZZfAL+upijWVnQdjsP1o6wxrTAFomJf?= =?us-ascii?Q?QzpYZHM2plDwEnGJSeMpiOnZ61Xk3b9LLfuQFgFdcCl/qWr8jOGbK+oOAjpu?= =?us-ascii?Q?luzj1F5qP3Tb75/lXBJH2aLxXCt15AyR2fnyu/wZGTKaPLYsINgI61hHqkEc?= =?us-ascii?Q?yAomhCdPTmKhhlaMNFgWoK1IkEF0nXhWNPtJ1hbk7et7BEcMVqkECPQWaUTB?= =?us-ascii?Q?6dHrLc1AVUYsWr4nszZmcsy6drf5TuYFle9cV3gYW/GSBvWHZzTkM9qFz8vz?= =?us-ascii?Q?N01phhKnx/nKDFEXcjyE8Vd/UjAgu4/BE3j17XmqGA3ux2gr5feNcMyesIVP?= =?us-ascii?Q?jCmJrsVGrsQAMZKvo/Dnxlp1yuqcZ9MbkOw5l0gA2Jz9mhoppW2xVLFg5YxT?= =?us-ascii?Q?iWjzgLebFOdheG7040NYot8Ev3MbLRKcTscp0oToqCVQ7O/gTpKpAUBH20WV?= =?us-ascii?Q?mcAoI7eExWPgGVjAXBdgUUdS?= Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-OriginatorOrg: Nvidia.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: DM6PR12MB3945.namprd12.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 272e52a5-af83-4799-3c43-08d8f8c9f18b X-MS-Exchange-CrossTenant-originalarrivaltime: 06 Apr 2021 07:02:29.6234 (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: 4WaRjasZFSP8UcFGaAtHS7oTAtWaFMSnJiix1WjTbwVNw48uyN1Z3EfXTtQlOpP2eDnJoPgYJmSQEGoA44Po+g== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR12MB4762 Subject: Re: [dpdk-dev] [PATCH] net/mlx5/windows: fix link speed calculation 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 Sender: "dev" > Subject: Re: [PATCH] net/mlx5/windows: fix link speed calculation >=20 > External email: Use caution opening links or attachments >=20 >=20 > There is no previous commit with net/mlx5/windows as title prefix. > In general, there is "on Windows" at the end of the title. > This way we can have the same formatting when fixing the common code for = a > Windows case. >=20 > 05/04/2021 18:18, Tal Shnaiderman: > > In Windows DevX returns the value of the current link speed in bps > > rate, the conversion to the Mbps rate expected by DPDK is incorrect > > and fixed. >=20 > "was" incorrect. >=20 > I think you should explain the confusion between Mbit/s and Mibit/s. >=20 > > - dev_link.link_speed =3D context_obj->mlx5_dev.link_speed / (1024 = * 1024); > > + dev_link.link_speed =3D context_obj->mlx5_dev.link_speed / (1000 = * > > + 1000); >=20 >=20 Thanks Thomas for the review, will address the comments in v2.