diff mbox series

[V1] tests/vhost_user_live_migration: delete dequeue-zero-copy parameter

Message ID 20210329055437.41653-1-weix.ling@intel.com (mailing list archive)
State Superseded
Headers
Series [V1] tests/vhost_user_live_migration: delete dequeue-zero-copy parameter |

Commit Message

Ling, WeiX March 29, 2021, 5:54 a.m. UTC 
  As DPDK community will remove support dequeue zero copy feature in
20.11,so remove dequeue-zero-copy parameter when start testpmd.

Signed-off-by: Ling Wei <weix.ling@intel.com>
---
 tests/TestSuite_vhost_user_live_migration.py | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

Comments

Ling, WeiX March 29, 2021, 5:57 a.m. UTC | #1 
> -----Original Message-----
> From: Ling, WeiX <weix.ling@intel.com>
> Sent: Monday, March 29, 2021 01:55 PM
> To: dts@dpdk.org
> Cc: Ling, WeiX <weix.ling@intel.com>
> Subject: [dts][PATCH V1] tests/vhost_user_live_migration: delete dequeue-
> zero-copy parameter
> 
Tested-by: Wei Ling <weix.ling@intel.com>
Wang, Yinan March 30, 2021, 2:37 a.m. UTC | #2 
Acked-by: Wang, Yinan <yinan.wang@intel.com>

> -----Original Message-----
> From: dts <dts-bounces@dpdk.org> On Behalf Of Ling, WeiX
> Sent: 2021?3?29? 13:57
> To: dts@dpdk.org
> Subject: Re: [dts] [PATCH V1] tests/vhost_user_live_migration: delete
> dequeue-zero-copy parameter
> 
> > -----Original Message-----
> > From: Ling, WeiX <weix.ling@intel.com>
> > Sent: Monday, March 29, 2021 01:55 PM
> > To: dts@dpdk.org
> > Cc: Ling, WeiX <weix.ling@intel.com>
> > Subject: [dts][PATCH V1] tests/vhost_user_live_migration: delete
> dequeue-
> > zero-copy parameter
> >
> Tested-by: Wei Ling <weix.ling@intel.com>
Tu, Lijuan March 30, 2021, 5:55 a.m. UTC | #3 
> As DPDK community will remove support dequeue zero copy feature in 20.11,so
> remove dequeue-zero-copy parameter when start testpmd.
> 
> Signed-off-by: Ling Wei <weix.ling@intel.com>

Test plan need to update at the same time, thanks
diff mbox series

Patch

diff --git a/tests/TestSuite_vhost_user_live_migration.py b/tests/TestSuite_vhost_user_live_migration.py
index 6d524db7..f73411e5 100644
--- a/tests/TestSuite_vhost_user_live_migration.py
+++ b/tests/TestSuite_vhost_user_live_migration.py
@@ -142,16 +142,13 @@  class TestVhostUserLiveMigration(TestCase):
         self.verify(len(self.core_list0) >= core_number and len(self.core_list1) >= core_number,
                     'There have not enough cores to start testpmd on duts')
 
-    def launch_testpmd_as_vhost_on_both_dut(self, zero_copy=False):
+    def launch_testpmd_as_vhost_on_both_dut(self):
         """
         start testpmd as vhost user on host_dut and backup_dut
         """
         self.get_core_list()
-        zero_copy_str = ''
-        if zero_copy is True:
-            zero_copy_str = ',dequeue-zero-copy=1'
         testcmd = self.testpmd_path + " "
-        vdev = ['eth_vhost0,iface=%s/vhost-net,queues=%d%s' % (self.base_dir, self.queue_number, zero_copy_str)]
+        vdev = ['eth_vhost0,iface=%s/vhost-net,queues=%d' % (self.base_dir, self.queue_number)]
         para = " -- -i --nb-cores=%d --rxq=%d --txq=%d" % (self.queue_number, self.queue_number, self.queue_number)
         eal_params_first = self.dut.create_eal_parameters(cores=self.core_list0, prefix='vhost', ports=[self.host_pci_info], vdevs=vdev)
         eal_params_secondary = self.dut.create_eal_parameters(cores=self.core_list1, prefix='vhost', ports=[self.backup_pci_info], vdevs=vdev)