test suite reviews and discussions
 help / color / mirror / Atom feed
From: "Peng, Yuan" <yuan.peng@intel.com>
To: "Xiao, QimaiX" <qimaix.xiao@intel.com>, "dts@dpdk.org" <dts@dpdk.org>
Subject: Re: [dts] [PATCH V3]tests/TestSuite_cvl_fdir: add cvl 25g max rules support
Date: Fri, 19 Jun 2020 08:21:10 +0000	[thread overview]
Message-ID: <BY5PR11MB3893190932E62FF8F340F0FEFE980@BY5PR11MB3893.namprd11.prod.outlook.com> (raw)
In-Reply-To: <MWHPR1101MB225408BC4AF7FFF9243FB9D7FB810@MWHPR1101MB2254.namprd11.prod.outlook.com>

Comment inline

-----Original Message-----
From: dts <dts-bounces@dpdk.org> On Behalf Of Xiao, QimaiX
Sent: Friday, June 12, 2020 10:47 AM
To: dts@dpdk.org
Subject: Re: [dts] [PATCH V3]tests/TestSuite_cvl_fdir: add cvl 25g max rules support

Tested-by: Xiao, QimaiX <qimaix.xiao@intel.com>

Regards,
Xiao Qimai

> -----Original Message-----
> From: Xiao, QimaiX <qimaix.xiao@intel.com>
> Sent: Friday, June 12, 2020 10:38 AM
> To: dts@dpdk.org
> Cc: Xiao, QimaiX <qimaix.xiao@intel.com>
> Subject: [dts][PATCH V3]tests/TestSuite_cvl_fdir: add cvl 25g max 
> rules support
> 
> * 1. cvl25g only support 14848 fdir rules
> * 2. fix max rules case failed caused python crash issue
> * 3. add steps for case test_port_start_stop_reset according to 
> testplan updation
> 
> Signed-off-by: Xiao Qimai <qimaix.xiao@intel.com>
> ---
>  tests/TestSuite_cvl_fdir.py | 32 +++++++++++++++++++++++---------
>  1 file changed, 23 insertions(+), 9 deletions(-)
> 
> diff --git a/tests/TestSuite_cvl_fdir.py b/tests/TestSuite_cvl_fdir.py 
> index
> 12d9ad8..f2252d0 100644
> --- a/tests/TestSuite_cvl_fdir.py
> +++ b/tests/TestSuite_cvl_fdir.py
> @@ -1722,7 +1722,7 @@ class TestCVLFdir(TestCase):
> 
>      def test_void_action(self):
>          rule = "flow create 0 ingress pattern eth dst is 
> 00:11:22:33:44:55 / ipv4 src is 192.168.0.20 dst is 192.168.0.21 ttl is 2 tos is 4 / end actions end"
> -        self.create_fdir_rule(rule, check_stats=False, msg="Invalid input action:
> Invalid argument")
> +        self.create_fdir_rule(rule, check_stats=False)
>          self.check_fdir_rule(stats=False)
> 
>      def _test_unsupported_action(self):
> @@ -2068,6 +2068,11 @@ class TestCVLFdir(TestCase):
>          self.check_fdir_rule(port_id=0, stats=True, rule_list=rule_li)
>          out2 = self.send_pkts_getouput(pkts=pkt, port_id=0, count=1)
>          rfc.check_mark(out2, pkt_num=1, check_param={"port_id": 0, "queue":
> 1, "mark_id": 0}, stats=True)
> +        rule2 = 'flow create 0 ingress pattern eth dst is 
> + 00:11:22:33:44:55 / ipv4
> src is 192.168.0.22 dst is 192.168.0.23 / end actions queue index 2 / 
> mark id 1 / end'
> +        rule_li2 = self.create_fdir_rule(rule=rule2, check_stats=True)
> +        self.check_fdir_rule(rule_list=rule_li+rule_li2)
> +        out3 =
> self.send_pkts_getouput(pkts='Ether(dst="00:11:22:33:44:55")/IP(src="1
> 92.1
> 68.0.22",dst="192.168.0.23") / Raw("x" * 80)', port_id=0, count=1)
> +        rfc.check_mark(out3, pkt_num=1, check_param={"port_id": 0,
> + "queue": 2, "mark_id": 1}, stats=True)
> 
>      def test_add_delete_rules(self):
>          self.pmd_output.execute_cmd("stop")
> @@ -2157,10 +2162,22 @@ class TestCVLFdir(TestCase):
>          rules = list()
>          pkt_pattern =
> 'Ether(dst="00:11:22:33:44:55")/IP(src="192.168.100.20",dst="192.168.%
> d.%d
> ")/Raw("x" * 80)'
>          pkts2 = list()
> -        for i in range(60):
> -            for j in range(256):
> -                rules.append(rule_pattern % (i, j))
> -                pkts2.append(pkt_pattern % (i, j))
> +        if self.nic in ['columbiaville_100g']:
> +            rule_li = list(map(str, range(15360)))
> +            pkts =
> ['Ether(dst="00:11:22:33:44:55")/IP(src="192.168.100.20",dst="192.168.
> 0.0")/
> Raw("x" * 80)',
> +
> 'Ether(dst="00:11:22:33:44:55")/IP(src="192.168.100.20",dst="192.168.59.255"
> )/Raw("x" * 80)']
> +            for i in range(60):
> +                for j in range(256):
> +                    rules.append(rule_pattern % (i, j))
> +                    pkts2.append(pkt_pattern % (i, j))
> +        else:
[Yuan]: I think it should identify the NIC type.
> +            rule_li = list(map(str, range(14848)))
> +            pkts =
> ['Ether(dst="00:11:22:33:44:55")/IP(src="192.168.100.20",dst="192.168.
> 0.0")/
> Raw("x" * 80)',
> +
> 'Ether(dst="00:11:22:33:44:55")/IP(src="192.168.100.20",dst="192.168.57.255"
> )/Raw("x" * 80)']
> +            for i in range(58):
> +                for j in range(256):
> +                    rules.append(rule_pattern % (i, j))
> +                    pkts2.append(pkt_pattern % (i, j))
>          cmd_path = '/tmp/test_max_rules'
>          cmd_li = map(lambda x: x + os.linesep, rules)
>          with open(cmd_path, 'w') as f:
> @@ -2174,15 +2191,12 @@ class TestCVLFdir(TestCase):
>                                                      self.portMask, 64, 64, cmd_path),
>                                                  eal_param="-w 
> %s,flow-mark-support=1 - w %s,flow-mark-support=1" % (
>                                                      self.pci0, self.pci1), socket=self.ports_socket)
> -            self.verify('Failed to create file' not in out, "create some rule
> failed: %s" % out)
> +            self.verify('Failed to create flow' not in out, "create 
> + some rule failed")
>              self.config_testpmd()
>              self.pmd_output.execute_cmd('start')
>              rule = "flow create 0 ingress pattern eth / ipv4 src is 
> 192.168.100.20 dst is 192.168.60.0 / end actions queue index 1 / mark / end"
>              self.create_fdir_rule(rule=rule, check_stats=False, 
> msg='Failed to create flow', validate=False)
> -            rule_li = list(map(str, range(15360)))
>              self.check_fdir_rule(port_id=0, stats=True, rule_list=rule_li)
> -            pkts =
> ['Ether(dst="00:11:22:33:44:55")/IP(src="192.168.100.20",dst="192.168.
> 0.0")/
> Raw("x" * 80)',
> -
> 'Ether(dst="00:11:22:33:44:55")/IP(src="192.168.100.20",dst="192.168.59.255"
> )/Raw("x" * 80)']
>              out1 = self.send_pkts_getouput(pkts=pkts, port_id=0, count=1)
>              rfc.check_mark(out1, pkt_num=2, check_param={"port_id": 
> 0,
> "queue": 1, "mark_id": 0}, stats=True)
>              self.pmd_output.execute_cmd("flow flush 0")
> --
> 1.8.3.1


      reply	other threads:[~2020-06-19  8:21 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-06-12  2:38 Xiao Qimai
2020-06-12  2:46 ` Xiao, QimaiX
2020-06-19  8:21   ` Peng, Yuan [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=BY5PR11MB3893190932E62FF8F340F0FEFE980@BY5PR11MB3893.namprd11.prod.outlook.com \
    --to=yuan.peng@intel.com \
    --cc=dts@dpdk.org \
    --cc=qimaix.xiao@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).