[V1] tests/dpdk_hugetlbfs_mount_size: Start testpmd with different pci

Message ID 20201110135646.1809176-1-yux.jiang@intel.com (mailing list archive)
State Accepted
Headers
Series [V1] tests/dpdk_hugetlbfs_mount_size: Start testpmd with different pci |

Commit Message

Yu Jiang Nov. 10, 2020, 1:56 p.m. UTC
  From: JiangYu <yux.jiang@intel.com>

1,Start testpmd with different pci
2,Umount huges in tear_down.

Signed-off-by: JiangYu <yux.jiang@intel.com>
---
 tests/TestSuite_dpdk_hugetlbfs_mount_size.py | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)
  

Comments

Yu Jiang Nov. 10, 2020, 5:59 a.m. UTC | #1
Tested-by: JiangYuX <yux.jiang@intel.com>

    Best Regards
    Jiang yu


> -----Original Message-----
> From: JiangYuX [mailto:yux.jiang@intel.com]
> Sent: Tuesday, November 10, 2020 9:57 PM
> To: dts@dpdk.org
> Cc: Jiang, YuX <yux.jiang@intel.com>
> Subject: [dts] [PATCH V1] tests/dpdk_hugetlbfs_mount_size: Start testpmd
> with different pci
  
Tu, Lijuan Nov. 11, 2020, 6:47 a.m. UTC | #2
> 1,Start testpmd with different pci
> 2,Umount huges in tear_down.
> 
> Signed-off-by: JiangYu <yux.jiang@intel.com>

Applied
  
Wang, Yinan Nov. 17, 2020, 5:58 a.m. UTC | #3
Acked-by: Wang, Yinan <yinan.wang@intel.com>

> -----Original Message-----
> From: dts <dts-bounces@dpdk.org> On Behalf Of Jiang, YuX
> Sent: 2020?11?10? 14:00
> To: Jiang, YuX <yux.jiang@intel.com>; dts@dpdk.org
> Subject: Re: [dts] [PATCH V1] tests/dpdk_hugetlbfs_mount_size: Start
> testpmd with different pci
> 
> Tested-by: JiangYuX <yux.jiang@intel.com>
> 
>     Best Regards
>     Jiang yu
> 
> 
> > -----Original Message-----
> > From: JiangYuX [mailto:yux.jiang@intel.com]
> > Sent: Tuesday, November 10, 2020 9:57 PM
> > To: dts@dpdk.org
> > Cc: Jiang, YuX <yux.jiang@intel.com>
> > Subject: [dts] [PATCH V1] tests/dpdk_hugetlbfs_mount_size: Start
> testpmd
> > with different pci
  

Patch

diff --git a/tests/TestSuite_dpdk_hugetlbfs_mount_size.py b/tests/TestSuite_dpdk_hugetlbfs_mount_size.py
index 25c4ed2..48fdc8e 100644
--- a/tests/TestSuite_dpdk_hugetlbfs_mount_size.py
+++ b/tests/TestSuite_dpdk_hugetlbfs_mount_size.py
@@ -207,7 +207,7 @@  class DpdkHugetlbfsMountSize(TestCase):
         self.logger.info("launch secondary testpmd")
         ttd_secondary = 'numactl --membind=%d %s -l %s -n %d  --legacy-mem --socket-mem %s --huge-dir' \
                             ' %s --file-prefix=%s -w %s -- -i --socket-num=%d --no-numa'
-        launch_ttd_secondary = ttd_secondary % (self.numa_id, self.app_path, self.core_list2, self.mem_channels, self.socket_mem2, MNT_PATH[1], vhost_name[1], self.pci_info_0, self.numa_id)
+        launch_ttd_secondary = ttd_secondary % (self.numa_id, self.app_path, self.core_list2, self.mem_channels, self.socket_mem2, MNT_PATH[1], vhost_name[1], self.pci_info_1, self.numa_id)
         self.session_secondary.send_expect(launch_ttd_secondary, "testpmd> ", 120)
         self.session_secondary.send_expect("set promisc all off", "testpmd> ", 120)
         self.session_secondary.send_expect("start", "testpmd> ", 120)
@@ -236,6 +236,7 @@  class DpdkHugetlbfsMountSize(TestCase):
         # start send packet and verify the session can receive the packet.
         self.send_pkg(0)
         self.verify_result(self.session_first)
+        self.send_pkg(1)
         self.verify_result(self.session_secondary)
         self.session_first.send_expect("quit", "#", 15)
         self.session_secondary.send_expect("quit", "#", 15)
@@ -262,6 +263,8 @@  class DpdkHugetlbfsMountSize(TestCase):
         """
         Run after each test case.
         """
+        # If case fails, the mount should be cancelled to avoid affecting next cases
+        self.umount_huge([MNT_PATH[0], MNT_PATH[1], MNT_PATH[2]])
         self.dut.kill_all()
         time.sleep(2)