From f59993186236727b473faea922999932b2643aed Mon Sep 17 00:00:00 2001 From: Dave Barach Date: Wed, 16 Jan 2019 08:21:16 -0500 Subject: DOC ONLY: update readme.md Remove the source code, replace with reference to the upstreamed patch. Change-Id: I6f5430b1e9804530afbec1805353852e1da745d6 Signed-off-by: Dave Barach --- extras/wireshark/diffs.txt | 12 -- extras/wireshark/packet-vpp.c | 432 ------------------------------------------ extras/wireshark/readme.md | 53 +++--- 3 files changed, 28 insertions(+), 469 deletions(-) delete mode 100644 extras/wireshark/diffs.txt delete mode 100644 extras/wireshark/packet-vpp.c diff --git a/extras/wireshark/diffs.txt b/extras/wireshark/diffs.txt deleted file mode 100644 index 66688d3311e..00000000000 --- a/extras/wireshark/diffs.txt +++ /dev/null @@ -1,12 +0,0 @@ -diff --git a/epan/dissectors/CMakeLists.txt b/epan/dissectors/CMakeLists.txt -index 6fe8356f4e..6294ab5504 100644 ---- a/epan/dissectors/CMakeLists.txt -+++ b/epan/dissectors/CMakeLists.txt -@@ -1812,6 +1812,7 @@ set(DISSECTOR_SRC - ${CMAKE_CURRENT_SOURCE_DIR}/packet-vnc.c - ${CMAKE_CURRENT_SOURCE_DIR}/packet-vntag.c - ${CMAKE_CURRENT_SOURCE_DIR}/packet-vp8.c -+ ${CMAKE_CURRENT_SOURCE_DIR}/packet-vpp.c - ${CMAKE_CURRENT_SOURCE_DIR}/packet-vrrp.c - ${CMAKE_CURRENT_SOURCE_DIR}/packet-vrt.c - ${CMAKE_CURRENT_SOURCE_DIR}/packet-vsip.c diff --git a/extras/wireshark/packet-vpp.c b/extras/wireshark/packet-vpp.c deleted file mode 100644 index 0327a88d7f4..00000000000 --- a/extras/wireshark/packet-vpp.c +++ /dev/null @@ -1,432 +0,0 @@ -/* packet-vpp.c - * - * Routines for the disassembly of fd.io vpp project - * dispatch captures - * - * Copyright (c) 2018 Cisco and/or its affiliates. - * Licensed under the Apache License, Version 2.0 (the "License"); - * you may not use this file except in compliance with the License. - * You may obtain a copy of the License at: - * - * http://www.apache.org/licenses/LICENSE-2.0 - * - * Unless required by applicable law or agreed to in writing, software - * distributed under the License is distributed on an "AS IS" BASIS, - * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. - * See the License for the specific language governing permissions and - * limitations under the License. - * - * This version is not to be upstreamed as-is, since it hooks up the - * vpp dissector to WTAP_ENCAP_USER13, a test encap type. - */ - -#include "config.h" - -#include -#include -#include -#include -#include -#include -#include -#include - -void proto_register_vpp(void); -void proto_reg_handoff_vpp(void); - -static int proto_vpp = -1; -static int proto_vpp_metadata = -1; -static int proto_vpp_opaque = -1; -static int proto_vpp_opaque2 = -1; -static int proto_vpp_trace = -1; -static int hf_vpp_nodename = -1; -static int hf_vpp_metadata = -1; -static int hf_vpp_buffer_index = -1; -static int hf_vpp_buffer_opaque = -1; -static int hf_vpp_buffer_opaque2 = -1; -static int hf_vpp_buffer_trace = -1; - -static gint ett_vpp = -1; -static gint ett_vpp_opaque = -1; -static gint ett_vpp_opaque2 = -1; -static gint ett_vpp_metadata = -1; -static gint ett_vpp_trace = -1; - -static dissector_handle_t vpp_dissector_handle; -static dissector_handle_t vpp_opaque_dissector_handle; -static dissector_handle_t vpp_opaque2_dissector_handle; -static dissector_handle_t vpp_metadata_dissector_handle; -static dissector_handle_t vpp_trace_dissector_handle; - -typedef enum - { - VLIB_NODE_PROTO_HINT_NONE = 0, - VLIB_NODE_PROTO_HINT_ETHERNET, - VLIB_NODE_PROTO_HINT_IP4, - VLIB_NODE_PROTO_HINT_IP6, - VLIB_NODE_PROTO_HINT_TCP, - VLIB_NODE_PROTO_HINT_UDP, - VLIB_NODE_N_PROTO_HINTS, - } vlib_node_proto_hint_t; - -static dissector_handle_t next_dissectors[VLIB_NODE_N_PROTO_HINTS]; - -/* List of next dissectors hints that we know about */ -#define foreach_next_dissector \ -_(VLIB_NODE_PROTO_HINT_ETHERNET, eth_maybefcs) \ -_(VLIB_NODE_PROTO_HINT_IP4, ip) \ -_(VLIB_NODE_PROTO_HINT_IP6, ipv6) \ -_(VLIB_NODE_PROTO_HINT_TCP, tcp) \ -_(VLIB_NODE_PROTO_HINT_UDP, udp) - -static void -add_multi_line_string_to_tree(proto_tree *tree, tvbuff_t *tvb, gint start, - gint len, int hf) -{ - gint next; - int line_len; - int data_len; - - while (len > 0) { - line_len = tvb_find_line_end(tvb, start, len, &next, FALSE); - data_len = next - start; - proto_tree_add_string(tree, hf, tvb, start, data_len, - tvb_format_stringzpad(tvb, start, line_len)); - start += data_len; - len -= data_len; - } -} - -static int -dissect_vpp_metadata (tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, - void* data _U_) -{ - int offset = 0; - proto_item *ti; - proto_tree *metadata_tree; - gint metadata_string_length; - - col_set_str(pinfo->cinfo, COL_PROTOCOL, "VPP-Metadata"); - col_clear(pinfo->cinfo, COL_INFO); - - ti = proto_tree_add_item(tree, proto_vpp_metadata, tvb, offset, -1, ENC_NA); - metadata_tree = proto_item_add_subtree(ti, ett_vpp_metadata); - - /* How long is the metadata string? */ - metadata_string_length = tvb_strsize (tvb, offset); - - add_multi_line_string_to_tree (metadata_tree, tvb, 0, - metadata_string_length, - hf_vpp_metadata); - return tvb_captured_length(tvb); -} - -static int -dissect_vpp_trace (tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, - void* data _U_) -{ - int offset = 0; - proto_item *ti; - proto_tree *trace_tree; - gint trace_string_length; - - col_set_str(pinfo->cinfo, COL_PROTOCOL, "VPP-Trace"); - col_clear(pinfo->cinfo, COL_INFO); - - ti = proto_tree_add_item(tree, proto_vpp_trace, tvb, offset, -1, ENC_NA); - trace_tree = proto_item_add_subtree(ti, ett_vpp_trace); - - /* How long is the trace string? */ - trace_string_length = tvb_strsize (tvb, offset); - - add_multi_line_string_to_tree (trace_tree, tvb, 0, - trace_string_length, - hf_vpp_buffer_trace); - return tvb_captured_length(tvb); -} - - -static int -dissect_vpp_opaque (tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, - void* data _U_) -{ - int offset = 0; - proto_item *ti; - proto_tree *opaque_tree; - gint opaque_string_length; - - col_set_str(pinfo->cinfo, COL_PROTOCOL, "VPP-Opaque"); - col_clear(pinfo->cinfo, COL_INFO); - - ti = proto_tree_add_item(tree, proto_vpp_opaque, tvb, offset, -1, ENC_NA); - opaque_tree = proto_item_add_subtree(ti, ett_vpp_opaque); - - opaque_string_length = tvb_strsize (tvb, offset); - add_multi_line_string_to_tree (opaque_tree, tvb, 0, opaque_string_length, - hf_vpp_buffer_opaque); - - return tvb_captured_length(tvb); -} - -static int -dissect_vpp_opaque2 (tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, - void* data _U_) -{ - int offset = 0; - proto_item *ti; - proto_tree *opaque2_tree; - gint opaque2_string_length; - - col_set_str(pinfo->cinfo, COL_PROTOCOL, "VPP-Opaque2"); - col_clear(pinfo->cinfo, COL_INFO); - - ti = proto_tree_add_item(tree, proto_vpp_opaque2, tvb, offset, -1, ENC_NA); - opaque2_tree = proto_item_add_subtree(ti, ett_vpp_opaque2); - - opaque2_string_length = tvb_strsize (tvb, offset); - add_multi_line_string_to_tree (opaque2_tree, tvb, 0, opaque2_string_length, - hf_vpp_buffer_opaque2); - - return tvb_captured_length(tvb); -} - - -static int -dissect_vpp(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void* data _U_) -{ - proto_item *ti; - proto_tree *vpp_tree; - tvbuff_t *metadata_tvb, *opaque_tvb, *opaque2_tvb, *eth_tvb, *trace_tvb; - int offset = 0; - guint8 major_version, minor_version, string_count, protocol_hint; - guint8 *name; - guint len; - guint8 maybe_protocol_id; - dissector_handle_t use_this_dissector; - - col_set_str(pinfo->cinfo, COL_PROTOCOL, "VPP"); - col_clear(pinfo->cinfo, COL_INFO); - - ti = proto_tree_add_item(tree, proto_vpp, tvb, offset, -1, ENC_NA); - vpp_tree = proto_item_add_subtree(ti, ett_vpp); - - major_version = tvb_get_guint8 (tvb, offset); - offset++; - - minor_version = tvb_get_guint8 (tvb, offset); - offset++; - - if (major_version != 1 || minor_version != 0) - ws_debug_printf ("WARNING: version mismatch (%d, %d)", - major_version, minor_version); - - /* Number of counted strings in this trace record */ - string_count = tvb_get_guint8 (tvb, offset); - offset++; - - /* - * Hint: protocol which should be at b->data[b->current_data] - * It will be a while before vpp sends useful hints for every - * possible node, see heuristic below - */ - protocol_hint = tvb_get_guint8 (tvb, offset); - offset++; - - /* Buffer Index */ - proto_tree_add_item(vpp_tree, hf_vpp_buffer_index, tvb, - offset, 4, ENC_BIG_ENDIAN); - offset += 4; - - /* Nodename */ - len = tvb_strsize (tvb, offset); - name = tvb_get_string_enc (wmem_packet_scope(), tvb, offset, len, - ENC_ASCII); - proto_tree_add_string (tree, hf_vpp_nodename, tvb, offset, len, name); - offset += len; - - /* Metadata */ - len = tvb_strsize (tvb, offset); - metadata_tvb = tvb_new_subset_remaining (tvb, offset); - call_dissector (vpp_metadata_dissector_handle, metadata_tvb, pinfo, tree); - offset += len; - - /* Opaque */ - len = tvb_strsize (tvb, offset); - opaque_tvb = tvb_new_subset_remaining (tvb, offset); - call_dissector (vpp_opaque_dissector_handle, opaque_tvb, pinfo, tree); - offset += len; - - /* Opaque2 */ - len = tvb_strsize (tvb, offset); - opaque2_tvb = tvb_new_subset_remaining (tvb, offset); - call_dissector (vpp_opaque2_dissector_handle, opaque2_tvb, pinfo, tree); - offset += len; - - /* Trace, if present */ - if (string_count > 4) - { - len = tvb_strsize (tvb, offset); - trace_tvb = tvb_new_subset_remaining (tvb, offset); - call_dissector (vpp_trace_dissector_handle, trace_tvb, pinfo, tree); - offset += len; - } - - eth_tvb = tvb_new_subset_remaining (tvb, offset); - - /* - * Delegate the rest of the packet dissection to the per-node - * next dissector in the foreach_node_to_dissector_pair list - * - * Failing that, pretend its an ethernet packet - */ - if (protocol_hint >= array_length(next_dissectors)) { - ws_debug_printf ("protocol_hint %d out of range (max %d)", - (int) protocol_hint, - (int) array_length(next_dissectors)); - protocol_hint = 0; - } - /* See setup for hint == 0 below */ - use_this_dissector = next_dissectors [protocol_hint]; - if (protocol_hint == 0) { - maybe_protocol_id = tvb_get_guint8 (tvb, offset); - - switch (maybe_protocol_id) { - case 0x45: - use_this_dissector = next_dissectors[VLIB_NODE_PROTO_HINT_IP4]; - break; - case 0x60: - use_this_dissector = next_dissectors[VLIB_NODE_PROTO_HINT_IP6]; - break; - default: - break; - } - } - call_dissector (use_this_dissector, eth_tvb, pinfo, tree); - return tvb_captured_length(tvb); -} - -void -proto_register_vpp(void) -{ - static hf_register_info vpp_hf[] = { - { &hf_vpp_buffer_index, - { "BufferIndex", "vpp.BufferIndex", FT_UINT32, BASE_HEX, NULL, 0x0, - NULL, HFILL }, - }, - { &hf_vpp_nodename, - { "NodeName", "vpp.NodeName", FT_STRINGZ, BASE_NONE, NULL, 0x0, - NULL, HFILL }, - }, - }; - - static hf_register_info metadata_hf[] = { - { &hf_vpp_metadata, - { "Metadata", "vpp.metadata", - FT_STRINGZ, BASE_NONE, NULL, 0x0, NULL, HFILL }, - }, - }; - - static hf_register_info opaque_hf[] = { - { &hf_vpp_buffer_opaque, - { "Opaque", "vpp.opaque", - FT_STRINGZ, BASE_NONE, NULL, 0x0, NULL, HFILL }, - }, - }; - - static hf_register_info opaque2_hf[] = { - { &hf_vpp_buffer_opaque2, - { "Opaque2", "vpp.opaque2", - FT_STRINGZ, BASE_NONE, NULL, 0x0, NULL, HFILL }, - }, - }; - - static hf_register_info trace_hf[] = { - { &hf_vpp_buffer_trace, - { "Trace", "vpp.trace", FT_STRINGZ, BASE_NONE, NULL, 0x0, - NULL, HFILL }, - }, - }; - - static gint *vpp_ett[] = { - &ett_vpp, - }; - static gint *ett_metadata[] = { - &ett_vpp_metadata, - }; - static gint *ett_opaque[] = { - &ett_vpp_opaque, - }; - static gint *ett_opaque2[] = { - &ett_vpp_opaque2, - }; - static gint *ett_trace[] = { - &ett_vpp_trace, - }; - - proto_vpp = proto_register_protocol("VPP Dispatch Trace", "VPP", "vpp"); - proto_register_field_array(proto_vpp, vpp_hf, array_length(vpp_hf)); - proto_register_subtree_array (vpp_ett, array_length(vpp_ett)); - register_dissector("vpp", dissect_vpp, proto_vpp); - - proto_vpp_metadata = proto_register_protocol("VPP Buffer Metadata", - "VPP-Metadata", - "vpp-metadata"); - proto_register_field_array(proto_vpp_metadata, metadata_hf, - array_length(metadata_hf)); - proto_register_subtree_array (ett_metadata, array_length(ett_metadata)); - register_dissector("vppMetadata", dissect_vpp_metadata, proto_vpp_metadata); - - proto_vpp_opaque = proto_register_protocol("VPP Buffer Opaque", "VPP-Opaque", - "vpp-opaque"); - proto_register_field_array(proto_vpp_opaque, opaque_hf, - array_length(opaque_hf)); - proto_register_subtree_array (ett_opaque, array_length(ett_opaque)); - register_dissector("vppOpaque", dissect_vpp_opaque, proto_vpp_opaque); - - proto_vpp_opaque2 = proto_register_protocol("VPP Buffer Opaque2", "VPP-Opaque2", - "vpp-opaque2"); - proto_register_field_array(proto_vpp_opaque2, opaque2_hf, - array_length(opaque2_hf)); - proto_register_subtree_array (ett_opaque2, array_length(ett_opaque2)); - register_dissector("vppOpaque2", dissect_vpp_opaque2, proto_vpp_opaque2); - - - proto_vpp_trace = proto_register_protocol("VPP Buffer Trace", "VPP-Trace", - "vpp-trace"); - proto_register_field_array(proto_vpp_trace, trace_hf, - array_length(trace_hf)); - proto_register_subtree_array (ett_trace, array_length(ett_trace)); - register_dissector("vppTrace", dissect_vpp_trace, proto_vpp_trace); - -#define _(idx,dname) next_dissectors[idx] = find_dissector (#dname); - foreach_next_dissector; -#undef _ - - /* if all else fails, dissect data as if ethernet MAC */ - next_dissectors[VLIB_NODE_PROTO_HINT_NONE] = - next_dissectors [VLIB_NODE_PROTO_HINT_ETHERNET]; -} - -void -proto_reg_handoff_vpp(void) -{ - vpp_dissector_handle = find_dissector("vpp"); - vpp_metadata_dissector_handle = find_dissector("vppMetadata"); - vpp_opaque_dissector_handle = find_dissector("vppOpaque"); - vpp_opaque2_dissector_handle = find_dissector("vppOpaque2"); - vpp_trace_dissector_handle = find_dissector("vppTrace"); - dissector_add_uint("wtap_encap", WTAP_ENCAP_VPP, vpp_dissector_handle); -} - -/* - * Editor modelines - http://www.wireshark.org/tools/modelines.html - * - * Local variables: - * c-basic-offset: 4 - * tab-width: 8 - * indent-tabs-mode: nil - * End: - * - * vi: set shiftwidth=4 tabstop=8 expandtab: - * :indentSize=4:tabSize=8:noTabs=true: - */ diff --git a/extras/wireshark/readme.md b/extras/wireshark/readme.md index d47806d53f2..fe4fdbfeb5a 100644 --- a/extras/wireshark/readme.md +++ b/extras/wireshark/readme.md @@ -1,25 +1,31 @@ How to build a vpp dispatch trace aware Wireshark ================================================= -At some point, we will upstream our vpp pcap dispatch trace dissector. -It's not finished - contributions welcome - and we have to work through -whatever issues will be discovered during the upstreaming process. +We have upstreamed our vpp pcap dispatch trace dissector. After +working through a laundry list of issues discovered during the +upstreaming process, it appears that the code is close to being +merged. See https://code.wireshark.org/review/#/c/31466. -On the other hand, it's ready for some tire-kicking. Here's how to build -wireshark +As of this writing, the simplest way to build a vpp dispatch trace +aware wireshark is to clone the wireshark repo, and apply the vpp +dissector patch. -Download and patch wireshark source code ------------------------------------------ -The wireshark git repo is large, so it takes a while to clone. +Download wireshark source code +------------------------------ + +The wireshark git repo is large, so it takes a while to clone. ``` git clone https://code.wireshark.org/review/wireshark - cp .../extras/wireshark/packet-vpp.c wireshark/epan/dissectors - patch -p1 < .../extras/wireshark/diffs.txt ``` -The small patch adds packet-vpp.c to the dissector list. +Download Gerrit 31466 using the URL shown above. If you have "git +review" set up, it's as simple as "git review -d 31466" in the wireshark +workspace. + +Alternatively, download a patch-file from the gerrit server and apply +the patch. Install prerequisite Debian packages ------------------------------------ @@ -36,13 +42,14 @@ to compile wireshark, beyond what's typically installed on an Ubuntu Compile Wireshark ----------------- -Mercifully, Wireshark uses cmake, so it's relatively easy to build, at -least on Ubuntu 18.04. - +Wireshark uses cmake, so it's relatively easy to build, at least on +Ubuntu 18.04. ``` $ cd wireshark - $ cmake -G Ninja + $ mkdir build + $ cd build + $ cmake -G Ninja ../ $ ninja -j 8 $ sudo ninja install ``` @@ -67,15 +74,11 @@ dispatch trace capture like so: Display in Wireshark -------------------- -Display /tmp/vppcapture in the vpp-enabled version of wireshark. With -any luck, normal version of wireshark will refuse to process vpp -dispatch trace pcap files because they won't understand the encap type. +Display /tmp/vppcapture in the vpp-enabled version of wireshark. +Normal version of wireshark will refuse to process vpp dispatch trace +pcap files because they won't understand the encap type. Set wireshark to filter on vpp.bufferindex to watch a single packet -traverse the forwarding graph. Otherwise, you'll see a vector of packets -in e.g. ip4-lookup, then a vector of packets in ip4-rewrite, etc. - - - - - +traverse the forwarding graph. Otherwise, you'll see a vector of +packets in e.g. ip4-lookup, then a vector of packets in ip4-rewrite, +etc. -- cgit 1.2.3-korg