[dpdk-dev] [PATCH] examples/vhost_crypto: fix session private mempool

[dpdk-dev] [PATCH] examples/vhost_crypto: fix session private mempool

[Fan Zhang]

This patch fixes the incorrect session private mempool passing
to cryptodev.

Fixes: ac5e42daca19 ("vhost/crypto: use separate session mempools")

Signed-off-by: Fan Zhang <roy.fan.zhang@intel.com>
---
 examples/vhost_crypto/main.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/examples/vhost_crypto/main.c b/examples/vhost_crypto/main.c
index 8bdcb3a9c..ac7118e7e 100644
--- a/examples/vhost_crypto/main.c
+++ b/examples/vhost_crypto/main.c
@@ -541,7 +541,7 @@ main(int argc, char *argv[])
 				rte_cryptodev_sym_get_private_session_size(
 				info->cid), 64, 0, NULL, NULL, NULL, NULL,
 				rte_lcore_to_socket_id(lo->lcore_id), 0);
-		if (!info->sess_priv_pool || info->sess_pool) {
+		if (!info->sess_priv_pool || !info->sess_pool) {
 			RTE_LOG(ERR, USER1, "Failed to create mempool");
 			goto error_exit;
 		}
@@ -562,7 +562,7 @@ main(int argc, char *argv[])
 
 		qp_conf.nb_descriptors = NB_CRYPTO_DESCRIPTORS;
 		qp_conf.mp_session = info->sess_pool;
-		qp_conf.mp_session_private = info->sess_pool;
+		qp_conf.mp_session_private = info->sess_priv_pool;
 
 		for (j = 0; j < dev_info.max_nb_queue_pairs; j++) {
 			ret = rte_cryptodev_queue_pair_setup(info->cid, j,
-- 
2.13.6

Re: [dpdk-dev] [PATCH] examples/vhost_crypto: fix session private mempool

[De Lara Guarch, Pablo]

> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Fan Zhang
> Sent: Tuesday, January 15, 2019 10:38 AM
> To: dev@dpdk.org
> Cc: maxime.coquelin@redhat.com
> Subject: [dpdk-dev] [PATCH] examples/vhost_crypto: fix session private
> mempool
> 
> This patch fixes the incorrect session private mempool passing to cryptodev.
> 
> Fixes: ac5e42daca19 ("vhost/crypto: use separate session mempools")
> 
> Signed-off-by: Fan Zhang <roy.fan.zhang@intel.com>

Acked-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>

Re: [dpdk-dev] [PATCH] examples/vhost_crypto: fix session private mempool

[Maxime Coquelin]

On 1/15/19 11:38 AM, Fan Zhang wrote:
> This patch fixes the incorrect session private mempool passing
> to cryptodev.
> 
> Fixes: ac5e42daca19 ("vhost/crypto: use separate session mempools")
> 
> Signed-off-by: Fan Zhang <roy.fan.zhang@intel.com>
> ---
>   examples/vhost_crypto/main.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)



Reviewed-by: Maxime Coquelin <maxime.coquelin@redhat.com>

Thanks,
Maxime

Re: [dpdk-dev] [PATCH] examples/vhost_crypto: fix session private mempool

[Maxime Coquelin]

On 1/15/19 11:38 AM, Fan Zhang wrote:
> This patch fixes the incorrect session private mempool passing
> to cryptodev.
> 
> Fixes: ac5e42daca19 ("vhost/crypto: use separate session mempools")
> 
> Signed-off-by: Fan Zhang <roy.fan.zhang@intel.com>
> ---
>   examples/vhost_crypto/main.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 

Applied to dpdk-next-virtio/master.

Thanks,
Maxime