From patchwork Tue Jan 24 11:21:38 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Amit Prakash Shukla X-Patchwork-Id: 122477 X-Patchwork-Delegate: thomas@monjalon.net Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id BDF2042478; Tue, 24 Jan 2023 12:23:04 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id ADCEB40223; Tue, 24 Jan 2023 12:23:04 +0100 (CET) Received: from mx0b-0016f401.pphosted.com (mx0b-0016f401.pphosted.com [67.231.156.173]) by mails.dpdk.org (Postfix) with ESMTP id AE59D40150 for ; Tue, 24 Jan 2023 12:23:02 +0100 (CET) Received: from pps.filterd (m0045851.ppops.net [127.0.0.1]) by mx0b-0016f401.pphosted.com (8.17.1.19/8.17.1.19) with ESMTP id 30OBIVrA006200 for ; Tue, 24 Jan 2023 03:23:02 -0800 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=marvell.com; h=from : to : cc : subject : date : message-id : in-reply-to : references : mime-version : content-transfer-encoding : content-type; s=pfpt0220; bh=cuC0P8KiIllibHCGLBiuZfWIub7JD9SABRP8cQ7SrBU=; b=aH0yWYt5ZKZe9LNDMNXZaiHEqIquUG8MqeAh2jjSwrTwzxNNxdaaLXMDSe/7BEBb3JAZ EkD3YT1Q0w9osB5ntJ7aVXITHatVJtmytE1+X1xHXvXePqh8m/Z9Ic+ko1+WFbBCL+r5 +FBOVFAzu5e3EaCP51AkfBmt7XeKVqxvP0T/U2nqE2myvU6wwar/SWVS2y4VpLnMciP9 6VrbekQ/JmTqx1HzLo31ambfriTjMjkmtS4CE1erNPR87EA8jdt6pSr//IW5uCXQzA2i mmx4bqJ67S4yv9V8Pg1PKRBvZk1EbUOLGnEwxd4a2q4bs6sLGacbcWZJPPMYcZXNbZk3 wA== Received: from dc5-exch02.marvell.com ([199.233.59.182]) by mx0b-0016f401.pphosted.com (PPS) with ESMTPS id 3n8gerfp26-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-SHA384 bits=256 verify=NOT) for ; Tue, 24 Jan 2023 03:23:02 -0800 Received: from DC5-EXCH02.marvell.com (10.69.176.39) by DC5-EXCH02.marvell.com (10.69.176.39) with Microsoft SMTP Server (TLS) id 15.0.1497.42; Tue, 24 Jan 2023 03:22:46 -0800 Received: from maili.marvell.com (10.69.176.80) by DC5-EXCH02.marvell.com (10.69.176.39) with Microsoft SMTP Server id 15.0.1497.42 via Frontend Transport; Tue, 24 Jan 2023 03:22:46 -0800 Received: from localhost.localdomain (unknown [10.28.36.157]) by maili.marvell.com (Postfix) with ESMTP id 111383F7068; Tue, 24 Jan 2023 03:22:44 -0800 (PST) From: Amit Prakash Shukla To: Jerin Jacob , Kiran Kumar K , Nithin Dabilpuram CC: , Amit Prakash Shukla Subject: [PATCH v4 3/3] l3fwd-graph: changes to configure pcap capture Date: Tue, 24 Jan 2023 16:51:38 +0530 Message-ID: <20230124112139.1426279-3-amitprakashs@marvell.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20230124112139.1426279-1-amitprakashs@marvell.com> References: <20230112100106.2180665-1-amitprakashs@marvell.com> <20230124112139.1426279-1-amitprakashs@marvell.com> MIME-Version: 1.0 X-Proofpoint-GUID: ukNsvt_mMrucVaEHeepcAhsEZHENClwk X-Proofpoint-ORIG-GUID: ukNsvt_mMrucVaEHeepcAhsEZHENClwk X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.219,Aquarius:18.0.930,Hydra:6.0.562,FMLib:17.11.122.1 definitions=2023-01-23_12,2023-01-24_01,2022-06-22_01 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 Added support to configure pcap capture. Signed-off-by: Amit Prakash Shukla --- v2: - Fixed code style issue - Fixed CI compilation issue on github-robot v3: - Code review suggestion from Stephen - Fixed potential memory leak v4: - Code review suggestion from Jerin doc/guides/sample_app_ug/l3_forward_graph.rst | 23 +++++++ examples/l3fwd-graph/main.c | 62 ++++++++++++++++++- 2 files changed, 83 insertions(+), 2 deletions(-) diff --git a/doc/guides/sample_app_ug/l3_forward_graph.rst b/doc/guides/sample_app_ug/l3_forward_graph.rst index 0a3e0d44ec..3043dba8e2 100644 --- a/doc/guides/sample_app_ug/l3_forward_graph.rst +++ b/doc/guides/sample_app_ug/l3_forward_graph.rst @@ -51,6 +51,9 @@ The application has a number of command line options similar to l3fwd:: [--max-pkt-len PKTLEN] [--no-numa] [--per-port-pool] + [--pcap-enable] + [--pcap-num-cap] + [--pcap-file-name] Where, @@ -69,6 +72,12 @@ Where, * ``--per-port-pool:`` Optional, set to use independent buffer pools per port. Without this option, single buffer pool is used for all ports. +* ``--pcap-enable:`` Optional, Enables packet capture in pcap format on each node with mbuf and node metadata. + +* ``--pcap-num-cap:`` Optional, Number of packets to be captured per core. + +* ``--pcap-file-name:`` Optional, Pcap filename to capture packets in. + For example, consider a dual processor socket platform with 8 physical cores, where cores 0-7 and 16-23 appear on socket 0, while cores 8-15 and 24-31 appear on socket 1. @@ -99,6 +108,20 @@ In this command: | | | | | +----------+-----------+-----------+-------------------------------------+ +To enable pcap trace on each graph, use following command: + +.. code-block:: console + + .//examples/dpdk-l3fwd-graph -l 1,2 -n 4 -- -p 0x3 --config="(0,0,1),(1,0,2)" --pcap-enable --pcap-num-cap= --pcap-file-name "" + +In this command: + +* The --pcap-enable option enables pcap trace on graph nodes. + +* The --pcap-num-cap option enables user to configure number packets to be captured per graph. Default 1024 packets per graph are captured. + +* The --pcap-file-name option enables user to give filename in which packets are to be captured. + Refer to the *DPDK Getting Started Guide* for general information on running applications and the Environment Abstraction Layer (EAL) options. diff --git a/examples/l3fwd-graph/main.c b/examples/l3fwd-graph/main.c index 6dcb6ee92b..5feeab4f0f 100644 --- a/examples/l3fwd-graph/main.c +++ b/examples/l3fwd-graph/main.c @@ -76,6 +76,12 @@ xmm_t val_eth[RTE_MAX_ETHPORTS]; /* Mask of enabled ports */ static uint32_t enabled_port_mask; +/* Pcap trace */ +static char pcap_filename[RTE_GRAPH_PCAP_FILE_SZ]; +static uint64_t packet_to_capture; +static int pcap_trace_enable; + + struct lcore_rx_queue { uint16_t port_id; uint8_t queue_id; @@ -261,7 +267,8 @@ print_usage(const char *prgname) " [--eth-dest=X,MM:MM:MM:MM:MM:MM]" " [--max-pkt-len PKTLEN]" " [--no-numa]" - " [--per-port-pool]\n\n" + " [--per-port-pool]" + " [--num-pkt-cap]\n\n" " -p PORTMASK: Hexadecimal bitmask of ports to configure\n" " -P : Enable promiscuous mode\n" @@ -270,10 +277,30 @@ print_usage(const char *prgname) "port X\n" " --max-pkt-len PKTLEN: maximum packet length in decimal (64-9600)\n" " --no-numa: Disable numa awareness\n" - " --per-port-pool: Use separate buffer pool per port\n\n", + " --per-port-pool: Use separate buffer pool per port\n" + " --pcap-enable: Enables pcap capture\n" + " --pcap-num-cap NUMPKT: Number of packets to capture\n" + " --pcap-file-name NAME: Pcap file name\n\n", prgname); } +static uint64_t +parse_num_pkt_cap(const char *num_pkt_cap) +{ + uint64_t num_pkt; + char *end = NULL; + + /* Parse decimal string */ + num_pkt = strtoull(num_pkt_cap, &end, 10); + if ((num_pkt_cap[0] == '\0') || (end == NULL) || (*end != '\0')) + return 0; + + if (num_pkt == 0) + return 0; + + return num_pkt; +} + static int parse_max_pkt_len(const char *pktlen) { @@ -404,6 +431,9 @@ static const char short_options[] = "p:" /* portmask */ #define CMD_LINE_OPT_NO_NUMA "no-numa" #define CMD_LINE_OPT_MAX_PKT_LEN "max-pkt-len" #define CMD_LINE_OPT_PER_PORT_POOL "per-port-pool" +#define CMD_LINE_OPT_PCAP_ENABLE "pcap-enable" +#define CMD_LINE_OPT_NUM_PKT_CAP "pcap-num-cap" +#define CMD_LINE_OPT_PCAP_FILENAME "pcap-file-name" enum { /* Long options mapped to a short option */ @@ -416,6 +446,9 @@ enum { CMD_LINE_OPT_NO_NUMA_NUM, CMD_LINE_OPT_MAX_PKT_LEN_NUM, CMD_LINE_OPT_PARSE_PER_PORT_POOL, + CMD_LINE_OPT_PARSE_PCAP_ENABLE, + CMD_LINE_OPT_PARSE_NUM_PKT_CAP, + CMD_LINE_OPT_PCAP_FILENAME_CAP, }; static const struct option lgopts[] = { @@ -424,6 +457,9 @@ static const struct option lgopts[] = { {CMD_LINE_OPT_NO_NUMA, 0, 0, CMD_LINE_OPT_NO_NUMA_NUM}, {CMD_LINE_OPT_MAX_PKT_LEN, 1, 0, CMD_LINE_OPT_MAX_PKT_LEN_NUM}, {CMD_LINE_OPT_PER_PORT_POOL, 0, 0, CMD_LINE_OPT_PARSE_PER_PORT_POOL}, + {CMD_LINE_OPT_PCAP_ENABLE, 0, 0, CMD_LINE_OPT_PARSE_PCAP_ENABLE}, + {CMD_LINE_OPT_NUM_PKT_CAP, 1, 0, CMD_LINE_OPT_PARSE_NUM_PKT_CAP}, + {CMD_LINE_OPT_PCAP_FILENAME, 1, 0, CMD_LINE_OPT_PCAP_FILENAME_CAP}, {NULL, 0, 0, 0}, }; @@ -498,6 +534,23 @@ parse_args(int argc, char **argv) per_port_pool = 1; break; + case CMD_LINE_OPT_PARSE_PCAP_ENABLE: + printf("Packet capture enabled\n"); + pcap_trace_enable = 1; + break; + + case CMD_LINE_OPT_PARSE_NUM_PKT_CAP: + packet_to_capture = parse_num_pkt_cap(optarg); + printf("Number of packets to capture: %"PRIu64"\n", + packet_to_capture); + break; + + case CMD_LINE_OPT_PCAP_FILENAME_CAP: + rte_strlcpy(pcap_filename, optarg, + sizeof(pcap_filename)); + printf("Pcap file name: %s\n", pcap_filename); + break; + default: print_usage(prgname); return -1; @@ -1027,6 +1080,11 @@ main(int argc, char **argv) memset(&graph_conf, 0, sizeof(graph_conf)); graph_conf.node_patterns = node_patterns; + /* Pcap config */ + graph_conf.pcap_enable = pcap_trace_enable; + graph_conf.num_pkt_to_capture = packet_to_capture; + graph_conf.pcap_filename = pcap_filename; + for (lcore_id = 0; lcore_id < RTE_MAX_LCORE; lcore_id++) { rte_graph_t graph_id; rte_edge_t i;