DPDK patches and discussions
 help / color / mirror / Atom feed
From: Kevin Laatz <kevin.laatz@intel.com>
To: dev@dpdk.org
Cc: cristian.dumitrescu@intel.com, jasvinder.singh@intel.com,
	Kevin Laatz <kevin.laatz@intel.com>
Subject: [dpdk-dev] [PATCH] examples/ip_pipeline: add link show to the cli
Date: Fri,  8 Jun 2018 17:30:53 +0100	[thread overview]
Message-ID: <20180608163053.37891-1-kevin.laatz@intel.com> (raw)

Add the functionality to track links in the application. This enables the
user to print the name, mac address and per-port statistics for each link
in the application.

Signed-off-by: Kevin Laatz <kevin.laatz@intel.com>
---
 examples/ip_pipeline/cli.c  | 103 ++++++++++++++++++++++++++++++++++++++++++++
 examples/ip_pipeline/link.c |   6 +++
 examples/ip_pipeline/link.h |   3 ++
 3 files changed, 112 insertions(+)

diff --git a/examples/ip_pipeline/cli.c b/examples/ip_pipeline/cli.c
index c9587f5..b7fba3b 100644
--- a/examples/ip_pipeline/cli.c
+++ b/examples/ip_pipeline/cli.c
@@ -9,6 +9,7 @@
 
 #include <rte_common.h>
 #include <rte_cycles.h>
+#include <rte_ethdev.h>
 
 #include "cli.h"
 #include "kni.h"
@@ -4355,6 +4356,103 @@ cmd_thread_pipeline_disable(char **tokens,
 	}
 }
 
+/* Print the link stats and info */
+static void
+print_link_info(struct link *link, char *out, size_t out_size)
+{
+	struct rte_eth_stats stats;
+	struct ether_addr mac_addr;
+	struct rte_eth_link eth_link;
+
+	memset(&stats, 0, sizeof(stats));
+	rte_eth_stats_get(link->port_id, &stats);
+
+	rte_eth_macaddr_get(link->port_id, &mac_addr);
+	rte_eth_link_get(link->port_id, &eth_link);
+
+	snprintf(out, out_size,
+		"\n"
+		"Link name: %s\n"
+		"\tPort %u MAC: %02"PRIx8":%02"PRIx8":%02"PRIx8
+		":%02"PRIx8":%02"PRIx8":%02"PRIx8"\n"
+		"\tLink Status: %s\n"
+		"\tLink Speed: %u\n"
+		"\n"
+		"\tRX info:\n"
+		"\t\tQueues: %u\n"
+		"\t\tPackets: %" PRIu64
+		"\t\tBytes: %" PRIu64"\n"
+		"\t\tErrors: %" PRIu64
+		"\t\tMissed: %" PRIu64
+		"\t\tNo-mbuf: %" PRIu64"\n"
+		"\tTX info:\n"
+		"\t\tQueues: %u\n"
+		"\t\tPackets: %" PRIu64
+		"\t\tBytes: %" PRIu64"\n"
+		"\t\tErrors: %" PRIu64"\n",
+		link->name,
+		link->port_id,
+		mac_addr.addr_bytes[0], mac_addr.addr_bytes[1],
+		mac_addr.addr_bytes[2], mac_addr.addr_bytes[3],
+		mac_addr.addr_bytes[4], mac_addr.addr_bytes[5],
+		eth_link.link_status == 1 ? "UP" : "DOWN",
+		eth_link.link_speed,
+		link->n_rxq,
+		stats.ipackets,
+		stats.ibytes,
+		stats.ierrors,
+		stats.imissed,
+		stats.rx_nombuf,
+		link->n_txq,
+		stats.opackets,
+		stats.obytes,
+		stats.oerrors);
+}
+
+/*
+ * link show [<link_name>]
+ */
+static void
+cmd_link_show(char **tokens, uint32_t n_tokens, char *out, size_t out_size)
+{
+	struct link *link;
+	struct link_list *link_list;
+	char *link_name;
+	uint32_t i = 0;
+
+	if (n_tokens != 2 && n_tokens != 3) {
+		snprintf(out, out_size, MSG_ARG_MISMATCH, tokens[0]);
+		return;
+	}
+
+	link_list = links_get();
+
+	if (TAILQ_EMPTY(link_list)) {
+		snprintf(out, out_size, "No links - nothing to show\n");
+		return;
+	}
+
+	if (n_tokens == 2) {
+		TAILQ_FOREACH(link, link_list, node) {
+			size_t os = out_size - (i * strlen(out));
+			char *o = &out[i * strlen(out)];
+
+			if (os == 0) {
+				snprintf(out, out_size, MSG_CMD_FAIL,
+						"Output buffer too small");
+				return;
+			}
+
+			print_link_info(link, o, os);
+			i++;
+		}
+	} else {
+		link_name = tokens[2];
+		link = link_find(link_name);
+		print_link_info(link, out, out_size);
+	}
+}
+
 void
 cli_process(char *in, char *out, size_t out_size)
 {
@@ -4380,6 +4478,11 @@ cli_process(char *in, char *out, size_t out_size)
 	}
 
 	if (strcmp(tokens[0], "link") == 0) {
+		if (strcmp(tokens[1], "show") == 0) {
+			cmd_link_show(tokens, n_tokens, out, out_size);
+			return;
+		}
+
 		cmd_link(tokens, n_tokens, out, out_size);
 		return;
 	}
diff --git a/examples/ip_pipeline/link.c b/examples/ip_pipeline/link.c
index b8a431f..b11f0c1 100644
--- a/examples/ip_pipeline/link.c
+++ b/examples/ip_pipeline/link.c
@@ -36,6 +36,12 @@ link_find(const char *name)
 	return NULL;
 }
 
+struct link_list *
+links_get(void)
+{
+	return &link_list;
+}
+
 static struct rte_eth_conf port_conf_default = {
 	.link_speeds = 0,
 	.rxmode = {
diff --git a/examples/ip_pipeline/link.h b/examples/ip_pipeline/link.h
index 37d3dc4..10f9d51 100644
--- a/examples/ip_pipeline/link.h
+++ b/examples/ip_pipeline/link.h
@@ -30,6 +30,9 @@ link_init(void);
 struct link *
 link_find(const char *name);
 
+struct link_list *
+links_get(void);
+
 struct link_params_rss {
 	uint32_t queue_id[LINK_RXQ_RSS_MAX];
 	uint32_t n_queues;
-- 
2.9.5

             reply	other threads:[~2018-06-08 16:31 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-06-08 16:30 Kevin Laatz [this message]
2018-06-08 16:51 ` Dumitrescu, Cristian
2018-06-15 14:57   ` Laatz, Kevin
2018-06-15 15:31     ` Dumitrescu, Cristian
2018-06-22 14:31 ` [dpdk-dev] [PATCH v2] " Kevin Laatz

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20180608163053.37891-1-kevin.laatz@intel.com \
    --to=kevin.laatz@intel.com \
    --cc=cristian.dumitrescu@intel.com \
    --cc=dev@dpdk.org \
    --cc=jasvinder.singh@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).