diff options
| author | Suzuki K Poulose <suzuki.poulose@arm.com> | 2016-05-10 13:14:41 +0300 |
|---|---|---|
| committer | Will Deacon <will.deacon@arm.com> | 2016-05-11 12:11:37 +0300 |
| commit | 99aa036241ed4a08a71b627bb903b5d7c75d78c1 (patch) | |
| tree | 6a612396ddac86e340fd0eec78721e19395869f6 /arch/arm64 | |
| parent | 5bb1cc0ff9a6b68871970737e6c4c16919928d8b (diff) | |
| download | linux-99aa036241ed4a08a71b627bb903b5d7c75d78c1.tar.xz | |
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>
Diffstat (limited to 'arch/arm64')
| -rw-r--r-- | arch/arm64/kernel/smp.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/arch/arm64/kernel/smp.c b/arch/arm64/kernel/smp.c index dc9647521c59..678e0842cb3b 100644 --- a/arch/arm64/kernel/smp.c +++ b/arch/arm64/kernel/smp.c @@ -264,8 +264,6 @@ asmlinkage void secondary_start_kernel(void) pr_info("CPU%u: Booted secondary processor [%08x]\n", cpu, read_cpuid_id()); update_cpu_boot_status(CPU_BOOT_SUCCESS); - /* Make sure the status update is visible before we complete */ - smp_wmb(); set_cpu_online(cpu, true); complete(&cpu_running); |