Messages in this thread | | | Date | Wed, 09 Aug 2006 01:11:58 +1000 | From | Nick Piggin <> | Subject | Re: [RFC] NUMA futex hashing |
| |
Ulrich Drepper wrote: > On 8/8/06, Eric Dumazet <dada1@cosmosbay.com> wrote: > >> The validity of the virtual address is still tested by normal get_user() >> call.. If the memory was freed by a thread, then a normal EFAULT error >> will >> be reported... eventually. > > > This is indeed what should be done. Private futexes are the by far > more frequent case and I bet you'd see improvements when avoiding the > mm mutex even for normal machines since futexes really are everywhere. > For shared mutexes you end up doing two lookups and that's fine IMO > as long as the first lookup is fast.
The private futex's namespace is its virtual address, so I don't see how you can decouple that from the management of virtual addresses.
Let me get this straight: to insert a contended futex into your rbtree, you need to hold the mmap sem to ensure that address remains valid, then you need to take a lock which protects your rbtree. Then to wake up a process and remove the futex, you need to take the rbtree lock. Or to unmap any memory you also need to take the rbtree lock and ensure there are no futexes there.
So you just add another lock for no reason, or have I got a few screws loose myself? I don't see how you can significantly reduce lock cacheline bouncing in a futex heavy workload if you're just going to add another shared data structure. But if you can, sweet ;)
-- SUSE Labs, Novell Inc. Send instant messages to your online friends http://au.messenger.yahoo.com
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |