From b69f927d527bc7e17e67f2eecdda34f42165375a Mon Sep 17 00:00:00 2001 From: Fiona Trahe Date: Wed, 10 Jun 2020 20:53:08 +0200 Subject: [PATCH] cryptodev: add traces in multi-process path This patch adds traces to some Cryptodev functions that are used in primary/secondary context. Signed-off-by: Fiona Trahe Acked-by: Akhil Goyal --- lib/librte_cryptodev/rte_cryptodev.c | 22 ++++++++++++++++++++-- 1 file changed, 20 insertions(+), 2 deletions(-) diff --git a/lib/librte_cryptodev/rte_cryptodev.c b/lib/librte_cryptodev/rte_cryptodev.c index 33ced00e47..1dd795bcb5 100644 --- a/lib/librte_cryptodev/rte_cryptodev.c +++ b/lib/librte_cryptodev/rte_cryptodev.c @@ -717,8 +717,13 @@ rte_cryptodev_data_alloc(uint8_t dev_id, struct rte_cryptodev_data **data, mz = rte_memzone_reserve(mz_name, sizeof(struct rte_cryptodev_data), socket_id, 0); - } else + CDEV_LOG_DEBUG("PRIMARY:reserved memzone for %s (%p)", + mz_name, mz); + } else { mz = rte_memzone_lookup(mz_name); + CDEV_LOG_DEBUG("SECONDARY:looked up memzone for %s (%p)", + mz_name, mz); + } if (mz == NULL) return -ENOMEM; @@ -749,8 +754,14 @@ rte_cryptodev_data_free(uint8_t dev_id, struct rte_cryptodev_data **data) RTE_ASSERT(*data == mz->addr); *data = NULL; - if (rte_eal_process_type() == RTE_PROC_PRIMARY) + if (rte_eal_process_type() == RTE_PROC_PRIMARY) { + CDEV_LOG_DEBUG("PRIMARY:free memzone of %s (%p)", + mz_name, mz); return rte_memzone_free(mz); + } else { + CDEV_LOG_DEBUG("SECONDARY:don't free memzone of %s (%p)", + mz_name, mz); + } return 0; } @@ -807,8 +818,15 @@ rte_cryptodev_pmd_allocate(const char *name, int socket_id) cryptodev->data->dev_id = dev_id; cryptodev->data->socket_id = socket_id; cryptodev->data->dev_started = 0; + CDEV_LOG_DEBUG("PRIMARY:init data"); } + CDEV_LOG_DEBUG("Data for %s: dev_id %d, socket %d, started %d", + cryptodev->data->name, + cryptodev->data->dev_id, + cryptodev->data->socket_id, + cryptodev->data->dev_started); + /* init user callbacks */ TAILQ_INIT(&(cryptodev->link_intr_cbs)); -- 2.20.1