diff options
author | Tony Luck <tony.luck@intel.com> | 2010-06-30 10:46:16 -0700 |
---|---|---|
committer | Tony Luck <tony.luck@intel.com> | 2010-06-30 10:46:16 -0700 |
commit | b70f4e85bfc4d7000036355b714a92d5c574f1be (patch) | |
tree | 284c14c836412acdd0572ef020499749bca602d8 /arch | |
parent | 984bc9601f64fd341b8573021d7c999f1f1499a9 (diff) | |
download | op-kernel-dev-b70f4e85bfc4d7000036355b714a92d5c574f1be.zip op-kernel-dev-b70f4e85bfc4d7000036355b714a92d5c574f1be.tar.gz |
[IA64] Fix spinaphore down_spin()
Typo in down_spin() meant it only read the low 32 bits of the
"serve" value, instead of the full 64 bits. This results in the
system hanging when the values in ticket/serve get larger than
32-bits. A big enough system running the right test can hit this
in a just a few hours.
Broken since 883a3acf5b0d4782ac35981227a0d094e8b44850
[IA64] Re-implement spinaphores using ticket lock concepts
Reported via IRC by Bjorn Helgaas
Signed-off-by: Tony Luck <tony.luck@intel.com>
Diffstat (limited to 'arch')
-rw-r--r-- | arch/ia64/mm/tlb.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/ia64/mm/tlb.c b/arch/ia64/mm/tlb.c index 5dfd916..7b3cdc6 100644 --- a/arch/ia64/mm/tlb.c +++ b/arch/ia64/mm/tlb.c @@ -121,7 +121,7 @@ static inline void down_spin(struct spinaphore *ss) ia64_invala(); for (;;) { - asm volatile ("ld4.c.nc %0=[%1]" : "=r"(serve) : "r"(&ss->serve) : "memory"); + asm volatile ("ld8.c.nc %0=[%1]" : "=r"(serve) : "r"(&ss->serve) : "memory"); if (time_before(t, serve)) return; cpu_relax(); |