arm64: secondary_start_kernel: Remove unnecessary barrier
Remove the unnecessary smp_wmb(), which was added to make sure that the update_cpu_boot_status() completes before we mark the CPU online. But update_cpu_boot_status() already has dsb() (required for the failing CPUs) to ensure the correct behavior. Cc: Catalin Marinas <catalin.marinas@arm.com> Acked-by: Mark Rutland <mark.rutland@arm.com> Reported-by: Dennis Chen <dennis.chen@arm.com> Signed-off-by: Suzuki K Poulose <suzuki.poulose@arm.com> Signed-off-by: Will Deacon <will.deacon@arm.com>
This commit is contained in:
Родитель
5bb1cc0ff9
Коммит
99aa036241
|
@ -264,8 +264,6 @@ asmlinkage void secondary_start_kernel(void)
|
||||||
pr_info("CPU%u: Booted secondary processor [%08x]\n",
|
pr_info("CPU%u: Booted secondary processor [%08x]\n",
|
||||||
cpu, read_cpuid_id());
|
cpu, read_cpuid_id());
|
||||||
update_cpu_boot_status(CPU_BOOT_SUCCESS);
|
update_cpu_boot_status(CPU_BOOT_SUCCESS);
|
||||||
/* Make sure the status update is visible before we complete */
|
|
||||||
smp_wmb();
|
|
||||||
set_cpu_online(cpu, true);
|
set_cpu_online(cpu, true);
|
||||||
complete(&cpu_running);
|
complete(&cpu_running);
|
||||||
|
|
||||||
|
|
Загрузка…
Ссылка в новой задаче