From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by dpdk.org (Postfix) with ESMTP id 1B189B33E; Tue, 20 Dec 2016 22:26:15 +0100 (CET) Received: from fmsmga004.fm.intel.com ([10.253.24.48]) by orsmga103.jf.intel.com with ESMTP; 20 Dec 2016 13:26:14 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.33,380,1477983600"; d="scan'208";a="205095961" Received: from dwdohert-dpdk.ir.intel.com ([163.33.210.152]) by fmsmga004.fm.intel.com with ESMTP; 20 Dec 2016 13:26:13 -0800 To: Pablo de Lara References: <1481817632-183082-1-git-send-email-pablo.de.lara.guarch@intel.com> <1482168543-40289-1-git-send-email-pablo.de.lara.guarch@intel.com> <1482168543-40289-2-git-send-email-pablo.de.lara.guarch@intel.com> Cc: dev@dpdk.org, stable@dpdk.org From: Declan Doherty Message-ID: Date: Tue, 20 Dec 2016 21:18:19 +0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.1.1 MIME-Version: 1.0 In-Reply-To: <1482168543-40289-2-git-send-email-pablo.de.lara.guarch@intel.com> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [dpdk-stable] [PATCH v3 1/4] crypto/aesni_mb: fix incorrect crypto session X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 20 Dec 2016 21:26:16 -0000 On 19/12/16 17:29, Pablo de Lara wrote: > 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 > --- > 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; > Acked-by: Declan Doherty