diff options
| author | Peter Zijlstra <[email protected]> | 2018-09-05 10:41:58 +0200 |
|---|---|---|
| committer | Thomas Gleixner <[email protected]> | 2018-09-06 23:38:35 +0200 |
| commit | e2c631ba75a7e727e8db0a9d30a06bfd434adb3a (patch) | |
| tree | 49136573c4a2503cd7ba0658b83a03a250cffd09 /tools/perf/util/trace-event-scripting.c | |
| parent | c43c5e9f524ec914e7e202f9c5ab91779629ccc6 (diff) | |
clocksource: Revert "Remove kthread"
I turns out that the silly spawn kthread from worker was actually needed.
clocksource_watchdog_kthread() cannot be called directly from
clocksource_watchdog_work(), because clocksource_select() calls
timekeeping_notify() which uses stop_machine(). One cannot use
stop_machine() from a workqueue() due lock inversions wrt CPU hotplug.
Revert the patch but add a comment that explain why we jump through such
apparently silly hoops.
Fixes: 7197e77abcb6 ("clocksource: Remove kthread")
Reported-by: Siegfried Metz <[email protected]>
Signed-off-by: Peter Zijlstra (Intel) <[email protected]>
Signed-off-by: Thomas Gleixner <[email protected]>
Tested-by: Niklas Cassel <[email protected]>
Tested-by: Kevin Shanahan <[email protected]>
Tested-by: [email protected]
Tested-by: Siegfried Metz <[email protected]>
Cc: [email protected]
Cc: [email protected]
Cc: [email protected]
Cc: [email protected]
Cc: [email protected]
Link: https://lkml.kernel.org/r/[email protected]
Diffstat (limited to 'tools/perf/util/trace-event-scripting.c')
0 files changed, 0 insertions, 0 deletions