diff options
| author | Oleg Nesterov <oleg@redhat.com> | 2013-07-18 20:47:12 +0200 |
|---|---|---|
| committer | Steven Rostedt <rostedt@goodmis.org> | 2013-07-18 21:31:33 -0400 |
| commit | cd458ba9d5a5592d37b5145e560071e91ea762ac (patch) | |
| tree | d6a560b077696e68584995ba23e10556fcebcd8d /drivers/hid/hid-input.c | |
| parent | 7710b639953b791610f0022a7d52d9801c93b969 (diff) | |
| download | op-kernel-dev-cd458ba9d5a5592d37b5145e560071e91ea762ac.zip op-kernel-dev-cd458ba9d5a5592d37b5145e560071e91ea762ac.tar.gz | |
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 'drivers/hid/hid-input.c')
0 files changed, 0 insertions, 0 deletions
