From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from out4-smtp.messagingengine.com (out4-smtp.messagingengine.com [66.111.4.28]) by dpdk.org (Postfix) with ESMTP id 25C57235 for ; Tue, 21 Nov 2017 14:26:29 +0100 (CET) Received: from compute1.internal (compute1.nyi.internal [10.202.2.41]) by mailout.nyi.internal (Postfix) with ESMTP id CF4BE207FE; Tue, 21 Nov 2017 08:26:28 -0500 (EST) Received: from frontend2 ([10.202.2.161]) by compute1.internal (MEProxy); Tue, 21 Nov 2017 08:26:28 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=fridaylinux.org; h=cc:date:from:in-reply-to:message-id:references:subject:to :x-me-sender:x-me-sender:x-sasl-enc; s=fm1; bh=OK3DueqT1RP8G+nmd NryL0TmM9yWqrTcPS1g0wuBVZo=; b=kS57PzwHK1T+n/Wgsju7WZd9haXyN4ibO vs5+87k475Oq77fEr3PYcojGYTJYt6j1fsijnzSHcPdbrIf+xhPH9NGS5pztFCUy wxNsB5OSQCW5KQ4vnVsBMbxv8VJlIbPb0tQ9oEsF+LVZfYnviGgND6G08sDPqwAl ZrKhE5vF5KtO1wCtwnmYBWJfOdAs90CWkiS8JvMoYQ/qv32fNe1BSda1bmh3dRJM Z8vutdwU2LkI0hVE6AUUfEPE54EBsnoHdizQE/CiqjgyjI/+sWFIflKGuluB0o4l H03eR4VneYQ4Xhbb0BHh4iSb27WLIJlk3DaNoqJzvUtPFcaOD1O5A== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:date:from:in-reply-to:message-id :references:subject:to:x-me-sender:x-me-sender:x-sasl-enc; s= fm1; bh=OK3DueqT1RP8G+nmdNryL0TmM9yWqrTcPS1g0wuBVZo=; b=FvYgI8Rn KJ+xLAdj1aipHalPBFHOlpoNDUzNsz4mA88E9oL1B415murqv9Icve1AVtFzSTrv RxGUk7XMklzrGhlPW6AUV/cBeOkoWn1LudsB9sb5D2ZxJQEmlJg3Hohkc3gxzJXs OrfqVDu1J7B8Ac/4I7IQH4RtMh99M3rogrp2rssao+F0aTuECXVOldd70v/2G8eG nYIbw8CevlxJcy2zOrPP37vuYe6NiH7Me76u+ot2a32MbUzA9Hs+LWdRVNPysbMb weO2G/fgdNG8c4kMuLRYCcc316VRUINnbdUPM0iyzKWBL6lxNty+xpMfpQ/VxpgU eZV3qxZq8kq2yw== X-ME-Sender: Received: from localhost.localdomain (unknown [180.158.62.0]) by mail.messagingengine.com (Postfix) with ESMTPA id DB64C24631; Tue, 21 Nov 2017 08:26:26 -0500 (EST) From: Yuanhan Liu To: Sergio Gonzalez Monroy Cc: dpdk stable Date: Tue, 21 Nov 2017 21:17:34 +0800 Message-Id: <1511270333-31002-112-git-send-email-yliu@fridaylinux.org> X-Mailer: git-send-email 2.7.4 In-Reply-To: <1511270333-31002-1-git-send-email-yliu@fridaylinux.org> References: <1511270333-31002-1-git-send-email-yliu@fridaylinux.org> Subject: [dpdk-stable] patch 'crypto/aesni_mb: fix invalid session error' has been queued to stable release 17.08.1 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, 21 Nov 2017 13:26:29 -0000 Hi, FYI, your patch has been queued to stable release 17.08.1 Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet. It will be pushed if I get no objections before 11/24/17. So please shout if anyone has objections. Thanks. --yliu --- >>From b98e5dee6d6cb8b05e12d89a6150d305825e3083 Mon Sep 17 00:00:00 2001 From: Sergio Gonzalez Monroy Date: Thu, 7 Sep 2017 12:50:36 +0100 Subject: [PATCH] crypto/aesni_mb: fix invalid session error [ upstream commit 6fcd9fdf379b8218202e7213623da9b7359bb518 ] Setting an invalid session in the crypto op results in SEGFAULT because the JOB user_data was never set to the crypto op. Fixes: 0f548b50a160 ("crypto/aesni_mb: process crypto op on dequeue") Signed-off-by: Sergio Gonzalez Monroy --- drivers/crypto/aesni_mb/rte_aesni_mb_pmd.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/drivers/crypto/aesni_mb/rte_aesni_mb_pmd.c b/drivers/crypto/aesni_mb/rte_aesni_mb_pmd.c index 16e1451..f64ff61 100644 --- a/drivers/crypto/aesni_mb/rte_aesni_mb_pmd.c +++ b/drivers/crypto/aesni_mb/rte_aesni_mb_pmd.c @@ -626,13 +626,16 @@ flush_mb_mgr(struct aesni_mb_qp *qp, struct rte_crypto_op **ops, } static inline JOB_AES_HMAC * -set_job_null_op(JOB_AES_HMAC *job) +set_job_null_op(JOB_AES_HMAC *job, struct rte_crypto_op *op) { job->chain_order = HASH_CIPHER; job->cipher_mode = NULL_CIPHER; job->hash_alg = NULL_HASH; job->cipher_direction = DECRYPT; + /* Set user data to be crypto operation data struct */ + job->user_data = op; + return job; } @@ -670,7 +673,7 @@ aesni_mb_pmd_dequeue_burst(void *queue_pair, struct rte_crypto_op **ops, retval = set_mb_job_params(job, qp, op); if (unlikely(retval != 0)) { qp->stats.dequeue_err_count++; - set_job_null_op(job); + set_job_null_op(job, op); } /* Submit job to multi-buffer for processing */ -- 2.7.4