diff options
author | Tobias Klauser <tklauser@distanz.ch> | 2017-01-17 09:35:01 -0500 |
---|---|---|
committer | Thomas Gleixner <tglx@linutronix.de> | 2017-01-18 05:55:09 -0500 |
commit | 0fec9557fd0c5349e3bd1a2141612a60bc20bb71 (patch) | |
tree | 549969ab682cb758e8c439602adc931ecddc5244 /kernel | |
parent | 4205e4786d0b9fc3b4fec7b1910cf645a0468307 (diff) |
cpu/hotplug: Remove unused but set variable in _cpu_down()
After the recent removal of the hotplug notifiers the variable 'hasdied' in
_cpu_down() is set but no longer read, leading to the following GCC warning
when building with 'make W=1':
kernel/cpu.c:767:7: warning: variable ‘hasdied’ set but not used [-Wunused-but-set-variable]
Fix it by removing the variable.
Fixes: 530e9b76ae8f ("cpu/hotplug: Remove obsolete cpu hotplug register/unregister functions")
Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Cc: rt@linutronix.de
Link: http://lkml.kernel.org/r/20170117143501.20893-1-tklauser@distanz.ch
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Diffstat (limited to 'kernel')
-rw-r--r-- | kernel/cpu.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/kernel/cpu.c b/kernel/cpu.c index c47506357519..0a5f630f5c54 100644 --- a/kernel/cpu.c +++ b/kernel/cpu.c | |||
@@ -764,7 +764,6 @@ static int __ref _cpu_down(unsigned int cpu, int tasks_frozen, | |||
764 | { | 764 | { |
765 | struct cpuhp_cpu_state *st = per_cpu_ptr(&cpuhp_state, cpu); | 765 | struct cpuhp_cpu_state *st = per_cpu_ptr(&cpuhp_state, cpu); |
766 | int prev_state, ret = 0; | 766 | int prev_state, ret = 0; |
767 | bool hasdied = false; | ||
768 | 767 | ||
769 | if (num_online_cpus() == 1) | 768 | if (num_online_cpus() == 1) |
770 | return -EBUSY; | 769 | return -EBUSY; |
@@ -809,7 +808,6 @@ static int __ref _cpu_down(unsigned int cpu, int tasks_frozen, | |||
809 | cpuhp_kick_ap_work(cpu); | 808 | cpuhp_kick_ap_work(cpu); |
810 | } | 809 | } |
811 | 810 | ||
812 | hasdied = prev_state != st->state && st->state == CPUHP_OFFLINE; | ||
813 | out: | 811 | out: |
814 | cpu_hotplug_done(); | 812 | cpu_hotplug_done(); |
815 | return ret; | 813 | return ret; |