cryptodev: allocate driver structure statically
[dpdk.git] / drivers / crypto / scheduler / scheduler_pmd.c
index b189fbd..3170f7f 100644 (file)
@@ -33,6 +33,7 @@
 #include <rte_hexdump.h>
 #include <rte_cryptodev.h>
 #include <rte_cryptodev_pmd.h>
+#include <rte_cryptodev_vdev.h>
 #include <rte_vdev.h>
 #include <rte_malloc.h>
 #include <rte_cpuflags.h>
 #include "rte_cryptodev_scheduler.h"
 #include "scheduler_pmd_private.h"
 
+uint8_t cryptodev_driver_id;
+
 struct scheduler_init_params {
        struct rte_crypto_vdev_init_params def_p;
        uint32_t nb_slaves;
-       uint8_t slaves[RTE_CRYPTODEV_SCHEDULER_MAX_NB_SLAVES];
        enum rte_cryptodev_scheduler_mode mode;
        uint32_t enable_ordering;
+       uint64_t wcmask;
+       char slave_names[RTE_CRYPTODEV_SCHEDULER_MAX_NB_SLAVES]
+                       [RTE_CRYPTODEV_SCHEDULER_NAME_MAX_LEN];
 };
 
 #define RTE_CRYPTODEV_VDEV_NAME                        ("name")
