[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Date: Sun, 5 May 2019 17:12:43 +0300
From: Ido Schimmel <idosch@...sch.org>
To: netdev@...r.kernel.org
Cc: stephen@...workplumber.org, jiri@...lanox.com, mlxsw@...lanox.com,
Ido Schimmel <idosch@...lanox.com>
Subject: [PATCH iproute2-master] devlink: Fix monitor command
From: Ido Schimmel <idosch@...lanox.com>
The command is supposed to allow users to filter events related to
certain objects, but returns an error when an object is specified:
# devlink mon dev
Command "dev" not found
Fix this by allowing the command to process the specified objects.
Example:
# devlink/devlink mon dev &
# echo "10 1" > /sys/bus/netdevsim/new_device
[dev,new] netdevsim/netdevsim10
# devlink/devlink mon port &
# echo "11 1" > /sys/bus/netdevsim/new_device
[port,new] netdevsim/netdevsim11/0: type notset flavour physical
[port,new] netdevsim/netdevsim11/0: type eth netdev eth1 flavour physical
# devlink/devlink mon &
# echo "12 1" > /sys/bus/netdevsim/new_device
[dev,new] netdevsim/netdevsim12
[port,new] netdevsim/netdevsim12/0: type notset flavour physical
[port,new] netdevsim/netdevsim12/0: type eth netdev eth2 flavour physical
Fixes: a3c4b484a1ed ("add devlink tool")
Signed-off-by: Ido Schimmel <idosch@...lanox.com>
---
devlink/devlink.c | 6 +-----
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/devlink/devlink.c b/devlink/devlink.c
index dc6e73fec20c..6a4ce58b9ee9 100644
--- a/devlink/devlink.c
+++ b/devlink/devlink.c
@@ -3858,12 +3858,8 @@ static int cmd_mon(struct dl *dl)
if (dl_argv_match(dl, "help")) {
cmd_mon_help();
return 0;
- } else if (dl_no_arg(dl)) {
- dl_arg_inc(dl);
- return cmd_mon_show(dl);
}
- pr_err("Command \"%s\" not found\n", dl_argv(dl));
- return -ENOENT;
+ return cmd_mon_show(dl);
}
struct dpipe_field {
--
2.20.1
Powered by blists - more mailing lists