diff options
| author |   Chen, Kenneth W <kenneth.w.chen@intel.com> | 2006-04-07 17:12:54 -0700 |
|---|---|---|
| committer |   Tony Luck <tony.luck@intel.com> | 2006-04-07 22:39:49 -0700 |
| commit | cfab9d0e1da8e08a39759d0fc3bf5e40f0ac2d55 (patch) | |
| tree | e7dd4bbb97374b758bcb169b635b997c12893565 /include/asm-ia64 | |
| parent | [IA64] for_each_possible_cpu: ia64 (diff) | |
| download | linux-dev-cfab9d0e1da8e08a39759d0fc3bf5e40f0ac2d55.tar.xz linux-dev-cfab9d0e1da8e08a39759d0fc3bf5e40f0ac2d55.zip | |
[IA64] fix bug in ia64 __mutex_fastpath_trylock
The parenthesis around "likely" used in ia64 __mutex_fastpath_trylock
is incorrect, and it leads to broken mutex_trylock. Here is the
patch that fixed the bug. I removed the likely altogether because
there is no branch and gcc does a reasonable job at predicating the
return value.
Signed-off-by: Ken Chen <kenneth.w.chen@intel.com>
Signed-off-by: Tony Luck <tony.luck@intel.com>
Diffstat (limited to 'include/asm-ia64')
| -rw-r--r-- | include/asm-ia64/mutex.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/asm-ia64/mutex.h b/include/asm-ia64/mutex.h index 5a3224f6af38..bed73a643a56 100644 --- a/include/asm-ia64/mutex.h +++ b/include/asm-ia64/mutex.h @@ -84,7 +84,7 @@ __mutex_fastpath_unlock(atomic_t *count, void (*fail_fn)(atomic_t *)) static inline int __mutex_fastpath_trylock(atomic_t *count, int (*fail_fn)(atomic_t *)) { - if (likely(cmpxchg_acq(count, 1, 0)) == 1) + if (cmpxchg_acq(count, 1, 0) == 1) return 1; return 0; } |