From patchwork Tue Jul 5 08:16:47 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Lingli Chen X-Patchwork-Id: 113673 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 A2BE1A0542; Tue, 5 Jul 2022 10:54:12 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 745AD4021D; Tue, 5 Jul 2022 10:54:12 +0200 (CEST) Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by mails.dpdk.org (Postfix) with ESMTP id B66B4400D7 for ; Tue, 5 Jul 2022 10:54:10 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1657011250; x=1688547250; h=from:to:cc:subject:date:message-id:mime-version: content-transfer-encoding; bh=/M0B29t2CVyArRZmef2xUv0cJjEx/2kQIh9eGoZlZqc=; b=fyI0v58QI4UsUay8TtkYUCJ5ZkKApOPFh7By8mZ6MEgoxG4eZDAOIFOF EFzc9rxARyxHHfmX5T9HB1/gLrBTApprNFiHfqP2RXnwwATvCiw6ua4R6 mHLzc5avZYxDOMUo6rAnQCoZ0v/HIE4K73OWZkBCTxseVQNY7gfl5k8np C9ykDxsEuMaHgC4TEE28KWaS/qpD2Y83RHb03wu41YfxmyCuEDioLO/lN V6ryOW7DKh0hmCtFgtKLwxuKFa0DWl6zSxWolw0lnRcw8b0p1CYGZB3sB Qa8d7wKNN0n1eJjf6Ee1J276jaMHYd3C3HHRn2GJILVcXml4vPctURU3X w==; X-IronPort-AV: E=McAfee;i="6400,9594,10398"; a="308835443" X-IronPort-AV: E=Sophos;i="5.92,245,1650956400"; d="scan'208";a="308835443" Received: from orsmga007.jf.intel.com ([10.7.209.58]) by fmsmga101.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 05 Jul 2022 01:54:06 -0700 X-IronPort-AV: E=Sophos;i="5.92,245,1650956400"; d="scan'208";a="590313952" Received: from unknown (HELO localhost.localdomain) ([10.239.252.99]) by orsmga007-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 05 Jul 2022 01:54:04 -0700 From: Lingli Chen To: dts@dpdk.org Cc: Lingli Chen Subject: [dts][PATCH V1] tests/distributor: modify script add novector path expect Date: Tue, 5 Jul 2022 04:16:47 -0400 Message-Id: <20220705081647.35533-1-linglix.chen@intel.com> X-Mailer: git-send-email 2.17.1 MIME-Version: 1.0 X-BeenThere: dts@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: test suite reviews and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dts-bounces@dpdk.org The “--force-max-simd-bitwidth=64” is relatively new to DPDK (novector code path) and severely limits the performance of the burst mode of the distributor. So modify script to add novector path expect Signed-off-by: Lingli Chen --- test_plans/distributor_test_plan.rst | 4 ++++ tests/TestSuite_distributor.py | 12 +++++++++--- 2 files changed, 13 insertions(+), 3 deletions(-) diff --git a/test_plans/distributor_test_plan.rst b/test_plans/distributor_test_plan.rst index 7e61bef4..12fd9e9f 100644 --- a/test_plans/distributor_test_plan.rst +++ b/test_plans/distributor_test_plan.rst @@ -40,6 +40,10 @@ Compared CPU cycles for normal distributor and burst API Verify burst distributor API cost much less cycles then normal version +The novector path severely limits the performance of the burst mode of the distributor. +so novector: expect single mode > burst mode +vector: expect single mode > burst mode * 2 + Test case: Distribute with maximum workers ========================================== Start distributor sample with 63(0xeffffffffffffffff0) workers diff --git a/tests/TestSuite_distributor.py b/tests/TestSuite_distributor.py index 7b41eb01..6878edbd 100644 --- a/tests/TestSuite_distributor.py +++ b/tests/TestSuite_distributor.py @@ -65,9 +65,15 @@ class TestDistributor(TestCase): ) self.dut.send_expect("quit", "# ") self.verify("Test OK" in out, "Test failed") - self.verify( - cycles_single > cycles_burst * 2, "Burst performance should be much better" - ) + if "force-max-simd-bitwidth=64" in eal_para: + self.verify( + cycles_single > cycles_burst, "Burst performance should be much better" + ) + else: + self.verify( + cycles_single > cycles_burst * 2, + "Burst performance should be much better", + ) def test_perf_distributor(self): """