From 3299068145e1a432aa9a2b28fdefb02566daa981 Mon Sep 17 00:00:00 2001 From: Sunil K Pandey Date: Tue, 9 Dec 2025 08:57:44 -0800 Subject: [PATCH] nptl: Optimize trylock for high cache contention workloads (BZ #33704) Check lock availability before acquisition to reduce cache line bouncing. Significantly improves trylock throughput on multi-core systems under heavy contention. Tested on x86_64. Fixes BZ #33704. Co-authored-by: Alex M Wells Reviewed-by: Wilco Dijkstra (cherry picked from commit 63716823dbad9482e09972907ae98e9cb00f9b86) --- nptl/pthread_mutex_trylock.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/nptl/pthread_mutex_trylock.c b/nptl/pthread_mutex_trylock.c index 3bba464409..5d2637d1b0 100644 --- a/nptl/pthread_mutex_trylock.c +++ b/nptl/pthread_mutex_trylock.c @@ -56,7 +56,8 @@ __pthread_mutex_trylock (pthread_mutex_t *mutex) return 0; } - if (lll_trylock (mutex->__data.__lock) == 0) + if (atomic_load_relaxed (&(mutex->__data.__lock)) == 0 + && lll_trylock (mutex->__data.__lock) == 0) { /* Record the ownership. */ mutex->__data.__owner = id; @@ -79,7 +80,10 @@ __pthread_mutex_trylock (pthread_mutex_t *mutex) /*FALL THROUGH*/ case PTHREAD_MUTEX_ADAPTIVE_NP: case PTHREAD_MUTEX_ERRORCHECK_NP: - if (lll_trylock (mutex->__data.__lock) != 0) + /* Mutex type is already loaded, lock check overhead should + be minimal. */ + if (atomic_load_relaxed (&(mutex->__data.__lock)) != 0 + || lll_trylock (mutex->__data.__lock) != 0) break; /* Record the ownership. */