@@ -56,6 +61,8 @@ struct scheduler_init_params {
 #define RTE_CRYPTODEV_VDEV_MAX_NB_QP_ARG       ("max_nb_queue_pairs")
 #define RTE_CRYPTODEV_VDEV_MAX_NB_SESS_ARG     ("max_nb_sessions")
 #define RTE_CRYPTODEV_VDEV_SOCKET_ID           ("socket_id")
+#define RTE_CRYPTODEV_VDEV_COREMASK            ("coremask")
+#define RTE_CRYPTODEV_VDEV_CORELIST            ("corelist")
 
 const char *scheduler_valid_params[] = {
        RTE_CRYPTODEV_VDEV_NAME,
@@ -64,7 +71,9 @@ const char *scheduler_valid_params[] = {
        RTE_CRYPTODEV_VDEV_ORDERING,
        RTE_CRYPTODEV_VDEV_MAX_NB_QP_ARG,
        RTE_CRYPTODEV_VDEV_MAX_NB_SESS_ARG,
-       RTE_CRYPTODEV_VDEV_SOCKET_ID
+       RTE_CRYPTODEV_VDEV_SOCKET_ID,
+       RTE_CRYPTODEV_VDEV_COREMASK,
+       RTE_CRYPTODEV_VDEV_CORELIST
 };
 
 struct scheduler_parse_map {
@@ -78,7 +87,9 @@ const struct scheduler_parse_map scheduler_mode_map[] = {
        {RTE_STR(SCHEDULER_MODE_NAME_PKT_SIZE_DISTR),
                        CDEV_SCHED_MODE_PKT_SIZE_DISTR},
        {RTE_STR(SCHEDULER_MODE_NAME_FAIL_OVER),
-                       CDEV_SCHED_MODE_FAILOVER}
+                       CDEV_SCHED_MODE_FAILOVER},
+       {RTE_STR(SCHEDULER_MODE_NAME_MULTI_CORE),
+                       CDEV_SCHED_MODE_MULTICORE}
 };
 
 const struct scheduler_parse_map scheduler_ordering_map[] = {
@@ -86,75 +97,51 @@ const struct scheduler_parse_map scheduler_ordering_map[] = {
                {"disable", 0}
 };
 
-static int
-attach_init_slaves(uint8_t scheduler_id,
-               const uint8_t *slaves, const uint8_t nb_slaves)
-{
-       uint8_t i;
-
-       for (i = 0; i < nb_slaves; i++) {
-               struct rte_cryptodev *dev =
-                               rte_cryptodev_pmd_get_dev(slaves[i]);
-               int status = rte_cryptodev_scheduler_slave_attach(
-                               scheduler_id, slaves[i]);
-
-               if (status < 0 || !dev) {
-                       CS_LOG_ERR("Failed to attach slave cryptodev "
-                                       "%u.\n", slaves[i]);
-                       return status;
-               }
-
-               RTE_LOG(INFO, PMD, "  Attached Slave %s\n", dev->data->name);
-       }
-
-       return 0;
-}
-
 static int
 cryptodev_scheduler_create(const char *name,
-       struct scheduler_init_params *init_params)
+               struct rte_vdev_device *vdev,
+               struct scheduler_init_params *init_params)
 {
-       char crypto_dev_name[RTE_CRYPTODEV_NAME_MAX_LEN] = {0};
        struct rte_cryptodev *dev;
        struct scheduler_ctx *sched_ctx;
        uint32_t i;
        int ret;
 
-       if (init_params->def_p.name[0] == '\0') {
-               ret = rte_cryptodev_pmd_create_dev_name(
-                               crypto_dev_name,
-                               RTE_STR(CRYPTODEV_NAME_SCHEDULER_PMD));
+       if (init_params->def_p.name[0] == '\0')
+               snprintf(init_params->def_p.name,
+                               sizeof(init_params->def_p.name),
+                               "%s", name);
 
-               if (ret < 0) {
-                       CS_LOG_ERR("failed to create unique name");
-                       return ret;
-               }
-       } else {
-               strncpy(crypto_dev_name, init_params->def_p.name,
-                               RTE_CRYPTODEV_NAME_MAX_LEN - 1);
-       }
-
-       dev = rte_cryptodev_pmd_virtual_dev_init(crypto_dev_name,
+       dev = rte_cryptodev_vdev_pmd_init(init_params->def_p.name,
                        sizeof(struct scheduler_ctx),
-                       init_params->def_p.socket_id);
+                       init_params->def_p.socket_id,
+                       vdev);
        if (dev == NULL) {
                CS_LOG_ERR("driver %s: failed to create cryptodev vdev",
                        name);
                return -EFAULT;
        }
 
-       dev->dev_type = RTE_CRYPTODEV_SCHEDULER_PMD;
+       dev->driver_id = cryptodev_driver_id;
        dev->dev_ops = rte_crypto_scheduler_pmd_ops;
 
        sched_ctx = dev->data->dev_private;
        sched_ctx->max_nb_queue_pairs =
                        init_params->def_p.max_nb_queue_pairs;
 
-       ret = attach_init_slaves(dev->data->dev_id, init_params->slaves,
-                       init_params->nb_slaves);
-       if (ret < 0) {
-               rte_cryptodev_pmd_release_device(dev);
-               return ret;
+       if (init_params->mode == CDEV_SCHED_MODE_MULTICORE) {
+               uint16_t i;
+
+               sched_ctx->nb_wc = 0;
+
+               for (i = 0; i < RTE_CRYPTODEV_SCHEDULER_MAX_NB_WORKER_CORES; i++) {
+                       if (init_params->wcmask & (1ULL << i)) {
+                               sched_ctx->wc_pool[sched_ctx->nb_wc++] = i;
+                               RTE_LOG(INFO, PMD,
+                                       "  Worker core[%u]=%u added\n",
+                                       sched_ctx->nb_wc-1, i);
+                       }
+               }
        }
 
        if (init_params->mode > CDEV_SCHED_MODE_USERDEFINED &&
@@ -189,18 +176,56 @@ cryptodev_scheduler_create(const char *name,
                break;
        }
 
+       for (i = 0; i < init_params->nb_slaves; i++) {
+               sched_ctx->init_slave_names[sched_ctx->nb_init_slaves] =
+                       rte_zmalloc_socket(
+                               NULL,
+                               RTE_CRYPTODEV_SCHEDULER_NAME_MAX_LEN, 0,
+                               SOCKET_ID_ANY);
+
+               if (!sched_ctx->init_slave_names[
+                               sched_ctx->nb_init_slaves]) {
+                       CS_LOG_ERR("driver %s: Insufficient memory",
+                                       name);
+                       return -ENOMEM;
+               }
+
+               strncpy(sched_ctx->init_slave_names[
+                                       sched_ctx->nb_init_slaves],
+                               init_params->slave_names[i],
+                               RTE_CRYPTODEV_SCHEDULER_NAME_MAX_LEN - 1);
+
+               sched_ctx->nb_init_slaves++;
+       }
+
+       /*
+        * Initialize capabilities structure as an empty structure,
+        * in case device information is requested when no slaves are attached
+        */
+       sched_ctx->capabilities = rte_zmalloc_socket(NULL,
+                       sizeof(struct rte_cryptodev_capabilities),
+                       0, SOCKET_ID_ANY);
+
+       if (!sched_ctx->capabilities) {
+               RTE_LOG(ERR, PMD, "Not enough memory for capability "
+                               "information\n");
+               return -ENOMEM;
+       }
+
        return 0;
 }
 
 static int
-cryptodev_scheduler_remove(const char *name)
+cryptodev_scheduler_remove(struct rte_vdev_device *vdev)
 {
+       const char *name;
        struct rte_cryptodev *dev;
        struct scheduler_ctx *sched_ctx;
 
-       if (name == NULL)
+       if (vdev == NULL)
                return -EINVAL;
 
+       name = rte_vdev_device_name(vdev);
        dev = rte_cryptodev_pmd_get_named_dev(name);
        if (dev == NULL)
                return -EINVAL;
@@ -221,22 +246,6 @@ cryptodev_scheduler_remove(const char *name)
        return 0;
 }
 
-static uint8_t
-number_of_sockets(void)
-{
-       int sockets = 0;
-       int i;
-       const struct rte_memseg *ms = rte_eal_get_physmem_layout();
-
-       for (i = 0; ((i < RTE_MAX_MEMSEG) && (ms[i].addr != NULL)); i++) {
-               if (sockets < ms[i].socket_id)
-                       sockets = ms[i].socket_id;
-       }
-
-       /* Number of sockets = maximum socket_id + 1 */
-       return ++sockets;
-}
-
 /** Parse integer from integer argument */
 static int
 parse_integer_arg(const char *key __rte_unused,
@@ -253,6 +262,43 @@ parse_integer_arg(const char *key __rte_unused,
        return 0;
 }
 
+/** Parse integer from hexadecimal integer argument */
+static int
+parse_coremask_arg(const char *key __rte_unused,
+               const char *value, void *extra_args)
+{
+       struct scheduler_init_params *params = extra_args;
+
+       params->wcmask = strtoull(value, NULL, 16);
+
+       return 0;
+}
+
+/** Parse integer from list of integers argument */
+static int
+parse_corelist_arg(const char *key __rte_unused,
+               const char *value, void *extra_args)
+{
+       struct scheduler_init_params *params = extra_args;
+
+       params->wcmask = 0ULL;
+
+       const char *token = value;
+
+       while (isdigit(token[0])) {
+               char *rval;
+               unsigned int core = strtoul(token, &rval, 10);
+
+               params->wcmask |= 1ULL << core;
+               token = (const char *)rval;
+               if (token[0] == '\0')
+                       break;
+               token++;
+       }
+
+       return 0;
+}
+
 /** Parse name */
 static int
 parse_name_arg(const char *key __rte_unused,
@@ -278,21 +324,14 @@ parse_slave_arg(const char *key __rte_unused,
                const char *value, void *extra_args)
 {
        struct scheduler_init_params *param = extra_args;
-       struct rte_cryptodev *dev =
-                       rte_cryptodev_pmd_get_named_dev(value);
 
-       if (!dev) {
-               RTE_LOG(ERR, PMD, "Invalid slave name %s.\n", value);
-               return -EINVAL;
-       }
-
-       if (param->nb_slaves >= RTE_CRYPTODEV_SCHEDULER_MAX_NB_SLAVES - 1) {
+       if (param->nb_slaves >= RTE_CRYPTODEV_SCHEDULER_MAX_NB_SLAVES) {
                CS_LOG_ERR("Too many slaves.\n");
                return -ENOMEM;
        }
 
-       param->slaves[param->nb_slaves] = dev->data->dev_id;
-       param->nb_slaves++;
+       strncpy(param->slave_names[param->nb_slaves++], value,
+                       RTE_CRYPTODEV_SCHEDULER_NAME_MAX_LEN - 1);
 
        return 0;
 }
@@ -379,6 +418,18 @@ scheduler_parse_init_params(struct scheduler_init_params *params,
                if (ret < 0)
                        goto free_kvlist;
 
+               ret = rte_kvargs_process(kvlist, RTE_CRYPTODEV_VDEV_COREMASK,
+                               &parse_coremask_arg,
+                               params);
+               if (ret < 0)
+                       goto free_kvlist;
+
+               ret = rte_kvargs_process(kvlist, RTE_CRYPTODEV_VDEV_CORELIST,
+                               &parse_corelist_arg,
+                               params);
+               if (ret < 0)
+                       goto free_kvlist;
+
                ret = rte_kvargs_process(kvlist, RTE_CRYPTODEV_VDEV_NAME,
                                &parse_name_arg,
                                &params->def_p);
@@ -399,12 +450,6 @@ scheduler_parse_init_params(struct scheduler_init_params *params,
                                &parse_ordering_arg, params);
                if (ret < 0)
                        goto free_kvlist;
-
-               if (params->def_p.socket_id >= number_of_sockets()) {
-                       CDEV_LOG_ERR("Invalid socket id specified to create "
-                               "the virtual crypto device on");
-                       goto free_kvlist;
-               }
        }
 
 free_kvlist:
@@ -413,7 +458,7 @@ free_kvlist:
 }
 
 static int
-cryptodev_scheduler_probe(const char *name, const char *input_args)
+cryptodev_scheduler_probe(struct rte_vdev_device *vdev)
 {
        struct scheduler_init_params init_params = {
                .def_p = {
@@ -423,14 +468,21 @@ cryptodev_scheduler_probe(const char *name, const char *input_args)
                        ""
                },
                .nb_slaves = 0,
-               .slaves = {0},
                .mode = CDEV_SCHED_MODE_NOT_SET,
-               .enable_ordering = 0
+               .enable_ordering = 0,
+               .slave_names = { {0} }
        };
+       const char *name;
 
-       scheduler_parse_init_params(&init_params, input_args);
+       name = rte_vdev_device_name(vdev);
+       if (name == NULL)
+               return -EINVAL;
 
-       RTE_LOG(INFO, PMD, "Initialising %s on NUMA node %d\n", name,
+       scheduler_parse_init_params(&init_params,
+                                   rte_vdev_device_args(vdev));
+
+       RTE_LOG(INFO, PMD, "Initialising %s on NUMA node %d\n",
+                       name,
                        init_params.def_p.socket_id);
        RTE_LOG(INFO, PMD, "  Max number of queue pairs = %d\n",
                        init_params.def_p.max_nb_queue_pairs);
@@ -439,8 +491,13 @@ cryptodev_scheduler_probe(const char *name, const char *input_args)
        if (init_params.def_p.name[0] != '\0')
                RTE_LOG(INFO, PMD, "  User defined name = %s\n",
                        init_params.def_p.name);
+       if (init_params.wcmask != 0)
+               RTE_LOG(INFO, PMD, "  workers core mask = %"PRIx64"\n",
+                       init_params.wcmask);
 
-       return cryptodev_scheduler_create(name, &init_params);
+       return cryptodev_scheduler_create(name,
+                                       vdev,
+                                       &init_params);
 }
 
 static struct rte_vdev_driver cryptodev_scheduler_pmd_drv = {
@@ -448,6 +505,8 @@ static struct rte_vdev_driver cryptodev_scheduler_pmd_drv = {
        .remove = cryptodev_scheduler_remove
 };
 
+static struct cryptodev_driver scheduler_crypto_drv;
+
 RTE_PMD_REGISTER_VDEV(CRYPTODEV_NAME_SCHEDULER_PMD,
        cryptodev_scheduler_pmd_drv);
 RTE_PMD_REGISTER_PARAM_STRING(CRYPTODEV_NAME_SCHEDULER_PMD,
@@ -455,3 +514,6 @@ RTE_PMD_REGISTER_PARAM_STRING(CRYPTODEV_NAME_SCHEDULER_PMD,
        "max_nb_sessions=<int> "
        "socket_id=<int> "
        "slave=<name>");
+RTE_PMD_REGISTER_CRYPTO_DRIVER(scheduler_crypto_drv,
+               cryptodev_scheduler_pmd_drv,
+               cryptodev_driver_id);