DPDK patches and discussions
 help / color / Atom feed
From: "Trahe, Fiona" <fiona.trahe@intel.com>
To: "Kusztal, ArkadiuszX" <arkadiuszx.kusztal@intel.com>,
	Anoob Joseph <anoobj@marvell.com>, "dev@dpdk.org" <dev@dpdk.org>
Cc: "akhil.goyal@nxp.com" <akhil.goyal@nxp.com>,
	Shally Verma <shallyv@marvell.com>,
	"Trahe, Fiona" <fiona.trahe@intel.com>
Subject: Re: [dpdk-dev] [PATCH v4] cryptodev: extend api of asymmetric crypto by	sessionless
Date: Mon, 21 Oct 2019 10:44:22 +0000
Message-ID: <348A99DA5F5B7549AA880327E580B435A2179ECD@IRSMSX101.ger.corp.intel.com> (raw)
In-Reply-To: <06EE24DD0B19E248B53F6DC8657831551B2A0503@hasmsx109.ger.corp.intel.com>

Hi Anoob, Akhil,
I know this patch is already applied, and that's ok.
Just want to update with my thoughts on terminology of sessionless vs session-less.
Arek suggested below using session-less for all except C code, for consistency with Symmetric crypto.
And has done in a few places in API and documentation. 
However, Symmetric crypto has a mix of sessionless/Sessionless/session-less/session_less.
And sessionless is used for not just code but also feature flag matrix and
performance tool command line options.
So I propose we use sessionless in all places except the few that already have session-less.
I don't think it worth another asymmetric crypto API patch to change those.
But we plan to use sessionless throughout the QAT asymmetric crypto PMD and asymm test code.
Fiona
 


