diff options
author | John Lo <loj@cisco.com> | 2019-08-01 15:15:36 -0400 |
---|---|---|
committer | Dave Barach <openvpp@barachs.net> | 2019-08-01 22:53:14 +0000 |
commit | 778ce573180618d4f694f91bd662d92b3eada725 (patch) | |
tree | 08363730d419f9f35e0307337647272b9ad301b3 /src/vnet | |
parent | 49568afd2096600f5d6745385d81d873c8893e29 (diff) |
interface: fix pcap tx/rx trace cli handling
Provide default packet_to_capture value. Display interface name
correctly for "pcap tx/rx trace status".
Type: fix
Signed-off-by: John Lo <loj@cisco.com>
Change-Id: I7064d0dbea236a9aff68bba7fbaf2c4a73b16c6f
Signed-off-by: John Lo <loj@cisco.com>
Diffstat (limited to 'src/vnet')
-rw-r--r-- | src/vnet/interface_cli.c | 13 | ||||
-rw-r--r-- | src/vnet/interface_output.c | 2 |
2 files changed, 5 insertions, 10 deletions
diff --git a/src/vnet/interface_cli.c b/src/vnet/interface_cli.c index f990cdeeb5d..437854476b4 100644 --- a/src/vnet/interface_cli.c +++ b/src/vnet/interface_cli.c @@ -1700,8 +1700,6 @@ pcap_trace_command_internal (vlib_main_t * vm, unformat_input_t * input, vlib_cli_command_t * cmd, int rx_tx) { -#define PCAP_DEF_PKT_TO_CAPTURE (1000) - unformat_input_t _line_input, *line_input = &_line_input; u8 *filename; u8 *chroot_filename = 0; @@ -1722,6 +1720,8 @@ pcap_trace_command_internal (vlib_main_t * vm, if (vm->pcap[rx_tx].pcap_enable == 0) { enabled = 1; + vm->pcap[rx_tx].pcap_main.n_packets_to_capture = + PCAP_DEF_PKT_TO_CAPTURE; } else { @@ -1812,9 +1812,7 @@ pcap_trace_command_internal (vlib_main_t * vm, { vlib_cli_output (vm, "max is %d for any interface to file %s", - vm->pcap[rx_tx].pcap_main.n_packets_to_capture ? - vm->pcap[rx_tx].pcap_main.n_packets_to_capture - : PCAP_DEF_PKT_TO_CAPTURE, + vm->pcap[rx_tx].pcap_main.n_packets_to_capture, vm->pcap[rx_tx].pcap_main.file_name ? (u8 *) vm->pcap[rx_tx].pcap_main.file_name : (u8 *) "/tmp/vpe.pcap"); @@ -1822,10 +1820,7 @@ pcap_trace_command_internal (vlib_main_t * vm, else { vlib_cli_output (vm, "max is %d for interface %U to file %s", - vm->pcap[rx_tx].pcap_main.n_packets_to_capture - ? vm->pcap[rx_tx]. - pcap_main.n_packets_to_capture : - PCAP_DEF_PKT_TO_CAPTURE, + vm->pcap[rx_tx].pcap_main.n_packets_to_capture, format_vnet_sw_if_index_name, vnm, vm->pcap[rx_tx].pcap_sw_if_index, vm->pcap[rx_tx]. diff --git a/src/vnet/interface_output.c b/src/vnet/interface_output.c index 884c00d93cf..b4832bf2bba 100644 --- a/src/vnet/interface_output.c +++ b/src/vnet/interface_output.c @@ -1407,7 +1407,7 @@ pcap_drop_trace_command_fn (vlib_main_t * vm, clib_memset (&im->pcap_main, 0, sizeof (im->pcap_main)); im->pcap_main.file_name = (char *) im->pcap_filename; - im->pcap_main.n_packets_to_capture = 100; + im->pcap_main.n_packets_to_capture = PCAP_DEF_PKT_TO_CAPTURE; if (im->pcap_pkts_to_capture) im->pcap_main.n_packets_to_capture = im->pcap_pkts_to_capture; |