On 06/01/2011 07:46 AM, Christoph Lameter wrote:
> +#define cmpxchg8b_local(ptr, o1, o2, n1, n2) \
> +({ \
> + char __ret; \
> + __typeof__(o2) __dummy; \
> + __typeof__(*(ptr)) __old1 = (o1); \
> + __typeof__(o2) __old2 = (o2); \
> + __typeof__(*(ptr)) __new1 = (n1); \
> + __typeof__(o2) __new2 = (n2); \
> + asm volatile("cmpxchg8b %2; setz %1" \
> + : "=d"(__dummy), "=a"(__ret), "m+" (*ptr)\
> + : "a" (__old), "d"(__old2), \
> + "b" (__new1), "c" (__new2), \
> + : "memory"); \
> + __ret; })
Another syntax error... did you even compile-test any of your patches on
32 bits?
> +#
> +# CMPXCHG_DOUBLE needs to be set to enable the kernel to use cmpxchg16/8b
> +# for cmpxchg_double if it find processor flags that indicate that the
> +# capabilities are available. CMPXCHG_DOUBLE only compiles in
> +# detection support. It needs to be set if there is a chance that processor
> +# supports these instructions.
> +#
> +config CMPXCHG_DOUBLE
> + def_bool GENERIC_CPU || X86_GENERIC || !M386
> +
Still wrong.
-hpa
--
H. Peter Anvin, Intel Open Source Technology Center
I work for Intel. I don't speak on their behalf.
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@xxxxxxxxxx For more info on Linux MM,
see: http://www.linux-mm.org/ .
Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/
Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>