diff options

author	Steven Rostedt (VMware) <rostedt@goodmis.org>	2017-01-30 19:27:10 -0500
committer	Steven Rostedt (VMware) <rostedt@goodmis.org>	2017-01-31 09:13:49 -0500
commit	79c6f448c8b79c321e4a1f31f98194e4f6b6cae7 (patch)
tree	370efda701f03cccf21e02bb1fdd3b852547d75c /tools/perf/builtin-lock.c
parent	0c744ea4f77d72b3dcebb7a8f2684633ec79be88 (diff)

tracing: Fix hwlat kthread migration

The hwlat tracer creates a kernel thread at start of the tracer. It is pinned to a single CPU and will move to the next CPU after each period of running. If the user modifies the migration thread's affinity, it will not change after that happens. The original code created the thread at the first instance it was called, but later was changed to destroy the thread after the tracer was finished, and would not be created until the next instance of the tracer was established. The code that initialized the affinity was only called on the initial instantiation of the tracer. After that, it was not initialized, and the previous affinity did not match the current newly created one, making it appear that the user modified the thread's affinity when it did not, and the thread failed to migrate again. Cc: stable@vger.kernel.org Fixes: 0330f7aa8ee6 ("tracing: Have hwlat trace migrate across tracing_cpumask CPUs") Signed-off-by: Steven Rostedt (VMware) <rostedt@goodmis.org>

Diffstat (limited to 'tools/perf/builtin-lock.c')


context:
space:
mode: