]> git.baikalelectronics.ru Git - kernel.git/commit
tracing/x86/xen: Remove zero data size trace events trace_xen_mmu_flush_tlb{_all}
authorSteven Rostedt (VMware) <rostedt@goodmis.org>
Wed, 9 May 2018 18:36:09 +0000 (14:36 -0400)
committerSteven Rostedt (VMware) <rostedt@goodmis.org>
Mon, 14 May 2018 21:02:30 +0000 (17:02 -0400)
commit00d937977bd2681990d86600711bfa4d4520f828
tree5d4fe81b9dd5555baac5e86ebd231a928cf2b126
parent615c104f45639f1e339dd4274ad98a44ca85e158
tracing/x86/xen: Remove zero data size trace events trace_xen_mmu_flush_tlb{_all}

Doing an audit of trace events, I discovered two trace events in the xen
subsystem that use a hack to create zero data size trace events. This is not
what trace events are for. Trace events add memory footprint overhead, and
if all you need to do is see if a function is hit or not, simply make that
function noinline and use function tracer filtering.

Worse yet, the hack used was:

 __array(char, x, 0)

Which creates a static string of zero in length. There's assumptions about
such constructs in ftrace that this is a dynamic string that is nul
terminated. This is not the case with these tracepoints and can cause
problems in various parts of ftrace.

Nuke the trace events!

Link: http://lkml.kernel.org/r/20180509144605.5a220327@gandalf.local.home
Cc: stable@vger.kernel.org
Fixes: 425126c3c6248 ("xen/mmu: Use Xen specific TLB flush instead of the generic one.")
Reviewed-by: Juergen Gross <jgross@suse.com>
Signed-off-by: Steven Rostedt (VMware) <rostedt@goodmis.org>
arch/x86/xen/mmu.c
arch/x86/xen/mmu_pv.c
include/trace/events/xen.h