Clean up dpdk plugin rx/tx pcap tracing

Needed a spinlock to protect the data vector. Cleaned up debug cli so
the output makes sense, and so that various parameters exist in one
place. Removed a nonsense memset-to-zero which led to ultra-confusing
results.

Change-Id: I91cd14ce7fe84fd2eceab86e016b5ee001993be4
Signed-off-by: Dave Barach <dbarach@cisco.com>

1 files changed, 25 insertions, 29 deletions

diff --git a/src/plugins/dpdk/device/cli.c b/src/plugins/dpdk/device/cli.c
index 2a49771ef26..40ec32325e4 100644
--- a/src/plugins/dpdk/device/cli.c
+++ b/src/plugins/dpdk/device/cli.c
@@ -147,7 +147,7 @@ pcap_trace_command_internal (vlib_main_t * vm,
 		    clib_error_report (error);
 		  else
 		    vlib_cli_output (vm, "saved to %s...",
-				     dm->pcap[rx_tx].pcap_filename);
+				     dm->pcap[rx_tx].pcap_main.file_name);
 		}
 
 	      dm->pcap[rx_tx].pcap_enable = 0;
@@ -211,23 +211,25 @@ pcap_trace_command_internal (vlib_main_t * vm,
 	    {
 	      vlib_cli_output
 		(vm, "max is %d for any interface to file %s",
-		 dm->pcap_pkts_to_capture ?
-		 dm->pcap[rx_tx].pcap_pkts_to_capture
+		 dm->pcap[rx_tx].pcap_main.n_packets_to_capture ?
+		 dm->pcap[rx_tx].pcap_main.n_packets_to_capture
 		 : PCAP_DEF_PKT_TO_CAPTURE,
-		 dm->pcap_filename ?
-		 dm->pcap[rx_tx].pcap_filename : (u8 *) "/tmp/vpe.pcap");
+		 dm->pcap[rx_tx].pcap_main.file_name ?
+		 (u8 *) dm->pcap[rx_tx].pcap_main.file_name :
+		 (u8 *) "/tmp/vpe.pcap");
 	    }
 	  else
 	    {
 	      vlib_cli_output (vm, "max is %d for interface %U to file %s",
-			       dm->pcap[rx_tx].pcap_pkts_to_capture
-			       ? dm->pcap_pkts_to_capture
-			       : PCAP_DEF_PKT_TO_CAPTURE,
+			       dm->pcap[rx_tx].pcap_main.n_packets_to_capture
+			       ? dm->pcap[rx_tx].
+			       pcap_main.n_packets_to_capture :
+			       PCAP_DEF_PKT_TO_CAPTURE,
 			       format_vnet_sw_if_index_name, dm->vnet_main,
 			       dm->pcap_sw_if_index,
-			       dm->pcap[rx_tx].pcap_filename
-			       ? dm->pcap[rx_tx].pcap_filename : (u8 *)
-			       "/tmp/vpe.pcap");
+			       dm->pcap[rx_tx].
+			       pcap_main.file_name ? (u8 *) dm->pcap[rx_tx].
+			       pcap_main.file_name : (u8 *) "/tmp/vpe.pcap");
 	    }
 
 	  if (dm->pcap[rx_tx].pcap_enable == 0)
@@ -262,34 +264,28 @@ pcap_trace_command_internal (vlib_main_t * vm,
       /* Since no error, save configured values. */
       if (chroot_filename)
 	{
-	  if (dm->pcap[rx_tx].pcap_filename)
-	    vec_free (dm->pcap[rx_tx].pcap_filename);
+	  if (dm->pcap[rx_tx].pcap_main.file_name)
+	    vec_free (dm->pcap[rx_tx].pcap_main.file_name);
 	  vec_add1 (chroot_filename, 0);
-	  dm->pcap[rx_tx].pcap_filename = chroot_filename;
+	  dm->pcap[rx_tx].pcap_main.file_name = (char *) chroot_filename;
 	}
 
       if (max)
-	dm->pcap[rx_tx].pcap_pkts_to_capture = max;
-
+	dm->pcap[rx_tx].pcap_main.n_packets_to_capture = max;
 
       if (enabled)
 	{
-	  if (dm->pcap[rx_tx].pcap_filename == 0)
-	    dm->pcap[rx_tx].pcap_filename = format (0, "/tmp/vpe.pcap%c", 0);
-
-	  memset (&dm->pcap[rx_tx].pcap_main, 0,
-		  sizeof (dm->pcap[rx_tx].pcap_main));
-	  dm->pcap[rx_tx].pcap_main.file_name =
-	    (char *) dm->pcap[rx_tx].pcap_filename;
-	  dm->pcap[rx_tx].pcap_main.n_packets_to_capture
-	    = PCAP_DEF_PKT_TO_CAPTURE;
-	  if (dm->pcap[rx_tx].pcap_pkts_to_capture)
-	    dm->pcap[rx_tx].pcap_main.n_packets_to_capture
-	      = dm->pcap[rx_tx].pcap_pkts_to_capture;
+	  if (dm->pcap[rx_tx].pcap_main.file_name == 0)
+	    dm->pcap[rx_tx].pcap_main.file_name
+	      = (char *) format (0, "/tmp/vpe.pcap%c", 0);
 
+	  dm->pcap[rx_tx].pcap_main.n_packets_captured = 0;
 	  dm->pcap[rx_tx].pcap_main.packet_type = PCAP_PACKET_TYPE_ethernet;
+	  if (dm->pcap[rx_tx].pcap_main.lock == 0)
+	    clib_spinlock_init (&(dm->pcap[rx_tx].pcap_main.lock));
 	  dm->pcap[rx_tx].pcap_enable = 1;
-	  vlib_cli_output (vm, "pcap tx capture on...");
+	  vlib_cli_output (vm, "pcap %s capture on...",
+			   rx_tx == VLIB_RX ? "rx" : "tx");
 	}
     }
   else if (chroot_filename)