[PATCH] Fix wrong dmb
Nicolas Pitre
nico at fluxnic.net
Tue Jan 12 13:43:33 EST 2010
On Tue, 12 Jan 2010, Russell King - ARM Linux wrote:
> The __kuser_cmpxchg code uses an ARMv6 dmb instruction, rather than
> one based upon the architecture being built for. Switch to using
> the macro provided for this purpose, which also eliminates the
> need for an ifdef.
>
> Signed-off-by: Russell King <rmk+kernel at arm.linux.org.uk>
Acked-by: Nicolas Pitre <nico at fluxnic.net>
>
> diff --git a/arch/arm/kernel/entry-armv.S b/arch/arm/kernel/entry-armv.S
> index d2903e3..6c5cf36 100644
> --- a/arch/arm/kernel/entry-armv.S
> +++ b/arch/arm/kernel/entry-armv.S
> @@ -957,9 +957,7 @@ kuser_cmpxchg_fixup:
>
> #else
>
> -#ifdef CONFIG_SMP
> - mcr p15, 0, r0, c7, c10, 5 @ dmb
> -#endif
> + smp_dmb
> 1: ldrex r3, [r2]
> subs r3, r3, r0
> strexeq r3, r1, [r2]
>
More information about the linux-arm-kernel
mailing list