From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 9AFFC41DB8; Thu, 2 Mar 2023 21:23:37 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 4976842D63; Thu, 2 Mar 2023 21:22:15 +0100 (CET) Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by mails.dpdk.org (Postfix) with ESMTP id 3968742D33 for ; Thu, 2 Mar 2023 21:22:06 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1677788526; x=1709324526; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=p8ENaeeRE0HqMWbEYSs3ihOj54vg71NUCVc6wydhFZI=; b=mTeSv9jSIutMdmaNTf5l+yHPePiw+krowc5kcqgRZE8DezzaVfGEJDNa eu1wAF1h18ZzoKcA9iBXCdzVqdvk8M7/cIdHUCtUWbA6i6RCoF8KjWE1A zvsYSI6ZlWUdpCR/b7zGq12OhM49ZusAMqDuqFZHONTIX2PXru5GCAPWv XAbw65XQmEaGIaQFRVEK7ZyFDJ6GQOAzLanooKEZCY9B8bUP0sNNdozoa J0rblXOoZKaaILlu796RHNfFj7ch1QB6/ChpMGucyhlN7jzw8duhuFSRU jGftqRqIfLZQp5cYFpz5Cs8SCVB4xbOZzJu9Nr+VHMYC2GFSZn/oOHoOA A==; X-IronPort-AV: E=McAfee;i="6500,9779,10637"; a="334871227" X-IronPort-AV: E=Sophos;i="5.98,228,1673942400"; d="scan'208";a="334871227" Received: from orsmga008.jf.intel.com ([10.7.209.65]) by fmsmga103.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 02 Mar 2023 12:22:05 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6500,9779,10637"; a="705398133" X-IronPort-AV: E=Sophos;i="5.98,228,1673942400"; d="scan'208";a="705398133" Received: from unknown (HELO csl-npg-qt0.la.intel.com) ([10.233.181.103]) by orsmga008.jf.intel.com with ESMTP; 02 Mar 2023 12:22:05 -0800 From: Hernan Vargas To: dev@dpdk.org, gakhil@marvell.com, trix@redhat.com, maxime.coquelin@redhat.com Cc: nicolas.chautru@intel.com, qi.z.zhang@intel.com, Hernan Vargas Subject: [PATCH v3 14/17] test/bbdev: remove iteration count check Date: Thu, 2 Mar 2023 12:22:08 -0800 Message-Id: <20230302202211.170017-15-hernan.vargas@intel.com> X-Mailer: git-send-email 2.37.1 In-Reply-To: <20230302202211.170017-1-hernan.vargas@intel.com> References: <20230302202211.170017-1-hernan.vargas@intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org To make the test compatible with devices that do not support early termination, the iteration count assert can be removed. Signed-off-by: Hernan Vargas Reviewed-by: Maxime Coquelin --- app/test-bbdev/test_bbdev_perf.c | 24 ++++++++---------------- 1 file changed, 8 insertions(+), 16 deletions(-) diff --git a/app/test-bbdev/test_bbdev_perf.c b/app/test-bbdev/test_bbdev_perf.c index 635c0a5eab48..da5c3a4bc9a7 100644 --- a/app/test-bbdev/test_bbdev_perf.c +++ b/app/test-bbdev/test_bbdev_perf.c @@ -2300,7 +2300,7 @@ validate_op_so_chain(struct rte_bbdev_op_data *op, static int validate_dec_op(struct rte_bbdev_dec_op **ops, const uint16_t n, - struct rte_bbdev_dec_op *ref_op, const int vector_mask) + struct rte_bbdev_dec_op *ref_op) { unsigned int i; int ret; @@ -2311,17 +2311,12 @@ validate_dec_op(struct rte_bbdev_dec_op **ops, const uint16_t n, struct rte_bbdev_op_turbo_dec *ops_td; struct rte_bbdev_op_data *hard_output; struct rte_bbdev_op_data *soft_output; - struct rte_bbdev_op_turbo_dec *ref_td = &ref_op->turbo_dec; for (i = 0; i < n; ++i) { ops_td = &ops[i]->turbo_dec; hard_output = &ops_td->hard_output; soft_output = &ops_td->soft_output; - if (vector_mask & TEST_BBDEV_VF_EXPECTED_ITER_COUNT) - TEST_ASSERT(ops_td->iter_count <= ref_td->iter_count, - "Returned iter_count (%d) > expected iter_count (%d)", - ops_td->iter_count, ref_td->iter_count); ret = check_dec_status_and_ordering(ops[i], i, ref_op->status); TEST_ASSERT_SUCCESS(ret, "Checking status and ordering for decoder failed"); @@ -3070,8 +3065,7 @@ dequeue_event_callback(uint16_t dev_id, if (test_vector.op_type == RTE_BBDEV_OP_TURBO_DEC) { struct rte_bbdev_dec_op *ref_op = tp->op_params->ref_dec_op; - ret = validate_dec_op(tp->dec_ops, num_ops, ref_op, - tp->op_params->vector_mask); + ret = validate_dec_op(tp->dec_ops, num_ops, ref_op); /* get the max of iter_count for all dequeued ops */ for (i = 0; i < num_ops; ++i) tp->iter_count = RTE_MAX( @@ -3683,8 +3677,7 @@ throughput_pmd_lcore_dec(void *arg) } if (test_vector.op_type != RTE_BBDEV_OP_NONE) { - ret = validate_dec_op(ops_deq, num_ops, ref_op, - tp->op_params->vector_mask); + ret = validate_dec_op(ops_deq, num_ops, ref_op); TEST_ASSERT_SUCCESS(ret, "Validation failed!"); } @@ -4743,7 +4736,7 @@ throughput_test(struct active_device *ad, static int latency_test_dec(struct rte_mempool *mempool, struct test_buffers *bufs, struct rte_bbdev_dec_op *ref_op, - int vector_mask, uint16_t dev_id, uint16_t queue_id, + uint16_t dev_id, uint16_t queue_id, const uint16_t num_to_process, uint16_t burst_sz, uint64_t *total_time, uint64_t *min_time, uint64_t *max_time, bool disable_et) { @@ -4809,8 +4802,7 @@ latency_test_dec(struct rte_mempool *mempool, *total_time += last_time; if (test_vector.op_type != RTE_BBDEV_OP_NONE) { - ret = validate_dec_op(ops_deq, burst_sz, ref_op, - vector_mask); + ret = validate_dec_op(ops_deq, burst_sz, ref_op); TEST_ASSERT_SUCCESS(ret, "Validation failed!"); } @@ -5175,9 +5167,9 @@ validation_latency_test(struct active_device *ad, if (op_type == RTE_BBDEV_OP_TURBO_DEC) iter = latency_test_dec(op_params->mp, bufs, - op_params->ref_dec_op, op_params->vector_mask, - ad->dev_id, queue_id, num_to_process, - burst_sz, &total_time, &min_time, &max_time, latency_flag); + op_params->ref_dec_op, ad->dev_id, queue_id, + num_to_process, burst_sz, &total_time, + &min_time, &max_time, latency_flag); else if (op_type == RTE_BBDEV_OP_LDPC_ENC) iter = latency_test_ldpc_enc(op_params->mp, bufs, op_params->ref_enc_op, ad->dev_id, queue_id, -- 2.37.1