test suite reviews and discussions
 help / color / mirror / Atom feed
From: "Wang, Yinan" <yinan.wang@intel.com>
To: "Xiao, QimaiX" <qimaix.xiao@intel.com>, "dts@dpdk.org" <dts@dpdk.org>
Cc: "Xiao, QimaiX" <qimaix.xiao@intel.com>
Subject: Re: [dts] [PATCH V1 01/11]loopback_multi_paths_port_restart: update	script according to testplan's update
Date: Wed, 25 Mar 2020 08:22:15 +0000	[thread overview]
Message-ID: <bc8b9959f0e84f3eaa1b386a86dcdd13@intel.com> (raw)
In-Reply-To: <1585123857-130281-1-git-send-email-qimaix.xiao@intel.com>

Acked-by: Wang, Yinan <yinan.wang@intel.com>

> -----Original Message-----
> From: dts <dts-bounces@dpdk.org> On Behalf Of Xiao Qimai
> Sent: 2020年3月25日 16:11
> To: dts@dpdk.org
> Cc: Xiao, QimaiX <qimaix.xiao@intel.com>
> Subject: [dts] [PATCH V1 01/11]loopback_multi_paths_port_restart: update
> script according to testplan's update
> 
> Signed-off-by: Xiao Qimai <qimaix.xiao@intel.com>
> ---
>  .../TestSuite_loopback_multi_paths_port_restart.py | 85
> +++++++++++++++-------
>  1 file changed, 57 insertions(+), 28 deletions(-)
> 
> diff --git a/tests/TestSuite_loopback_multi_paths_port_restart.py
> b/tests/TestSuite_loopback_multi_paths_port_restart.py
> index 0b54b8d..5f1faad 100644
> --- a/tests/TestSuite_loopback_multi_paths_port_restart.py
> +++ b/tests/TestSuite_loopback_multi_paths_port_restart.py
> @@ -121,15 +121,21 @@ class TestLoopbackPortRestart(TestCase):
>                  break
>              time.sleep(3)
>              loop = loop + 1
> -
>          self.verify("down" not in port_status, "port can not up after restart")
> 
> -    def port_restart(self):
> -        self.vhost.send_expect("stop", "testpmd> ", 120)
> -        self.vhost.send_expect("port stop 0", "testpmd> ", 120)
> -        self.check_port_throughput_after_port_stop()
> -        self.vhost.send_expect("clear port stats all", "testpmd> ", 120)
> -        self.vhost.send_expect("port start all", "testpmd> ", 120)
> +    def port_restart(self, restart_times=1):
> +        if restart_times == 1:
> +            self.vhost.send_expect("stop", "testpmd> ", 120)
> +            self.vhost.send_expect("port stop 0", "testpmd> ", 120)
> +            self.check_port_throughput_after_port_stop()
> +            self.vhost.send_expect("clear port stats all", "testpmd> ", 120)
> +            self.vhost.send_expect("port start all", "testpmd> ", 120)
> +        else:
> +            for i in range(restart_times):
> +                self.vhost.send_expect("stop", "testpmd> ", 120)
> +                self.vhost.send_expect("port stop 0", "testpmd> ", 120)
> +                self.vhost.send_expect("clear port stats all", "testpmd> ",
> 120)
> +                self.vhost.send_expect("port start all", "testpmd> ",
> + 120)
>          self.check_port_link_status_after_port_restart()
>          self.vhost.send_expect("set burst 1", "testpmd> ", 120)
>          self.vhost.send_expect("start tx_first 1", "testpmd> ", 120) @@
> -156,7 +162,7 @@ class TestLoopbackPortRestart(TestCase):
>          self.verify(Mpps > 0, "%s can not receive packets" %
> self.running_case)
>          return Mpps
> 
> -    def send_and_verify(self, case_info, frame_size):
> +    def send_and_verify(self, case_info, frame_size, restart_times=1):
>          """
>          start to send packets and calculate the average throughput
>          """
> @@ -166,7 +172,7 @@ class TestLoopbackPortRestart(TestCase):
>          Mpps = self.calculate_avg_throughput()
>          self.update_table_info(case_info, frame_size, Mpps, "Before
> Restart")
> 
> -        self.port_restart()
> +        self.port_restart(restart_times)
>          Mpps = self.calculate_avg_throughput()
>          self.update_table_info(case_info, frame_size, Mpps, "After Restart
> and set burst to 1")
> 
> @@ -184,7 +190,7 @@ class TestLoopbackPortRestart(TestCase):
>          self.dut.close_session(self.vhost)
>          self.dut.close_session(self.virtio_user)
> 
> -    def test_vhost_loopback_virtio11_mergeable_mac(self):
> +    def test_loopback_test_with_packed_ring_mergeable_path(self):
>          """
>          performance for [frame_sizes] and restart port on virtio1.1
> mergeable path
>          """
> @@ -193,11 +199,11 @@ class TestLoopbackPortRestart(TestCase):
>          for frame_size in self.frame_sizes:
>              self.start_vhost_testpmd()
>              self.start_virtio_user_testpmd(pmd_arg)
> -            self.send_and_verify("virtio1.1 mergeable", frame_size)
> +            self.send_and_verify("packed ring mergeable", frame_size)
>              self.close_all_testpmd()
>          self.result_table_print()
> 
> -    def test_vhost_loopback_virtio11_normal_mac(self):
> +    def test_loopback_test_with_packed_ring_nonmergeable_path(self):
>          """
>          performance for [frame_sizes] and restart port ob virtio1.1 normal
> path
>          """
> @@ -206,63 +212,86 @@ class TestLoopbackPortRestart(TestCase):
>          for frame_size in self.frame_sizes:
>              self.start_vhost_testpmd()
>              self.start_virtio_user_testpmd(pmd_arg)
> -            self.send_and_verify("virtio1.1 normal", frame_size)
> +            self.send_and_verify("packed ring non-mergeable", frame_size)
> +            self.close_all_testpmd()
> +        self.result_table_print()
> +
> +    def
> test_lookback_test_with_packed_ring_inorder_mergeable_path(self):
> +        pmd_arg = {"version": "packed_vq=1,mrg_rxbuf=1,in_order=1",
> +                   "path": "--tx-offloads=0x0 --enable-hw-vlan-strip"}
> +        for frame_size in self.frame_sizes:
> +            self.start_vhost_testpmd()
> +            self.start_virtio_user_testpmd(pmd_arg)
> +            self.send_and_verify("packed ring non-mergeable",
> + frame_size)
>              self.close_all_testpmd()
>          self.result_table_print()
> 
> -    def test_vhost_loopback_virtiouser_inorder_mergeable_mac(self):
> +    def
> test_lookback_test_with_packed_ring_inorder_nonmergeable_path(self):
>          """
>          performance for [frame_sizes] and restart port on inorder mergeable
> path
>          """
> -        pmd_arg = {"version": "packed_vq=0,in_order=1,mrg_rxbuf=1 ",
> -                          "path": "--tx-offloads=0x0
> --enable-hw-vlan-strip "}
> +        pmd_arg = {"version": "packed_vq=1,mrg_rxbuf=0,in_order=1",
> +                          "path": "--tx-offloads=0x0
> + --enable-hw-vlan-strip"}
>          for frame_size in self.frame_sizes:
>              self.start_vhost_testpmd()
>              self.start_virtio_user_testpmd(pmd_arg)
> -            self.send_and_verify("inorder mergeable", frame_size)
> +            self.send_and_verify("packed ring inorder non-mergeable",
> + frame_size)
>              self.close_all_testpmd()
>          self.result_table_print()
> 
> -    def test_vhost_loopback_virtiouser_inorder_mergeable_off_mac(self):
> +    def test_lookback_test_with_split_ring_inorder_mergeable_path(self):
>          """
>          performance for [frame_sizes] and restart port on inorder normal
> path
>          """
> +        pmd_arg = {"version": "packed_vq=0,in_order=1,mrg_rxbuf=1",
> +                          "path": "--tx-offloads=0x0
> --enable-hw-vlan-strip "}
> +        for frame_size in self.frame_sizes:
> +            self.start_vhost_testpmd()
> +            self.start_virtio_user_testpmd(pmd_arg)
> +            self.send_and_verify("split ring inorder mergeable", frame_size)
> +            self.close_all_testpmd()
> +        self.result_table_print()
> +
> +    def
> test_lookback_test_with_split_ring_inorder_nonmergeable_path(self):
> +        """
> +        performance for [frame_sizes] and restart port on virtio normal path
> +        """
>          pmd_arg = {"version": "packed_vq=0,in_order=1,mrg_rxbuf=0 ",
>                            "path": "--tx-offloads=0x0
> --enable-hw-vlan-strip "}
>          for frame_size in self.frame_sizes:
>              self.start_vhost_testpmd()
>              self.start_virtio_user_testpmd(pmd_arg)
> -            self.send_and_verify("inorder normal", frame_size)
> +            self.send_and_verify("split ring inorder non-mergeable",
> + frame_size)
>              self.close_all_testpmd()
>          self.result_table_print()
> 
> -    def test_vhost_loopback_virtiouser_mergeable_mac(self):
> +    def test_lookback_test_with_split_ring_mergeable_path(self):
>          """
> -        performance for [frame_sizes] and restart port on virtio mergeable
> path
> +        performance for [frame_sizes] and restart port on virtio normal
> + path
>          """
> -        pmd_arg = {"version": "packed_vq=0,in_order=0,mrg_rxbuf=1 ",
> +        pmd_arg = {"version": "packed_vq=0,in_order=0,mrg_rxbuf=1",
>                            "path": "--tx-offloads=0x0
> --enable-hw-vlan-strip "}
>          for frame_size in self.frame_sizes:
>              self.start_vhost_testpmd()
>              self.start_virtio_user_testpmd(pmd_arg)
> -            self.send_and_verify("virtiouser mergeable", frame_size)
> +            self.send_and_verify("split ring mergeable", frame_size,
> + restart_times=100)
>              self.close_all_testpmd()
>          self.result_table_print()
> 
> -    def test_vhost_loopback_virtiouser_normal_mac(self):
> +    def test_lookback_test_with_split_ring_nonmergeable_path(self):
>          """
>          performance for [frame_sizes] and restart port on virtio normal path
>          """
> -        pmd_arg = {"version": "packed_vq=0,in_order=0,mrg_rxbuf=0 ",
> +        pmd_arg = {"version": "packed_vq=0,in_order=0,mrg_rxbuf=0",
>                            "path": "--tx-offloads=0x0
> --enable-hw-vlan-strip "}
>          for frame_size in self.frame_sizes:
>              self.start_vhost_testpmd()
>              self.start_virtio_user_testpmd(pmd_arg)
> -            self.send_and_verify("virtiouser normal", frame_size)
> +            self.send_and_verify("split ring non-mergeable",
> + frame_size)
>              self.close_all_testpmd()
>          self.result_table_print()
> 
> -    def test_vhost_loopback_virtiouser_vector_rx_mac(self):
> +    def test_loopback_test_with_split_ring_vector_rx_path(self):
>          """
>          performance for frame_sizes and restart port on virtio vector rx
>          """
> @@ -271,7 +300,7 @@ class TestLoopbackPortRestart(TestCase):
>          for frame_size in self.frame_sizes:
>              self.start_vhost_testpmd()
>              self.start_virtio_user_testpmd(pmd_arg)
> -            self.send_and_verify("virtiouser vector_rx", frame_size)
> +            self.send_and_verify("split ring vector_rx", frame_size)
>              self.close_all_testpmd()
>          self.result_table_print()
> 
> --
> 1.8.3.1


  parent reply	other threads:[~2020-03-25  8:22 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-03-25  8:10 Xiao Qimai
2020-03-25  8:10 ` [dts] [PATCH V1 02/11]loopback_multi_queues: " Xiao Qimai
2020-03-25  8:23   ` Wang, Yinan
2020-03-31  3:00   ` Xiao, QimaiX
2020-03-25  8:10 ` [dts] [PATCH V1 03/11]pvp_virtio_user_2M_hugepages: " Xiao Qimai
2020-03-25  8:23   ` Wang, Yinan
2020-03-31  3:00   ` Xiao, QimaiX
2020-03-25  8:10 ` [dts] [PATCH V1 04/11]pvp_virtio_user_4k_pages: " Xiao Qimai
2020-03-25  8:20   ` Wang, Yinan
2020-03-31  3:00   ` Xiao, QimaiX
2020-03-25  8:10 ` [dts] [PATCH V1 05/11]vhost_enqueue_interrupt: " Xiao Qimai
2020-03-25  8:22   ` Wang, Yinan
2020-03-31  3:00   ` Xiao, QimaiX
2020-03-25  8:10 ` [dts] [PATCH V1 06/11]vhost_event_idx_interrupt: " Xiao Qimai
2020-03-25  8:21   ` Wang, Yinan
2020-03-31  2:59   ` Xiao, QimaiX
2020-03-25  8:10 ` [dts] [PATCH V1 07/11]vhost_user_live_migration: " Xiao Qimai
2020-03-25  8:22   ` Wang, Yinan
2020-03-31  2:59   ` Xiao, QimaiX
2020-03-25  8:10 ` [dts] [PATCH V1 08/11]vhost_virtio_pmd_interrupt: " Xiao Qimai
2020-03-25  8:22   ` Wang, Yinan
2020-03-31  3:00   ` Xiao, QimaiX
2020-03-25  8:10 ` [dts] [PATCH V1 09/11]vhost_virtio_user_interrupt: " Xiao Qimai
2020-03-25  8:22   ` Wang, Yinan
2020-03-31  3:00   ` Xiao, QimaiX
2020-03-25  8:10 ` [dts] [PATCH V1 10/11]virtio_event_idx_interrupt: " Xiao Qimai
2020-03-25  8:22   ` Wang, Yinan
2020-03-31  2:59   ` Xiao, QimaiX
2020-03-25  8:10 ` [dts] [PATCH V1 11/11]virtio_pvp_regression: " Xiao Qimai
2020-03-25  8:22   ` Wang, Yinan
2020-03-31  2:59   ` Xiao, QimaiX
2020-03-25  8:22 ` Wang, Yinan [this message]
2020-03-31  3:00 ` [dts] [PATCH V1 01/11]loopback_multi_paths_port_restart: " Xiao, QimaiX
2020-03-31  3:06 ` Tu, Lijuan

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=bc8b9959f0e84f3eaa1b386a86dcdd13@intel.com \
    --to=yinan.wang@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).