git.droids-corp.org / dpdk.git / blobdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw | inline | side by side
net/ice/base: introduce and use for each bit iterator
[dpdk.git] / drivers / crypto / scheduler / rte_cryptodev_scheduler.c
diff --git a/drivers/crypto/scheduler/rte_cryptodev_scheduler.c b/drivers/crypto/scheduler/rte_cryptodev_scheduler.c
index ed574cc..730504d 100644 (file)
--- a/drivers/crypto/scheduler/rte_cryptodev_scheduler.c
+++ b/drivers/crypto/scheduler/rte_cryptodev_scheduler.c
@@ -1,6 +1,7 @@
 /* SPDX-License-Identifier: BSD-3-Clause
  * Copyright(c) 2017 Intel Corporation
  */
+#include <rte_string_fns.h>
 #include <rte_reorder.h>
 #include <rte_cryptodev.h>
 #include <rte_cryptodev_pmd.h>
@@ -168,30 +169,30 @@ rte_cryptodev_scheduler_slave_attach(uint8_t scheduler_id, uint8_t slave_id)
        uint32_t i;
 
        if (!dev) {
-               CS_LOG_ERR("Operation not supported");
+               CR_SCHED_LOG(ERR, "Operation not supported");
                return -ENOTSUP;
        }
 
-       if (dev->driver_id != cryptodev_driver_id) {
-               CS_LOG_ERR("Operation not supported");
+       if (dev->driver_id != cryptodev_scheduler_driver_id) {
+               CR_SCHED_LOG(ERR, "Operation not supported");
                return -ENOTSUP;
        }
 
        if (dev->data->dev_started) {
-               CS_LOG_ERR("Illegal operation");
+               CR_SCHED_LOG(ERR, "Illegal operation");
                return -EBUSY;
        }
 
        sched_ctx = dev->data->dev_private;
        if (sched_ctx->nb_slaves >=
                        RTE_CRYPTODEV_SCHEDULER_MAX_NB_SLAVES) {
-               CS_LOG_ERR("Too many slaves attached");
+               CR_SCHED_LOG(ERR, "Too many slaves attached");
                return -ENOMEM;
        }
 
        for (i = 0; i < sched_ctx->nb_slaves; i++)
                if (sched_ctx->slaves[i].dev_id == slave_id) {
-                       CS_LOG_ERR("Slave already added");
+                       CR_SCHED_LOG(ERR, "Slave already added");
                        return -ENOTSUP;
                }
 
@@ -208,7 +209,7 @@ rte_cryptodev_scheduler_slave_attach(uint8_t scheduler_id, uint8_t slave_id)
                slave->driver_id = 0;
                sched_ctx->nb_slaves--;
 
-               CS_LOG_ERR("capabilities update failed");
+               CR_SCHED_LOG(ERR, "capabilities update failed");
                return -ENOTSUP;
        }
 
@@ -227,17 +228,17 @@ rte_cryptodev_scheduler_slave_detach(uint8_t scheduler_id, uint8_t slave_id)
        uint32_t i, slave_pos;
 
        if (!dev) {
-               CS_LOG_ERR("Operation not supported");
+               CR_SCHED_LOG(ERR, "Operation not supported");
                return -ENOTSUP;
        }
 
