netperf: stop tcpdump process during performance testing

This patch was wrongly dropped in original refactor.

Tcpdump brings extra latency, it causes the guest to batch
more bytes per packet. We should stop tcpdump to drop the
effect, it will be restarted at the beginning of next case.
Reported-by: NWenli Quan <wquan@redhat.com>
Signed-off-by: NAmos Kong <akong@redhat.com>
Acked-by: NWenli Quan <wquan@redhat.com>

tests/netperf.py

@@ -207,6 +207,14 @@ def run_netperf(test, params, env):
             username = params_tmp["username"]
             env_setup(i, ip_dict[i], username, shell_port, password)
 
+    if "tcpdump" in env and env["tcpdump"].is_alive():
+        # Stop the background tcpdump process
+        try:
+            logging.debug("Stopping the background tcpdump")
+            env["tcpdump"].close()
+        except Exception:
+            pass
+
     error.context("Start netperf testing", logging.info)
     start_test(server_ip, server_ctl, host, clients, test.resultsdir,
                l=int(params.get('l')),