mutex: have non-spinning mutexes on s390 by default
authorHeiko Carstens <heiko.carstens@de.ibm.com>
Thu, 9 Apr 2009 16:48:34 +0000 (18:48 +0200)
committerIngo Molnar <mingo@elte.hu>
Thu, 9 Apr 2009 17:28:24 +0000 (19:28 +0200)
commit36cd3c9f925b9307236505ae7ad1ad7ac4d4357c
treed9be68502d0f11b2259427e9ee8320891367143c
parent022624a758dc9489388a99ad29577b4c8c09237c
mutex: have non-spinning mutexes on s390 by default

Impact: performance regression fix for s390

The adaptive spinning mutexes will not always do what one would expect on
virtualized architectures like s390. Especially the cpu_relax() loop in
mutex_spin_on_owner might hurt if the mutex holding cpu has been scheduled
away by the hypervisor.

We would end up in a cpu_relax() loop when there is no chance that the
state of the mutex changes until the target cpu has been scheduled again by
the hypervisor.

For that reason we should change the default behaviour to no-spin on s390.

We do have an instruction which allows to yield the current cpu in favour of
a different target cpu. Also we have an instruction which allows us to figure
out if the target cpu is physically backed.

However we need to do some performance tests until we can come up with
a solution that will do the right thing on s390.

Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>
Acked-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Martin Schwidefsky <schwidefsky@de.ibm.com>
Cc: Christian Borntraeger <borntraeger@de.ibm.com>
LKML-Reference: <20090409184834.7a0df7b2@osiris.boeblingen.de.ibm.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
arch/Kconfig
arch/s390/Kconfig
kernel/mutex.c