Message ID | 20190718160943.10724-1-arkadiuszx.kusztal@intel.com (mailing list archive) |
---|---|
Headers |
Return-Path: <dev-bounces@dpdk.org> X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 924E62BA8; Thu, 18 Jul 2019 18:10:31 +0200 (CEST) Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by dpdk.org (Postfix) with ESMTP id 1958B152A for <dev@dpdk.org>; Thu, 18 Jul 2019 18:10:29 +0200 (CEST) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by fmsmga103.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 18 Jul 2019 09:10:28 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.64,278,1559545200"; d="scan'208";a="187815270" Received: from akusztax-mobl.ger.corp.intel.com ([10.104.14.174]) by fmsmga001.fm.intel.com with ESMTP; 18 Jul 2019 09:10:27 -0700 From: Arek Kusztal <arkadiuszx.kusztal@intel.com> To: dev@dpdk.org Cc: akhil.goyal@nxp.com, fiona.trahe@intel.com, shallyv@marvell.com, damianx.nowak@intel.com, Arek Kusztal <arkadiuszx.kusztal@intel.com> Date: Thu, 18 Jul 2019 18:09:36 +0200 Message-Id: <20190718160943.10724-1-arkadiuszx.kusztal@intel.com> X-Mailer: git-send-email 2.19.1.windows.1 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [dpdk-dev] [PATCH v5 0/7] Rework API for RSA algorithm in asymmetric crypto X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 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> |
Series |
Rework API for RSA algorithm in asymmetric crypto
|
|
Message
Arkadiusz Kusztal
July 18, 2019, 4:09 p.m. UTC
Ramoved patches that was not acked by Shally. So open things: 1. Creating padding struct. 2. Padding parameters (seedlen, optional label etc). 3. Leading zeroes questions. 4. Random number requirements. 5. Capabilities. 6. Verify signature field when none padding. 7. Padding none. 8. Signature input format. Arek Kusztal (7): cryptodev: change RSA API comments about primes cryptodev: add cipher field to RSA op crypto/openssl: add cipher field to openssl RSA implementation test: add cipher field to RSA test cryptodev: remove RSA PKCS1 BT0 padding openssl: remove RSA PKCS1_5 BT0 padding test: remove RSA PKCS1_5 BT0 padding from test cases app/test/test_cryptodev_asym.c | 13 ++++--- drivers/crypto/openssl/rte_openssl_pmd.c | 12 +++---- lib/librte_cryptodev/rte_crypto_asym.h | 61 +++++++++++++++++++++----------- 3 files changed, 55 insertions(+), 31 deletions(-)
Comments
> -----Original Message----- > From: Arek Kusztal <arkadiuszx.kusztal@intel.com> > Sent: Thursday, July 18, 2019 9:40 PM > To: dev@dpdk.org > Cc: akhil.goyal@nxp.com; fiona.trahe@intel.com; Shally Verma > <shallyv@marvell.com>; damianx.nowak@intel.com; Arek Kusztal > <arkadiuszx.kusztal@intel.com> > Subject: [EXT] [PATCH v5 0/7] Rework API for RSA algorithm in asymmetric > crypto > > External Email > > ---------------------------------------------------------------------- > Ramoved patches that was not acked by Shally. > So open things: > 1. Creating padding struct. > 2. Padding parameters (seedlen, optional label etc). > 3. Leading zeroes questions. > 4. Random number requirements. > 5. Capabilities. > 6. Verify signature field when none padding. > 7. Padding none. > 8. Signature input format. > > Arek Kusztal (7): > cryptodev: change RSA API comments about primes > cryptodev: add cipher field to RSA op > crypto/openssl: add cipher field to openssl RSA implementation > test: add cipher field to RSA test > cryptodev: remove RSA PKCS1 BT0 padding > openssl: remove RSA PKCS1_5 BT0 padding > test: remove RSA PKCS1_5 BT0 padding from test cases > > app/test/test_cryptodev_asym.c | 13 ++++--- > drivers/crypto/openssl/rte_openssl_pmd.c | 12 +++---- > lib/librte_cryptodev/rte_crypto_asym.h | 61 +++++++++++++++++++++--- > -------- > 3 files changed, 55 insertions(+), 31 deletions(-) > > -- > 2.1.0 With minor feedback on description on Patchv5 2/7, Series-acked-by: Shally Verma <shallyv@marvell.com>
> > ---------------------------------------------------------------------- > > Ramoved patches that was not acked by Shally. > > So open things: > > 1. Creating padding struct. > > 2. Padding parameters (seedlen, optional label etc). > > 3. Leading zeroes questions. > > 4. Random number requirements. > > 5. Capabilities. > > 6. Verify signature field when none padding. > > 7. Padding none. > > 8. Signature input format. > > > > Arek Kusztal (7): > > cryptodev: change RSA API comments about primes > > cryptodev: add cipher field to RSA op > > crypto/openssl: add cipher field to openssl RSA implementation > > test: add cipher field to RSA test > > cryptodev: remove RSA PKCS1 BT0 padding > > openssl: remove RSA PKCS1_5 BT0 padding > > test: remove RSA PKCS1_5 BT0 padding from test cases > > > > app/test/test_cryptodev_asym.c | 13 ++++--- > > drivers/crypto/openssl/rte_openssl_pmd.c | 12 +++---- > > lib/librte_cryptodev/rte_crypto_asym.h | 61 +++++++++++++++++++++--- > > -------- > > 3 files changed, 55 insertions(+), 31 deletions(-) > > > > -- > > 2.1.0 > With minor feedback on description on Patchv5 2/7, > Series-acked-by: Shally Verma <shallyv@marvell.com> Comment on 2/7 shall be taken in next series for next release as suggested by Shally. Applied to dpdk-next-crypto Thanks.
19/07/2019 14:51, Akhil Goyal: > > > Arek Kusztal (7): > > > cryptodev: change RSA API comments about primes > > > cryptodev: add cipher field to RSA op > > > crypto/openssl: add cipher field to openssl RSA implementation > > > test: add cipher field to RSA test > > > cryptodev: remove RSA PKCS1 BT0 padding > > > openssl: remove RSA PKCS1_5 BT0 padding > > > test: remove RSA PKCS1_5 BT0 padding from test cases > > > > > With minor feedback on description on Patchv5 2/7, > > Series-acked-by: Shally Verma <shallyv@marvell.com> > > Comment on 2/7 shall be taken in next series for next release as suggested by Shally. > > Applied to dpdk-next-crypto A commit in cryptodev is removing a field which is removed in PMDs in next patches. Guess what? It breaks the compilation in the middle. Anyway, there is no reason for this split, so I squash the patches.
Hi Thomas, > -----Original Message----- > From: Thomas Monjalon [mailto:thomas@monjalon.net] > Sent: Friday, July 19, 2019 6:23 PM > To: Kusztal, ArkadiuszX <arkadiuszx.kusztal@intel.com> > Cc: dev@dpdk.org; Akhil Goyal <akhil.goyal@nxp.com>; Shally Verma > <shallyv@marvell.com>; Trahe, Fiona <fiona.trahe@intel.com>; Nowak, > DamianX <damianx.nowak@intel.com>; Ayuj Verma > <ayverma@marvell.com>; Kanaka Durga Kotamarthy > <kkotamarthy@marvell.com>; Sunila Sahu <ssahu@marvell.com> > Subject: Re: [dpdk-dev] [EXT] [PATCH v5 0/7] Rework API for RSA algorithm in > asymmetric crypto > > 19/07/2019 14:51, Akhil Goyal: > > > > Arek Kusztal (7): > > > > cryptodev: change RSA API comments about primes > > > > cryptodev: add cipher field to RSA op > > > > crypto/openssl: add cipher field to openssl RSA implementation > > > > test: add cipher field to RSA test > > > > cryptodev: remove RSA PKCS1 BT0 padding > > > > openssl: remove RSA PKCS1_5 BT0 padding > > > > test: remove RSA PKCS1_5 BT0 padding from test cases > > > > > > > With minor feedback on description on Patchv5 2/7, > > > Series-acked-by: Shally Verma <shallyv@marvell.com> > > > > Comment on 2/7 shall be taken in next series for next release as suggested > by Shally. > > > > Applied to dpdk-next-crypto > > A commit in cryptodev is removing a field which is removed in PMDs in next > patches. Guess what? It breaks the compilation in the middle. > Anyway, there is no reason for this split, so I squash the patches. [AK] Sorry for that. In future I will squash such commits into one then. > > > >