From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by dpdk.org (Postfix) with ESMTP id B895B5A6A for ; Sat, 1 Jul 2017 03:09:45 +0200 (CEST) Received: from fmsmga005.fm.intel.com ([10.253.24.32]) by fmsmga103.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 30 Jun 2017 18:09:45 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.40,289,1496127600"; d="scan'208";a="121072660" Received: from silpixa00399464.ir.intel.com (HELO silpixa00399464.ger.corp.intel.com) ([10.237.222.157]) by fmsmga005.fm.intel.com with ESMTP; 30 Jun 2017 18:09:43 -0700 From: Pablo de Lara To: declan.doherty@intel.com, zbigniew.bodek@caviumnetworks.com, jerin.jacob@caviumnetworks.com, akhil.goyal@nxp.com, hemant.agrawal@nxp.com, fiona.trahe@intel.com, john.griffin@intel.com, deepak.k.jain@intel.com Cc: dev@dpdk.org, Slawomir Mrozowicz , Pablo de Lara Date: Fri, 30 Jun 2017 18:09:30 +0100 Message-Id: <20170630170933.56631-9-pablo.de.lara.guarch@intel.com> X-Mailer: git-send-email 2.9.4 In-Reply-To: <20170630170933.56631-1-pablo.de.lara.guarch@intel.com> References: <20170526184538.27292-1-michalx.k.jastrzebski@intel.com> <20170630170933.56631-1-pablo.de.lara.guarch@intel.com> Subject: [dpdk-dev] [PATCH v2 08/11] cryptodev: remove mempool from session X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 01 Jul 2017 01:09:46 -0000 From: Slawomir Mrozowicz Mempool pointer can be obtained from the object itself, which means that it is not required to actually store the pointer in the session. Signed-off-by: Slawomir Mrozowicz Signed-off-by: Pablo de Lara --- doc/guides/rel_notes/release_17_08.rst | 1 + lib/librte_cryptodev/rte_cryptodev.c | 7 +++---- lib/librte_cryptodev/rte_cryptodev.h | 6 ------ 3 files changed, 4 insertions(+), 10 deletions(-) diff --git a/doc/guides/rel_notes/release_17_08.rst b/doc/guides/rel_notes/release_17_08.rst index 04bd3d5..6215584 100644 --- a/doc/guides/rel_notes/release_17_08.rst +++ b/doc/guides/rel_notes/release_17_08.rst @@ -162,6 +162,7 @@ API Changes the new parameter ``device id``. * ``dev_id`` field has been removed from ``rte_cryptodev_sym_session`` structure. * ``driver_id`` field has been removed from ``rte_cryptodev_sym_session`` structure. + * Mempool pointer ``mp`` has been removed from ``rte_cryptodev_sym_session`` structure. ABI Changes diff --git a/lib/librte_cryptodev/rte_cryptodev.c b/lib/librte_cryptodev/rte_cryptodev.c index d2772fd..cc42c25 100644 --- a/lib/librte_cryptodev/rte_cryptodev.c +++ b/lib/librte_cryptodev/rte_cryptodev.c @@ -1026,8 +1026,6 @@ rte_cryptodev_sym_session_init(struct rte_mempool *mp, { memset(sess, 0, mp->elt_size); - sess->mp = mp; - if (dev->dev_ops->session_initialize) (*dev->dev_ops->session_initialize)(mp, sess); } @@ -1065,7 +1063,7 @@ rte_cryptodev_sym_session_create(uint8_t dev_id, dev_id); /* Return session to mempool */ - rte_mempool_put(sess->mp, _sess); + rte_mempool_put(dev->data->session_pool, _sess); return NULL; } @@ -1137,7 +1135,8 @@ rte_cryptodev_sym_session_free(uint8_t dev_id, dev->dev_ops->session_clear(dev, (void *)sess->_private); /* Return session to mempool */ - rte_mempool_put(sess->mp, (void *)sess); + struct rte_mempool *mp = rte_mempool_from_obj(sess); + rte_mempool_put(mp, (void *)sess); return NULL; } diff --git a/lib/librte_cryptodev/rte_cryptodev.h b/lib/librte_cryptodev/rte_cryptodev.h index 7d574f1..044a4aa 100644 --- a/lib/librte_cryptodev/rte_cryptodev.h +++ b/lib/librte_cryptodev/rte_cryptodev.h @@ -797,12 +797,6 @@ rte_cryptodev_enqueue_burst(uint8_t dev_id, uint16_t qp_id, /** Cryptodev symmetric crypto session */ struct rte_cryptodev_sym_session { RTE_STD_C11 - struct { - struct rte_mempool *mp; - /**< Mempool session allocated from */ - } __rte_aligned(8); - /**< Public symmetric session details */ - __extension__ char _private[0]; /**< Private session material */ }; -- 2.9.4