Skip to content
Snippets Groups Projects
Commit f061d351 authored by Darren Hart's avatar Darren Hart Committed by Ingo Molnar
Browse files

futex: remove the pointer math from double_unlock_hb


Impact: simplify code

I mistakenly included the pointer value ordering in the
double_unlock_hb() in my previous patch. It's only necessary
in the double_lock_hb() function. This patch removes it.

Signed-off-by: default avatarDarren Hart <dvhltc@us.ibm.com>
Acked-by: default avatarPeter Zijlstra <peterz@infradead.org>
Cc: Rusty Russell <rusty@rustcorp.com.au>
LKML-Reference: <20090312221118.11146.68610.stgit@Aeon>
Signed-off-by: default avatarIngo Molnar <mingo@elte.hu>
parent e4dc5b7a
Branches
Tags
No related merge requests found
...@@ -658,15 +658,9 @@ double_lock_hb(struct futex_hash_bucket *hb1, struct futex_hash_bucket *hb2) ...@@ -658,15 +658,9 @@ double_lock_hb(struct futex_hash_bucket *hb1, struct futex_hash_bucket *hb2)
static inline void 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)
{ {
if (hb1 <= hb2) {
spin_unlock(&hb2->lock);
if (hb1 < hb2)
spin_unlock(&hb1->lock);
} else { /* hb1 > hb2 */
spin_unlock(&hb1->lock); spin_unlock(&hb1->lock);
spin_unlock(&hb2->lock); spin_unlock(&hb2->lock);
} }
}
/* /*
* Wake up waiters matching bitset queued on this futex (uaddr). * Wake up waiters matching bitset queued on this futex (uaddr).
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment