diff options
author | Rasmus Villemoes <linux@rasmusvillemoes.dk> | 2015-09-09 23:24:01 +0200 |
---|---|---|
committer | Steven Rostedt <rostedt@goodmis.org> | 2015-10-01 09:05:17 -0400 |
commit | 79ac6ef521075d0f40805df77e8890c55f538fe4 (patch) | |
tree | cf353ec399027ddd9f93a958004cc61777c1d9a9 /kernel/trace/ftrace.c | |
parent | 37aea98b84c0ce2ac638510fefeed9f8f920bd34 (diff) |
tracing: Use kstrdup_const instead of private implementation
The kernel now has kstrdup_const/kfree_const for reusing .rodata
(typically string literals) when possible; there's no reason to
duplicate that logic in the tracing system. Moreover, as the comment
above core_kernel_data states, it may not always return true for
.rodata - that is for example the case on x86_64, where we thus end up
kstrdup'ing all the passed-in strings.
Arguably, testing for .rodata explicitly (as kstrdup_const does) is
also more correct: I don't think one is supposed to be able to change
the name after creating the event_subsystem by passing the address of
a static char (but non-const) array.
Link: http://lkml.kernel.org/r/1441833841-12955-1-git-send-email-linux@rasmusvillemoes.dk
Signed-off-by: Rasmus Villemoes <linux@rasmusvillemoes.dk>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Diffstat (limited to 'kernel/trace/ftrace.c')
0 files changed, 0 insertions, 0 deletions