diff options
author | Dave Jones <davej@redhat.com> | 2012-07-25 11:39:08 -0400 |
---|---|---|
committer | Steven Rostedt <rostedt@goodmis.org> | 2012-11-19 15:25:09 -0500 |
commit | bf3071f5a054db9e5bab873355d27a7330ce5187 (patch) | |
tree | 91ac4cf8705b3d50b9fc5555ea5ebc7127f53aa7 /kernel | |
parent | 717a9ef7f355480686cdbac3f32d6075437a923e (diff) |
tracing: Remove unnecessary WARN_ONCE's from tracing_buffers_splice_read
WARN shouldn't be used as a means of communicating failure to a userspace programmer.
Link: http://lkml.kernel.org/r/20120725153908.GA25203@redhat.com
Signed-off-by: Dave Jones <davej@redhat.com>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Diffstat (limited to 'kernel')
-rw-r--r-- | kernel/trace/trace.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/kernel/trace/trace.c b/kernel/trace/trace.c index 64ad9bc4275b..5bc35907cc6e 100644 --- a/kernel/trace/trace.c +++ b/kernel/trace/trace.c | |||
@@ -4275,13 +4275,11 @@ tracing_buffers_splice_read(struct file *file, loff_t *ppos, | |||
4275 | return -ENOMEM; | 4275 | return -ENOMEM; |
4276 | 4276 | ||
4277 | if (*ppos & (PAGE_SIZE - 1)) { | 4277 | if (*ppos & (PAGE_SIZE - 1)) { |
4278 | WARN_ONCE(1, "Ftrace: previous read must page-align\n"); | ||
4279 | ret = -EINVAL; | 4278 | ret = -EINVAL; |
4280 | goto out; | 4279 | goto out; |
4281 | } | 4280 | } |
4282 | 4281 | ||
4283 | if (len & (PAGE_SIZE - 1)) { | 4282 | if (len & (PAGE_SIZE - 1)) { |
4284 | WARN_ONCE(1, "Ftrace: splice_read should page-align\n"); | ||
4285 | if (len < PAGE_SIZE) { | 4283 | if (len < PAGE_SIZE) { |
4286 | ret = -EINVAL; | 4284 | ret = -EINVAL; |
4287 | goto out; | 4285 | goto out; |