> -----Original Message-----
> From: Kusztal, ArkadiuszX
> Sent: Thursday, October 10, 2019 9:28 AM
> To: Anoob Joseph <anoobj@marvell.com>; dev@dpdk.org
> Cc: akhil.goyal@nxp.com; Trahe, Fiona <fiona.trahe@intel.com>; Shally Verma
> <shallyv@marvell.com>
> Subject: RE: [dpdk-dev] [PATCH v4] cryptodev: extend api of asymmetric crypto by sessionless
> 
> Hi Anoob,
> 
> > -----Original Message-----
> > From: Anoob Joseph [mailto:anoobj@marvell.com]
> > Sent: Thursday, October 10, 2019 5:43 AM
> > To: Kusztal, ArkadiuszX <arkadiuszx.kusztal@intel.com>; dev@dpdk.org
> > Cc: akhil.goyal@nxp.com; Trahe, Fiona <fiona.trahe@intel.com>; Shally
> > Verma <shallyv@marvell.com>
> > Subject: RE: [dpdk-dev] [PATCH v4] cryptodev: extend api of asymmetric
> > crypto by sessionless
> >
> > Hi Arek,
> >
> > Shouldn't we update release notes also?
> Yes, good point, I will update.
> >
> > Also a minor observation, what is the usage that we should stick to?
> > "session-less" or "sessionless". I see that we use it interchangeably.
> Yes, I think it would be good if could unify this.
> Symmetric API states:
> "Session-less API crypto operation parameters"
> So we can stick to that in everything that is not C identifier I suppose.
> I will send v5 then.
> >
> > Thanks,
> > Anoob
> >
> > > -----Original Message-----
> > > From: dev <dev-bounces@dpdk.org> On Behalf Of Arek Kusztal
> > > Sent: Wednesday, October 9, 2019 9:39 PM
> > > To: dev@dpdk.org
> > > Cc: akhil.goyal@nxp.com; fiona.trahe@intel.com; Shally Verma
> > > <shallyv@marvell.com>; Anoob Joseph <anoobj@marvell.com>; Arek
> > Kusztal
> > > <arkadiuszx.kusztal@intel.com>
> > > Subject: [dpdk-dev] [PATCH v4] cryptodev: extend api of asymmetric
> > > crypto by sessionless
> > >
> > > This commit adds asymmetric session-less option to rte_crypto_asym_op.
> > > Feature flag for sessionless is added to rte_cryptodev.
> > >
> > > Signed-off-by: Arek Kusztal <arkadiuszx.kusztal@intel.com>
> > > ---
> > > v2:
> > > - added union
> > > v3:
> > > - added entry into feature matrix
> > > - added information to prog_guide
> > > v4:
> > > - fixed formatting
> > > - fixed grammar
> > >
> > >  doc/guides/cryptodevs/features/default.ini | 1 +
> > >  doc/guides/prog_guide/cryptodev_lib.rst    | 9 ++++++++-
> > >  lib/librte_cryptodev/rte_crypto_asym.h     | 9 +++++++--
> > >  lib/librte_cryptodev/rte_cryptodev.h       | 2 ++
> > >  4 files changed, 18 insertions(+), 3 deletions(-)
> > >
> > > diff --git a/doc/guides/cryptodevs/features/default.ini
> > > b/doc/guides/cryptodevs/features/default.ini
> > > index d3ee1af..b7f9a0a 100644
> > > --- a/doc/guides/cryptodevs/features/default.ini
> > > +++ b/doc/guides/cryptodevs/features/default.ini
> > > @@ -26,6 +26,7 @@ OOP LB  In LB  Out     =
> > >  RSA PRIV OP KEY EXP    =
> > >  RSA PRIV OP KEY QT     =
> > >  Digest encrypted       =
> > > +Asymmetric sessionless =
> > >
> > >  ;
> > >  ; Supported crypto algorithms of a default crypto driver.
> > > diff --git a/doc/guides/prog_guide/cryptodev_lib.rst
> > > b/doc/guides/prog_guide/cryptodev_lib.rst
> > > index 9719944..4584834 100644
> > > --- a/doc/guides/prog_guide/cryptodev_lib.rst
> > > +++ b/doc/guides/prog_guide/cryptodev_lib.rst
> > > @@ -876,7 +876,14 @@ private asymmetric session data. Once this is
> > > done, session should be freed usin
> > >
> > >  Asymmetric Sessionless Support
> > >  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> > > -Currently asymmetric crypto framework does not support sessionless.
> > > +Asymmetric crypto framework supports sessionless operations.
> > > +
> > > +Fields that should be set by user are:
> > > +
> > > +Member xform of struct rte_crypto_asym_op should point to the user
> > > created rte_crypto_asym_xform.
> > > +Note that this rte_crypto_asym_xform should be immutable for the
> > > lifetime of associated crypto_op.
> > > +
> > > +Member sess_type of rte_crypto_op should be set to
> > > RTE_CRYPTO_OP_SESSIONLESS.
> > >
> > >  Transforms and Transform Chaining
> > >  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> > > diff --git a/lib/librte_cryptodev/rte_crypto_asym.h
> > > b/lib/librte_cryptodev/rte_crypto_asym.h
> > > index 4fbef2f..0d34ce8 100644
> > > --- a/lib/librte_cryptodev/rte_crypto_asym.h
> > > +++ b/lib/librte_cryptodev/rte_crypto_asym.h
> > > @@ -522,8 +522,13 @@ struct rte_crypto_dsa_op_param {
> > >   *
> > >   */
> > >  struct rte_crypto_asym_op {
> > > -	struct rte_cryptodev_asym_session *session;
> > > -	/**< Handle for the initialised session context */
> > > +	RTE_STD_C11
> > > +	union {
> > > +		struct rte_cryptodev_asym_session *session;
> > > +		/**< Handle for the initialised session context */
> > > +		struct rte_crypto_asym_xform *xform;
> > > +		/**< Session-less API crypto operation parameters */
> > > +	};
> > >
> > >  	__extension__
> > >  	union {
> > > diff --git a/lib/librte_cryptodev/rte_cryptodev.h
> > > b/lib/librte_cryptodev/rte_cryptodev.h
> > > index e175b83..c6ffa3b 100644
> > > --- a/lib/librte_cryptodev/rte_cryptodev.h
> > > +++ b/lib/librte_cryptodev/rte_cryptodev.h
> > > @@ -448,6 +448,8 @@ rte_cryptodev_asym_get_xform_enum(enum
> > > rte_crypto_asym_xform_type *xform_enum,  /**< Support RSA Private Key
> > > OP with CRT (quintuple) Keys */
> > >  #define RTE_CRYPTODEV_FF_DIGEST_ENCRYPTED		(1ULL << 19)
> > >  /**< Support encrypted-digest operations where digest is appended to
> > > data */
> > > +#define RTE_CRYPTODEV_FF_ASYM_SESSIONLESS		(1ULL << 20)
> > > +/**< Support asymmetric session-less operations */
> > >
> > >
> > >  /**
> > > --
> > > 2.1.0


      reply index

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-10-09 16:08 Arek Kusztal
2019-10-10  3:42 ` Anoob Joseph
2019-10-10  8:27   ` Kusztal, ArkadiuszX
2019-10-21 10:44     ` Trahe, Fiona [this message]

Reply instructions:

You may reply publically to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=348A99DA5F5B7549AA880327E580B435A2179ECD@IRSMSX101.ger.corp.intel.com \
    --to=fiona.trahe@intel.com \
    --cc=akhil.goyal@nxp.com \
    --cc=anoobj@marvell.com \
    --cc=arkadiuszx.kusztal@intel.com \
    --cc=dev@dpdk.org \
    --cc=shallyv@marvell.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

DPDK patches and discussions

Archives are clonable:
	git clone --mirror http://inbox.dpdk.org/dev/0 dev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 dev dev/ http://inbox.dpdk.org/dev \
		dev@dpdk.org
	public-inbox-index dev


Newsgroup available over NNTP:
	nntp://inbox.dpdk.org/inbox.dpdk.dev


AGPL code for this site: git clone https://public-inbox.org/ public-inbox