From: Thomas Monjalon <thomas.monjalon@6wind.com>
To: Fiona Trahe <fiona.trahe@intel.com>
Cc: dev@dpdk.org
Subject: Re: [dpdk-dev] [PATCH v9 4/5] cryptodev: change burst API to be crypto op oriented
Date: Thu, 10 Mar 2016 17:03:04 +0100 [thread overview]
Message-ID: <1546962.OW49KDJ4kE@xps13> (raw)
In-Reply-To: <1457624473-11138-5-git-send-email-fiona.trahe@intel.com>
2016-03-10 15:41, Fiona Trahe:
> lib/librte_cryptodev/rte_crypto_sym.h | 374 ++++++------
Sorry there is another issue:
rte_crypto_sym.h:622: warning: argument 'op' of command @param is not found in the argument list of __rte_crypto_sym_op_attach_sym_session(struct rte_crypto_sym_op *sym_op, struct rte_cryptodev_sym_session *sess)
rte_crypto_sym.h:628: warning: The following parameters of __rte_crypto_sym_op_attach_sym_session(struct rte_crypto_sym_op *sym_op, struct rte_cryptodev_sym_session *sess) are not documented:
parameter 'sym_op'
rte_crypto_sym.h:592: warning: unable to resolve reference to `rte_cryptodev_session_create' for \ref command
I've fixed it with the following changes:
* The length of the data pointed to by this field is
* set up for the session in the @ref
* rte_crypto_auth_xform structure as part of the @ref
- * rte_cryptodev_session_create function call. This
- * length must not exceed 240 bytes.
+ * rte_cryptodev_sym_session_create function call.
+ * This length must not exceed 240 bytes.
/**
* Attach a session to a symmetric crypto operation
*
- * @param op crypto operation
+ * @param sym_op crypto operation
* @param sess cryptodev session
*/
next prev parent reply other threads:[~2016-03-10 16:04 UTC|newest]
Thread overview: 62+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-01-30 13:07 [dpdk-dev] [PATCH] cryptodev: API change to rte_crypto_op bursts Declan Doherty
2016-02-08 17:50 ` Trahe, Fiona
2016-02-19 11:01 ` [dpdk-dev] [PATCH v2 0/2] cryptodev API changes Declan Doherty
2016-02-19 11:01 ` [dpdk-dev] [PATCH v2 1/2] cryptodev: API tidy and changes to support future extensions Declan Doherty
2016-02-19 11:01 ` [dpdk-dev] [PATCH v2 2/2] cryptodev: change burst API to be crypto op oriented Declan Doherty
2016-02-22 11:17 ` Trahe, Fiona
2016-02-22 18:23 ` Trahe, Fiona
2016-02-22 18:56 ` Trahe, Fiona
2016-02-26 17:30 ` [dpdk-dev] [PATCH v3 0/2] cryptodev API changes Declan Doherty
2016-02-26 17:30 ` [dpdk-dev] [PATCH v3 1/2] cryptodev: API tidy and changes to support future extensions Declan Doherty
2016-02-26 17:30 ` [dpdk-dev] [PATCH v3 2/2] cryptodev: change burst API to be crypto op oriented Declan Doherty
2016-02-29 16:00 ` [dpdk-dev] [PATCH v3 0/2] cryptodev API changes Declan Doherty
2016-02-29 16:52 ` [dpdk-dev] [PATCH v4 " Declan Doherty
2016-02-29 16:52 ` [dpdk-dev] [PATCH v4 1/2] cryptodev: API tidy and changes to support future extensions Declan Doherty
2016-03-04 14:43 ` Thomas Monjalon
2016-02-29 16:52 ` [dpdk-dev] [PATCH v4 2/2] cryptodev: change burst API to be crypto op oriented Declan Doherty
2016-02-29 17:47 ` [dpdk-dev] [PATCH v4 0/2] cryptodev API changes Trahe, Fiona
2016-03-04 17:17 ` [dpdk-dev] [PATCH v5 " Fiona Trahe
2016-03-04 17:38 ` Thomas Monjalon
2016-03-04 17:43 ` Trahe, Fiona
2016-03-04 17:45 ` Thomas Monjalon
2016-03-04 18:01 ` Trahe, Fiona
2016-03-04 17:39 ` Trahe, Fiona
2016-03-15 6:48 ` Cao, Min
2016-03-04 17:17 ` [dpdk-dev] [PATCH v5 1/2] This patch splits symmetric specific definitions and functions away from the common crypto APIs to facilitate the future extension and expansion of the cryptodev framework, in order to allow asymmetric crypto operations to be introduced at a later date, as well as to clean the logical structure of the public includes. The patch also introduces the _sym prefix to symmetric specific structure and functions to improve clarity in the API Fiona Trahe
2016-03-04 17:17 ` [dpdk-dev] [PATCH v5 2/2] This patch modifies the crypto burst enqueue/dequeue APIs to operate on bursts rte_crypto_op's rather than the current implementation which operates on rte_mbuf bursts, this simplifies the burst processing in the crypto PMDs and the use of crypto operations in general Fiona Trahe
2016-03-04 18:29 ` [dpdk-dev] [PATCH v6 0/2] cryptodev API changes Fiona Trahe
2016-03-07 11:50 ` [dpdk-dev] [PATCH v7 " Fiona Trahe
2016-03-07 13:23 ` De Lara Guarch, Pablo
2016-03-07 13:53 ` Jain, Deepak K
2016-03-10 13:42 ` [dpdk-dev] [PATCH v8 0/5] " Fiona Trahe
2016-03-10 14:05 ` De Lara Guarch, Pablo
2016-03-10 15:41 ` [dpdk-dev] [PATCH v9 " Fiona Trahe
2016-03-10 16:14 ` Thomas Monjalon
2016-03-10 15:41 ` [dpdk-dev] [PATCH v9 1/5] cryptodev: code cleanup Fiona Trahe
2016-03-10 15:41 ` [dpdk-dev] [PATCH v9 2/5] cryptodev: refactor to partition common from symmetric-specific code Fiona Trahe
2016-03-10 15:41 ` [dpdk-dev] [PATCH v9 3/5] cryptodev: remove unused phys_addr field from key Fiona Trahe
2016-03-10 15:41 ` [dpdk-dev] [PATCH v9 4/5] cryptodev: change burst API to be crypto op oriented Fiona Trahe
2016-03-10 16:03 ` Thomas Monjalon [this message]
2016-03-10 16:13 ` Trahe, Fiona
2016-03-10 15:41 ` [dpdk-dev] [PATCH v9 5/5] mbuf_offload: remove library Fiona Trahe
2016-03-14 8:59 ` [dpdk-dev] [PATCH v8 0/5] cryptodev API changes Cao, Min
2016-03-10 13:42 ` [dpdk-dev] [PATCH v8 1/5] cryptodev: code cleanup Fiona Trahe
2016-03-10 13:42 ` [dpdk-dev] [PATCH v8 2/5] cryptodev: refactor to partition common from symmetric-specific code Fiona Trahe
2016-03-10 13:42 ` [dpdk-dev] [PATCH v8 3/5] cryptodev: remove unused phys_addr field from key Fiona Trahe
2016-03-10 13:43 ` [dpdk-dev] [PATCH v8 4/5] cryptodev: change burst API to be crypto op oriented Fiona Trahe
2016-03-10 14:03 ` Thomas Monjalon
2016-03-10 13:43 ` [dpdk-dev] [PATCH v8 5/5] mbuf_offload: remove library Fiona Trahe
2016-03-15 5:21 ` [dpdk-dev] [PATCH v7 0/2] cryptodev API changes Cao, Min
2016-03-07 11:50 ` [dpdk-dev] [PATCH v7 1/2] cryptodev: API tidy and changes to support future extensions Fiona Trahe
2016-03-08 14:10 ` Thomas Monjalon
2016-03-10 10:30 ` Trahe, Fiona
2016-03-07 11:50 ` [dpdk-dev] [PATCH v7 2/2] cryptodev: change burst API to be crypto op oriented Fiona Trahe
2016-03-08 14:32 ` Thomas Monjalon
2016-03-09 12:55 ` Trahe, Fiona
2016-03-10 10:28 ` Trahe, Fiona
2016-03-15 6:46 ` [dpdk-dev] [PATCH v6 0/2] cryptodev API changes Cao, Min
2016-03-04 18:29 ` [dpdk-dev] [PATCH v6 1/2] cryptodev: API tidy and changes to support future extensions Fiona Trahe
2016-03-04 18:29 ` [dpdk-dev] [PATCH v6 2/2] cryptodev: change burst API to be crypto op oriented Fiona Trahe
2016-03-15 6:57 ` [dpdk-dev] [PATCH v4 0/2] cryptodev API changes Cao, Min
2016-03-15 7:07 ` [dpdk-dev] [PATCH v3 " Cao, Min
2016-03-15 7:48 ` [dpdk-dev] [PATCH v2 " Cao, Min
Reply instructions:
You may reply publicly 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=1546962.OW49KDJ4kE@xps13 \
--to=thomas.monjalon@6wind.com \
--cc=dev@dpdk.org \
--cc=fiona.trahe@intel.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
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).