From mboxrd@z Thu Jan 1 00:00:00 1970 From: Declan Doherty Subject: [PATCH] aesni_mb: strict-aliasing rule compilation fix Date: Sat, 30 Jan 2016 13:08:31 +0000 Message-ID: <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 mga14.intel.com (mga14.intel.com [192.55.52.115]) by dpdk.org (Postfix) with ESMTP id 494888EA1 for ; Sat, 30 Jan 2016 14:10:06 +0100 (CET) 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" When compiling the AESNI_MB PMD with GCC on Centos 6 a "dereferencing poi= nter =E2=80=98obj_p=E2=80=99 does break strict-aliasing rules" warning occurs = in the get_session() function. This patch fixes this build warning. Signed-off-by: Declan Doherty --- 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