[dpdk-dev] [PATCH] crypto/scheduler: fix slave init attach

Fan Zhang roy.fan.zhang at intel.com
Wed Apr 19 15:42:17 CEST 2017


Fixes: 881851676940 ("crypto/scheduler: improve commandline parsing")

This patch fix the slaves attaching initially.

Originally, the scheduler will try to attach the slaves passed by the
cmdline option in the initial stage, this may break the sequence
of the slaves inside the scheduler.

Signed-off-by: Fan Zhang <roy.fan.zhang at intel.com>
---
 drivers/crypto/scheduler/scheduler_pmd.c | 21 ---------------------
 1 file changed, 21 deletions(-)

diff --git a/drivers/crypto/scheduler/scheduler_pmd.c b/drivers/crypto/scheduler/scheduler_pmd.c
index f2a1d2a..2a92d1e 100644
--- a/drivers/crypto/scheduler/scheduler_pmd.c
+++ b/drivers/crypto/scheduler/scheduler_pmd.c
@@ -160,27 +160,6 @@ cryptodev_scheduler_create(const char *name,
 	}
 
 	for (i = 0; i < init_params->nb_slaves; i++) {
-		struct rte_cryptodev *slave_dev =
-				rte_cryptodev_pmd_get_named_dev(
-					init_params->slave_names[i]);
-
-		/* if the slave dev exists in the system, attach it */
-		if (slave_dev) {
-			int ret = rte_cryptodev_scheduler_slave_attach(
-				dev->data->dev_id,
-				slave_dev->data->dev_id);
-
-			if (ret < 0)
-				return ret;
-
-			CS_LOG_INFO("Scheduler %s attached slave %s\n",
-					dev->data->name,
-					init_params->slave_names[i]);
-
-			continue;
-		}
-
-		/* if the slave doesn't exist, store and attach later*/
 		sched_ctx->init_slave_names[sched_ctx->nb_init_slaves] =
 			rte_zmalloc_socket(
 				NULL,
-- 
2.7.4



More information about the dev mailing list