diff options
| author |   Jesper Dangaard Brouer <brouer@redhat.com> | 2015-05-21 12:16:56 +0200 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2015-05-22 23:59:16 -0400 |
| commit | 4020726479ff799318a5aa188b81d79df86a0ea3 (patch) | |
| tree | dd8a0a338ffebf50049f9da4b57af53af17c8ff2 /net/core/pktgen.c | |
| parent | pktgen: document ability to add same device to several threads (diff) | |
| download | linux-dev-4020726479ff799318a5aa188b81d79df86a0ea3.tar.xz linux-dev-4020726479ff799318a5aa188b81d79df86a0ea3.zip | |
pktgen: make /proc/net/pktgen/pgctrl report fail on invalid input
Giving /proc/net/pktgen/pgctrl an invalid command just returns shell
success and prints a warning in dmesg. This is not very useful for
shell scripting, as it can only detect the error by parsing dmesg.
Instead return -EINVAL when the command is unknown, as this provides
userspace shell scripting a way of detecting this.
Also bump version tag to 2.75, because (1) reading /proc/net/pktgen/pgctrl
output this version number which would allow to detect this small
semantic change, and (2) because the pktgen version tag have not been
updated since 2010.
Signed-off-by: Jesper Dangaard Brouer <brouer@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/core/pktgen.c')
| -rw-r--r-- | net/core/pktgen.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/net/core/pktgen.c b/net/core/pktgen.c index 167e59243fea..676550f34560 100644 --- a/net/core/pktgen.c +++ b/net/core/pktgen.c @@ -177,7 +177,7 @@ #include <asm/dma.h> #include <asm/div64.h> /* do_div */ -#define VERSION "2.74" +#define VERSION "2.75" #define IP_NAME_SZ 32 #define MAX_MPLS_LABELS 16 /* This is the max label stack depth */ #define MPLS_STACK_BOTTOM htonl(0x00000100) @@ -512,7 +512,7 @@ static ssize_t pgctrl_write(struct file *file, const char __user *buf, pktgen_reset_all_threads(pn); else - pr_warn("Unknown command: %s\n", data); + return -EINVAL; return count; } |