[dpdk-dev] [PATCH 3/3] testpmd: Add commands to test administrative link up and down of PMD

[Ouyang Changchun]

This patch adds commands to test the functionality of administrative link up and down of PMD in testpmd.

Signed-off-by: Ouyang Changchun <changchun.ouyang at intel.com>
---
 app/test-pmd/cmdline.c | 78 ++++++++++++++++++++++++++++++++++++++++++++++++++
 app/test-pmd/testpmd.c | 14 +++++++++
 app/test-pmd/testpmd.h |  2 ++
 3 files changed, 94 insertions(+)

diff --git a/app/test-pmd/cmdline.c b/app/test-pmd/cmdline.c
index 6030192..9dcf475 100644
--- a/app/test-pmd/cmdline.c
+++ b/app/test-pmd/cmdline.c
@@ -3844,6 +3844,82 @@ cmdline_parse_inst_t cmd_start_tx_first = {
 	},
 };
 
+/* *** LINK UP ADMINISTRATIVELY *** */
+struct cmd_admin_link_up_result {
+	cmdline_fixed_string_t admin;
+	cmdline_fixed_string_t link_up;
+	cmdline_fixed_string_t port;
+	uint8_t port_id;
+};
+
+cmdline_parse_token_string_t cmd_admin_link_up_admin =
+	TOKEN_STRING_INITIALIZER(struct cmd_admin_link_up_result, admin, "admin");
+cmdline_parse_token_string_t cmd_admin_link_up_link_up =
+	TOKEN_STRING_INITIALIZER(struct cmd_admin_link_up_result, link_up, "link-up");
+cmdline_parse_token_string_t cmd_admin_link_up_port =
+	TOKEN_STRING_INITIALIZER(struct cmd_admin_link_up_result, port, "port");
+cmdline_parse_token_num_t cmd_admin_link_up_port_id =
+	TOKEN_NUM_INITIALIZER(struct cmd_admin_link_up_result, port_id, UINT8);
+
+static void cmd_admin_link_up_parsed(__attribute__((unused)) void *parsed_result,
+			     __attribute__((unused)) struct cmdline *cl,
+			     __attribute__((unused)) void *data)
+{
+	struct cmd_admin_link_up_result *res = parsed_result;
+	dev_admin_link_up(res->port_id);
+}
+
+cmdline_parse_inst_t cmd_admin_link_up = {
+	.f = cmd_admin_link_up_parsed,
+	.data = NULL,
+	.help_str = "admin link-up port (port id)",
+	.tokens = {
+		(void *)&cmd_admin_link_up_admin,
+		(void *)&cmd_admin_link_up_link_up,
+		(void *)&cmd_admin_link_up_port,
+		(void *)&cmd_admin_link_up_port_id,
+		NULL,
+	},
+};
+
+/* *** LINK DOWN ADMINISTRATIVELY *** */
+struct cmd_admin_link_down_result {
+	cmdline_fixed_string_t admin;
+	cmdline_fixed_string_t link_down;
+	cmdline_fixed_string_t port;
+	uint8_t port_id;
+};
+
+cmdline_parse_token_string_t cmd_admin_link_down_admin =
+	TOKEN_STRING_INITIALIZER(struct cmd_admin_link_down_result, admin, "admin");
+cmdline_parse_token_string_t cmd_admin_link_down_link_down =
+	TOKEN_STRING_INITIALIZER(struct cmd_admin_link_down_result, link_down, "link-down");
+cmdline_parse_token_string_t cmd_admin_link_down_port =
+	TOKEN_STRING_INITIALIZER(struct cmd_admin_link_down_result, port, "port");
+cmdline_parse_token_num_t cmd_admin_link_down_port_id =
+	TOKEN_NUM_INITIALIZER(struct cmd_admin_link_down_result, port_id, UINT8);
+
+static void cmd_admin_link_down_parsed(__attribute__((unused)) void *parsed_result,
+			     __attribute__((unused)) struct cmdline *cl,
+			     __attribute__((unused)) void *data)
+{
+	struct cmd_admin_link_down_result *res = parsed_result;
+	dev_admin_link_down(res->port_id);
+}
+
+cmdline_parse_inst_t cmd_admin_link_down = {
+	.f = cmd_admin_link_down_parsed,
+	.data = NULL,
+	.help_str = "admin link-down port (port id)",
+	.tokens = {
+		(void *)&cmd_admin_link_down_admin,
+		(void *)&cmd_admin_link_down_link_down,
+		(void *)&cmd_admin_link_down_port,
+		(void *)&cmd_admin_link_down_port_id,
+		NULL,
+	},
+};
+
 /* *** SHOW CFG *** */
 struct cmd_showcfg_result {
 	cmdline_fixed_string_t show;
@@ -6055,6 +6131,8 @@ cmdline_parse_ctx_t main_ctx[] = {
 	(cmdline_parse_inst_t *)&cmd_showcfg,
 	(cmdline_parse_inst_t *)&cmd_start,
 	(cmdline_parse_inst_t *)&cmd_start_tx_first,
+	(cmdline_parse_inst_t *)&cmd_admin_link_up,
+	(cmdline_parse_inst_t *)&cmd_admin_link_down,
 	(cmdline_parse_inst_t *)&cmd_reset,
 	(cmdline_parse_inst_t *)&cmd_set_numbers,
 	(cmdline_parse_inst_t *)&cmd_set_txpkts,
diff --git a/app/test-pmd/testpmd.c b/app/test-pmd/testpmd.c
index bc38305..9e9997f 100644
--- a/app/test-pmd/testpmd.c
+++ b/app/test-pmd/testpmd.c
@@ -1208,6 +1208,20 @@ stop_packet_forwarding(void)
 	test_done = 1;
 }
 
+void
+dev_admin_link_up(portid_t pid)
+{
+	if (rte_eth_dev_admin_link_up((uint8_t)pid) < 0)
+		printf("\nAdmin link up fail.\n");
+}
+
+void
+dev_admin_link_down(portid_t pid)
+{
+	if (rte_eth_dev_admin_link_down((uint8_t)pid) < 0)
+		printf("\nAdmin link down fail.\n");
+}
+
 static int
 all_ports_started(void)
 {
diff --git a/app/test-pmd/testpmd.h b/app/test-pmd/testpmd.h
index c7998a6..90c306d 100644
--- a/app/test-pmd/testpmd.h
+++ b/app/test-pmd/testpmd.h
@@ -499,6 +499,8 @@ char *list_pkt_forwarding_modes(void);
 void set_pkt_forwarding_mode(const char *fwd_mode);
 void start_packet_forwarding(int with_tx_first);
 void stop_packet_forwarding(void);
+void dev_admin_link_up(portid_t pid);
+void dev_admin_link_down(portid_t pid);
 void init_port_config(void);
 int init_port_dcb_config(portid_t pid,struct dcb_config *dcb_conf);
 int start_port(portid_t pid);
-- 
1.9.0