From patchwork Sat Oct 9 08:11:44 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Jiale, SongX" X-Patchwork-Id: 100907 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 7596AA0C41; Sat, 9 Oct 2021 09:58:12 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 48D6540040; Sat, 9 Oct 2021 09:58:12 +0200 (CEST) Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by mails.dpdk.org (Postfix) with ESMTP id EBD284003C for ; Sat, 9 Oct 2021 09:58:09 +0200 (CEST) X-IronPort-AV: E=McAfee;i="6200,9189,10131"; a="224053384" X-IronPort-AV: E=Sophos;i="5.85,360,1624345200"; d="scan'208";a="224053384" Received: from orsmga008.jf.intel.com ([10.7.209.65]) by fmsmga102.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 09 Oct 2021 00:58:08 -0700 X-IronPort-AV: E=Sophos;i="5.85,360,1624345200"; d="scan'208";a="489767964" Received: from unknown (HELO dpdk-zhaohy-t.sh.intel.com) ([10.240.183.68]) by orsmga008-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 09 Oct 2021 00:58:07 -0700 From: Jiale Song To: dts@dpdk.org Cc: Jiale Song Date: Sat, 9 Oct 2021 16:11:44 +0800 Message-Id: <1633767105-231037-1-git-send-email-songx.jiale@intel.com> X-Mailer: git-send-email 1.8.3.1 Subject: [dts] [PATCH V1 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 Signed-off-by: Jiale Song --- test_plans/cvl_switch_filter_test_plan.rst | 8 ++------ tests/TestSuite_cvl_switch_filter.py | 5 +---- 2 files changed, 3 insertions(+), 10 deletions(-) mode change 100644 => 100755 test_plans/cvl_switch_filter_test_plan.rst 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 old mode 100644 new mode 100755 index b8f7e14..0f04371 --- a/test_plans/cvl_switch_filter_test_plan.rst +++ b/test_plans/cvl_switch_filter_test_plan.rst @@ -51,11 +51,7 @@ for device's flow director engine. 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 fdir table first. Therefore, to test switch filter in non-pipeline mode, we need to -fill the fdir table first, and then the rules are created in the switch filter table. The capacity of fdir -table is 16K, of which 14K is shared by all pfs and vfs, and the remaining 2K is gurantee for pfs. If 4*25G -NIC, the gurantee for each pf is 512. If 2*100G NIC, the gurantee of each pf is 1024. so 1 pf can create at -most 14848 rules on 4 ports card and 15360 rules on 2 ports card. +will be created in the switch table first. Pattern and input set --------------------- @@ -181,7 +177,7 @@ Prerequisites 6. Launch dpdk with the following arguments in non-pipeline mode:: - ./x86_64-native-linuxapp-gcc/app/testpmd -c 0xf -n 4 -w 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 -w 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 5ec8b18..36420d1 --- a/tests/TestSuite_cvl_switch_filter.py +++ b/tests/TestSuite_cvl_switch_filter.py @@ -2607,7 +2607,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 @@ -2784,9 +2784,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]