-       if (dev->driver_id != cryptodev_driver_id) {
-               CS_LOG_ERR("Operation not supported");
+       if (dev->driver_id != cryptodev_scheduler_driver_id) {
+               CR_SCHED_LOG(ERR, "Operation not supported");
                return -ENOTSUP;
        }
 
        if (dev->data->dev_started) {
-               CS_LOG_ERR("Illegal operation");
+               CR_SCHED_LOG(ERR, "Illegal operation");
                return -EBUSY;
        }
 
@@ -247,12 +248,12 @@ rte_cryptodev_scheduler_slave_detach(uint8_t scheduler_id, uint8_t slave_id)
                if (sched_ctx->slaves[slave_pos].dev_id == slave_id)
                        break;
        if (slave_pos == sched_ctx->nb_slaves) {
-               CS_LOG_ERR("Cannot find slave");
+               CR_SCHED_LOG(ERR, "Cannot find slave");
                return -ENOTSUP;
        }
 
        if (sched_ctx->ops.slave_detach(dev, slave_id) < 0) {
-               CS_LOG_ERR("Failed to detach slave");
+               CR_SCHED_LOG(ERR, "Failed to detach slave");
                return -ENOTSUP;
        }
 
@@ -265,7 +266,7 @@ rte_cryptodev_scheduler_slave_detach(uint8_t scheduler_id, uint8_t slave_id)
        sched_ctx->nb_slaves--;
 
        if (update_scheduler_capability(sched_ctx) < 0) {
-               CS_LOG_ERR("capabilities update failed");
+               CR_SCHED_LOG(ERR, "capabilities update failed");
                return -ENOTSUP;
        }
 
@@ -284,17 +285,17 @@ rte_cryptodev_scheduler_mode_set(uint8_t scheduler_id,
        struct scheduler_ctx *sched_ctx;
 
        if (!dev) {
-               CS_LOG_ERR("Operation not supported");
+               CR_SCHED_LOG(ERR, "Operation not supported");
                return -ENOTSUP;
        }
 
-       if (dev->driver_id != cryptodev_driver_id) {
-               CS_LOG_ERR("Operation not supported");
+       if (dev->driver_id != cryptodev_scheduler_driver_id) {
+               CR_SCHED_LOG(ERR, "Operation not supported");
                return -ENOTSUP;
        }
 
        if (dev->data->dev_started) {
-               CS_LOG_ERR("Illegal operation");
+               CR_SCHED_LOG(ERR, "Illegal operation");
                return -EBUSY;
        }
 
@@ -306,34 +307,34 @@ rte_cryptodev_scheduler_mode_set(uint8_t scheduler_id,
        switch (mode) {
        case CDEV_SCHED_MODE_ROUNDROBIN:
                if (rte_cryptodev_scheduler_load_user_scheduler(scheduler_id,
-                               roundrobin_scheduler) < 0) {
-                       CS_LOG_ERR("Failed to load scheduler");
+                               crypto_scheduler_roundrobin) < 0) {
+                       CR_SCHED_LOG(ERR, "Failed to load scheduler");
                        return -1;
                }
                break;
        case CDEV_SCHED_MODE_PKT_SIZE_DISTR:
                if (rte_cryptodev_scheduler_load_user_scheduler(scheduler_id,
-                               pkt_size_based_distr_scheduler) < 0) {
-                       CS_LOG_ERR("Failed to load scheduler");
+                               crypto_scheduler_pkt_size_based_distr) < 0) {
+                       CR_SCHED_LOG(ERR, "Failed to load scheduler");
                        return -1;
                }
                break;
        case CDEV_SCHED_MODE_FAILOVER:
                if (rte_cryptodev_scheduler_load_user_scheduler(scheduler_id,
-                               failover_scheduler) < 0) {
-                       CS_LOG_ERR("Failed to load scheduler");
+                               crypto_scheduler_failover) < 0) {
+                       CR_SCHED_LOG(ERR, "Failed to load scheduler");
                        return -1;
                }
                break;
        case CDEV_SCHED_MODE_MULTICORE:
                if (rte_cryptodev_scheduler_load_user_scheduler(scheduler_id,
-                               multicore_scheduler) < 0) {
-                       CS_LOG_ERR("Failed to load scheduler");
+                               crypto_scheduler_multicore) < 0) {
+                       CR_SCHED_LOG(ERR, "Failed to load scheduler");
                        return -1;
                }
                break;
        default:
-               CS_LOG_ERR("Not yet supported");
+               CR_SCHED_LOG(ERR, "Not yet supported");
                return -ENOTSUP;
        }
 
