From patchwork Wed Dec 14 23:33:46 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Chautru, Nicolas" X-Patchwork-Id: 120904 X-Patchwork-Delegate: maxime.coquelin@redhat.com Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 8E3BBA0543; Thu, 15 Dec 2022 00:37:12 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id D766242D1F; Thu, 15 Dec 2022 00:37:01 +0100 (CET) Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by mails.dpdk.org (Postfix) with ESMTP id C8C0E40A8A for ; Thu, 15 Dec 2022 00:36:57 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1671061018; x=1702597018; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=eOKSX27gE0a/1xJaps4WwFSfAYfU2ku6uokKRLGERxs=; b=TElg5APeU2d6FdIiBchCA+UiZxr4L2LF2Vire4oPUOEySHt6b0AHL8JJ CA7Hf4wXnCndx+8KVZ/b5SZbzqLjwhwewrwvPVvUVlIWHTY5NPV04gHU2 xVYD4vpeaCWm69yCdas3ZXhlZcOPjczjLHz9Yiy+/d+ecRZlR7d7ZLJc7 mjaVqF0hUg7c1OryIa8Sf7I5jhDulg5CyXvMi5J1X49Z1Tl9cT9zrCC8H 6wGykV21J0E2gFsHbOgWfK9WU9R9EVmZR9ALMFLozIboIsScePeBwLGGy hrBdNDzRp5bphSSwfl2FP5P5ZWOHj3Zc3tznLbCmUsftIOw7ssu3XHfe3 w==; X-IronPort-AV: E=McAfee;i="6500,9779,10561"; a="306181671" X-IronPort-AV: E=Sophos;i="5.96,245,1665471600"; d="scan'208";a="306181671" Received: from orsmga006.jf.intel.com ([10.7.209.51]) by orsmga101.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 14 Dec 2022 15:36:57 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6500,9779,10561"; a="626966066" X-IronPort-AV: E=Sophos;i="5.96,245,1665471600"; d="scan'208";a="626966066" Received: from spr-npg-bds1-eec2.sn.intel.com (HELO spr-npg-bds1-eec2..) ([10.233.181.123]) by orsmga006.jf.intel.com with ESMTP; 14 Dec 2022 15:36:43 -0800 From: Nicolas Chautru To: dev@dpdk.org, maxime.coquelin@redhat.com Cc: hernan.vargas@intel.com, Nicolas Chautru Subject: [PATCH v1 3/3] app/bbdev-test: remove offload cost optional build flag Date: Wed, 14 Dec 2022 23:33:46 +0000 Message-Id: <20221214233346.391973-4-nicolas.chautru@intel.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20221214233346.391973-1-nicolas.chautru@intel.com> References: <20221214233346.391973-1-nicolas.chautru@intel.com> MIME-Version: 1.0 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 Remove the optional build flag for bbdev-test application so that it is enabled by default and having to maintain single build variant. Signed-off-by: Nicolas Chautru Reviewed-by: Maxime Coquelin --- app/test-bbdev/test_bbdev_perf.c | 23 ----------------------- 1 file changed, 23 deletions(-) diff --git a/app/test-bbdev/test_bbdev_perf.c b/app/test-bbdev/test_bbdev_perf.c index b2096525ea..3818b74c79 100644 --- a/app/test-bbdev/test_bbdev_perf.c +++ b/app/test-bbdev/test_bbdev_perf.c @@ -166,7 +166,6 @@ struct thread_params { struct rte_bbdev_fft_op *fft_ops[MAX_BURST]; }; -#ifdef RTE_BBDEV_OFFLOAD_COST /* Stores time statistics */ struct test_time_stats { /* Stores software enqueue total working time */ @@ -188,7 +187,6 @@ struct test_time_stats { /* Stores maximum value of dequeue working time */ uint64_t deq_max_time; }; -#endif typedef int (test_case_function)(struct active_device *ad, struct test_op_params *op_params); @@ -4916,7 +4914,6 @@ validation_test(struct active_device *ad, struct test_op_params *op_params) return validation_latency_test(ad, op_params, false); } -#ifdef RTE_BBDEV_OFFLOAD_COST static int get_bbdev_queue_stats(uint16_t dev_id, uint16_t queue_id, struct rte_bbdev_stats *stats) @@ -5367,19 +5364,11 @@ offload_latency_test_ldpc_enc(struct rte_mempool *mempool, return i; } -#endif static int offload_cost_test(struct active_device *ad, struct test_op_params *op_params) { -#ifndef RTE_BBDEV_OFFLOAD_COST - RTE_SET_USED(ad); - RTE_SET_USED(op_params); - printf("Offload latency test is disabled.\n"); - printf("Set RTE_BBDEV_OFFLOAD_COST to 'y' to turn the test on.\n"); - return TEST_SKIPPED; -#else int iter; uint16_t burst_sz = op_params->burst_sz; const uint16_t num_to_process = op_params->num_to_process; @@ -5492,10 +5481,8 @@ offload_cost_test(struct active_device *ad, stats.dequeue_err_count); return TEST_SUCCESS; -#endif } -#ifdef RTE_BBDEV_OFFLOAD_COST static int offload_latency_empty_q_test_dec(uint16_t dev_id, uint16_t queue_id, const uint16_t num_to_process, uint16_t burst_sz, @@ -5563,19 +5550,10 @@ offload_latency_empty_q_test_enc(uint16_t dev_id, uint16_t queue_id, return i; } -#endif - static int offload_latency_empty_q_test(struct active_device *ad, struct test_op_params *op_params) { -#ifndef RTE_BBDEV_OFFLOAD_COST - RTE_SET_USED(ad); - RTE_SET_USED(op_params); - printf("Offload latency empty dequeue test is disabled.\n"); - printf("Set RTE_BBDEV_OFFLOAD_COST to 'y' to turn the test on.\n"); - return TEST_SKIPPED; -#else int iter; uint64_t deq_total_time, deq_min_time, deq_max_time; uint16_t burst_sz = op_params->burst_sz; @@ -5625,7 +5603,6 @@ offload_latency_empty_q_test(struct active_device *ad, rte_get_tsc_hz()); return TEST_SUCCESS; -#endif } static int