From: "Wang, Yinan" <yinan.wang@intel.com>
To: "Chen, LingliX" <linglix.chen@intel.com>, "dts@dpdk.org" <dts@dpdk.org>
Cc: "Chen, LingliX" <linglix.chen@intel.com>
Subject: Re: [dts] [PATCH V1] tests/vhost_cbdma: remove one case sync with testplan
Date: Fri, 27 Aug 2021 09:07:25 +0000 [thread overview]
Message-ID: <BYAPR11MB2648C14D8BAC954CE7D18A208FC89@BYAPR11MB2648.namprd11.prod.outlook.com> (raw)
In-Reply-To: <20210827164435.18136-1-linglix.chen@intel.com>
Acked-by: Yinan Wang <yinan.wang@intel.com>
> -----Original Message-----
> From: dts <dts-bounces@dpdk.org> On Behalf Of Lingli Chen
> Sent: 2021?8?28? 0:45
> To: dts@dpdk.org
> Cc: Chen, LingliX <linglix.chen@intel.com>
> Subject: [dts] [PATCH V1] tests/vhost_cbdma: remove one case sync with
> testplan
>
> remove test_check_threshold_value_with_cbdma sync with testplan.
>
> Signed-off-by: Lingli Chen <linglix.chen@intel.com>
> ---
> tests/TestSuite_vhost_cbdma.py | 46 +++-------------------------------
> 1 file changed, 3 insertions(+), 43 deletions(-)
>
> diff --git a/tests/TestSuite_vhost_cbdma.py
> b/tests/TestSuite_vhost_cbdma.py
> index 16fef645..c44b692a 100644
> --- a/tests/TestSuite_vhost_cbdma.py
> +++ b/tests/TestSuite_vhost_cbdma.py
> @@ -318,49 +318,9 @@ class TestVirTioVhostCbdma(TestCase):
> self.handle_expected(mode_list=self.mode_list)
> self.handle_results(mode_list=self.mode_list)
>
> - def test_check_threshold_value_with_cbdma(self):
> - """
> - Test Case3: CBDMA threshold value check
> - """
> - used_cbdma_num = 4
> - params = '--nb-cores=1 --rxq=2 --txq=2'
> - dmathr = [512, 4096]
> - vid_dict = {}
> - self.get_cbdma_ports_info_and_bind_to_dpdk(used_cbdma_num)
> - self.dut.restore_interfaces()
> - # launch vhost, Check the cbdma threshold value for each vhost port
> can be config correct from vhost log
> - vhost_vdev = [f"'eth_vhost0,iface=vhost-
> net0,queues=2,client=1,dmas=[txq0@{self.used_cbdma[0]};txq1@{self.used
> _cbdma[1]}],dmathr={dmathr[0]}'", \
> - f"'eth_vhost1,iface=vhost-
> net1,queues=2,client=1,dmas=[txq0@{self.used_cbdma[2]};txq1@{self.used
> _cbdma[3]}],dmathr={dmathr[1]}'"]
> - virtio_dev0 = f"net_virtio_user0,mac=00:01:02:03:04:05,path=./vhost-
> net0,queues=2,server=1,packed_vq=0,mrg_rxbuf=1,in_order=0,queue_size=
> 4096"
> - virtio_dev1 = f"net_virtio_user1,mac=00:01:02:03:04:05,path=./vhost-
> net1,queues=2,server=1,packed_vq=0,mrg_rxbuf=1,in_order=0,queue_size=
> 4096"
> - vdev_params = '{} --vdev {}'.format(vhost_vdev[0], vhost_vdev[1])
> - allow_pci = []
> - for index in range(used_cbdma_num):
> - allow_pci.append(self.cbdma_dev_infos[index])
> - self.pmdout_vhost_user.start_testpmd(cores=self.cores[0:2],
> param=params, vdevs=[vdev_params], ports=allow_pci, prefix="vhost",
> fixed_prefix=True)
> - self.vhost_user.send_expect('start', 'testpmd> ', 120)
> - # vid0,qid0,dma2,threshold:4096
> - self.launch_testpmd_as_virtio_user1(params, self.cores[2:4],
> dev=virtio_dev1)
> - vid_dict[dmathr[1]] = 0
> - # vid1,qid0,dma0,threshold:512
> - self.launch_testpmd_as_virtio_user(params, self.cores[4:6],
> dev=virtio_dev0)
> - vid_dict[dmathr[0]] = 1
> - # Check the cbdma threshold value for each vhost port can be config
> correct from vhost log
> - out = self.vhost_user.get_session_before(timeout=2)
> - self.vhost_user.send_expect("quit", "# ")
> - self.virtio_user.send_expect("quit", "# ")
> - self.virtio_user1.send_expect("quit", "# ")
> - pattern = re.compile(r'dma parameters:
> vid\S+,qid\d+,dma\d+,threshold:\d+')
> - return_param = re.findall(pattern, out)
> - self.logger.info("Actual Info:" + str(return_param))
> - check_value = 0
> - for dma in dmathr:
> - check_value +=
> len(re.findall('vid{},\S+threshold:{}'.format(vid_dict[dma], dma),
> str(return_param)))
> - self.verify(check_value == used_cbdma_num, "Check failed: Actual
> value:{}".format(return_param))
> -
> def
> test_perf_pvp_packed_all_path_with_cbdma_vhost_enqueue_operations(s
> elf):
> """
> - Test Case 4: PVP packed ring all path with DMA-accelerated vhost
> enqueue
> + Test Case 3: PVP packed ring all path with DMA-accelerated vhost
> enqueue
> """
> self.test_target = self.running_case
> self.expected_throughput =
> self.get_suite_cfg()['expected_throughput'][self.test_target]
> @@ -406,7 +366,7 @@ class TestVirTioVhostCbdma(TestCase):
>
> def
> test_perf_packed_dynamic_queue_number_cbdma_vhost_enqueue_operat
> ions(self):
> """
> - Test Case5: Packed ring dynamic queue number test for DMA-
> accelerated vhost Tx operations
> + Test Case4: Packed ring dynamic queue number test for DMA-
> accelerated vhost Tx operations
> """
> self.test_target = self.running_case
> self.expected_throughput =
> self.get_suite_cfg()['expected_throughput'][self.test_target]
> @@ -466,7 +426,7 @@ class TestVirTioVhostCbdma(TestCase):
>
> def test_perf_compare_pvp_split_ring_performance(self):
> """
> - Test Case6: Compare PVP split ring performance between CPU copy,
> CBDMA copy and Sync copy
> + Test Case5: Compare PVP split ring performance between CPU copy,
> CBDMA copy and Sync copy
> """
> used_cbdma_num = 1
> queue = 1
> --
> 2.32.0
next prev parent reply other threads:[~2021-08-27 9:07 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-08-27 16:44 Lingli Chen
2021-08-27 9:07 ` Wang, Yinan [this message]
2021-09-27 8:35 ` Chen, LingliX
2021-09-28 3:24 ` 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=BYAPR11MB2648C14D8BAC954CE7D18A208FC89@BYAPR11MB2648.namprd11.prod.outlook.com \
--to=yinan.wang@intel.com \
--cc=dts@dpdk.org \
--cc=linglix.chen@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).