@@ -347,12 +348,12 @@ rte_cryptodev_scheduler_mode_get(uint8_t scheduler_id)
        struct scheduler_ctx *sched_ctx;
 
        if (!dev) {
-               CS_LOG_ERR("Operation not supported");
+               CR_SCHED_LOG(ERR, "Operation not supported");
                return -ENOTSUP;
        }
 
-       if (dev->driver_id != cryptodev_driver_id) {
-               CS_LOG_ERR("Operation not supported");
+       if (dev->driver_id != cryptodev_scheduler_driver_id) {
+               CR_SCHED_LOG(ERR, "Operation not supported");
                return -ENOTSUP;
        }
 
@@ -369,17 +370,17 @@ rte_cryptodev_scheduler_ordering_set(uint8_t scheduler_id,
        struct scheduler_ctx *sched_ctx;
 
        if (!dev) {
-               CS_LOG_ERR("Operation not supported");
+               CR_SCHED_LOG(ERR, "Operation not supported");
                return -ENOTSUP;
        }
 
-       if (dev->driver_id != cryptodev_driver_id) {
-               CS_LOG_ERR("Operation not supported");
+       if (dev->driver_id != cryptodev_scheduler_driver_id) {
+               CR_SCHED_LOG(ERR, "Operation not supported");
                return -ENOTSUP;
        }
 
        if (dev->data->dev_started) {
-               CS_LOG_ERR("Illegal operation");
+               CR_SCHED_LOG(ERR, "Illegal operation");
                return -EBUSY;
        }
 
@@ -397,12 +398,12 @@ rte_cryptodev_scheduler_ordering_get(uint8_t scheduler_id)
        struct scheduler_ctx *sched_ctx;
 
        if (!dev) {
-               CS_LOG_ERR("Operation not supported");
+               CR_SCHED_LOG(ERR, "Operation not supported");
                return -ENOTSUP;
        }
 
-       if (dev->driver_id != cryptodev_driver_id) {
-               CS_LOG_ERR("Operation not supported");
+       if (dev->driver_id != cryptodev_scheduler_driver_id) {
+               CR_SCHED_LOG(ERR, "Operation not supported");
                return -ENOTSUP;
        }
 
@@ -419,40 +420,39 @@ rte_cryptodev_scheduler_load_user_scheduler(uint8_t scheduler_id,
        struct scheduler_ctx *sched_ctx;
 
        if (!dev) {
-               CS_LOG_ERR("Operation not supported");
+               CR_SCHED_LOG(ERR, "Operation not supported");
                return -ENOTSUP;
        }
 
-       if (dev->driver_id != cryptodev_driver_id) {
-               CS_LOG_ERR("Operation not supported");
+       if (dev->driver_id != cryptodev_scheduler_driver_id) {
+               CR_SCHED_LOG(ERR, "Operation not supported");
                return -ENOTSUP;
        }
 
        if (dev->data->dev_started) {
-               CS_LOG_ERR("Illegal operation");
+               CR_SCHED_LOG(ERR, "Illegal operation");
                return -EBUSY;
        }
 
        sched_ctx = dev->data->dev_private;
 
        if (strlen(scheduler->name) > RTE_CRYPTODEV_NAME_MAX_LEN - 1) {
-               CS_LOG_ERR("Invalid name %s, should be less than "
-                               "%u bytes.\n", scheduler->name,
+               CR_SCHED_LOG(ERR, "Invalid name %s, should be less than "
+                               "%u bytes.", scheduler->name,
                                RTE_CRYPTODEV_NAME_MAX_LEN);
                return -EINVAL;
        }
-       snprintf(sched_ctx->name, sizeof(sched_ctx->name), "%s",
-                       scheduler->name);
+       strlcpy(sched_ctx->name, scheduler->name, sizeof(sched_ctx->name));
 
        if (strlen(scheduler->description) >
                        RTE_CRYPTODEV_SCHEDULER_DESC_MAX_LEN - 1) {
-               CS_LOG_ERR("Invalid description %s, should be less than "
-                               "%u bytes.\n", scheduler->description,
+               CR_SCHED_LOG(ERR, "Invalid description %s, should be less than "
+                               "%u bytes.", scheduler->description,
                                RTE_CRYPTODEV_SCHEDULER_DESC_MAX_LEN - 1);
                return -EINVAL;
        }
-       snprintf(sched_ctx->description, sizeof(sched_ctx->description), "%s",
-                       scheduler->description);
+       strlcpy(sched_ctx->description, scheduler->description,
+               sizeof(sched_ctx->description));
 
        /* load scheduler instance operations functions */
        sched_ctx->ops.config_queue_pair = scheduler->ops->config_queue_pair;
@@ -473,7 +473,7 @@ rte_cryptodev_scheduler_load_user_scheduler(uint8_t scheduler_id,
                int ret = (*sched_ctx->ops.create_private_ctx)(dev);
 
                if (ret < 0) {
-                       CS_LOG_ERR("Unable to create scheduler private "
+                       CR_SCHED_LOG(ERR, "Unable to create scheduler private "
                                        "context");
                        return ret;
                }
@@ -492,12 +492,12 @@ rte_cryptodev_scheduler_slaves_get(uint8_t scheduler_id, uint8_t *slaves)
        uint32_t nb_slaves = 0;
 
        if (!dev) {
-               CS_LOG_ERR("Operation not supported");
+               CR_SCHED_LOG(ERR, "Operation not supported");
                return -ENOTSUP;
        }
 
-       if (dev->driver_id != cryptodev_driver_id) {
-               CS_LOG_ERR("Operation not supported");
+       if (dev->driver_id != cryptodev_scheduler_driver_id) {
+               CR_SCHED_LOG(ERR, "Operation not supported");
                return -ENOTSUP;
        }
 
@@ -525,17 +525,17 @@ rte_cryptodev_scheduler_option_set(uint8_t scheduler_id,
 
        if (option_type == CDEV_SCHED_OPTION_NOT_SET ||
                        option_type >= CDEV_SCHED_OPTION_COUNT) {
-               CS_LOG_ERR("Invalid option parameter");
+               CR_SCHED_LOG(ERR, "Invalid option parameter");
                return -EINVAL;
        }
 
        if (!option) {
-               CS_LOG_ERR("Invalid option parameter");
+               CR_SCHED_LOG(ERR, "Invalid option parameter");
                return -EINVAL;
        }
 
        if (dev->data->dev_started) {
-               CS_LOG_ERR("Illegal operation");
+               CR_SCHED_LOG(ERR, "Illegal operation");
                return -EBUSY;
        }
 
@@ -555,17 +555,17 @@ rte_cryptodev_scheduler_option_get(uint8_t scheduler_id,
        struct scheduler_ctx *sched_ctx;
 
        if (!dev) {
-               CS_LOG_ERR("Operation not supported");
+               CR_SCHED_LOG(ERR, "Operation not supported");
                return -ENOTSUP;
        }
 
        if (!option) {
-               CS_LOG_ERR("Invalid option parameter");
+               CR_SCHED_LOG(ERR, "Invalid option parameter");
                return -EINVAL;
        }
 
-       if (dev->driver_id != cryptodev_driver_id) {
-               CS_LOG_ERR("Operation not supported");
+       if (dev->driver_id != cryptodev_scheduler_driver_id) {
+               CR_SCHED_LOG(ERR, "Operation not supported");
                return -ENOTSUP;
        }
 
@@ -575,3 +575,6 @@ rte_cryptodev_scheduler_option_get(uint8_t scheduler_id,
 
        return (*sched_ctx->ops.option_get)(dev, option_type, option);
 }
+
+
+RTE_LOG_REGISTER(scheduler_logtype_driver, pmd.crypto.scheduler, INFO);
DPDK repo used for reviews