From: Amr Mokhtar <amr.mokhtar@intel.com>
To: dev@dpdk.org
Cc: thomas@monjalon.net, ferruh.yigit@intel.com,
Amr Mokhtar <amr.mokhtar@intel.com>
Subject: [dpdk-dev] [PATCH] bb/turbo_sw: fix exported dynamic log type
Date: Mon, 5 Feb 2018 23:33:49 +0000 [thread overview]
Message-ID: <1517873629-83062-1-git-send-email-amr.mokhtar@intel.com> (raw)
This patch fixes shared library compilation due to undefined
reference to an exported variable 'bbdev_logtype'.
In this fix, the logtype is converted to static in the bbdev lib,
in bbdev null pmd and turbo sw pmd.
Fixes: b8cfe2c9aed2 ("bb/turbo_sw: add software turbo driver")
Cc: thomas@monjalon.net
Signed-off-by: Amr Mokhtar <amr.mokhtar@intel.com>
---
drivers/bbdev/null/bbdev_null.c | 30 +++--
drivers/bbdev/turbo_sw/bbdev_turbo_software.c | 95 +++++++++-------
lib/librte_bbdev/rte_bbdev.c | 154 +++++++++++++-------------
lib/librte_bbdev/rte_bbdev.h | 28 +----
lib/librte_bbdev/rte_bbdev_op.h | 68 +-----------
lib/librte_bbdev/rte_bbdev_version.map | 1 -
6 files changed, 156 insertions(+), 220 deletions(-)
diff --git a/drivers/bbdev/null/bbdev_null.c b/drivers/bbdev/null/bbdev_null.c
index b23d766..3b5482a 100644
--- a/drivers/bbdev/null/bbdev_null.c
+++ b/drivers/bbdev/null/bbdev_null.c
@@ -15,6 +15,13 @@
#define DRIVER_NAME bbdev_null
+/* NULL BBDev logging ID */
+static int bbdev_null_logtype;
+
+/* Helper macro for logging */
+#define BBDEV_NULL_LOG(level, fmt, ...) \
+ rte_log(RTE_LOG_ ## level, bbdev_null_logtype, fmt "\n", ##__VA_ARGS__)
+
/* Initialisation params structure that can be used by null BBDEV driver */
struct bbdev_null_params {
int socket_id; /*< Null BBDEV socket */
@@ -66,7 +73,7 @@ info_get(struct rte_bbdev *dev, struct rte_bbdev_driver_info *dev_info)
dev_info->cpu_flag_reqs = NULL;
dev_info->min_alignment = 0;
- rte_bbdev_log_debug("got device info from %u", dev->data->dev_id);
+ BBDEV_NULL_LOG(DEBUG, "got device info from %u", dev->data->dev_id);
}
/* Release queue */
@@ -81,7 +88,7 @@ q_release(struct rte_bbdev *dev, uint16_t q_id)
dev->data->queues[q_id].queue_private = NULL;
}
- rte_bbdev_log_debug("released device queue %u:%u",
+ BBDEV_NULL_LOG(DEBUG, "released device queue %u:%u",
dev->data->dev_id, q_id);
return 0;
}
@@ -100,19 +107,19 @@ q_setup(struct rte_bbdev *dev, uint16_t q_id,
q = rte_zmalloc_socket(RTE_STR(DRIVER_NAME), sizeof(*q),
RTE_CACHE_LINE_SIZE, queue_conf->socket);
if (q == NULL) {
- rte_bbdev_log(ERR, "Failed to allocate queue memory");
+ BBDEV_NULL_LOG(ERR, "Failed to allocate queue memory");
return -ENOMEM;
}
q->processed_pkts = rte_ring_create(ring_name, queue_conf->queue_size,
queue_conf->socket, RING_F_SP_ENQ | RING_F_SC_DEQ);
if (q->processed_pkts == NULL) {
- rte_bbdev_log(ERR, "Failed to create ring");
+ BBDEV_NULL_LOG(ERR, "Failed to create ring");
goto free_q;
}
dev->data->queues[q_id].queue_private = q;
- rte_bbdev_log_debug("setup device queue %s", ring_name);
+ BBDEV_NULL_LOG(DEBUG, "setup device queue %s", ring_name);
return 0;
free_q:
@@ -194,7 +201,7 @@ parse_u16_arg(const char *key, const char *value, void *extra_args)
errno = 0;
result = strtoul(value, NULL, 0);
if ((result >= (1 << 16)) || (errno != 0)) {
- rte_bbdev_log(ERR, "Invalid value %lu for %s", result, key);
+ BBDEV_NULL_LOG(ERR, "Invalid value %lu for %s", result, key);
return -ERANGE;
}
*u16 = (uint16_t)result;
@@ -227,7 +234,7 @@ parse_bbdev_null_params(struct bbdev_null_params *params,
goto exit;
if (params->socket_id >= RTE_MAX_NUMA_NODES) {
- rte_bbdev_log(ERR, "Invalid socket, must be < %u",
+ BBDEV_NULL_LOG(ERR, "Invalid socket, must be < %u",
RTE_MAX_NUMA_NODES);
goto exit;
}
@@ -296,7 +303,7 @@ null_bbdev_probe(struct rte_vdev_device *vdev)
input_args = rte_vdev_device_args(vdev);
parse_bbdev_null_params(&init_params, input_args);
- rte_bbdev_log_debug("Init %s on NUMA node %d with max queues: %d",
+ BBDEV_NULL_LOG(DEBUG, "Init %s on NUMA node %d with max queues: %d",
name, init_params.socket_id, init_params.queues_num);
return null_bbdev_create(vdev, &init_params);
@@ -335,12 +342,11 @@ RTE_PMD_REGISTER_PARAM_STRING(DRIVER_NAME,
BBDEV_NULL_MAX_NB_QUEUES_ARG"=<int> "
BBDEV_NULL_SOCKET_ID_ARG"=<int>");
-int bbdev_logtype;
RTE_INIT(null_bbdev_init_log);
static void
null_bbdev_init_log(void)
{
- bbdev_logtype = rte_log_register("pmd.bbdev.null");
- if (bbdev_logtype >= 0)
- rte_log_set_level(bbdev_logtype, RTE_LOG_NOTICE);
+ bbdev_null_logtype = rte_log_register("pmd.bbdev.null");
+ if (bbdev_null_logtype >= 0)
+ rte_log_set_level(bbdev_null_logtype, RTE_LOG_NOTICE);
}
diff --git a/drivers/bbdev/turbo_sw/bbdev_turbo_software.c b/drivers/bbdev/turbo_sw/bbdev_turbo_software.c
index 981da6e..4b5a611 100644
--- a/drivers/bbdev/turbo_sw/bbdev_turbo_software.c
+++ b/drivers/bbdev/turbo_sw/bbdev_turbo_software.c
@@ -46,6 +46,14 @@ static const char * const turbo_sw_valid_params[] = {
TURBO_SW_SOCKET_ID_ARG
};
+/* Turbo SW PMD logging ID */
+static int bbdev_turbo_sw_logtype;
+
+/* Helper macro for logging */
+#define BBDEV_TURBO_SW_LOG(level, fmt, ...) \
+ rte_log(RTE_LOG_ ## level, bbdev_turbo_sw_logtype, fmt "\n", \
+ ##__VA_ARGS__)
+
/* queue */
struct turbo_sw_queue {
/* Ring for processed (encoded/decoded) operations which are ready to
@@ -168,7 +176,8 @@ info_get(struct rte_bbdev *dev, struct rte_bbdev_driver_info *dev_info)
dev_info->cpu_flag_reqs = &cpu_flag;
dev_info->min_alignment = 64;
- rte_bbdev_log_debug("got device info from %u\n", dev->data->dev_id);
+ BBDEV_TURBO_SW_LOG(DEBUG, "got device info from %u\n",
+ dev->data->dev_id);
}
/* Release queue */
@@ -190,7 +199,7 @@ q_release(struct rte_bbdev *dev, uint16_t q_id)
dev->data->queues[q_id].queue_private = NULL;
}
- rte_bbdev_log_debug("released device queue %u:%u",
+ BBDEV_TURBO_SW_LOG(DEBUG, "released device queue %u:%u",
dev->data->dev_id, q_id);
return 0;
}
@@ -208,7 +217,7 @@ q_setup(struct rte_bbdev *dev, uint16_t q_id,
q = rte_zmalloc_socket(RTE_STR(DRIVER_NAME), sizeof(*q),
RTE_CACHE_LINE_SIZE, queue_conf->socket);
if (q == NULL) {
- rte_bbdev_log(ERR, "Failed to allocate queue memory");
+ BBDEV_TURBO_SW_LOG(ERR, "Failed to allocate queue memory");
return -ENOMEM;
}
@@ -216,7 +225,7 @@ q_setup(struct rte_bbdev *dev, uint16_t q_id,
ret = snprintf(name, RTE_RING_NAMESIZE, RTE_STR(DRIVER_NAME)"_enc_out%u:%u",
dev->data->dev_id, q_id);
if ((ret < 0) || (ret >= (int)RTE_RING_NAMESIZE)) {
- rte_bbdev_log(ERR,
+ BBDEV_TURBO_SW_LOG(ERR,
"Creating queue name for device %u queue %u failed",
dev->data->dev_id, q_id);
return -ENAMETOOLONG;
@@ -225,7 +234,7 @@ q_setup(struct rte_bbdev *dev, uint16_t q_id,
((MAX_TB_SIZE >> 3) + 3) * sizeof(*q->enc_out) * 3,
RTE_CACHE_LINE_SIZE, queue_conf->socket);
if (q->enc_out == NULL) {
- rte_bbdev_log(ERR,
+ BBDEV_TURBO_SW_LOG(ERR,
"Failed to allocate queue memory for %s", name);
goto free_q;
}
@@ -235,7 +244,7 @@ q_setup(struct rte_bbdev *dev, uint16_t q_id,
RTE_STR(DRIVER_NAME)"_enc_in%u:%u", dev->data->dev_id,
q_id);
if ((ret < 0) || (ret >= (int)RTE_RING_NAMESIZE)) {
- rte_bbdev_log(ERR,
+ BBDEV_TURBO_SW_LOG(ERR,
"Creating queue name for device %u queue %u failed",
dev->data->dev_id, q_id);
return -ENAMETOOLONG;
@@ -244,7 +253,7 @@ q_setup(struct rte_bbdev *dev, uint16_t q_id,
(MAX_CB_SIZE >> 3) * sizeof(*q->enc_in),
RTE_CACHE_LINE_SIZE, queue_conf->socket);
if (q->enc_in == NULL) {
- rte_bbdev_log(ERR,
+ BBDEV_TURBO_SW_LOG(ERR,
"Failed to allocate queue memory for %s", name);
goto free_q;
}
@@ -253,7 +262,7 @@ q_setup(struct rte_bbdev *dev, uint16_t q_id,
ret = snprintf(name, RTE_RING_NAMESIZE, RTE_STR(DRIVER_NAME)"_ag%u:%u",
dev->data->dev_id, q_id);
if ((ret < 0) || (ret >= (int)RTE_RING_NAMESIZE)) {
- rte_bbdev_log(ERR,
+ BBDEV_TURBO_SW_LOG(ERR,
"Creating queue name for device %u queue %u failed",
dev->data->dev_id, q_id);
return -ENAMETOOLONG;
@@ -262,7 +271,7 @@ q_setup(struct rte_bbdev *dev, uint16_t q_id,
MAX_CB_SIZE * 10 * sizeof(*q->ag),
RTE_CACHE_LINE_SIZE, queue_conf->socket);
if (q->ag == NULL) {
- rte_bbdev_log(ERR,
+ BBDEV_TURBO_SW_LOG(ERR,
"Failed to allocate queue memory for %s", name);
goto free_q;
}
@@ -271,7 +280,7 @@ q_setup(struct rte_bbdev *dev, uint16_t q_id,
ret = snprintf(name, RTE_RING_NAMESIZE, RTE_STR(DRIVER_NAME)"_cb%u:%u",
dev->data->dev_id, q_id);
if ((ret < 0) || (ret >= (int)RTE_RING_NAMESIZE)) {
- rte_bbdev_log(ERR,
+ BBDEV_TURBO_SW_LOG(ERR,
"Creating queue name for device %u queue %u failed",
dev->data->dev_id, q_id);
return -ENAMETOOLONG;
@@ -280,7 +289,7 @@ q_setup(struct rte_bbdev *dev, uint16_t q_id,
(6144 >> 3) * sizeof(*q->code_block),
RTE_CACHE_LINE_SIZE, queue_conf->socket);
if (q->code_block == NULL) {
- rte_bbdev_log(ERR,
+ BBDEV_TURBO_SW_LOG(ERR,
"Failed to allocate queue memory for %s", name);
goto free_q;
}
@@ -290,7 +299,7 @@ q_setup(struct rte_bbdev *dev, uint16_t q_id,
RTE_STR(DRIVER_NAME)"_deint_input%u:%u",
dev->data->dev_id, q_id);
if ((ret < 0) || (ret >= (int)RTE_RING_NAMESIZE)) {
- rte_bbdev_log(ERR,
+ BBDEV_TURBO_SW_LOG(ERR,
"Creating queue name for device %u queue %u failed",
dev->data->dev_id, q_id);
return -ENAMETOOLONG;
@@ -299,7 +308,7 @@ q_setup(struct rte_bbdev *dev, uint16_t q_id,
MAX_KW * sizeof(*q->deint_input),
RTE_CACHE_LINE_SIZE, queue_conf->socket);
if (q->deint_input == NULL) {
- rte_bbdev_log(ERR,
+ BBDEV_TURBO_SW_LOG(ERR,
"Failed to allocate queue memory for %s", name);
goto free_q;
}
@@ -309,7 +318,7 @@ q_setup(struct rte_bbdev *dev, uint16_t q_id,
RTE_STR(DRIVER_NAME)"_deint_output%u:%u",
dev->data->dev_id, q_id);
if ((ret < 0) || (ret >= (int)RTE_RING_NAMESIZE)) {
- rte_bbdev_log(ERR,
+ BBDEV_TURBO_SW_LOG(ERR,
"Creating queue name for device %u queue %u failed",
dev->data->dev_id, q_id);
return -ENAMETOOLONG;
@@ -318,7 +327,7 @@ q_setup(struct rte_bbdev *dev, uint16_t q_id,
MAX_KW * sizeof(*q->deint_output),
RTE_CACHE_LINE_SIZE, queue_conf->socket);
if (q->deint_output == NULL) {
- rte_bbdev_log(ERR,
+ BBDEV_TURBO_SW_LOG(ERR,
"Failed to allocate queue memory for %s", name);
goto free_q;
}
@@ -328,7 +337,7 @@ q_setup(struct rte_bbdev *dev, uint16_t q_id,
RTE_STR(DRIVER_NAME)"_adapter_output%u:%u",
dev->data->dev_id, q_id);
if ((ret < 0) || (ret >= (int)RTE_RING_NAMESIZE)) {
- rte_bbdev_log(ERR,
+ BBDEV_TURBO_SW_LOG(ERR,
"Creating queue name for device %u queue %u failed",
dev->data->dev_id, q_id);
return -ENAMETOOLONG;
@@ -337,7 +346,7 @@ q_setup(struct rte_bbdev *dev, uint16_t q_id,
MAX_CB_SIZE * 6 * sizeof(*q->adapter_output),
RTE_CACHE_LINE_SIZE, queue_conf->socket);
if (q->adapter_output == NULL) {
- rte_bbdev_log(ERR,
+ BBDEV_TURBO_SW_LOG(ERR,
"Failed to allocate queue memory for %s", name);
goto free_q;
}
@@ -346,7 +355,7 @@ q_setup(struct rte_bbdev *dev, uint16_t q_id,
ret = snprintf(name, RTE_RING_NAMESIZE, RTE_STR(DRIVER_NAME)"%u:%u",
dev->data->dev_id, q_id);
if ((ret < 0) || (ret >= (int)RTE_RING_NAMESIZE)) {
- rte_bbdev_log(ERR,
+ BBDEV_TURBO_SW_LOG(ERR,
"Creating queue name for device %u queue %u failed",
dev->data->dev_id, q_id);
return -ENAMETOOLONG;
@@ -354,14 +363,14 @@ q_setup(struct rte_bbdev *dev, uint16_t q_id,
q->processed_pkts = rte_ring_create(name, queue_conf->queue_size,
queue_conf->socket, RING_F_SP_ENQ | RING_F_SC_DEQ);
if (q->processed_pkts == NULL) {
- rte_bbdev_log(ERR, "Failed to create ring for %s", name);
+ BBDEV_TURBO_SW_LOG(ERR, "Failed to create ring for %s", name);
goto free_q;
}
q->type = queue_conf->op_type;
dev->data->queues[q_id].queue_private = q;
- rte_bbdev_log_debug("setup device queue %s", name);
+ BBDEV_TURBO_SW_LOG(DEBUG, "setup device queue %s", name);
return 0;
free_q:
@@ -391,19 +400,19 @@ is_enc_input_valid(const uint16_t k, const int32_t k_idx,
const uint16_t in_length)
{
if (k_idx < 0) {
- rte_bbdev_log(ERR, "K Index is invalid");
+ BBDEV_TURBO_SW_LOG(ERR, "K Index is invalid");
return -1;
}
if (in_length - (k >> 3) < 0) {
- rte_bbdev_log(ERR,
+ BBDEV_TURBO_SW_LOG(ERR,
"Mismatch between input length (%u bytes) and K (%u bits)",
in_length, k);
return -1;
}
if (k > MAX_CB_SIZE) {
- rte_bbdev_log(ERR, "CB size (%u) is too big, max: %d",
+ BBDEV_TURBO_SW_LOG(ERR, "CB size (%u) is too big, max: %d",
k, MAX_CB_SIZE);
return -1;
}
@@ -418,19 +427,19 @@ static inline int
is_dec_input_valid(int32_t k_idx, int16_t kw, int16_t in_length)
{
if (k_idx < 0) {
- rte_bbdev_log(ERR, "K index is invalid");
+ BBDEV_TURBO_SW_LOG(ERR, "K index is invalid");
return -1;
}
if (in_length - kw < 0) {
- rte_bbdev_log(ERR,
+ BBDEV_TURBO_SW_LOG(ERR,
"Mismatch between input length (%u) and kw (%u)",
in_length, kw);
return -1;
}
if (kw > MAX_KW) {
- rte_bbdev_log(ERR, "Input length (%u) is too big, max: %d",
+ BBDEV_TURBO_SW_LOG(ERR, "Input length (%u) is too big, max: %d",
kw, MAX_KW);
return -1;
}
@@ -523,7 +532,7 @@ process_enc_cb(struct turbo_sw_queue *q, struct rte_bbdev_enc_op *op,
turbo_resp.output_win_2 = out2;
if (bblib_turbo_encoder(&turbo_req, &turbo_resp) != 0) {
op->status |= 1 << RTE_BBDEV_DRV_ERROR;
- rte_bbdev_log(ERR, "Turbo Encoder failed");
+ BBDEV_TURBO_SW_LOG(ERR, "Turbo Encoder failed");
return;
}
@@ -578,7 +587,7 @@ process_enc_cb(struct turbo_sw_queue *q, struct rte_bbdev_enc_op *op,
if (bblib_rate_match_dl(&rm_req, &rm_resp) != 0) {
op->status |= 1 << RTE_BBDEV_DRV_ERROR;
- rte_bbdev_log(ERR, "Rate matching failed");
+ BBDEV_TURBO_SW_LOG(ERR, "Rate matching failed");
return;
}
enc->output.length += rm_resp.OutputLen;
@@ -641,14 +650,14 @@ enqueue_enc_one_op(struct turbo_sw_queue *q, struct rte_bbdev_enc_op *op)
op->status = 0;
if (total_left > MAX_TB_SIZE >> 3) {
- rte_bbdev_log(ERR, "TB size (%u) is too big, max: %d",
+ BBDEV_TURBO_SW_LOG(ERR, "TB size (%u) is too big, max: %d",
total_left, MAX_TB_SIZE);
op->status = 1 << RTE_BBDEV_DATA_ERROR;
return;
}
if (m_in == NULL || m_out == NULL) {
- rte_bbdev_log(ERR, "Invalid mbuf pointer");
+ BBDEV_TURBO_SW_LOG(ERR, "Invalid mbuf pointer");
op->status = 1 << RTE_BBDEV_DATA_ERROR;
return;
}
@@ -695,7 +704,7 @@ enqueue_enc_one_op(struct turbo_sw_queue *q, struct rte_bbdev_enc_op *op)
/* check if all input data was processed */
if (total_left != 0) {
op->status |= 1 << RTE_BBDEV_DATA_ERROR;
- rte_bbdev_log(ERR,
+ BBDEV_TURBO_SW_LOG(ERR,
"Mismatch between mbuf length and included CBs sizes");
}
}
@@ -859,7 +868,7 @@ process_dec_cb(struct turbo_sw_queue *q, struct rte_bbdev_dec_op *op,
adapter_req.isinverted = 0;
else {
op->status |= 1 << RTE_BBDEV_DRV_ERROR;
- rte_bbdev_log(ERR, "LLR format wasn't specified");
+ BBDEV_TURBO_SW_LOG(ERR, "LLR format wasn't specified");
return;
}
@@ -871,7 +880,7 @@ process_dec_cb(struct turbo_sw_queue *q, struct rte_bbdev_dec_op *op,
out = (uint8_t *)rte_pktmbuf_append(m_out, (k >> 3));
if (out == NULL) {
op->status |= 1 << RTE_BBDEV_DATA_ERROR;
- rte_bbdev_log(ERR, "Too little space in output mbuf");
+ BBDEV_TURBO_SW_LOG(ERR, "Too little space in output mbuf");
return;
}
/* rte_bbdev_op_data.offset can be different than the offset of the
@@ -898,7 +907,7 @@ process_dec_cb(struct turbo_sw_queue *q, struct rte_bbdev_dec_op *op,
dec->iter_count = RTE_MAX(iter_cnt, dec->iter_count);
} else {
op->status |= 1 << RTE_BBDEV_DATA_ERROR;
- rte_bbdev_log(ERR, "Turbo Decoder failed");
+ BBDEV_TURBO_SW_LOG(ERR, "Turbo Decoder failed");
return;
}
}
@@ -919,7 +928,7 @@ enqueue_dec_one_op(struct turbo_sw_queue *q, struct rte_bbdev_dec_op *op)
op->status = 0;
if (m_in == NULL || m_out == NULL) {
- rte_bbdev_log(ERR, "Invalid mbuf pointer");
+ BBDEV_TURBO_SW_LOG(ERR, "Invalid mbuf pointer");
op->status = 1 << RTE_BBDEV_DATA_ERROR;
return;
}
@@ -965,7 +974,7 @@ enqueue_dec_one_op(struct turbo_sw_queue *q, struct rte_bbdev_dec_op *op)
}
if (total_left != 0) {
op->status |= 1 << RTE_BBDEV_DATA_ERROR;
- rte_bbdev_log(ERR,
+ BBDEV_TURBO_SW_LOG(ERR,
"Mismatch between mbuf length and included Circular buffer sizes");
}
}
@@ -1055,7 +1064,8 @@ parse_u16_arg(const char *key, const char *value, void *extra_args)
errno = 0;
result = strtoul(value, NULL, 0);
if ((result >= (1 << 16)) || (errno != 0)) {
- rte_bbdev_log(ERR, "Invalid value %lu for %s", result, key);
+ BBDEV_TURBO_SW_LOG(ERR, "Invalid value %lu for %s",
+ result, key);
return -ERANGE;
}
*u16 = (uint16_t)result;
@@ -1087,7 +1097,7 @@ parse_turbo_sw_params(struct turbo_sw_params *params, const char *input_args)
goto exit;
if (params->socket_id >= RTE_MAX_NUMA_NODES) {
- rte_bbdev_log(ERR, "Invalid socket, must be < %u",
+ BBDEV_TURBO_SW_LOG(ERR, "Invalid socket, must be < %u",
RTE_MAX_NUMA_NODES);
goto exit;
}
@@ -1155,7 +1165,7 @@ turbo_sw_bbdev_probe(struct rte_vdev_device *vdev)
input_args = rte_vdev_device_args(vdev);
parse_turbo_sw_params(&init_params, input_args);
- rte_bbdev_log_debug(
+ BBDEV_TURBO_SW_LOG(DEBUG,
"Initialising %s on NUMA node %d with max queues: %d\n",
name, init_params.socket_id, init_params.queues_num);
@@ -1195,12 +1205,11 @@ RTE_PMD_REGISTER_PARAM_STRING(DRIVER_NAME,
TURBO_SW_MAX_NB_QUEUES_ARG"=<int> "
TURBO_SW_SOCKET_ID_ARG"=<int>");
-int bbdev_logtype;
RTE_INIT(null_bbdev_init_log);
static void
null_bbdev_init_log(void)
{
- bbdev_logtype = rte_log_register("pmd.bbdev.turbo_sw");
- if (bbdev_logtype >= 0)
- rte_log_set_level(bbdev_logtype, RTE_LOG_NOTICE);
+ bbdev_turbo_sw_logtype = rte_log_register("pmd.bbdev.turbo_sw");
+ if (bbdev_turbo_sw_logtype >= 0)
+ rte_log_set_level(bbdev_turbo_sw_logtype, RTE_LOG_NOTICE);
}
diff --git a/lib/librte_bbdev/rte_bbdev.c b/lib/librte_bbdev/rte_bbdev.c
index 8a053e3..17a00d0 100644
--- a/lib/librte_bbdev/rte_bbdev.c
+++ b/lib/librte_bbdev/rte_bbdev.c
@@ -27,10 +27,17 @@
#define DEV_NAME "BBDEV"
+/* BBDev library logging ID */
+static int bbdev_logtype;
+
+/* Helper macro for logging */
+#define BBDEV_LOG(level, fmt, ...) \
+ rte_log(RTE_LOG_ ## level, bbdev_logtype, fmt "\n", ##__VA_ARGS__)
+
/* Helper macro to check dev_id is valid */
#define VALID_DEV_OR_RET_ERR(dev, dev_id) do { \
if (dev == NULL) { \
- rte_bbdev_log(ERR, "device %u is invalid", dev_id); \
+ BBDEV_LOG(ERR, "device %u is invalid", dev_id); \
return -ENODEV; \
} \
} while (0)
@@ -38,7 +45,7 @@
/* Helper macro to check dev_ops is valid */
#define VALID_DEV_OPS_OR_RET_ERR(dev, dev_id) do { \
if (dev->dev_ops == NULL) { \
- rte_bbdev_log(ERR, "NULL dev_ops structure in device %u", \
+ BBDEV_LOG(ERR, "NULL dev_ops structure in device %u", \
dev_id); \
return -ENODEV; \
} \
@@ -47,7 +54,7 @@
/* Helper macro to check that driver implements required function pointer */
#define VALID_FUNC_OR_RET_ERR(func, dev_id) do { \
if (func == NULL) { \
- rte_bbdev_log(ERR, "device %u does not support %s", \
+ BBDEV_LOG(ERR, "device %u does not support %s", \
dev_id, #func); \
return -ENOTSUP; \
} \
@@ -56,7 +63,7 @@
/* Helper macro to check that queue is valid */
#define VALID_QUEUE_OR_RET_ERR(queue_id, dev) do { \
if (queue_id >= dev->data->num_queues) { \
- rte_bbdev_log(ERR, "Invalid queue_id %u for device %u", \
+ BBDEV_LOG(ERR, "Invalid queue_id %u for device %u", \
queue_id, dev->data->dev_id); \
return -ERANGE; \
} \
@@ -113,7 +120,7 @@ rte_bbdev_data_alloc(void)
} else
mz = rte_memzone_lookup(MZ_RTE_BBDEV_DATA);
if (mz == NULL) {
- rte_bbdev_log(CRIT,
+ BBDEV_LOG(CRIT,
"Cannot allocate memzone for bbdev port data");
return -ENOMEM;
}
@@ -168,18 +175,18 @@ rte_bbdev_allocate(const char *name)
uint16_t dev_id;
if (name == NULL) {
- rte_bbdev_log(ERR, "Invalid null device name");
+ BBDEV_LOG(ERR, "Invalid null device name");
return NULL;
}
if (rte_bbdev_get_named_dev(name) != NULL) {
- rte_bbdev_log(ERR, "Device \"%s\" is already allocated", name);
+ BBDEV_LOG(ERR, "Device \"%s\" is already allocated", name);
return NULL;
}
dev_id = find_free_dev_id();
if (dev_id == RTE_BBDEV_MAX_DEVS) {
- rte_bbdev_log(ERR, "Reached maximum number of devices");
+ BBDEV_LOG(ERR, "Reached maximum number of devices");
return NULL;
}
@@ -193,7 +200,7 @@ rte_bbdev_allocate(const char *name)
bbdev->data = find_bbdev_data(name);
if (bbdev->data == NULL) {
- rte_bbdev_log(ERR,
+ BBDEV_LOG(ERR,
"Max BBDevs already allocated in multi-process environment!");
return NULL;
}
@@ -204,7 +211,7 @@ rte_bbdev_allocate(const char *name)
ret = snprintf(bbdev->data->name, RTE_BBDEV_NAME_MAX_LEN, "%s", name);
if ((ret < 0) || (ret >= RTE_BBDEV_NAME_MAX_LEN)) {
- rte_bbdev_log(ERR, "Copying device name \"%s\" failed", name);
+ BBDEV_LOG(ERR, "Copying device name \"%s\" failed", name);
return NULL;
}
@@ -213,7 +220,8 @@ rte_bbdev_allocate(const char *name)
num_devs++;
- rte_bbdev_log_debug("Initialised device %s (id = %u). Num devices = %u",
+ BBDEV_LOG(DEBUG,
+ "Initialised device %s (id = %u). Num devices = %u",
name, dev_id, num_devs);
return bbdev;
@@ -226,7 +234,7 @@ rte_bbdev_release(struct rte_bbdev *bbdev)
struct rte_bbdev_callback *cb, *next;
if (bbdev == NULL) {
- rte_bbdev_log(ERR, "NULL bbdev");
+ BBDEV_LOG(ERR, "NULL bbdev");
return -ENODEV;
}
dev_id = bbdev->data->dev_id;
@@ -247,7 +255,7 @@ rte_bbdev_release(struct rte_bbdev *bbdev)
num_devs--;
bbdev->state = RTE_BBDEV_UNUSED;
- rte_bbdev_log_debug(
+ BBDEV_LOG(DEBUG,
"Un-initialised device id = %u. Num devices = %u",
dev_id, num_devs);
return 0;
@@ -259,7 +267,7 @@ rte_bbdev_get_named_dev(const char *name)
unsigned int i;
if (name == NULL) {
- rte_bbdev_log(ERR, "NULL driver name");
+ BBDEV_LOG(ERR, "NULL driver name");
return NULL;
}
@@ -310,7 +318,7 @@ rte_bbdev_setup_queues(uint16_t dev_id, uint16_t num_queues, int socket_id)
VALID_DEV_OPS_OR_RET_ERR(dev, dev_id);
if (dev->data->started) {
- rte_bbdev_log(ERR,
+ BBDEV_LOG(ERR,
"Device %u cannot be configured when started",
dev_id);
return -EBUSY;
@@ -322,7 +330,7 @@ rte_bbdev_setup_queues(uint16_t dev_id, uint16_t num_queues, int socket_id)
dev->dev_ops->info_get(dev, &dev_info);
if ((num_queues == 0) || (num_queues > dev_info.max_num_queues)) {
- rte_bbdev_log(ERR,
+ BBDEV_LOG(ERR,
"Device %u supports 0 < N <= %u queues, not %u",
dev_id, dev_info.max_num_queues, num_queues);
return -EINVAL;
@@ -334,7 +342,7 @@ rte_bbdev_setup_queues(uint16_t dev_id, uint16_t num_queues, int socket_id)
for (i = 0; i < dev->data->num_queues; i++) {
int ret = dev->dev_ops->queue_release(dev, i);
if (ret < 0) {
- rte_bbdev_log(ERR,
+ BBDEV_LOG(ERR,
"Device %u queue %u release failed",
dev_id, i);
return ret;
@@ -344,7 +352,7 @@ rte_bbdev_setup_queues(uint16_t dev_id, uint16_t num_queues, int socket_id)
if (dev->dev_ops->close) {
ret = dev->dev_ops->close(dev);
if (ret < 0) {
- rte_bbdev_log(ERR,
+ BBDEV_LOG(ERR,
"Device %u couldn't be closed",
dev_id);
return ret;
@@ -358,7 +366,7 @@ rte_bbdev_setup_queues(uint16_t dev_id, uint16_t num_queues, int socket_id)
sizeof(dev->data->queues[0]), RTE_CACHE_LINE_SIZE,
dev->data->socket_id);
if (dev->data->queues == NULL) {
- rte_bbdev_log(ERR,
+ BBDEV_LOG(ERR,
"calloc of %u queues for device %u on socket %i failed",
num_queues, dev_id, dev->data->socket_id);
return -ENOMEM;
@@ -370,14 +378,14 @@ rte_bbdev_setup_queues(uint16_t dev_id, uint16_t num_queues, int socket_id)
if (dev->dev_ops->setup_queues) {
ret = dev->dev_ops->setup_queues(dev, num_queues, socket_id);
if (ret < 0) {
- rte_bbdev_log(ERR,
+ BBDEV_LOG(ERR,
"Device %u memory configuration failed",
dev_id);
goto error;
}
}
- rte_bbdev_log_debug("Device %u set up with %u queues", dev_id,
+ BBDEV_LOG(DEBUG, "Device %u set up with %u queues", dev_id,
num_queues);
return 0;
@@ -398,7 +406,7 @@ rte_bbdev_intr_enable(uint16_t dev_id)
VALID_DEV_OPS_OR_RET_ERR(dev, dev_id);
if (dev->data->started) {
- rte_bbdev_log(ERR,
+ BBDEV_LOG(ERR,
"Device %u cannot be configured when started",
dev_id);
return -EBUSY;
@@ -407,16 +415,16 @@ rte_bbdev_intr_enable(uint16_t dev_id)
if (dev->dev_ops->intr_enable) {
ret = dev->dev_ops->intr_enable(dev);
if (ret < 0) {
- rte_bbdev_log(ERR,
+ BBDEV_LOG(ERR,
"Device %u interrupts configuration failed",
dev_id);
return ret;
}
- rte_bbdev_log_debug("Enabled interrupts for dev %u", dev_id);
+ BBDEV_LOG(DEBUG, "Enabled interrupts for dev %u", dev_id);
return 0;
}
- rte_bbdev_log(ERR, "Device %u doesn't support interrupts", dev_id);
+ BBDEV_LOG(ERR, "Device %u doesn't support interrupts", dev_id);
return -ENOTSUP;
}
@@ -437,7 +445,7 @@ rte_bbdev_queue_configure(uint16_t dev_id, uint16_t queue_id,
VALID_QUEUE_OR_RET_ERR(queue_id, dev);
if (dev->data->queues[queue_id].started || dev->data->started) {
- rte_bbdev_log(ERR,
+ BBDEV_LOG(ERR,
"Queue %u of device %u cannot be configured when started",
queue_id, dev_id);
return -EBUSY;
@@ -467,24 +475,24 @@ rte_bbdev_queue_configure(uint16_t dev_id, uint16_t queue_id,
}
}
if (ret == 0) {
- rte_bbdev_log(ERR, "Invalid operation type");
+ BBDEV_LOG(ERR, "Invalid operation type");
return -EINVAL;
}
if (conf->queue_size > dev_info.queue_size_lim) {
- rte_bbdev_log(ERR,
+ BBDEV_LOG(ERR,
"Size (%u) of queue %u of device %u must be: <= %u",
conf->queue_size, queue_id, dev_id,
dev_info.queue_size_lim);
return -EINVAL;
}
if (!rte_is_power_of_2(conf->queue_size)) {
- rte_bbdev_log(ERR,
+ BBDEV_LOG(ERR,
"Size (%u) of queue %u of device %u must be a power of 2",
conf->queue_size, queue_id, dev_id);
return -EINVAL;
}
if (conf->priority > dev_info.max_queue_priority) {
- rte_bbdev_log(ERR,
+ BBDEV_LOG(ERR,
"Priority (%u) of queue %u of bdev %u must be <= %u",
conf->priority, queue_id, dev_id,
dev_info.max_queue_priority);
@@ -496,7 +504,7 @@ rte_bbdev_queue_configure(uint16_t dev_id, uint16_t queue_id,
if (dev->data->queues[queue_id].queue_private != NULL) {
ret = dev->dev_ops->queue_release(dev, queue_id);
if (ret < 0) {
- rte_bbdev_log(ERR, "Device %u queue %u release failed",
+ BBDEV_LOG(ERR, "Device %u queue %u release failed",
dev_id, queue_id);
return ret;
}
@@ -506,7 +514,7 @@ rte_bbdev_queue_configure(uint16_t dev_id, uint16_t queue_id,
ret = dev->dev_ops->queue_setup(dev, queue_id, (conf != NULL) ?
conf : &dev_info.default_queue_conf);
if (ret < 0) {
- rte_bbdev_log(ERR,
+ BBDEV_LOG(ERR,
"Device %u queue %u setup failed", dev_id,
queue_id);
return ret;
@@ -522,7 +530,7 @@ rte_bbdev_queue_configure(uint16_t dev_id, uint16_t queue_id,
if (op_type_str == NULL)
return -EINVAL;
- rte_bbdev_log_debug("Configured dev%uq%u (size=%u, type=%s, prio=%u)",
+ BBDEV_LOG(DEBUG, "Configured dev%uq%u (size=%u, type=%s, prio=%u)",
dev_id, queue_id, stored_conf->queue_size, op_type_str,
stored_conf->priority);
@@ -539,14 +547,14 @@ rte_bbdev_start(uint16_t dev_id)
VALID_DEV_OPS_OR_RET_ERR(dev, dev_id);
if (dev->data->started) {
- rte_bbdev_log_debug("Device %u is already started", dev_id);
+ BBDEV_LOG(DEBUG, "Device %u is already started", dev_id);
return 0;
}
if (dev->dev_ops->start) {
int ret = dev->dev_ops->start(dev);
if (ret < 0) {
- rte_bbdev_log(ERR, "Device %u start failed", dev_id);
+ BBDEV_LOG(ERR, "Device %u start failed", dev_id);
return ret;
}
}
@@ -557,7 +565,7 @@ rte_bbdev_start(uint16_t dev_id)
dev->data->queues[i].started = true;
dev->data->started = true;
- rte_bbdev_log_debug("Started device %u", dev_id);
+ BBDEV_LOG(DEBUG, "Started device %u", dev_id);
return 0;
}
@@ -570,7 +578,7 @@ rte_bbdev_stop(uint16_t dev_id)
VALID_DEV_OPS_OR_RET_ERR(dev, dev_id);
if (!dev->data->started) {
- rte_bbdev_log_debug("Device %u is already stopped", dev_id);
+ BBDEV_LOG(DEBUG, "Device %u is already stopped", dev_id);
return 0;
}
@@ -578,7 +586,7 @@ rte_bbdev_stop(uint16_t dev_id)
dev->dev_ops->stop(dev);
dev->data->started = false;
- rte_bbdev_log_debug("Stopped device %u", dev_id);
+ BBDEV_LOG(DEBUG, "Stopped device %u", dev_id);
return 0;
}
@@ -595,7 +603,7 @@ rte_bbdev_close(uint16_t dev_id)
if (dev->data->started) {
ret = rte_bbdev_stop(dev_id);
if (ret < 0) {
- rte_bbdev_log(ERR, "Device %u stop failed", dev_id);
+ BBDEV_LOG(ERR, "Device %u stop failed", dev_id);
return ret;
}
}
@@ -604,7 +612,7 @@ rte_bbdev_close(uint16_t dev_id)
for (i = 0; i < dev->data->num_queues; i++) {
ret = dev->dev_ops->queue_release(dev, i);
if (ret < 0) {
- rte_bbdev_log(ERR, "Device %u queue %u release failed",
+ BBDEV_LOG(ERR, "Device %u queue %u release failed",
dev_id, i);
return ret;
}
@@ -614,7 +622,7 @@ rte_bbdev_close(uint16_t dev_id)
if (dev->dev_ops->close) {
ret = dev->dev_ops->close(dev);
if (ret < 0) {
- rte_bbdev_log(ERR, "Device %u close failed", dev_id);
+ BBDEV_LOG(ERR, "Device %u close failed", dev_id);
return ret;
}
}
@@ -623,7 +631,7 @@ rte_bbdev_close(uint16_t dev_id)
dev->data->queues = NULL;
dev->data->num_queues = 0;
- rte_bbdev_log_debug("Closed device %u", dev_id);
+ BBDEV_LOG(DEBUG, "Closed device %u", dev_id);
return 0;
}
@@ -638,7 +646,7 @@ rte_bbdev_queue_start(uint16_t dev_id, uint16_t queue_id)
VALID_QUEUE_OR_RET_ERR(queue_id, dev);
if (dev->data->queues[queue_id].started) {
- rte_bbdev_log_debug("Queue %u of device %u already started",
+ BBDEV_LOG(DEBUG, "Queue %u of device %u already started",
queue_id, dev_id);
return 0;
}
@@ -646,14 +654,14 @@ rte_bbdev_queue_start(uint16_t dev_id, uint16_t queue_id)
if (dev->dev_ops->queue_start) {
int ret = dev->dev_ops->queue_start(dev, queue_id);
if (ret < 0) {
- rte_bbdev_log(ERR, "Device %u queue %u start failed",
+ BBDEV_LOG(ERR, "Device %u queue %u start failed",
dev_id, queue_id);
return ret;
}
}
dev->data->queues[queue_id].started = true;
- rte_bbdev_log_debug("Started queue %u of device %u", queue_id, dev_id);
+ BBDEV_LOG(DEBUG, "Started queue %u of device %u", queue_id, dev_id);
return 0;
}
@@ -668,7 +676,7 @@ rte_bbdev_queue_stop(uint16_t dev_id, uint16_t queue_id)
VALID_QUEUE_OR_RET_ERR(queue_id, dev);
if (!dev->data->queues[queue_id].started) {
- rte_bbdev_log_debug("Queue %u of device %u already stopped",
+ BBDEV_LOG(DEBUG, "Queue %u of device %u already stopped",
queue_id, dev_id);
return 0;
}
@@ -676,14 +684,14 @@ rte_bbdev_queue_stop(uint16_t dev_id, uint16_t queue_id)
if (dev->dev_ops->queue_stop) {
int ret = dev->dev_ops->queue_stop(dev, queue_id);
if (ret < 0) {
- rte_bbdev_log(ERR, "Device %u queue %u stop failed",
+ BBDEV_LOG(ERR, "Device %u queue %u stop failed",
dev_id, queue_id);
return ret;
}
}
dev->data->queues[queue_id].started = false;
- rte_bbdev_log_debug("Stopped queue %u of device %u", queue_id, dev_id);
+ BBDEV_LOG(DEBUG, "Stopped queue %u of device %u", queue_id, dev_id);
return 0;
}
@@ -701,7 +709,7 @@ get_stats_from_queues(struct rte_bbdev *dev, struct rte_bbdev_stats *stats)
stats->enqueue_err_count += q_stats->enqueue_err_count;
stats->dequeue_err_count += q_stats->dequeue_err_count;
}
- rte_bbdev_log_debug("Got stats on %u", dev->data->dev_id);
+ BBDEV_LOG(DEBUG, "Got stats on %u", dev->data->dev_id);
}
static void
@@ -714,7 +722,7 @@ reset_stats_in_queues(struct rte_bbdev *dev)
memset(q_stats, 0, sizeof(*q_stats));
}
- rte_bbdev_log_debug("Reset stats on %u", dev->data->dev_id);
+ BBDEV_LOG(DEBUG, "Reset stats on %u", dev->data->dev_id);
}
int
@@ -726,7 +734,7 @@ rte_bbdev_stats_get(uint16_t dev_id, struct rte_bbdev_stats *stats)
VALID_DEV_OPS_OR_RET_ERR(dev, dev_id);
if (stats == NULL) {
- rte_bbdev_log(ERR, "NULL stats structure");
+ BBDEV_LOG(ERR, "NULL stats structure");
return -EINVAL;
}
@@ -736,7 +744,7 @@ rte_bbdev_stats_get(uint16_t dev_id, struct rte_bbdev_stats *stats)
else
get_stats_from_queues(dev, stats);
- rte_bbdev_log_debug("Retrieved stats of device %u", dev_id);
+ BBDEV_LOG(DEBUG, "Retrieved stats of device %u", dev_id);
return 0;
}
@@ -753,7 +761,7 @@ rte_bbdev_stats_reset(uint16_t dev_id)
else
reset_stats_in_queues(dev);
- rte_bbdev_log_debug("Reset stats of device %u", dev_id);
+ BBDEV_LOG(DEBUG, "Reset stats of device %u", dev_id);
return 0;
}
@@ -766,7 +774,7 @@ rte_bbdev_info_get(uint16_t dev_id, struct rte_bbdev_info *dev_info)
VALID_FUNC_OR_RET_ERR(dev->dev_ops->info_get, dev_id);
if (dev_info == NULL) {
- rte_bbdev_log(ERR, "NULL dev info structure");
+ BBDEV_LOG(ERR, "NULL dev info structure");
return -EINVAL;
}
@@ -781,7 +789,7 @@ rte_bbdev_info_get(uint16_t dev_id, struct rte_bbdev_info *dev_info)
/* Copy data maintained by device driver layer */
dev->dev_ops->info_get(dev, &dev_info->drv);
- rte_bbdev_log_debug("Retrieved info of device %u", dev_id);
+ BBDEV_LOG(DEBUG, "Retrieved info of device %u", dev_id);
return 0;
}
@@ -795,7 +803,7 @@ rte_bbdev_queue_info_get(uint16_t dev_id, uint16_t queue_id,
VALID_QUEUE_OR_RET_ERR(queue_id, dev);
if (queue_info == NULL) {
- rte_bbdev_log(ERR, "NULL queue info structure");
+ BBDEV_LOG(ERR, "NULL queue info structure");
return -EINVAL;
}
@@ -804,7 +812,7 @@ rte_bbdev_queue_info_get(uint16_t dev_id, uint16_t queue_id,
queue_info->conf = dev->data->queues[queue_id].conf;
queue_info->started = dev->data->queues[queue_id].started;
- rte_bbdev_log_debug("Retrieved info of queue %u of device %u",
+ BBDEV_LOG(DEBUG, "Retrieved info of queue %u of device %u",
queue_id, dev_id);
return 0;
}
@@ -860,12 +868,12 @@ rte_bbdev_op_pool_create(const char *name, enum rte_bbdev_op_type type,
const char *op_type_str;
if (name == NULL) {
- rte_bbdev_log(ERR, "NULL name for op pool");
+ BBDEV_LOG(ERR, "NULL name for op pool");
return NULL;
}
if (type >= RTE_BBDEV_OP_TYPE_COUNT) {
- rte_bbdev_log(ERR,
+ BBDEV_LOG(ERR,
"Invalid op type (%u), should be less than %u",
type, RTE_BBDEV_OP_TYPE_COUNT);
return NULL;
@@ -875,7 +883,7 @@ rte_bbdev_op_pool_create(const char *name, enum rte_bbdev_op_type type,
cache_size, sizeof(struct rte_bbdev_op_pool_private),
NULL, NULL, bbdev_op_init, &type, socket_id, 0);
if (mp == NULL) {
- rte_bbdev_log(ERR,
+ BBDEV_LOG(ERR,
"Failed to create op pool %s (num ops=%u, op size=%u) with error: %s",
name, num_elements, get_bbdev_op_size(type),
rte_strerror(rte_errno));
@@ -886,7 +894,7 @@ rte_bbdev_op_pool_create(const char *name, enum rte_bbdev_op_type type,
if (op_type_str == NULL)
return NULL;
- rte_bbdev_log_debug(
+ BBDEV_LOG(DEBUG,
"Op pool %s created for %u ops (type=%s, cache=%u, socket=%u, size=%u)",
name, num_elements, op_type_str, cache_size, socket_id,
get_bbdev_op_size(type));
@@ -906,14 +914,14 @@ rte_bbdev_callback_register(uint16_t dev_id, enum rte_bbdev_event_type event,
VALID_DEV_OR_RET_ERR(dev, dev_id);
if (event >= RTE_BBDEV_EVENT_MAX) {
- rte_bbdev_log(ERR,
+ BBDEV_LOG(ERR,
"Invalid event type (%u), should be less than %u",
event, RTE_BBDEV_EVENT_MAX);
return -EINVAL;
}
if (cb_fn == NULL) {
- rte_bbdev_log(ERR, "NULL callback function");
+ BBDEV_LOG(ERR, "NULL callback function");
return -EINVAL;
}
@@ -952,14 +960,14 @@ rte_bbdev_callback_unregister(uint16_t dev_id, enum rte_bbdev_event_type event,
VALID_DEV_OR_RET_ERR(dev, dev_id);
if (event >= RTE_BBDEV_EVENT_MAX) {
- rte_bbdev_log(ERR,
+ BBDEV_LOG(ERR,
"Invalid event type (%u), should be less than %u",
event, RTE_BBDEV_EVENT_MAX);
return -EINVAL;
}
if (cb_fn == NULL) {
- rte_bbdev_log(ERR,
+ BBDEV_LOG(ERR,
"NULL callback function cannot be unregistered");
return -EINVAL;
}
@@ -995,17 +1003,17 @@ rte_bbdev_pmd_callback_process(struct rte_bbdev *dev,
struct rte_bbdev_callback dev_cb;
if (dev == NULL) {
- rte_bbdev_log(ERR, "NULL device");
+ BBDEV_LOG(ERR, "NULL device");
return;
}
if (dev->data == NULL) {
- rte_bbdev_log(ERR, "NULL data structure");
+ BBDEV_LOG(ERR, "NULL data structure");
return;
}
if (event >= RTE_BBDEV_EVENT_MAX) {
- rte_bbdev_log(ERR,
+ BBDEV_LOG(ERR,
"Invalid event type (%u), should be less than %u",
event, RTE_BBDEV_EVENT_MAX);
return;
@@ -1065,12 +1073,12 @@ rte_bbdev_queue_intr_ctl(uint16_t dev_id, uint16_t queue_id, int epfd, int op,
intr_handle = dev->intr_handle;
if (!intr_handle || !intr_handle->intr_vec) {
- rte_bbdev_log(ERR, "Device %u intr handle unset\n", dev_id);
+ BBDEV_LOG(ERR, "Device %u intr handle unset\n", dev_id);
return -ENOTSUP;
}
if (queue_id >= RTE_MAX_RXTX_INTR_VEC_ID) {
- rte_bbdev_log(ERR, "Device %u queue_id %u is too big\n",
+ BBDEV_LOG(ERR, "Device %u queue_id %u is too big\n",
dev_id, queue_id);
return -ENOTSUP;
}
@@ -1078,7 +1086,7 @@ rte_bbdev_queue_intr_ctl(uint16_t dev_id, uint16_t queue_id, int epfd, int op,
vec = intr_handle->intr_vec[queue_id];
ret = rte_intr_rx_ctl(intr_handle, epfd, op, vec, data);
if (ret && (ret != -EEXIST)) {
- rte_bbdev_log(ERR,
+ BBDEV_LOG(ERR,
"dev %u q %u int ctl error op %d epfd %d vec %u\n",
dev_id, queue_id, op, epfd, vec);
return ret;
@@ -1100,13 +1108,11 @@ rte_bbdev_op_type_str(enum rte_bbdev_op_type op_type)
if (op_type < RTE_BBDEV_OP_TYPE_COUNT)
return op_types[op_type];
- rte_bbdev_log(ERR, "Invalid operation type");
+ BBDEV_LOG(ERR, "Invalid operation type");
return NULL;
}
-int bbdev_logtype;
-
RTE_INIT(rte_bbdev_init_log);
static void
rte_bbdev_init_log(void)
diff --git a/lib/librte_bbdev/rte_bbdev.h b/lib/librte_bbdev/rte_bbdev.h
index 37a0d05..3a8f97f 100644
--- a/lib/librte_bbdev/rte_bbdev.h
+++ b/lib/librte_bbdev/rte_bbdev.h
@@ -462,12 +462,7 @@ rte_bbdev_enqueue_enc_ops(uint16_t dev_id, uint16_t queue_id,
{
struct rte_bbdev *dev = &rte_bbdev_devices[dev_id];
struct rte_bbdev_queue_data *q_data = &dev->data->queues[queue_id];
- uint16_t n = dev->enqueue_enc_ops(q_data, ops, num_ops);
-
- rte_bbdev_log_verbose("%u encode ops enqueued to dev%u,q%u.\n",
- num_ops, dev_id, queue_id);
-
- return n;
+ return dev->enqueue_enc_ops(q_data, ops, num_ops);
}
/**
@@ -497,12 +492,7 @@ rte_bbdev_enqueue_dec_ops(uint16_t dev_id, uint16_t queue_id,
{
struct rte_bbdev *dev = &rte_bbdev_devices[dev_id];
struct rte_bbdev_queue_data *q_data = &dev->data->queues[queue_id];
- uint16_t n = dev->enqueue_dec_ops(q_data, ops, num_ops);
-
- rte_bbdev_log_verbose("%u decode ops enqueued to dev%u,q%u.\n",
- num_ops, dev_id, queue_id);
-
- return n;
+ return dev->enqueue_dec_ops(q_data, ops, num_ops);
}
/**
@@ -532,12 +522,7 @@ rte_bbdev_dequeue_enc_ops(uint16_t dev_id, uint16_t queue_id,
{
struct rte_bbdev *dev = &rte_bbdev_devices[dev_id];
struct rte_bbdev_queue_data *q_data = &dev->data->queues[queue_id];
- uint16_t n = dev->dequeue_enc_ops(q_data, ops, num_ops);
-
- rte_bbdev_log_verbose("%u encode ops dequeued to dev%u,q%u\n",
- n, dev_id, queue_id);
-
- return n;
+ return dev->dequeue_enc_ops(q_data, ops, num_ops);
}
/**
@@ -568,12 +553,7 @@ rte_bbdev_dequeue_dec_ops(uint16_t dev_id, uint16_t queue_id,
{
struct rte_bbdev *dev = &rte_bbdev_devices[dev_id];
struct rte_bbdev_queue_data *q_data = &dev->data->queues[queue_id];
- uint16_t n = dev->dequeue_dec_ops(q_data, ops, num_ops);
-
- rte_bbdev_log_verbose("%u decode ops dequeued to dev%u,q%u\n",
- n, dev_id, queue_id);
-
- return n;
+ return dev->dequeue_dec_ops(q_data, ops, num_ops);
}
/** Definitions of device event types */
diff --git a/lib/librte_bbdev/rte_bbdev_op.h b/lib/librte_bbdev/rte_bbdev_op.h
index c0c7d73..9a80c64 100644
--- a/lib/librte_bbdev/rte_bbdev_op.h
+++ b/lib/librte_bbdev/rte_bbdev_op.h
@@ -27,58 +27,6 @@ extern "C" {
#define RTE_BBDEV_MAX_CODE_BLOCKS 64
-extern int bbdev_logtype;
-
-/**
- * Helper macro for logging
- *
- * @param level
- * Log level: EMERG, ALERT, CRIT, ERR, WARNING, NOTICE, INFO, or DEBUG
- * @param fmt
- * The format string, as in printf(3).
- * @param ...
- * The variable arguments required by the format string.
- *
- * @return
- * - 0 on success
- * - Negative on error
- */
-#define rte_bbdev_log(level, fmt, ...) \
- rte_log(RTE_LOG_ ## level, bbdev_logtype, fmt "\n", ##__VA_ARGS__)
-
-/**
- * Helper macro for debug logging with extra source info
- *
- * @param fmt
- * The format string, as in printf(3).
- * @param ...
- * The variable arguments required by the format string.
- *
- * @return
- * - 0 on success
- * - Negative on error
- */
-#define rte_bbdev_log_debug(fmt, ...) \
- rte_bbdev_log(DEBUG, RTE_STR(__LINE__) ":%s() " fmt, __func__, \
- ##__VA_ARGS__)
-
-/**
- * Helper macro for extra conditional logging from datapath
- *
- * @param fmt
- * The format string, as in printf(3).
- * @param ...
- * The variable arguments required by the format string.
- *
- * @return
- * - 0 on success
- * - Negative on error
- */
-#define rte_bbdev_log_verbose(fmt, ...) \
- (void)((RTE_LOG_DEBUG <= RTE_LOG_DP_LEVEL) ? \
- rte_log(RTE_LOG_DEBUG, \
- bbdev_logtype, ": " fmt "\n", ##__VA_ARGS__) : 0)
-
/** Flags for turbo decoder operation and capability structure */
enum rte_bbdev_op_td_flag_bitmasks {
/**< If sub block de-interleaving is to be performed. */
@@ -547,9 +495,6 @@ rte_bbdev_enc_op_alloc_bulk(struct rte_mempool *mempool,
if (unlikely(ret < 0))
return ret;
- rte_bbdev_log_verbose("%u encode ops allocated from %s\n",
- num_ops, mempool->name);
-
return 0;
}
@@ -585,9 +530,6 @@ rte_bbdev_dec_op_alloc_bulk(struct rte_mempool *mempool,
if (unlikely(ret < 0))
return ret;
- rte_bbdev_log_verbose("%u encode ops allocated from %s\n",
- num_ops, mempool->name);
-
return 0;
}
@@ -604,11 +546,8 @@ rte_bbdev_dec_op_alloc_bulk(struct rte_mempool *mempool,
static inline void
rte_bbdev_dec_op_free_bulk(struct rte_bbdev_dec_op **ops, unsigned int num_ops)
{
- if (num_ops > 0) {
+ if (num_ops > 0)
rte_mempool_put_bulk(ops[0]->mempool, (void **)ops, num_ops);
- rte_bbdev_log_verbose("%u decode ops freed to %s\n", num_ops,
- ops[0]->mempool->name);
- }
}
/**
@@ -624,11 +563,8 @@ rte_bbdev_dec_op_free_bulk(struct rte_bbdev_dec_op **ops, unsigned int num_ops)
static inline void
rte_bbdev_enc_op_free_bulk(struct rte_bbdev_enc_op **ops, unsigned int num_ops)
{
- if (num_ops > 0) {
+ if (num_ops > 0)
rte_mempool_put_bulk(ops[0]->mempool, (void **)ops, num_ops);
- rte_bbdev_log_verbose("%u encode ops freed to %s\n", num_ops,
- ops[0]->mempool->name);
- }
}
#ifdef __cplusplus
diff --git a/lib/librte_bbdev/rte_bbdev_version.map b/lib/librte_bbdev/rte_bbdev_version.map
index 737c339..d3b81ea 100644
--- a/lib/librte_bbdev/rte_bbdev_version.map
+++ b/lib/librte_bbdev/rte_bbdev_version.map
@@ -1,7 +1,6 @@
EXPERIMENTAL {
global:
- bbdev_logtype;
rte_bbdev_allocate;
rte_bbdev_callback_register;
rte_bbdev_callback_unregister;
--
2.7.4
reply other threads:[~2018-02-05 23:33 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1517873629-83062-1-git-send-email-amr.mokhtar@intel.com \
--to=amr.mokhtar@intel.com \
--cc=dev@dpdk.org \
--cc=ferruh.yigit@intel.com \
--cc=thomas@monjalon.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).