Commit 88f502fe authored by Ingo Molnar's avatar Ingo Molnar

futex: remove the pointer math from double_unlock_hb, fix

Impact: fix double unlock crash

Thomas Gleixner noticed that the simplified double_unlock_hb()
became ... too unsophisticated: in the hb1 == hb2 case it will
do a double unlock.
Reported-by: default avatarThomas Gleixner <tglx@linutronix.de>
Cc: Darren Hart <dvhltc@us.ibm.com>
LKML-Reference: <20090312221118.11146.68610.stgit@Aeon>
Signed-off-by: default avatarIngo Molnar <mingo@elte.hu>
parent f061d351
...@@ -659,7 +659,8 @@ static inline void ...@@ -659,7 +659,8 @@ static inline void
double_unlock_hb(struct futex_hash_bucket *hb1, struct futex_hash_bucket *hb2) double_unlock_hb(struct futex_hash_bucket *hb1, struct futex_hash_bucket *hb2)
{ {
spin_unlock(&hb1->lock); spin_unlock(&hb1->lock);
spin_unlock(&hb2->lock); if (hb1 != hb2)
spin_unlock(&hb2->lock);
} }
/* /*
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment