From mboxrd@z Thu Jan 1 00:00:00 1970 From: Declan Doherty Subject: [PATCH v2] aesni_mb: strict-aliasing rule compilation fix Date: Mon, 15 Feb 2016 17:06:07 +0000 Message-ID: <1455555967-31084-1-git-send-email-declan.doherty@intel.com> References: <1454159311-5924-1-git-send-email-declan.doherty@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable To: dev@dpdk.org Return-path: Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by dpdk.org (Postfix) with ESMTP id EC8CB37A6 for ; Mon, 15 Feb 2016 18:08:39 +0100 (CET) In-Reply-To: <1454159311-5924-1-git-send-email-declan.doherty@intel.com> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Fixes: 924e84f87306 ("aesni_mb: add driver for multi buffer based crypto"= ) When compiling the AESNI_MB PMD with GCC 4.4.7 on Centos 6.7 a "dereferen= cing pointer =E2=80=98obj_p=E2=80=99 does break strict-aliasing rules" warning= occurs in the get_session() function. Signed-off-by: Declan Doherty --- Adding "Fixes" details to patch drivers/crypto/aesni_mb/rte_aesni_mb_pmd.c | 15 +++++++++------ 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/drivers/crypto/aesni_mb/rte_aesni_mb_pmd.c b/drivers/crypto/= aesni_mb/rte_aesni_mb_pmd.c index d8ccf05..18ce176 100644 --- a/drivers/crypto/aesni_mb/rte_aesni_mb_pmd.c +++ b/drivers/crypto/aesni_mb/rte_aesni_mb_pmd.c @@ -299,7 +299,7 @@ aesni_mb_set_session_parameters(const struct aesni_mb= _ops *mb_ops, static struct aesni_mb_session * get_session(struct aesni_mb_qp *qp, struct rte_crypto_op *crypto_op) { - struct aesni_mb_session *sess; + struct aesni_mb_session *sess =3D NULL; =20 if (crypto_op->type =3D=3D RTE_CRYPTO_OP_WITH_SESSION) { if (unlikely(crypto_op->session->type !=3D @@ -308,16 +308,19 @@ get_session(struct aesni_mb_qp *qp, struct rte_cryp= to_op *crypto_op) =20 sess =3D (struct aesni_mb_session *)crypto_op->session->_private; } else { - struct rte_cryptodev_session *c_sess =3D NULL; + void *_sess =3D NULL; =20 - if (rte_mempool_get(qp->sess_mp, (void **)&c_sess)) + if (rte_mempool_get(qp->sess_mp, (void **)&_sess)) return NULL; =20 - sess =3D (struct aesni_mb_session *)c_sess->_private; + sess =3D (struct aesni_mb_session *) + ((struct rte_cryptodev_session *)_sess)->_private; =20 if (unlikely(aesni_mb_set_session_parameters(qp->ops, - sess, crypto_op->xform) !=3D 0)) - return NULL; + sess, crypto_op->xform) !=3D 0)) { + rte_mempool_put(qp->sess_mp, _sess); + sess =3D NULL; + } } =20 return sess; --=20 2.5.0