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 581CCA0C47; Sun, 31 Oct 2021 20:08:17 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id B76204068E; Sun, 31 Oct 2021 20:08:16 +0100 (CET) Received: from mx0b-0016f401.pphosted.com (mx0a-0016f401.pphosted.com [67.231.148.174]) by mails.dpdk.org (Postfix) with ESMTP id 73C8440689 for ; Sun, 31 Oct 2021 20:08:15 +0100 (CET) Received: from pps.filterd (m0045849.ppops.net [127.0.0.1]) by mx0a-0016f401.pphosted.com (8.16.1.2/8.16.1.2) with ESMTP id 19VE4LhF019441; Sun, 31 Oct 2021 12:08:14 -0700 Received: from nam04-bn8-obe.outbound.protection.outlook.com (mail-bn8nam08lp2040.outbound.protection.outlook.com [104.47.74.40]) by mx0a-0016f401.pphosted.com (PPS) with ESMTPS id 3c1vk38p4s-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Sun, 31 Oct 2021 12:08:14 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=iZaGt0J9zM3NwMOE858F93c/ztgdb0jAXtGQ+BR0N1u+dQxV+zLdQWqpFexofIeFp7sRX9Yd3v2+8SPi34DIuwk87Gche/S2Rb240pu49U8LWcB9SJI1QH0UyhWQo8ZhC6vPmoTDbYeV9bGVmUMh+YypJPgamNOnugLvSQIUWXeDJl4zV530Y2SOpR1pLIyABazGfc0LK7oe7W28xi1rwNaREUcic+yVhz4D4Xp4mF1ZVIbUlagYZ9IFMnOgtUW+41Y6RzVXlk9TnqJ0R4vf+sUxLxlrnG3caJk07sHWINowS542XfW1gzvjTjtW/4H1duHffFzLdo0Z+ARf19MhOw== 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=w/YZXM6tmHbs62L8qwr/uL5jpvDKp2ugsUHY94q9ZuA=; b=RT+ybDvA4euxKSSI8mj1y3yLYFjvfdFd/1YkizCoJl5phlkqVaasxeyshmhvg8Y6SsmkOgR0uFk1t8+mr4xVbrgTx2aZqLm/IG3qxD0PBjEwpirCyD2cMEv80iUz7VKmTqpLbpgPMOs/O0zC7TNgOGtmLzgA7gQ6/WqxUdkfIBLlq24Pc2u+qUUoKtl1Wj2aHf0F6OauTeMQx5hGQuACwzratGxsh3Qc15nTqlOxEDjqhvSF9AgaTX8RthIIWrfwkhVrtgc9rhZUQwpYMb6q2xhx+TCVkSmsiesbtv/veO39MAwc4A4tDSQiQma2yEFc2TOLSeYkfMdsq/fZJ/IsMA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=marvell.com; dmarc=pass action=none header.from=marvell.com; dkim=pass header.d=marvell.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=marvell.onmicrosoft.com; s=selector1-marvell-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=w/YZXM6tmHbs62L8qwr/uL5jpvDKp2ugsUHY94q9ZuA=; b=iLqv3exEspB34nzh5JkKg5nZyxyE/yh3CIcYBHnyQaL5zpiR+9SWtYM/Yau4bSPJ1IfWFrebGRVaHMwUGm68iWylrfgA/SyIJ6bZhFI10tq0d5K7ZZ8lfWXsKv5OL7vAbVXSUxtzor1Pwfp6gaF6knHwgHrHcZfBtOqe0g4pFTU= Received: from CO6PR18MB4484.namprd18.prod.outlook.com (2603:10b6:5:359::9) by CO6PR18MB4403.namprd18.prod.outlook.com (2603:10b6:5:35d::14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4649.15; Sun, 31 Oct 2021 19:08:10 +0000 Received: from CO6PR18MB4484.namprd18.prod.outlook.com ([fe80::c41e:707:3f91:71b8]) by CO6PR18MB4484.namprd18.prod.outlook.com ([fe80::c41e:707:3f91:71b8%8]) with mapi id 15.20.4649.019; Sun, 31 Oct 2021 19:08:09 +0000 From: Akhil Goyal To: Tal Shnaiderman , "dev@dpdk.org" , NBU-Contact-Thomas Monjalon , "bruce.richardson@intel.com" , "Yigit, Ferruh" CC: Matan Azrad , Raslan Darawsheh , Asaf Penso , "declan.doherty@intel.com" , Slava Ovsiienko , Eilon Greenstein Thread-Topic: [EXT] [PATCH v3 5/5] crypto/mlx5: support on Windows Thread-Index: AQHXyXzs4Y/U/fwPkUyXQko6sENJSavmtbOAgAAlvYCABlKBAIAAUn3g Date: Sun, 31 Oct 2021 19:08:09 +0000 Message-ID: References: <20211017094133.18988-2-talshn@nvidia.com> <20211025084617.4952-1-talshn@nvidia.com> <20211025084617.4952-6-talshn@nvidia.com> In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 8266d40b-da18-4790-eb63-08d99ca1c785 x-ms-traffictypediagnostic: CO6PR18MB4403: x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:9508; x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: QLl4gWA88thVunzMhRd/gC0rrTPT6x328TOqMgb5A+weu50dxBfh/NTpGC45THFrfu2jPiBQd76rTTKDmOQ1WFBCP6fbqtVI/9SrlU9GMXEFpXo9H0NYQAkRlkU4k17oOaSKN+KgGZeh9IhRZqfN3PGN2a8kSIZFHdm0hEByd0GtzoSK7j2mGFNjtHO9ncsBX2bgZqJ48g9pxPl77y0K1tHM4EtFGMoeN2STBW/uhOO7Jcd54FXHBvC+FUvgfPpoI6XkI3NzE+odo0IszyRUxzspBMOXlLfzGgKz4nSq3ZMWjYc4/mTF5SQaSvMO1AJmboM2GNEHL3O1YKXFRhmGLS0KdLYpY+R5FbiKw0FXZKFo3XO4M50Q9X8yjVk4FW8CUF7iJiOE1vXVfpaN97+cp3pdvQBMa+36NjlHAijRm8glK0Z0rK1GT55HDy9RFOu8zYpeYBTQeUMos55X/bPZtUicTGtwl6rTeeWV5Ceof5wW8ls7lyhmZ2fzoxavxOy1YbGmDVzWUehgTcNbxklKneCDrzojEd43J7QynxQaU07khlSK/7GHszdHOr8tG2h4n+6kVFc5uyW36faRKWkbvkC9pZDY4u20Bk4idUXCF3boAdOKVRNp6op08MA+XRb6GD4mwS+9u7eKi5Q2FfaIwBA6VKMbvUVJ3vmxFIYEaNIs9IF2td/RgQaMfszr1axQnddajp53Af/Ih31qcAGWzA== x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:CO6PR18MB4484.namprd18.prod.outlook.com; PTR:; CAT:NONE; SFS:(4636009)(366004)(2906002)(5660300002)(55016002)(508600001)(9686003)(8676002)(8936002)(4326008)(86362001)(38070700005)(52536014)(76116006)(7416002)(71200400001)(186003)(38100700002)(66946007)(54906003)(66476007)(66556008)(83380400001)(7696005)(316002)(64756008)(33656002)(66446008)(110136005)(122000001)(6506007); DIR:OUT; SFP:1101; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?sA7ocTGkH+AVOL0Tf1E+Oe63eM0YHfXS8+1Iy69IiwKJaF3GB7a/9KjzVcD1?= =?us-ascii?Q?8rXRzzsU/BoObY36UdRYCl/nNf223Dw79zguKkRKm9xOowgxFi8qMDFju45D?= =?us-ascii?Q?Hi8VoSodPecKnL739A5xaAc+X6BQw3AWoGV4L/+z4DDTODW+Oh6qHhHJYwHk?= =?us-ascii?Q?gvU7WHAiKq6Kki08jqcIJ5UK/VKtU9osjVr57eaAl3BtIHMyWTX9NhWuAsh6?= =?us-ascii?Q?OMDAxufKNmuecSbGHeRgDy+opHC7gAxLitt9F+Htx1GpoFDkz1tLsZ5vFLRG?= =?us-ascii?Q?Zui8wRoNl9eC/3uIST8vabJ1VkrRVuXj+q/2PjC9D3CAxKPeVYkGNo4Qo3sY?= =?us-ascii?Q?MWqRpnzMVmavPIFck0npytK6GtpY8TTtufux7wdS+zq1G+xEpT39yGbKhD6F?= =?us-ascii?Q?VLcszvDPmK3CJggB09GlnoXTAAsTHYCIOcU8SbZvNAD/3cFtFP07PDd2/iJE?= =?us-ascii?Q?LsbRRAEBy1IDN/koGRbwTaNi6HzRFvbJ/Zom2sMtaPChTksV+I5/w2Pzu8T9?= =?us-ascii?Q?SXVLtru/GsGFnfV684/3ri+/Aqomo0NyX6xwGP7jlnICElwJNJNrio/R8wq4?= =?us-ascii?Q?HQ+W8nxnoIh9qwWyeAMmPHbE5KtsDFs3j9Xj63NQ1/l+5XRrDs+U77e9i+6u?= =?us-ascii?Q?gWQBS8hBacb6gFPTEHdEKGxOV83RlQbC7wNHD+kQUo+CyKVLIvTnaGOqiISJ?= =?us-ascii?Q?RUvtWJFXAavKxWOfksk9MJ1E7E6ONuHImhaR/tLeBQLQIpQWlzVoy4xOEk0g?= =?us-ascii?Q?cqIwnex+VhdBgZ9OUTXByKJivG6F4bgQPCDcMmzcbjkokg+XWBTo+kJ6+XbU?= =?us-ascii?Q?Rk5Jm6B5jnG53NbsOjFvKhgNLouX1GMQe/fS1SijpA2I9eU9FsahruQhIS6G?= =?us-ascii?Q?9HezTACwcUr2hwgikhYAxEMPPEDnJNDpoQacXPLMS5RwokYQ2CdgPEMMRBIf?= =?us-ascii?Q?MgtT9StgFZ3gUD/RT9RRmg0Rc8Dzj+AjqJpkZqmY2b0AC0D9vqzbMvuCaWOR?= =?us-ascii?Q?JdGfg9lSRJIGQ4EivzHN3EtdGAa6t9qEfEaUIlCTXbhBhoS8JbAeVC4rNi2g?= =?us-ascii?Q?JP5cn24MFJvfC4PV2NqlqfQAS/wwAyY2jfP8JK5CeuNYfe+T1wEWVqWUUaL8?= =?us-ascii?Q?38ELsqk+LgvObd5avR4rZKDaGDz++382w5GAD/vY7v75Ns5THwgyzcd+W9It?= =?us-ascii?Q?RZRsNd0Ng01kQjedzKr5KHlYSk6Hyq0rY6maXcC34/xiJ476Jk3vWJrx0S6s?= =?us-ascii?Q?YZ78n09zAWMPYA7S4Ft1zbNl+ovJjzT3S58WzFWGw/6YG6s3Q3X9JTa5QgWq?= =?us-ascii?Q?aeuNaaqDcwHDmvptONHxqBk/uWQaB3pNKpMzVbe9tGNCgkNh3MR+mgy5w+Ex?= =?us-ascii?Q?ZgDNn7Fll0++Op6TS+tqLo7G1mUnDTOaqF4c6IFow2xaNRPrcPhTNLQhRRmw?= =?us-ascii?Q?tbCqaq82yd/y9yteUKHyXNAJpGmy+f3W32pv6/ewD/ZhrwSZO2zvje5/MQ51?= =?us-ascii?Q?HhLZIgF8MN4liwrBz4Qp/G5GGJw1t5GG88w9rFjKGXnMXoX9KnbUpIvd6BTa?= =?us-ascii?Q?9XFoNJMlLVNjd117zyJt2w3dDco8o/wAdtCPVBQZZaqwqXKC9a8mxXsRU3qh?= =?us-ascii?Q?rQtBlJACSR5747dgXg+n1tubNJuypOYgpboqmyQ9DDtwz1AADd5a0884IBk8?= =?us-ascii?Q?z+MY45iQYRqU1+zrqIV5QMl4E58=3D?= Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-OriginatorOrg: marvell.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: CO6PR18MB4484.namprd18.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 8266d40b-da18-4790-eb63-08d99ca1c785 X-MS-Exchange-CrossTenant-originalarrivaltime: 31 Oct 2021 19:08:09.5887 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 70e1fb47-1155-421d-87fc-2e58f638b6e0 X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: eDpokZQyllsZgjCTEICX1BxeK5016eeOFK+q+WzAyzXm9kOCCIW/Qcfjx5ZUL6aGyR+3n5Hur64fRSiQMVTTxg== X-MS-Exchange-Transport-CrossTenantHeadersStamped: CO6PR18MB4403 X-Proofpoint-ORIG-GUID: 7ySXZi9HebZfwzEVnh99w8Z155Ugq7P1 X-Proofpoint-GUID: 7ySXZi9HebZfwzEVnh99w8Z155Ugq7P1 X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.182.1,Aquarius:18.0.790,Hydra:6.0.425,FMLib:17.0.607.475 definitions=2021-10-31_05,2021-10-29_03,2020-04-07_01 Subject: Re: [dpdk-dev] [EXT] [PATCH v3 5/5] crypto/mlx5: support on Windows 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" > > > > > > > Add support for mlx5 crypto pmd on Windows OS. > > > > Add changes to release note and pmd guide. > > > > > > > > Signed-off-by: Tal Shnaiderman > > > > Acked-by: Matan Azrad > > > > --- > > > > doc/guides/cryptodevs/mlx5.rst | 15 ++++++++++++--- > > > > doc/guides/rel_notes/release_21_11.rst | 1 + > > > > drivers/common/mlx5/version.map | 2 +- > > > > drivers/common/mlx5/windows/mlx5_common_os.c | 2 +- > > > > drivers/crypto/mlx5/meson.build | 4 ++-- > > > > 5 files changed, 17 insertions(+), 7 deletions(-) > > > > > > > > diff --git a/doc/guides/cryptodevs/mlx5.rst > > > > b/doc/guides/cryptodevs/mlx5.rst index 68bfdf3a83..7338c0c493 > 100644 > > > > --- a/doc/guides/cryptodevs/mlx5.rst > > > > +++ b/doc/guides/cryptodevs/mlx5.rst > > > > @@ -39,12 +39,12 @@ or to access the hardware components directly. > > > > There are different levels of objects and bypassing abilities. > > > > To get the best performances: > > > > > > > > -- Verbs is a complete high-level generic API. > > > > -- Direct Verbs is a device-specific API. > > > > +- Verbs is a complete high-level generic API (Linux only). > > > > +- Direct Verbs is a device-specific API (Linux only). > > > > - DevX allows to access firmware objects. > > > > > > > > Enabling ``librte_crypto_mlx5`` causes DPDK applications -to be > > > > linked against libibverbs. > > > > +to be linked against libibverbs on Linux OS. > > > > > > > > In order to move the device to crypto operational mode, credential > > > > and KEK (Key Encrypting Key) should be set as the first step. > > > > @@ -155,8 +155,17 @@ Limitations > > > > Prerequisites > > > > ------------- > > > > > > > > +Linux Prerequisites > > > > +~~~~~~~~~~~~~~~~~~~ > > > > + > > > > - Mellanox OFED version: **5.3** > > > > see :doc:`../../nics/mlx5` guide for more Mellanox OFED details. > > > > > > > > - Compilation can be done also with rdma-core v15+. > > > > see :doc:`../../nics/mlx5` guide for more rdma-core details. > > > > + > > > > +Windows Prerequisites > > > > +~~~~~~~~~~~~~~~~~~~~~ > > > > + > > > > +- Mellanox WINOF-2 version: **2.60** or higher. > > > > + see :doc:`../../nics/mlx5` guide for more Mellanox WINOF-2 detai= ls. > > > > > > I do not see a way to cross compile this driver with MinGW. How are w= e > > > expected to Test compilation for such cases? Do we need to build it > > > over Windows? I personally don't Think it is a good idea. > > > > > > Others please suggest. > > > > Yes, this driver can be built only using clang on Windows, net/mlx5 beh= aves > > the same way. > > > > DPDK CI has a compilation test for Windows [1], it will fail if a patch= is > > introduced that caused compilation to fail on Windows. > > This dependency is not good. At times we have dependent series and=20 we cannot just rely on CI to do compilation. Please remove this dependency as soon as possible. This series is applied on dpdk-next-crypto for now. But would need to take this patch out in future if there are frequent=20 Compilation breaks. We cannot ask developers working on common Code base to check compilation of the patches explicitly on Windows Machine. There may be many people who do not work on Windows Machine. >=20 > Any other comments? If not, can this series be merged?