Messages in this thread Patch in this message | | | From | Andi Kleen <> | Subject | [PATCH 25/29] x86, tsx: Add adaption support for spinlocks | Date | Fri, 22 Mar 2013 18:25:19 -0700 |
| |
From: Andi Kleen <ak@linux.intel.com>
Add adaptation support for ticket spinlocks. Each spinlock keeps a skip count on how often to skip elision. This is controlled by the abort rate. The actual adaptation algorithm is generic and shared with other lock types. This avoids us having to tune each spinlock individually for elision.
This adds 2 bytes to a 32bit spinlock for systems configured for more than 256 CPUs, so it's now 6 bytes instead of 4. For systems with less than 256 CPUs it expands spinlock_t from 2 to 4 bytes.
I played around with various schemes to merge the elision count with the main tickets, but they all reduced the maximum CPU count support too much and caused other problems. So in the end I just added the additional 2 bytes.
Signed-off-by: Andi Kleen <ak@linux.intel.com> --- arch/x86/include/asm/spinlock_types.h | 3 +++ arch/x86/kernel/rtm-locks.c | 10 ++++++++-- 2 files changed, 11 insertions(+), 2 deletions(-)
diff --git a/arch/x86/include/asm/spinlock_types.h b/arch/x86/include/asm/spinlock_types.h index ad0ad07..614159c 100644 --- a/arch/x86/include/asm/spinlock_types.h +++ b/arch/x86/include/asm/spinlock_types.h @@ -24,6 +24,9 @@ typedef struct arch_spinlock { __ticket_t head, tail; } tickets; }; +#ifdef CONFIG_RTM_LOCKS + short elision_adapt; +#endif } arch_spinlock_t; #define __ARCH_SPIN_LOCK_UNLOCKED { { 0 } } diff --git a/arch/x86/kernel/rtm-locks.c b/arch/x86/kernel/rtm-locks.c index a313a81..1651049 100644 --- a/arch/x86/kernel/rtm-locks.c +++ b/arch/x86/kernel/rtm-locks.c @@ -76,16 +76,22 @@ static DEFINE_PER_CPU(bool, cli_elided); static struct static_key spinlock_elision = STATIC_KEY_INIT_TRUE; module_param(spinlock_elision, static_key, 0644); +static __read_mostly struct elision_config spinlock_elision_config = + DEFAULT_ELISION_CONFIG; +TUNE_ELISION_CONFIG(spinlock, spinlock_elision_config); + static int rtm_spin_trylock(struct arch_spinlock *lock) { - if (elide_lock(spinlock_elision, !__ticket_spin_is_locked(lock))) + if (elide_lock_adapt(spinlock_elision, !__ticket_spin_is_locked(lock), + &lock->elision_adapt, &spinlock_elision_config)) return 1; return __ticket_spin_trylock(lock); } static inline void rtm_spin_lock(struct arch_spinlock *lock) { - if (!elide_lock(spinlock_elision, !__ticket_spin_is_locked(lock))) + if (!elide_lock_adapt(spinlock_elision, !__ticket_spin_is_locked(lock), + &lock->elision_adapt, &spinlock_elision_config)) __ticket_spin_lock(lock); } -- 1.7.7.6
| |