mm, tracing: Fix kmem_cache_free trace event to not print stale pointers

The update to kmem_cache_free trace event added printing of the slab name in
the trace event. But it only stores the pointer of the name which will be
printed as a string when the event is read some time in the future. This is
dangerous because the name could be freed in the mean time and when reading
the trace event it would try to dereference the string name by the pointer
to the name that has been freed.

Instead, use the trace event helper macros __string(), __assign_str(), and
__get_str() that are for this very case.

Cc: Jacob Wen <jian.w.wen@oracle.com>
Fixes: 3544de8ee6e4 ("mm, tracing: record slab name for kmem_cache_free()")
Signed-off-by: Steven Rostedt (VMware) <rostedt@goodmis.org>

diff --git a/include/trace/events/kmem.h b/include/trace/events/kmem.h
index 40845b0d5dad79cd0df4427417c634749471c6e3..3a60b6b6db32faa6e2d2aed900ca9a78e4786245 100644 (file)
--- a/include/trace/events/kmem.h
+++ b/include/trace/events/kmem.h
@@ -144,17 +144,17 @@ TRACE_EVENT(kmem_cache_free,
        TP_STRUCT__entry(
                __field(        unsigned long,  call_site       )
                __field(        const void *,   ptr             )
-               __field(        const char *,   name            )
+               __string(       name,   name    )
        ),
 
        TP_fast_assign(
                __entry->call_site      = call_site;
                __entry->ptr            = ptr;
-               __entry->name           = name;
+               __assign_str(name, name);
        ),
 
        TP_printk("call_site=%pS ptr=%p name=%s",
-                 (void *)__entry->call_site, __entry->ptr, __entry->name)
+                 (void *)__entry->call_site, __entry->ptr, __get_str(name))
 );
 
 TRACE_EVENT(mm_page_free,