Re: [dts] [PATCH V1]tests/pvp_multi_paths_vhost_single_core_performance: optimize eal parameters

["Tu, Lijuan" <lijuan.tu@intel.com>]

Applied, thanks

> -----Original Message-----
> From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of Xiao Qimai
> Sent: Thursday, January 23, 2020 2:02 PM
> To: dts@dpdk.org
> Cc: Xiao, QimaiX <qimaix.xiao@intel.com>
> Subject: [dts] [PATCH
> V1]tests/pvp_multi_paths_vhost_single_core_performance: optimize eal
> parameters
> 
> Signed-off-by: Xiao Qimai <qimaix.xiao@intel.com>
> ---
>  ...vp_multi_paths_vhost_single_core_performance.py | 26 +++++++++--------
> -----
>  1 file changed, 11 insertions(+), 15 deletions(-)
> 
> diff --git
> a/tests/TestSuite_pvp_multi_paths_vhost_single_core_performance.py
> b/tests/TestSuite_pvp_multi_paths_vhost_single_core_performance.py
> index 160ab78..94c1759 100644
> --- a/tests/TestSuite_pvp_multi_paths_vhost_single_core_performance.py
> +++ b/tests/TestSuite_pvp_multi_paths_vhost_single_core_performance.py
> @@ -57,10 +57,6 @@ class TestPVPMultiPathVhostPerformance(TestCase):
>          self.core_list_host = self.core_list[3:5]
>          self.core_mask_user = utils.create_mask(self.core_list_user)
>          self.core_mask_host = utils.create_mask(self.core_list_host)
> -        if len(set([int(core['socket']) for core in self.dut.cores])) == 1:
> -            self.socket_mem = '1024'
> -        else:
> -            self.socket_mem = '1024,1024'
> 
>          self.out_path = '/tmp'
>          out = self.tester.send_expect('ls -d %s' % self.out_path, '# ') @@ -126,11
> +122,11 @@ class TestPVPMultiPathVhostPerformance(TestCase):
>          self.dut.send_expect("rm -rf ./vhost-net*", "#")
>          self.dut.send_expect("killall -s INT testpmd", "#")
>          self.dut.send_expect("killall -s INT qemu-system-x86_64", "#")
> -        command_line_client = "./x86_64-native-linuxapp-gcc/app/testpmd -
> n %d -c %s --socket-mem " + \
> -                              " %s --legacy-mem --no-pci --file-prefix=vhost --vdev " + \
> -                              "'net_vhost0,iface=vhost-net,queues=1' -- -i --nb-cores=1 --
> txd=1024 --rxd=1024"
> -        command_line_client = command_line_client % (
> -            self.dut.get_memory_channels(), self.core_mask_host,
> self.socket_mem)
> +        eal_param = self.dut.create_eal_parameters(socket=self.ports_socket,
> prefix='vhost',
> +                                                   no_pci=True,
> +                                                   vdevs=['net_vhost0,iface=vhost-net,queues=1'])
> +        eal_param += " --single-file-segments"
> +        command_line_client = "./%s/app/testpmd " % self.target + eal_param
> + " -- -i --nb-cores=1 --txd=1024 --rxd=1024"
>          self.vhost.send_expect(command_line_client, "testpmd> ", 120)
>          self.vhost.send_expect("set fwd mac", "testpmd> ", 120)
>          self.vhost.send_expect("start", "testpmd> ", 120) @@ -139,12 +135,12
> @@ class TestPVPMultiPathVhostPerformance(TestCase):
>          """
>          start testpmd on virtio
>          """
> -        command_line_user = "./x86_64-native-linuxapp-gcc/app/testpmd -
> n %d -c %s " + \
> -                            " --socket-mem %s --legacy-mem --file-prefix=virtio " + \
> -                            "--vdev=virtio_user0,mac=00:01:02:03:04:05,path=./vhost-
> net,queues=1,%s " + \
> -                            "-- -i %s --nb-cores=2 --txd=1024 --rxd=1024"
> -        command_line_user = command_line_user % (
> -            self.dut.get_memory_channels(), self.core_mask_user,
> self.socket_mem, args["version"], args["path"])
> +        eal_param = self.dut.create_eal_parameters(socket=self.ports_socket,
> prefix='virtio',
> +
> vdevs=['virtio_user0,mac=00:01:02:03:04:05,path=./vhost-net,queues=1,%s' %
> +                                                          args["version"]])
> +        eal_param += " --single-file-segments"
> +        command_line_user = "./%s/app/testpmd " % self.target + eal_param +
> " -- -i %s --nb-cores=2 --txd=1024 --rxd=1024" % \
> +                            args["path"]
>          self.vhost_user.send_expect(command_line_user, "testpmd> ", 120)
>          self.vhost_user.send_expect("set fwd io", "testpmd> ", 120)
>          self.vhost_user.send_expect("start", "testpmd> ", 120)
> --
> 1.8.3.1