ia64: cleanup remove_siblinginfo()

YuryNorov · YuryNorov · commit a570e68fabec · 2022-06-03T06:52:58.000-07:00
remove_siblinginfo() initialises variable 'last', but never uses it.
Drop unneeded code.

CC: Ingo Molnar &lt;mingo@kernel.org&gt;
CC: Peter Zijlstra &lt;peterz@infradead.org&gt;
CC: Valentin Schneider &lt;vschneid@redhat.com&gt;
CC: linux-ia64@vger.kernel.org
CC: linux-kernel@vger.kernel.org
Signed-off-by: Yury Norov &lt;yury.norov@gmail.com&gt;
Acked-by: Andrew Morton &lt;akpm@linux-foundation.org&gt;
diff --git a/arch/ia64/kernel/smpboot.c b/arch/ia64/kernel/smpboot.c
@@ -576,17 +576,13 @@ clear_cpu_sibling_map(int cpu)
 static void
 remove_siblinginfo(int cpu)
 {
-	int last = 0;
-
 	if (cpu_data(cpu)->threads_per_core == 1 &&
 	    cpu_data(cpu)->cores_per_socket == 1) {
 		cpumask_clear_cpu(cpu, &cpu_core_map[cpu]);
 		cpumask_clear_cpu(cpu, &per_cpu(cpu_sibling_map, cpu));
 		return;
 	}
 
-	last = (cpumask_weight(&cpu_core_map[cpu]) == 1 ? 1 : 0);
-
 	/* remove it from all sibling map's */
 	clear_cpu_sibling_map(cpu);
 }

Original file line number	Diff line number	Diff line change
`@@ -576,17 +576,13 @@ clear_cpu_sibling_map(int cpu)`
`576`	`576`	`static void`
`577`	`577`	`remove_siblinginfo(int cpu)`
`578`	`578`	`{`
`579`		`- int last = 0;`
`580`		`-`
`581`	`579`	`if (cpu_data(cpu)->threads_per_core == 1 &&`
`582`	`580`	`cpu_data(cpu)->cores_per_socket == 1) {`
`583`	`581`	`cpumask_clear_cpu(cpu, &cpu_core_map[cpu]);`
`584`	`582`	`cpumask_clear_cpu(cpu, &per_cpu(cpu_sibling_map, cpu));`
`585`	`583`	`return;`
`586`	`584`	`}`
`587`	`585`
`588`		`- last = (cpumask_weight(&cpu_core_map[cpu]) == 1 ? 1 : 0);`
`589`		`-`
`590`	`586`	`/* remove it from all sibling map's */`
`591`	`587`	`clear_cpu_sibling_map(cpu);`
`592`	`588`	`}`