diff options
author | Oleg Nesterov <oleg@redhat.com> | 2013-07-18 18:47:12 (GMT) |
---|---|---|
committer | Steven Rostedt <rostedt@goodmis.org> | 2013-07-19 01:31:33 (GMT) |
commit | cd458ba9d5a5592d37b5145e560071e91ea762ac (patch) | |
tree | d6a560b077696e68584995ba23e10556fcebcd8d /mm/internal.h | |
parent | 7710b639953b791610f0022a7d52d9801c93b969 (diff) | |
download | linux-fsl-qoriq-cd458ba9d5a5592d37b5145e560071e91ea762ac.tar.xz |
tracing: Do not (ab)use trace_seq in event_id_read()
event_id_read() has no reason to kmalloc "struct trace_seq"
(more than PAGE_SIZE!), it can use a small buffer instead.
Note: "if (*ppos) return 0" looks strange and even wrong,
simple_read_from_buffer() handles ppos != 0 case corrrectly.
And it seems that almost every user of trace_seq in this file
should be converted too. Unless you use seq_open(), trace_seq
buys nothing compared to the raw buffer, but it needs a bit
more memory and code.
Link: http://lkml.kernel.org/r/20130718184712.GA4786@redhat.com
Signed-off-by: Oleg Nesterov <oleg@redhat.com>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Diffstat (limited to 'mm/internal.h')
0 files changed, 0 insertions, 0 deletions