From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id E842045BE5; Mon, 28 Oct 2024 03:22:21 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 6BD9D40B8C; Mon, 28 Oct 2024 03:21:57 +0100 (CET) Received: from mail-pl1-f175.google.com (mail-pl1-f175.google.com [209.85.214.175]) by mails.dpdk.org (Postfix) with ESMTP id 84FEA4066F for ; Mon, 28 Oct 2024 03:21:46 +0100 (CET) Received: by mail-pl1-f175.google.com with SMTP id d9443c01a7336-20cb47387ceso32180735ad.1 for ; Sun, 27 Oct 2024 19:21:46 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1730082106; x=1730686906; darn=dpdk.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=glCqBIq6hUrPBN7nC8uIsJYHIiXFMLCfhNidhPzB8W8=; b=0txj5Grr1peOoqpuegDJE1lSAWGbCqHmDGa/b9WJzGLKHejzuCN8CBqTMvQWM8J1nl P6IrsdHku3Xb1MrDh+8vadmpvAhummvhKmgmeDGIqv/Ml7M8y8IATmQVv6m0Q/lF1A+o FzrmkNWeC7hBtUujeq0XDXqDuGkNKXTle9MnUXOffNsfpJF7sBzAH/OHX4+L4J9DJUnC zkjYLr3M72v7nui27h+eNcu+32YXrpJdbK+gRoxzIanX32WiTSGIPIkI/Qvi7OVMzhmH AVFb2jQXv/R0ttnfBC7C+N4BiIf/GG3nNZOe/fnGqemPcHpr5zLVfioF69qKIqK4JIy1 l+ag== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1730082106; x=1730686906; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=glCqBIq6hUrPBN7nC8uIsJYHIiXFMLCfhNidhPzB8W8=; b=YKh2DGOtEpAvqnyd0zpqrXG6G0CqCQkldJRbW7ToH7WGJUWWzTAI13pP8L/dJ6roOQ hZmZszX6+Efmd/TINU3oDE2bARCzb1lUjmqHng4ho6WdBAABpJEJVEWjZF1CQAIql1uO 3hzOgdjv2/Ck5fBKaSSXzZ0ciKjfwtIg/W63ZSEJ6veXxXut9g/tFYZq3H+YYoTTVhbk NzCrSzpREvkxPVvhrKmJMFZvGVFMkEJ9mPXuY5j8KeSWHktRzRoPu1ikzMAYRpZiLUUw BNtL7X1I9nn8p4dWw1rLQ89SoWJKsZBkvAasiuowoT5kgimiQUCys1eaYY9h96ew/Z0k E3IQ== X-Gm-Message-State: AOJu0YxrbnJzV+JldLtgkHreSGZA7LRnGYEuUwuEx7iCaW4LwcRbVOXf b4BvKe/+7PuJYFGIu781PRLHFWcRG250aNVrbpOWkQuhcq3xfTtaUhFZYyDToLNck5JITpZ1T7S vfls= X-Google-Smtp-Source: AGHT+IFk9lpWnvJbLq/HDjJPSosUiDjhkEkbGYzVbXZMhT1+XeHpnvHmTNhOEDjB1xjemg+/PwdAfQ== X-Received: by 2002:a17:902:d581:b0:20c:bb1c:d216 with SMTP id d9443c01a7336-210c68b6e48mr89212925ad.21.1730082105643; Sun, 27 Oct 2024 19:21:45 -0700 (PDT) Received: from hermes.local (204-195-96-226.wavecable.com. [204.195.96.226]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-210bc04b8bbsm41095615ad.244.2024.10.27.19.21.45 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 27 Oct 2024 19:21:45 -0700 (PDT) From: Stephen Hemminger To: dev@dpdk.org Cc: Stephen Hemminger , Aman Singh Subject: [PATCH v10 6/8] test-pmd: add hex decode Date: Sun, 27 Oct 2024 19:19:32 -0700 Message-ID: <20241028022131.142609-7-stephen@networkplumber.org> X-Mailer: git-send-email 2.45.2 In-Reply-To: <20241028022131.142609-1-stephen@networkplumber.org> References: <20240312220129.70667-1-stephen@networkplumber.org> <20241028022131.142609-1-stephen@networkplumber.org> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org This adds new command: testpmd> set format hex which decodes packet in hex. Signed-off-by: Stephen Hemminger --- app/test-pmd/cmdline.c | 40 +++++++++++++++ app/test-pmd/config.c | 11 +++++ app/test-pmd/testpmd.c | 1 + app/test-pmd/testpmd.h | 7 +++ app/test-pmd/util.c | 54 ++++++++++++++++----- doc/guides/testpmd_app_ug/testpmd_funcs.rst | 13 +++++ 6 files changed, 115 insertions(+), 11 deletions(-) diff --git a/app/test-pmd/cmdline.c b/app/test-pmd/cmdline.c index a6dfa116c7..fb73087880 100644 --- a/app/test-pmd/cmdline.c +++ b/app/test-pmd/cmdline.c @@ -318,6 +318,9 @@ static void cmd_help_long_parsed(void *parsed_result, "set output (filename)\n" " Set the packet debug log file\n\n" + "set format (verbose|hex)\n" + " Set the format of packet log\\n" + "set log global|(type) (level)\n" " Set the log level.\n\n" @@ -4150,6 +4153,42 @@ static cmdline_parse_inst_t cmd_set_output = { }, }; +/* *** SET FORMAT OF PACKET LOG */ +struct cmd_set_format_result { + cmdline_fixed_string_t set; + cmdline_fixed_string_t format; + cmdline_fixed_string_t value; +}; + +static void +cmd_set_format_parsed(void *parsed_result, + __rte_unused struct cmdline *cl, + __rte_unused void *data) +{ + struct cmd_set_format_result *res = parsed_result; + + set_output_format(res->value); +} + +static cmdline_parse_token_string_t cmd_set_format_set = + TOKEN_STRING_INITIALIZER(struct cmd_set_format_result, set, "set"); +static cmdline_parse_token_string_t cmd_set_format_output = + TOKEN_STRING_INITIALIZER(struct cmd_set_format_result, format, "format"); +static cmdline_parse_token_string_t cmd_set_format_value = + TOKEN_STRING_INITIALIZER(struct cmd_set_format_result, value, "verbose#hex"); + +static cmdline_parse_inst_t cmd_set_format = { + .f = cmd_set_format_parsed, + .data = NULL, + .help_str = "set format verbose|hex", + .tokens = { + (void *)&cmd_set_format_set, + (void *)&cmd_set_format_output, + (void *)&cmd_set_format_value, + NULL, + }, +}; + /* *** SET LOG LEVEL CONFIGURATION *** */ struct cmd_set_log_result { @@ -13685,6 +13724,7 @@ static cmdline_parse_ctx_t builtin_ctx[] = { &cmd_read_rxd_txd, &cmd_stop, &cmd_mac_addr, + &cmd_set_format, &cmd_set_fwd_eth_peer, &cmd_set_qmap, &cmd_set_xstats_hide_zero, diff --git a/app/test-pmd/config.c b/app/test-pmd/config.c index d806cea3d3..86c18a7dc1 100644 --- a/app/test-pmd/config.c +++ b/app/test-pmd/config.c @@ -6340,6 +6340,17 @@ set_verbose_level(uint16_t vb_level) configure_rxtx_dump_callbacks(verbose_level); } +void +set_output_format(const char *mode) +{ + if (!strcmp(mode, "verbose")) + output_format = OUTPUT_MODE_VERBOSE; + else if (!strcmp(mode, "hex")) + output_format = OUTPUT_MODE_HEX; + else + fprintf(stderr, "Unknown output format '%s'\n", mode); +} + void set_output_file(const char *filename) { diff --git a/app/test-pmd/testpmd.c b/app/test-pmd/testpmd.c index 7790ba6ce0..f4fd51e46f 100644 --- a/app/test-pmd/testpmd.c +++ b/app/test-pmd/testpmd.c @@ -100,6 +100,7 @@ uint16_t verbose_level = 0; /**< Silent by default. */ RTE_ATOMIC(FILE *) output_file; /**< log to console by default. */ +enum output_mode output_format; /**< default to original mode. */ int testpmd_logtype; /**< Log type for testpmd logs */ /* use main core for command line ? */ diff --git a/app/test-pmd/testpmd.h b/app/test-pmd/testpmd.h index e5affd44a2..6c21fb16f8 100644 --- a/app/test-pmd/testpmd.h +++ b/app/test-pmd/testpmd.h @@ -488,6 +488,11 @@ enum dcb_mode_enable DCB_ENABLED }; +enum output_mode { + OUTPUT_MODE_VERBOSE = 0, + OUTPUT_MODE_HEX, +}; + extern uint8_t xstats_hide_zero; /**< Hide zero values for xstats display */ /* globals used for configuration */ @@ -495,6 +500,7 @@ extern uint8_t record_core_cycles; /**< Enables measurement of CPU cycles */ extern uint8_t record_burst_stats; /**< Enables display of RX and TX bursts */ extern uint16_t verbose_level; /**< Drives messages being displayed, if any. */ extern RTE_ATOMIC(FILE *) output_file; /**< Where packet data is written */ +extern enum output_mode output_format; /**< Format of packet decode */ extern int testpmd_logtype; /**< Log type for testpmd logs */ extern uint8_t interactive; extern uint8_t auto_start; @@ -1107,6 +1113,7 @@ void set_xstats_hide_zero(uint8_t on_off); void set_record_core_cycles(uint8_t on_off); void set_record_burst_stats(uint8_t on_off); void set_verbose_level(uint16_t vb_level); +void set_output_format(const char *mode); void set_output_file(const char *filename); void set_rx_pkt_segments(unsigned int *seg_lengths, unsigned int nb_segs); void set_rx_pkt_hdrs(unsigned int *seg_protos, unsigned int nb_segs); diff --git a/app/test-pmd/util.c b/app/test-pmd/util.c index 2446687090..130821fddb 100644 --- a/app/test-pmd/util.c +++ b/app/test-pmd/util.c @@ -16,6 +16,7 @@ #include "testpmd.h" #define MAX_STRING_LEN 8192 +#define MAX_DUMP_LEN 1024 #define MKDUMPSTR(buf, buf_size, cur_len, ...) \ do { \ @@ -67,9 +68,9 @@ get_timestamp(const struct rte_mbuf *mbuf) timestamp_dynfield_offset, rte_mbuf_timestamp_t *); } -static inline void -dump_pkt_burst(uint16_t port_id, uint16_t queue, struct rte_mbuf *pkts[], - uint16_t nb_pkts, int is_rx) +static void +dump_pkt_verbose(FILE *outf, uint16_t port_id, uint16_t queue, + struct rte_mbuf *pkts[], uint16_t nb_pkts, int is_rx) { struct rte_mbuf *mb; const struct rte_ether_hdr *eth_hdr; @@ -89,14 +90,6 @@ dump_pkt_burst(uint16_t port_id, uint16_t queue, struct rte_mbuf *pkts[], size_t buf_size = MAX_STRING_LEN; size_t cur_len = 0; uint64_t restore_info_dynflag; - FILE *outf; - - if (!nb_pkts) - return; - - outf = rte_atomic_load_explicit(&output_file, rte_memory_order_relaxed); - if (!outf) - return; restore_info_dynflag = rte_flow_restore_info_dynflag(); MKDUMPSTR(print_buf, buf_size, cur_len, @@ -303,6 +296,45 @@ dump_pkt_burst(uint16_t port_id, uint16_t queue, struct rte_mbuf *pkts[], fprintf(outf, "%s", print_buf); cur_len = 0; } +} + +static void +dump_pkt_hex(FILE *outf, uint16_t port_id, uint16_t queue, + struct rte_mbuf *pkts[], uint16_t nb_pkts, int is_rx) +{ + fprintf(outf, "port %u/queue %u: %s %u packets\n", port_id, queue, + is_rx ? "received" : "sent", (unsigned int) nb_pkts); + + for (uint16_t i = 0; i < nb_pkts; i++) { + rte_pktmbuf_dump(outf, pkts[i], MAX_DUMP_LEN); + fprintf(outf, "\n"); + } +} + + +static void +dump_pkt_burst(uint16_t port_id, uint16_t queue, struct rte_mbuf *pkts[], + uint16_t nb_pkts, int is_rx) +{ + FILE *outf; + + if (!nb_pkts) + return; + + outf = rte_atomic_load_explicit(&output_file, rte_memory_order_relaxed); + if (unlikely(!outf)) + return; + + switch (output_format) { + case OUTPUT_MODE_VERBOSE: + dump_pkt_verbose(outf, port_id, queue, pkts, nb_pkts, is_rx); + return; + case OUTPUT_MODE_HEX: + dump_pkt_hex(outf, port_id, queue, pkts, nb_pkts, is_rx); + break; + default: + return; + } fflush(outf); } diff --git a/doc/guides/testpmd_app_ug/testpmd_funcs.rst b/doc/guides/testpmd_app_ug/testpmd_funcs.rst index 6aef74ff45..088dffd9fd 100644 --- a/doc/guides/testpmd_app_ug/testpmd_funcs.rst +++ b/doc/guides/testpmd_app_ug/testpmd_funcs.rst @@ -691,6 +691,19 @@ Redirect the debug log:: testpmd> set output /tmp/packet.log +set format +~~~~~~~~~~ + +Chose the output format for packet debug log:: + + testpmd> set format verbose|hex + +Available formats are: + +* ``verbose`` print the packet meta data information +* ``hex`` print the mbuf flags and data in hex + + set verbose ~~~~~~~~~~~ -- 2.45.2