From: Andreas Mohr <andi@xxxxxxxxxxxxxxxxxxxxxxx> Use cpu_relax() in __acpi_acquire_global_lock() etc. [mingo@xxxxxxx: build fix] Signed-off-by: Andreas Mohr <andi@xxxxxxxx> Cc: "Brown, Len" <len.brown@xxxxxxxxx> Signed-off-by: Ingo Molnar <mingo@xxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxx> --- include/asm-i386/acpi.h | 15 +++++++++++---- include/asm-x86_64/acpi.h | 14 ++++++++++---- 2 files changed, 21 insertions(+), 8 deletions(-) diff -puN include/asm-i386/acpi.h~cpu_relax-use-in-acpi-lock include/asm-i386/acpi.h --- a/include/asm-i386/acpi.h~cpu_relax-use-in-acpi-lock +++ a/include/asm-i386/acpi.h @@ -31,6 +31,7 @@ #include <acpi/pdc_intel.h> #include <asm/system.h> /* defines cmpxchg */ +#include <asm/processor.h> /* cpu_relax() */ #define COMPILER_DEPENDENT_INT64 long long #define COMPILER_DEPENDENT_UINT64 unsigned long long @@ -61,11 +62,14 @@ static inline int __acpi_acquire_global_lock (unsigned int *lock) { unsigned int old, new, val; - do { + while (1) { old = *lock; new = (((old & ~0x3) + 2) + ((old >> 1) & 0x1)); val = cmpxchg(lock, old, new); - } while (unlikely (val != old)); + if (likely(val == old)) + break; + cpu_relax(); + } return (new < 3) ? -1 : 0; } @@ -73,11 +77,14 @@ static inline int __acpi_release_global_lock (unsigned int *lock) { unsigned int old, new, val; - do { + while (1) { old = *lock; new = old & ~0x3; val = cmpxchg(lock, old, new); - } while (unlikely (val != old)); + if (likely(val == old)) + break; + cpu_relax(); + } return old & 0x1; } diff -puN include/asm-x86_64/acpi.h~cpu_relax-use-in-acpi-lock include/asm-x86_64/acpi.h --- a/include/asm-x86_64/acpi.h~cpu_relax-use-in-acpi-lock +++ a/include/asm-x86_64/acpi.h @@ -59,11 +59,14 @@ static inline int __acpi_acquire_global_lock (unsigned int *lock) { unsigned int old, new, val; - do { + while (1) { old = *lock; new = (((old & ~0x3) + 2) + ((old >> 1) & 0x1)); val = cmpxchg(lock, old, new); - } while (unlikely (val != old)); + if (likely(val == old)) + break; + cpu_relax(); + } return (new < 3) ? -1 : 0; } @@ -71,11 +74,14 @@ static inline int __acpi_release_global_lock (unsigned int *lock) { unsigned int old, new, val; - do { + while (1) { old = *lock; new = old & ~0x3; val = cmpxchg(lock, old, new); - } while (unlikely (val != old)); + if (likely(val == old)) + break; + cpu_relax(); + } return old & 0x1; } _ - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html