Messages in this thread | | | Date | Thu, 27 Dec 2012 09:35:44 -0500 | From | Rik van Riel <> | Subject | Re: [RFC PATCH 3/3 -v2] x86,smp: auto tune spinlock backoff delay factor |
| |
On 12/27/2012 09:27 AM, Eric Dumazet wrote: > On Wed, 2012-12-26 at 22:07 -0800, Michel Lespinasse wrote: > >> If we go with per-spinlock tunings, I feel we'll most likely want to >> add an associative cache in order to avoid the 1/16 chance (~6%) of >> getting 595Mbit/s instead of 982Mbit/s when there is a hash collision. >> >> I would still prefer if we could make up something that didn't require >> per-spinlock tunings, but it's not clear if that'll work. At least we >> now know of a simple enough workload to figure it out :) > > Even with a per spinlock tuning, we can find workloads where holding > time depends on the context. > > For example, complex qdisc hierarchy typically use different times on > enqueue and dequeue operations. > > So the hash sounds good to me, because the hash key could mix both lock > address and caller IP ( __builtin_return_address(1) in > ticket_spin_lock_wait())
The lock acquisition time depends on the holder of the lock, and what the CPUs ahead of us in line will do with the lock, not on the caller IP of the spinner.
Therefore, I am not convinced that hashing on the caller IP will add much, if anything, except increasing the chance that we end up not backing off when we should...
IMHO it would be good to try keeping this solution as simple as we can get away with.
-- All rights reversed
| |