From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by dpdk.org (Postfix) with ESMTP id 15AD7FADB for ; Thu, 9 Feb 2017 19:35:38 +0100 (CET) Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by fmsmga103.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 09 Feb 2017 10:35:36 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.35,137,1484035200"; d="scan'208";a="1105266172" Received: from silpixa00381633.ir.intel.com (HELO silpixa00381633.ger.corp.intel.com) ([10.237.222.114]) by fmsmga001.fm.intel.com with ESMTP; 09 Feb 2017 10:35:36 -0800 From: Fan Zhang To: dev@dpdk.org Cc: pablo.de.lara.guarch@intel.com Date: Thu, 9 Feb 2017 18:36:41 +0000 Message-Id: <1486665401-14923-1-git-send-email-roy.fan.zhang@intel.com> X-Mailer: git-send-email 2.7.4 Subject: [dpdk-dev] [PATCH v2] crypto/scheduler: fix initialization X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 09 Feb 2017 18:35:39 -0000 Fixes the wrong slave initialization issue on start-up Fixes: 100e4f7("add round-robin mode") Signed-off-by: Fan Zhang --- v2: remove committed lines of code not relating to the topic drivers/crypto/scheduler/scheduler_roundrobin.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/crypto/scheduler/scheduler_roundrobin.c b/drivers/crypto/scheduler/scheduler_roundrobin.c index 7abdd29..1f5448e 100644 --- a/drivers/crypto/scheduler/scheduler_roundrobin.c +++ b/drivers/crypto/scheduler/scheduler_roundrobin.c @@ -350,7 +350,6 @@ static int scheduler_start(struct rte_cryptodev *dev) { struct scheduler_ctx *sched_ctx = dev->data->dev_private; - uint16_t i; for (i = 0; i < dev->data->nb_queue_pairs; i++) { @@ -358,14 +357,13 @@ scheduler_start(struct rte_cryptodev *dev) struct rr_scheduler_qp_ctx *rr_qp_ctx = qp_ctx->private_qp_ctx; uint32_t j; - uint16_t qp_id = rr_qp_ctx->slaves[0].qp_id; memset(rr_qp_ctx->slaves, 0, MAX_SLAVES_NUM * sizeof(struct scheduler_slave)); for (j = 0; j < sched_ctx->nb_slaves; j++) { rr_qp_ctx->slaves[j].dev_id = - sched_ctx->slaves[i].dev_id; - rr_qp_ctx->slaves[j].qp_id = qp_id; + sched_ctx->slaves[j].dev_id; + rr_qp_ctx->slaves[j].qp_id = i; } rr_qp_ctx->nb_slaves = sched_ctx->nb_slaves; -- 2.7.4