From: Raslan Darawsheh <rasland@mellanox.com>
To: thomas@monjalon.net, jingjing.wu@intel.com, gaetan.rivet@6wind.com
Cc: dev@dpdk.org, salehals@mellanox.com
Subject: [dpdk-dev] [PATCH] app/testpmd: app/testpmd: add device removal command
Date: Wed, 23 Aug 2017 17:37:04 +0300 [thread overview]
Message-ID: <1503499024-12480-1-git-send-email-rasland@mellanox.com> (raw)
Added hotplug in testpmd, to be able to test hotplug function
in the PMD's.
Signed-off-by: Raslan Darawsheh <rasland@mellanox.com>
---
app/test-pmd/cmdline.c | 44 ++++++++++++++++++++++++++++++++++++++++++++
app/test-pmd/testpmd.c | 18 ++++++++++++++++++
app/test-pmd/testpmd.h | 1 +
3 files changed, 63 insertions(+)
diff --git a/app/test-pmd/cmdline.c b/app/test-pmd/cmdline.c
index cd8c358..b32a368 100644
--- a/app/test-pmd/cmdline.c
+++ b/app/test-pmd/cmdline.c
@@ -716,6 +716,9 @@ static void cmd_help_long_parsed(void *parsed_result,
"port config (port_id|all) l2-tunnel E-tag"
" (enable|disable)\n"
" Enable/disable the E-tag support.\n\n"
+
+ " device remove (device)\n"
+ " Remove a device"
);
}
@@ -1125,6 +1128,46 @@ cmdline_parse_inst_t cmd_operate_detach_port = {
},
};
+/* *** Remove a specified device *** */
+struct cmd_operate_device_remove_result {
+ cmdline_fixed_string_t device;
+ cmdline_fixed_string_t keyword;
+ cmdline_fixed_string_t identifier;
+};
+
+static void cmd_operate_device_remove_parsed(void *parsed_result,
+ __attribute__((unused)) struct cmdline *cl,
+ __attribute__((unused)) void *data)
+{
+ struct cmd_operate_device_remove_result *res = parsed_result;
+ if (!strcmp(res->keyword, "remove"))
+ device_remove((char *) res->identifier);
+ else
+ printf("Unknown parameter\n");
+}
+
+cmdline_parse_token_string_t cmd_operate_device_remove_device =
+ TOKEN_STRING_INITIALIZER(struct cmd_operate_device_remove_result,
+ device, "device");
+cmdline_parse_token_string_t cmd_operate_device_remove_keyword =
+ TOKEN_STRING_INITIALIZER(struct cmd_operate_device_remove_result,
+ keyword, "remove");
+cmdline_parse_token_string_t cmd_operate_device_remove_identifier =
+ TOKEN_STRING_INITIALIZER(struct cmd_operate_device_remove_result,
+ identifier, NULL);
+
+cmdline_parse_inst_t cmd_operate_device_remove = {
+ .f = cmd_operate_device_remove_parsed,
+ .data = NULL,
+ .help_str = "device remove <device>: (device)",
+ .tokens = {
+ (void *)&cmd_operate_device_remove_device,
+ (void *)&cmd_operate_device_remove_keyword,
+ (void *)&cmd_operate_device_remove_identifier,
+ NULL,
+ },
+};
+
/* *** configure speed for all ports *** */
struct cmd_config_speed_all {
cmdline_fixed_string_t port;
@@ -14276,6 +14319,7 @@ cmdline_parse_ctx_t main_ctx[] = {
(cmdline_parse_inst_t *)&cmd_operate_specific_port,
(cmdline_parse_inst_t *)&cmd_operate_attach_port,
(cmdline_parse_inst_t *)&cmd_operate_detach_port,
+ (cmdline_parse_inst_t *)&cmd_operate_device_remove,
(cmdline_parse_inst_t *)&cmd_config_speed_all,
(cmdline_parse_inst_t *)&cmd_config_speed_specific,
(cmdline_parse_inst_t *)&cmd_config_rx_tx,
diff --git a/app/test-pmd/testpmd.c b/app/test-pmd/testpmd.c
index 7d40139..a2e8526 100644
--- a/app/test-pmd/testpmd.c
+++ b/app/test-pmd/testpmd.c
@@ -1742,6 +1742,24 @@ detach_port(uint8_t port_id)
}
void
+device_remove(char *device) {
+ struct rte_devargs devargs;
+
+ if (device == NULL) {
+ printf("Invalid parameters are specified\n");
+ return;
+ }
+
+ rte_eal_devargs_parse(device, &devargs);
+
+ if (rte_eal_hotplug_remove(devargs.bus->name, devargs.name)) {
+ printf("Fail to remove device\n");
+ return;
+ }
+ printf("Device removed successfully\n");
+}
+
+void
pmd_test_exit(void)
{
portid_t pt_id;
diff --git a/app/test-pmd/testpmd.h b/app/test-pmd/testpmd.h
index c9d7739..0780f55 100644
--- a/app/test-pmd/testpmd.h
+++ b/app/test-pmd/testpmd.h
@@ -612,6 +612,7 @@ void stop_port(portid_t pid);
void close_port(portid_t pid);
void attach_port(char *identifier);
void detach_port(uint8_t port_id);
+void device_remove(char *device);
int all_ports_stopped(void);
int port_is_started(portid_t port_id);
void pmd_test_exit(void);
--
2.7.4
next reply other threads:[~2017-08-23 14:37 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-08-23 14:37 Raslan Darawsheh [this message]
2017-08-23 15:09 ` Gaëtan Rivet
2017-08-23 16:18 ` Thomas Monjalon
2017-08-25 7:53 ` Gaëtan Rivet
2017-08-25 8:55 ` Thomas Monjalon
2017-08-28 9:55 ` Gaëtan Rivet
2017-08-28 10:30 ` Ferruh Yigit
2017-08-28 11:00 ` Thomas Monjalon
2017-08-28 11:12 ` Gaëtan Rivet
2017-09-07 8:17 ` Wu, Jingjing
2017-11-28 22:02 ` Ferruh Yigit
2017-12-03 9:32 ` Raslan Darawsheh
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=1503499024-12480-1-git-send-email-rasland@mellanox.com \
--to=rasland@mellanox.com \
--cc=dev@dpdk.org \
--cc=gaetan.rivet@6wind.com \
--cc=jingjing.wu@intel.com \
--cc=salehals@mellanox.com \
--cc=thomas@monjalon.net \
/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).