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 F2D5FA0C3F; Wed, 12 May 2021 07:51:08 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 7B36C4003F; Wed, 12 May 2021 07:51:08 +0200 (CEST) Received: from NAM10-DM6-obe.outbound.protection.outlook.com (mail-dm6nam10on2045.outbound.protection.outlook.com [40.107.93.45]) by mails.dpdk.org (Postfix) with ESMTP id 0663E4003E for ; Wed, 12 May 2021 07:51:06 +0200 (CEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=dM22tfLWF+eq/6zdlCt75XLJ7+lj15J/Q93cl2tZhdGmLr6bJh1wTvYCxcNLOGemLTKZAeaBmgRIiFoY9Sgg6AGL4GCI/cV5p3V0ebOqU3SuqYdOJSrHjhQJ4DVMsNWZOqYCmBd0eT9WyAe9MgKwvDKO0ty7vmD8+0bot60BOX0/UOahFOYMDMBGoIxKvNghrTfb8dGLb9i0TL563g/3rNxDR/rEJo/6LpsYXuYOvvskg2oPI6Al2DB+okNxrcoPiLYJe6ASnnyJlP3uzIBofBxS3VHjdJ5Di54h60kICQfSn7BDgSVlKLTrYQVTm+38JLGPOnGgkfkJ4sojag/Mnw== 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=J13KsiV4bPF61LGskprMjyXSRUUw3ThJ2GXUgI+v3uQ=; b=SclddFNRvUIEFD/gb7kqwwgh9mF0h2eMfdoySgtdgdOYpTJIsBhflirfkJ3+g7bcdTkXjEIyebva0pKThr+T+pfOp0sNPDhvnfyiNLF2qICY7tMfpLDMQANgOQmZN4+f07/NcNrQBXMJrfArxoSSPntdZoi7iguqU6n+QC94ekD1/d7+Ygz2FGiWh51Zmy8jw1z8FHrrYxHxx9NuahqLm3yJ6On36OX21PpNXWcbdFKbsj31wz2LJKU0uxFSzJRk9XVlofb3h5Jart+tXbQEpHhluG4yt80CUp3TELNrW9hnXCCAEfWQxJuWedi6U/xxixfU89k/fiTvP6wRXzeFtw== 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=J13KsiV4bPF61LGskprMjyXSRUUw3ThJ2GXUgI+v3uQ=; b=mgMae2pB0PPFmkDwejVIoHzleyCT2Rh6OdyZUquXAUc5vEV9u4VV/0FK4olnWl8tLU08O3vVAi3sge9ysfyGah1Ao+bhulgnmlV3qpFWRSuzDAGMWt70Q0JI+tZRpYytaFcT8Iq6wMIZReZ6R4QAtZ8wNFbFIaJePKZerWiLbagjBKVMn6MMwTPmlUA28MX8gHXOePRKyz2OzaRWtoPKhPYVsupbpMnZ495KfiFHCHwdrjlV7I6kaJREHtOKDN8EymxI6aeOEIlrwq1OsML+RPCE6ye2GMmss4qH3IAn81GAjWxbb9W33OUzuLJFDRXiKxuvGIb7eJWUi4Y/yeIOdQ== Received: from DM4PR12MB5389.namprd12.prod.outlook.com (2603:10b6:5:39a::7) by DM4PR12MB5040.namprd12.prod.outlook.com (2603:10b6:5:38b::19) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4108.25; Wed, 12 May 2021 05:51:05 +0000 Received: from DM4PR12MB5389.namprd12.prod.outlook.com ([fe80::70df:ef1e:d98c:ce4]) by DM4PR12MB5389.namprd12.prod.outlook.com ([fe80::70df:ef1e:d98c:ce4%6]) with mapi id 15.20.4108.031; Wed, 12 May 2021 05:51:05 +0000 From: Matan Azrad To: Akhil Goyal , "dev@dpdk.org" , NBU-Contact-Thomas Monjalon CC: Suanming Mou , Shiri Kuzin Thread-Topic: [EXT] [PATCH v3 03/15] crypto/mlx5: support session operations Thread-Index: AQHXQSnFZsqHi0ROBkyLX1cinh9IRKrZjXoAgAFCeWCAABcSgIAAVLhQgANYqACAAMmmIA== Date: Wed, 12 May 2021 05:51:05 +0000 Message-ID: References: <20210429154712.2820159-1-matan@nvidia.com> <20210504210857.3398397-1-matan@nvidia.com> <20210504210857.3398397-4-matan@nvidia.com> In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: marvell.com; dkim=none (message not signed) header.d=none;marvell.com; dmarc=none action=none header.from=nvidia.com; x-originating-ip: [79.179.12.22] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: a86bf9a4-91ce-453a-173a-08d91509ef0d x-ms-traffictypediagnostic: DM4PR12MB5040: 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:8273; x-ms-exchange-senderadcheck: 1 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: TP6qT3ohGxJwzQECbNHqMt/oQH5UcGWYspzqtALziFUiQhIDHw3iTLC2MOvljge79HgLWvR3lDi59V4pqEhFiofy8Ych3mknjKQras+xPh2Kg9gTnCbCkA8FpiD4HsctGQLoUwtWbZTlaE6BaCyZj8XTYsCy/n1Zpa518Oa+/3drwB1cSUI+WD/A0MA8FE6UKAaUd8/8H1Ok7lqJ1tOzfydXcIU3TtCOYsU3BQ5pQcsvnK6LKxe6zjI1udV0/cQn2tr8Ir5XqSAbbaVaurktIkTRbKXW/ax15jTpwjHQyPVmUUdd15ntMvJWUB2pZgGXnr3kigeoIb4bUUKbEiAWOpTcmpTPaeSZH1DXYFaZHhqbtf4AJFzbfOWtDZnmauMw4inpUplPZqe2he5Z1ehZ/qQJDCLoGdhNDfS5q6AFA+WR/skf+Gw9+Ht+CvMM7uXYATEOHA5Yaqdr4mX1bTSGi/Hx1/M9RBELjPco336ZU4ephWH4vhZnfYnIv4ROL3/hbnN/O6/swcmcxcqkjWjpodJODsO6XBporCMpdNMv7u209p0xslpFuX3oz3Hz0CUUbTx7/aCZDHlMGdT0S4UKIOgfg+UVEcGoQo8bSiM90zY= x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:DM4PR12MB5389.namprd12.prod.outlook.com; PTR:; CAT:NONE; SFS:(4636009)(366004)(346002)(396003)(136003)(376002)(39860400002)(478600001)(122000001)(38100700002)(71200400001)(110136005)(9686003)(55016002)(8936002)(26005)(54906003)(66476007)(4326008)(66946007)(86362001)(107886003)(5660300002)(66446008)(7696005)(6506007)(64756008)(2906002)(52536014)(8676002)(33656002)(66556008)(76116006)(186003)(316002); DIR:OUT; SFP:1101; x-ms-exchange-antispam-messagedata: =?us-ascii?Q?A2IPCvBpCyGSJ/FQwpGZKRxa5SLu9mQ4LVB6CyPcb1BnUvOy8eDPhe6uZqrC?= =?us-ascii?Q?4iTka3gYJN5dNXm7RsR3T9Lh9yQD0rwKaTjtfmh0t6bLDNrRqxdrtnJgxZ40?= =?us-ascii?Q?wLe5Weg7or2lx34uLraa0dnVYXTjCzu1HWT2eUJidWGnl6eq8+A1cy/QBBcu?= =?us-ascii?Q?M7AGsGlqJFLEte0WM/QMFamXXYOS2JIPx5ZcFRKc9248jRPxbMo+NW5g49XJ?= =?us-ascii?Q?BX5wiGOfSVx6+fvUo0N1Y5lzVN+Ob5FlDsL8ir+SoGT+uWiEODvtE3xRuWIR?= =?us-ascii?Q?3haXZGs6nFQsFx89od0rAQupszB52DZ8uzLp39pAFERbs9Jjl+SHYMPQVlpM?= =?us-ascii?Q?jBvjdDcapqPM0Kvi4bkI8+aFt692xsqhRJ5wh9Nh/r6DAi/0V2IDxtP9AhXg?= =?us-ascii?Q?1Eu3xD0OEwCAvxmgovdS4WoXIjf0iwLhVR1l7nDhJpJDhNXT7eqeAOB78nd9?= =?us-ascii?Q?pJ5aZGxNLShBJDmjkpMPURbwdLId2lNV/YNUWUef5SDsfPCaDaQuB/3eDJ2D?= =?us-ascii?Q?aXFlG8dokAvSudlij2cdiG/l+tnYrbh8N6z3jMC9HyYd7hI3jogKD9prYUf7?= =?us-ascii?Q?QZxKVTLlY9J+5SfEoF9BitYZj5dcIqj10AZ2KEvIsp6hGiA0j0+OqB5UD5I3?= =?us-ascii?Q?OYlyJOAAz0xt6bqLH+Ywro+k/2xnKez1ESKc7M5rzGWjDiYhuwh6HOPUdnQT?= =?us-ascii?Q?ObA4GnOe89e/f+IN1z6fgLL27J3usngr2VHfFF8Hwq2Fx0w24ncbQUdvWnkj?= =?us-ascii?Q?PEIrizPsqngqLAiSvfnOYX6B4A/NqT2NO+m7aNWQ5YKjMjD4Z7cVTKPhZz9o?= =?us-ascii?Q?C5p4lVITfQ5M2+6Jm1YidU6N3dz0Z4YsraossxFE5vFTC4kmxF2OMeCa3K+f?= =?us-ascii?Q?ByNtjtyo/kKk3h5DEe7yVejILxAUxXaNTs3hkK/FWRYI5f41ML/z+aIHxChd?= =?us-ascii?Q?N7N98v7Tk4IAezaEdz2LcdMNjVZYip4GdBlRzEk+s+E3265XAAJVd+nUK7Lh?= =?us-ascii?Q?p5jq7GlWkT3LIgBDZ0Gd4hW6/5yJsjTrsfS+PmctInbjyrBK2fzsB8pw3Vfj?= =?us-ascii?Q?K3t9dodHkwXDM9o/SUL2kN8pc71yG0W951BGlCpITsVAh5nWL+uTvuJMOhvx?= =?us-ascii?Q?6ToCKQ+UcySDrNMZoMpZ2jBfE/ertdcN75QaN6+yiyXSyZLmHQGy4wEYyXa0?= =?us-ascii?Q?Ez+6YvBwzIjlRoFZXI/OZ9t+GRbJTgE81rH1CrRj7TK6vk1AXNHQUSPDztLi?= =?us-ascii?Q?r0J8dcij6WXpaap08Csr00clbC80+dfh9Q/HDFGKa2IZY+0G7SFp4fSkuCr3?= =?us-ascii?Q?rJc=3D?= 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: DM4PR12MB5389.namprd12.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: a86bf9a4-91ce-453a-173a-08d91509ef0d X-MS-Exchange-CrossTenant-originalarrivaltime: 12 May 2021 05:51:05.7892 (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: VSp98g5xL7N3aqS8LPR6KLjvTnrpFXpdk86bdK0Y1GnrCYuhYhsTUK0LDwzYk/tGucUj5tSbofFILTWJD9R6fw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM4PR12MB5040 Subject: Re: [dpdk-dev] [EXT] [PATCH v3 03/15] crypto/mlx5: support session operations 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" From: Akhil Goyal > > > > > > +static void > > > > > > +mlx5_crypto_sym_session_clear(struct rte_cryptodev *dev, > > > > > > + struct rte_cryptodev_sym_session *s= ess) { > > > > > > + struct mlx5_crypto_priv *priv =3D dev->data->dev_private; > > > > > > + struct mlx5_crypto_session *sess_private_data =3D > > > > > > + get_sym_session_private_data(sess, > > > > > > +dev->driver_id); > > > > > > + > > > > > > + if (unlikely(sess_private_data =3D=3D NULL)) { > > > > > > + DRV_LOG(ERR, "Failed to get session %p private da= ta.", > > > > > > + sess_private_data); > > > > > > + return; > > > > > > + } > > > > > > + mlx5_crypto_dek_destroy(priv, sess_private_data->dek); > > > > > > + DRV_LOG(DEBUG, "Session %p was cleared.", > > > > > > + sess_private_data); > > } > > > > > > > > > > Memory leakage, mempool is not freed. > > > > > > > > Yes, good catch, this part was missed. > > > > > > > > > IMO, this driver is not properly tested with the unit test app. > > > > > > > > The only app we tested until now is l2fwd_crypto and it works fine! > > > > We can add it to doc. > > > > > > > > > Please add a note in the documentation that it is tested with aut= otest. > > > > > > > > > > > > The next app we want to test with, is test-crypto-perf. > > > > > > > I would recommend to run the test app first. > > > It will catch most of your basic bugs like the one above. > > > > It is too late for this, will add the test adjustment later. >=20 > Can we postpone the PMD to next release. We can, but this is not our plan. We met all the DPDK rules to push it on time. > I believe test application makes > The PMD look robust as per the DPDK crypto PMD API usage. Every test will add robustness to the PMD. > I haven't seen a PMD getting merged without test app. compress/mlx5, vdpa/mlx5, regex/mlx5, net/mlx5, vdev_netvsc.... > And I apologize I did not mentioned it earlier, but it is kind of obvious= thing to > run test app before sending it to upstream. In fact, no, I added more than one PMD, no one require specific test. > L2fwd-crypto is not doing data validation hence you cannot be sure that i= t is > working fine as per other standard stacks like Linux. It is not do data validation, but we check that the packet payload return b= ack has the expected encrypted\decrypted data using open-ssl. Also for us it was mandatory requirement to check it. For us, the current validation is enough, we don't support a lot of things = in the crypto PMD currently, only one algorithm in the most basic modes. For sure we will continue to add tests and to increase robustness. Also adding more features is in plan for future.=20 If you postpone it, it yours, we don't agree with it. > Regards, > Akhil