DPDK patches and discussions
 help / color / mirror / Atom feed
From: Pablo de Lara <pablo.de.lara.guarch@intel.com>
To: declan.doherty@intel.com
Cc: dev@dpdk.org, Pablo de Lara <pablo.de.lara.guarch@intel.com>,
	stable@dpdk.org
Subject: [dpdk-dev] [PATCH v3 1/4] crypto/aesni_mb: fix incorrect crypto session
Date: Mon, 19 Dec 2016 17:29:00 +0000	[thread overview]
Message-ID: <1482168543-40289-2-git-send-email-pablo.de.lara.guarch@intel.com> (raw)
In-Reply-To: <1482168543-40289-1-git-send-email-pablo.de.lara.guarch@intel.com>

When using sessionless crypto operations, crypto session
is obtained from a pool of sessions, when processing the
operation. Once the operation is processed, the session
is put back in the pool, but for the AESNI MB PMD, this
session was not being saved in the operation and therefore,
it did not return to the session pool.

Fixes: 924e84f87306 ("aesni_mb: add driver for multi buffer based crypto")

CC: stable@dpdk.org
Signed-off-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
---
 drivers/crypto/aesni_mb/rte_aesni_mb_pmd.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/crypto/aesni_mb/rte_aesni_mb_pmd.c b/drivers/crypto/aesni_mb/rte_aesni_mb_pmd.c
index f07cd07..7443b47 100644
--- a/drivers/crypto/aesni_mb/rte_aesni_mb_pmd.c
+++ b/drivers/crypto/aesni_mb/rte_aesni_mb_pmd.c
@@ -322,6 +322,7 @@ get_session(struct aesni_mb_qp *qp, struct rte_crypto_op *op)
 			rte_mempool_put(qp->sess_mp, _sess);
 			sess = NULL;
 		}
+		op->sym->session = (struct rte_cryptodev_sym_session *)_sess;
 	}
 
 	return sess;
-- 
2.7.4

  reply	other threads:[~2016-12-19 17:27 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-12-15 16:00 [dpdk-dev] [PATCH v2 0/3] AESNI MB PMD updates Pablo de Lara
2016-12-15 16:00 ` [dpdk-dev] [PATCH v2 1/3] doc: update AESNI MB PMD guide Pablo de Lara
2016-12-15 16:00 ` [dpdk-dev] [PATCH v2 2/3] crypto/aesni_mb: add single operation functionality Pablo de Lara
2016-12-15 16:00 ` [dpdk-dev] [PATCH v2 3/3] doc: add missing supported algos for AESNI MB PMD Pablo de Lara
2016-12-19 17:28 ` [dpdk-dev] [PATCH v3 0/4] AESNI MB PMD updates Pablo de Lara
2016-12-19 17:29   ` Pablo de Lara [this message]
2016-12-20 21:18     ` [dpdk-dev] [PATCH v3 1/4] crypto/aesni_mb: fix incorrect crypto session Declan Doherty
2016-12-19 17:29   ` [dpdk-dev] [PATCH v3 2/4] doc: update AESNI MB PMD guide Pablo de Lara
2016-12-19 17:29   ` [dpdk-dev] [PATCH v3 3/4] crypto/aesni_mb: add single operation functionality Pablo de Lara
2016-12-20 21:16     ` Declan Doherty
2016-12-19 17:29   ` [dpdk-dev] [PATCH v3 4/4] doc: add missing supported algos for AESNI MB PMD Pablo de Lara
2016-12-26 16:29   ` [dpdk-dev] [PATCH v3 0/4] AESNI MB PMD updates De Lara Guarch, Pablo

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=1482168543-40289-2-git-send-email-pablo.de.lara.guarch@intel.com \
    --to=pablo.de.lara.guarch@intel.com \
    --cc=declan.doherty@intel.com \
    --cc=dev@dpdk.org \
    --cc=stable@dpdk.org \
    /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).