diff options
author | Thomas Gleixner <[email protected]> | 2017-01-31 23:58:39 +0100 |
---|---|---|
committer | Ingo Molnar <[email protected]> | 2017-02-01 08:37:27 +0100 |
commit | 1aa6cfd33df492939b0be15ebdbcff1f8ae5ddb6 (patch) | |
tree | a04da1bdff7300758f2b0abbc9585cafea8c6db5 /lib/test-string_helpers.c | |
parent | dd86e373e09fb16b83e8adf5c48c421a4ca76468 (diff) |
perf/x86/intel/uncore: Clean up hotplug conversion fallout
The recent conversion to the hotplug state machine kept two mechanisms from
the original code:
1) The first_init logic which adds the number of online CPUs in a package
to the refcount. That's wrong because the callbacks are executed for
all online CPUs.
Remove it so the refcounting is correct.
2) The on_each_cpu() call to undo box->init() in the error handling
path. That's bogus because when the prepare callback fails no box has
been initialized yet.
Remove it.
Signed-off-by: Thomas Gleixner <[email protected]>
Cc: Alexander Shishkin <[email protected]>
Cc: Arnaldo Carvalho de Melo <[email protected]>
Cc: Jiri Olsa <[email protected]>
Cc: Linus Torvalds <[email protected]>
Cc: Peter Zijlstra <[email protected]>
Cc: Sebastian Siewior <[email protected]>
Cc: Stephane Eranian <[email protected]>
Cc: Vince Weaver <[email protected]>
Cc: Yasuaki Ishimatsu <[email protected]>
Fixes: 1a246b9f58c6 ("perf/x86/intel/uncore: Convert to hotplug state machine")
Link: http://lkml.kernel.org/r/[email protected]
Signed-off-by: Ingo Molnar <[email protected]>
Diffstat (limited to 'lib/test-string_helpers.c')
0 files changed, 0 insertions, 0 deletions