From patchwork Tue Oct 26 07:31:45 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Jiale, SongX" X-Patchwork-Id: 102864 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 CC227A0C47; Tue, 26 Oct 2021 09:31:58 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id A29D74003E; Tue, 26 Oct 2021 09:31:58 +0200 (CEST) Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by mails.dpdk.org (Postfix) with ESMTP id 58D7B4003E for ; Tue, 26 Oct 2021 09:31:57 +0200 (CEST) X-IronPort-AV: E=McAfee;i="6200,9189,10148"; a="227297499" X-IronPort-AV: E=Sophos;i="5.87,182,1631602800"; d="scan'208";a="227297499" Received: from orsmga007.jf.intel.com ([10.7.209.58]) by fmsmga102.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 26 Oct 2021 00:31:55 -0700 X-IronPort-AV: E=Sophos;i="5.87,182,1631602800"; d="scan'208";a="486029125" Received: from unknown (HELO dpdk-zhaohy-t.sh.intel.com) ([10.240.183.68]) by orsmga007-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 26 Oct 2021 00:31:53 -0700 From: Jiale Song To: dts@dpdk.org Cc: Jiale Song Date: Tue, 26 Oct 2021 15:31:45 +0800 Message-Id: <1635233506-142944-1-git-send-email-songx.jiale@intel.com> X-Mailer: git-send-email 1.8.3.1 Subject: [dts] [PATCH V2 1/2] tests/cvl_switch_filter: delete the placeholder fdir rule 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 Sender: "dts" in non-pipeline mode, "priority" is ignored, a flow rule can be created as a flow director rule or a switch rule depends on its pattern/action. if a rule is supported by switch or fdir at the same time, it will be created in the switch table first. we no longer need to populate the fdir table first dpdk commit: e4a0a7599d974f05665fec3e4c251659f0b11453 Signed-off-by: Jiale Song --- test_plans/cvl_switch_filter_test_plan.rst | 2 +- tests/TestSuite_cvl_switch_filter.py | 5 +---- 2 files changed, 2 insertions(+), 5 deletions(-) mode change 100644 => 100755 tests/TestSuite_cvl_switch_filter.py diff --git a/test_plans/cvl_switch_filter_test_plan.rst b/test_plans/cvl_switch_filter_test_plan.rst index ae29e64e..9a53e9cb 100644 --- a/test_plans/cvl_switch_filter_test_plan.rst +++ b/test_plans/cvl_switch_filter_test_plan.rst @@ -181,7 +181,7 @@ Prerequisites 6. Launch dpdk with the following arguments in non-pipeline mode:: - ./x86_64-native-linuxapp-gcc/app/testpmd -c 0xf -n 4 -a 0000:18:00.0 --log-level="ice,8" -- -i --txq=16 --rxq=16 --cmdline-file=testpmd_fdir_rules + ./x86_64-native-linuxapp-gcc/app/testpmd -c 0xf -n 4 -a 0000:18:00.0 --log-level="ice,8" -- -i --txq=16 --rxq=16 testpmd> port config 0 rss-hash-key ipv4 1234abcd1234abcd1234abcd1234abcd1234abcd1234abcd1234abcd1234abcd1234abcd1234abcd1234abcd1234abcd1234abcd testpmd> set fwd rxonly testpmd> set verbose 1 diff --git a/tests/TestSuite_cvl_switch_filter.py b/tests/TestSuite_cvl_switch_filter.py old mode 100644 new mode 100755 index 4837f16c..59d95849 --- a/tests/TestSuite_cvl_switch_filter.py +++ b/tests/TestSuite_cvl_switch_filter.py @@ -3001,7 +3001,7 @@ class CVLSwitchFilterTest(TestCase): """ #Prepare testpmd EAL and parameters all_eal_param = self.dut.create_eal_parameters(cores='1S/4C/1T', ports=[0]) - command = self.path + all_eal_param + " --log-level=\"ice,8\" -- -i --rxq=16 --txq=16 --cmdline-file=%s" % self.fdir_file + command = self.path + all_eal_param + " --log-level=\"ice,8\" -- -i --rxq=16 --txq=16 " # command = "./%s/app/testpmd %s --log-level=\"ice,8\" -- -i %s" % (self.dut.target, all_eal_param, "--rxq=16 --txq=16") return command @@ -3178,9 +3178,6 @@ class CVLSwitchFilterTest(TestCase): out_lines = out.splitlines() res = filter(bool, map(p_spec.match, out_lines)) result = [i.group(1) for i in res] - if is_non_pipeline: - #remove 15360 fdir rules id - del result[:self.fdir_rule_number] if is_need_rss_rule: #remove rss rule id del result[0] From patchwork Tue Oct 26 07:31:46 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Jiale, SongX" X-Patchwork-Id: 102865 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 C64D0A0C4B; Tue, 26 Oct 2021 09:31:59 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id BD981410DA; Tue, 26 Oct 2021 09:31:59 +0200 (CEST) Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by mails.dpdk.org (Postfix) with ESMTP id 3173C4003E for ; Tue, 26 Oct 2021 09:31:58 +0200 (CEST) X-IronPort-AV: E=McAfee;i="6200,9189,10148"; a="227297510" X-IronPort-AV: E=Sophos;i="5.87,182,1631602800"; d="scan'208";a="227297510" Received: from orsmga007.jf.intel.com ([10.7.209.58]) by fmsmga102.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 26 Oct 2021 00:31:57 -0700 X-IronPort-AV: E=Sophos;i="5.87,182,1631602800"; d="scan'208";a="486029147" Received: from unknown (HELO dpdk-zhaohy-t.sh.intel.com) ([10.240.183.68]) by orsmga007-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 26 Oct 2021 00:31:55 -0700 From: Jiale Song To: dts@dpdk.org Cc: Jiale Song Date: Tue, 26 Oct 2021 15:31:46 +0800 Message-Id: <1635233506-142944-2-git-send-email-songx.jiale@intel.com> X-Mailer: git-send-email 1.8.3.1 In-Reply-To: <1635233506-142944-1-git-send-email-songx.jiale@intel.com> References: <1635233506-142944-1-git-send-email-songx.jiale@intel.com> Subject: [dts] [PATCH V2 2/2] tests/cvl_switch_filter_pppoe: delete the placeholder fdir rule 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 Sender: "dts" Signed-off-by: Jiale Song --- test_plans/cvl_switch_filter_pppoe_test_plan.rst | 2 +- tests/TestSuite_cvl_switch_filter_pppoe.py | 5 +---- 2 files changed, 2 insertions(+), 5 deletions(-) mode change 100644 => 100755 tests/TestSuite_cvl_switch_filter_pppoe.py diff --git a/test_plans/cvl_switch_filter_pppoe_test_plan.rst b/test_plans/cvl_switch_filter_pppoe_test_plan.rst index 897e8c6c..ceef7760 100644 --- a/test_plans/cvl_switch_filter_pppoe_test_plan.rst +++ b/test_plans/cvl_switch_filter_pppoe_test_plan.rst @@ -203,7 +203,7 @@ Prerequisites 6. Launch dpdk with the following arguments in non-pipeline mode:: - ./x86_64-native-linuxapp-gcc/app/testpmd -c 0xf -n 4 -a 0000:18:00.0 --log-level="ice,8" -- -i --txq=16 --rxq=16 --cmdline-file=testpmd_fdir_rules + ./x86_64-native-linuxapp-gcc/app/testpmd -c 0xf -n 4 -a 0000:18:00.0 --log-level="ice,8" -- -i --txq=16 --rxq=16 testpmd> port config 0 rss-hash-key ipv4 1234abcd1234abcd1234abcd1234abcd1234abcd1234abcd1234abcd1234abcd1234abcd1234abcd1234abcd1234abcd1234abcd testpmd> set fwd rxonly testpmd> set verbose 1 diff --git a/tests/TestSuite_cvl_switch_filter_pppoe.py b/tests/TestSuite_cvl_switch_filter_pppoe.py old mode 100644 new mode 100755 index a8058f94..c14ea0e3 --- a/tests/TestSuite_cvl_switch_filter_pppoe.py +++ b/tests/TestSuite_cvl_switch_filter_pppoe.py @@ -2296,7 +2296,7 @@ class CVLSwitchFilterPPPOETest(TestCase): """ #Prepare testpmd EAL and parameters all_eal_param = self.dut.create_eal_parameters(cores='1S/4C/1T', ports=[0]) - command = self.path + all_eal_param + " --log-level=\"ice,8\" -- -i --rxq=16 --txq=16 --cmdline-file=%s" % self.fdir_file + command = self.path + all_eal_param + " --log-level=\"ice,8\" -- -i --rxq=16 --txq=16 " # command = "./%s/app/testpmd %s --log-level=\"ice,8\" -- -i %s" % (self.dut.target, all_eal_param, "--rxq=16 --txq=16") return command @@ -2473,9 +2473,6 @@ class CVLSwitchFilterPPPOETest(TestCase): out_lines = out.splitlines() res = filter(bool, map(p_spec.match, out_lines)) result = [i.group(1) for i in res] - if is_non_pipeline: - #remove 15360 fdir rules id - del result[:self.fdir_rule_number] if is_need_rss_rule: #remove rss rule id del result[0]