diff options
author | Oleg Nesterov <[email protected]> | 2013-07-18 20:47:12 +0200 |
---|---|---|
committer | Steven Rostedt <[email protected]> | 2013-07-18 21:31:33 -0400 |
commit | cd458ba9d5a5592d37b5145e560071e91ea762ac (patch) | |
tree | d6a560b077696e68584995ba23e10556fcebcd8d /tools/perf/util/scripting-engines/trace-event-python.c | |
parent | 7710b639953b791610f0022a7d52d9801c93b969 (diff) |
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/[email protected]
Signed-off-by: Oleg Nesterov <[email protected]>
Signed-off-by: Steven Rostedt <[email protected]>
Diffstat (limited to 'tools/perf/util/scripting-engines/trace-event-python.c')
0 files changed, 0 insertions, 0 deletions