Re: [PATCH] FUTEX_WAKE_OP (pthread_cond_signal speedup)

2005-08-23 Thread Jakub Jelinek
On Tue, Aug 23, 2005 at 10:36:08AM -0400, Ingo Molnar wrote: > a detail: many of the futex_atomic_op_inuser() seem to be duplicated > across architectures. Might be worth putting into asm-generic, to avoid > the duplication? Those are stub files waiting for arch maintainers to actually implement

Re: [PATCH] FUTEX_WAKE_OP (pthread_cond_signal speedup)

2005-08-23 Thread Ingo Molnar
On Tue, 23 Aug 2005, Jakub Jelinek wrote: > Hi! > > ATM pthread_cond_signal is unnecessarily slow, because it wakes one > waiter (which at least on UP usually means an immediate context switch > to one of the waiter threads). This waiter wakes up and after a few > instructions it attempts to

[PATCH] FUTEX_WAKE_OP (pthread_cond_signal speedup)

2005-08-23 Thread Jakub Jelinek
Hi! ATM pthread_cond_signal is unnecessarily slow, because it wakes one waiter (which at least on UP usually means an immediate context switch to one of the waiter threads). This waiter wakes up and after a few instructions it attempts to acquire the cv internal lock, but that lock is still held

[PATCH] FUTEX_WAKE_OP (pthread_cond_signal speedup)

2005-08-23 Thread Jakub Jelinek
Hi! ATM pthread_cond_signal is unnecessarily slow, because it wakes one waiter (which at least on UP usually means an immediate context switch to one of the waiter threads). This waiter wakes up and after a few instructions it attempts to acquire the cv internal lock, but that lock is still held

Re: [PATCH] FUTEX_WAKE_OP (pthread_cond_signal speedup)

2005-08-23 Thread Ingo Molnar
On Tue, 23 Aug 2005, Jakub Jelinek wrote: Hi! ATM pthread_cond_signal is unnecessarily slow, because it wakes one waiter (which at least on UP usually means an immediate context switch to one of the waiter threads). This waiter wakes up and after a few instructions it attempts to acquire

Re: [PATCH] FUTEX_WAKE_OP (pthread_cond_signal speedup)

2005-08-23 Thread Jakub Jelinek
On Tue, Aug 23, 2005 at 10:36:08AM -0400, Ingo Molnar wrote: a detail: many of the futex_atomic_op_inuser() seem to be duplicated across architectures. Might be worth putting into asm-generic, to avoid the duplication? Those are stub files waiting for arch maintainers to actually implement