Dpdk: symmetric_mp polishing

Author: mcgov

microsoft/testsuites/dpdk/dpdksuite.py

@@ -24,17 +24,7 @@
 from lisa.features import Gpu, Infiniband, IsolatedResource, Sriov
 from lisa.operating_system import BSD, CBLMariner, Ubuntu, Windows
 from lisa.testsuite import TestResult, simple_requirement
-from lisa.tools import (
-    Echo,
-    Git,
-    Hugepages,
-    Ip,
-    Kill,
-    Lscpu,
-    Lsmod,
-    Make,
-    Modprobe,
-)
+from lisa.tools import Echo, Git, Hugepages, Ip, Kill, Lscpu, Lsmod, Make, Modprobe
 from lisa.tools.hugepages import HugePageSize
 from lisa.tools.lscpu import CpuArchitecture
 from lisa.util.constants import SIGINT
@@ -54,12 +44,12 @@
     generate_send_receive_run_info,
     init_nodes_concurrent,
     initialize_node_resources,
+    run_dpdk_symmetric_mp,
     run_testpmd_concurrent,
     verify_dpdk_build,
     verify_dpdk_l3fwd_ntttcp_tcp,
     verify_dpdk_send_receive,
     verify_dpdk_send_receive_multi_txrx_queue,
-    run_dpdk_symmetric_mp,
 )
 from microsoft.testsuites.dpdk.dpdkvpp import DpdkVpp
 
@@ -1007,4 +997,4 @@ def _force_dpdk_default_source(self, variables: Dict[str, Any]) -> None:
 
     def after_case(self, log: Logger, **kwargs: Any) -> None:
         environment: Environment = kwargs.pop("environment")
-        # do_parallel_cleanup(environment)
+        do_parallel_cleanup(environment)

microsoft/testsuites/dpdk/dpdkutil.py

@@ -55,12 +55,12 @@
     Downloader,
     GitDownloader,
     Installer,
+    PackageManagerInstall,
     TarDownloader,
     check_dpdk_support,
     is_url_for_git_repo,
     is_url_for_tarball,
     update_kernel_from_repo,
-    PackageManagerInstall,
 )
 from microsoft.testsuites.dpdk.dpdktestpmd import PACKAGE_MANAGER_SOURCE, DpdkTestpmd
 from microsoft.testsuites.dpdk.rdmacore import (
@@ -1375,7 +1375,8 @@ def run_dpdk_symmetric_mp(
         (
             "HN_DRIVER: netvsc_hotplug_retry(): Found matching MAC address, "
             f"adding device {test_nics[0].pci_device_name} "
-            f"network name {test_nics[0].lower} args"
+            f"network name {test_nics[0].lower} "
+            f"args mac={test_nics[0].mac_addr},mac={test_nics[1].mac_addr}"
         ),
         delta_only=True,
     )  # relying on compiler defaults here, not great.