From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <dev-bounces@dpdk.org>
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 <dev@dpdk.org>; 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 <matan@nvidia.com>
To: Akhil Goyal <gakhil@marvell.com>, "dev@dpdk.org" <dev@dpdk.org>,
 NBU-Contact-Thomas Monjalon <thomas@monjalon.net>
CC: Suanming Mou <suanmingm@nvidia.com>, Shiri Kuzin <shirik@nvidia.com>
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: <DM4PR12MB53891BB77E00C44C2173E4F9DF529@DM4PR12MB5389.namprd12.prod.outlook.com>
References: <20210429154712.2820159-1-matan@nvidia.com>
 <20210504210857.3398397-1-matan@nvidia.com>
 <20210504210857.3398397-4-matan@nvidia.com>
 <MW2PR18MB2284FF951FC954D2F6C6B395D8569@MW2PR18MB2284.namprd18.prod.outlook.com>
 <DM4PR12MB5389E947B65BA5C2682FB9D6DF559@DM4PR12MB5389.namprd12.prod.outlook.com>
 <MW2PR18MB2284B241B6D5464F86FE9CA2D8559@MW2PR18MB2284.namprd18.prod.outlook.com>
 <DM4PR12MB538975B3B1647DF054320D9FDF559@DM4PR12MB5389.namprd12.prod.outlook.com>
 <MW2PR18MB2284E56315847F54CC1AC160D8539@MW2PR18MB2284.namprd18.prod.outlook.com>
In-Reply-To: <MW2PR18MB2284E56315847F54CC1AC160D8539@MW2PR18MB2284.namprd18.prod.outlook.com>
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: <DM4PR12MB50409209826468364C0E8CCFDF529@DM4PR12MB5040.namprd12.prod.outlook.com>
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 <dev.dpdk.org>
List-Unsubscribe: <https://mails.dpdk.org/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://mails.dpdk.org/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <https://mails.dpdk.org/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
Errors-To: dev-bounces@dpdk.org
Sender: "dev" <dev-bounces@dpdk.org>



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