]> git.baikalelectronics.ru Git - kernel.git/commit
Revert "locking/mutex: Complain upon mutex API misuse in IRQ contexts"
authorDavidlohr Bueso <dave@stgolabs.net>
Tue, 10 Dec 2019 22:05:23 +0000 (14:05 -0800)
committerIngo Molnar <mingo@kernel.org>
Tue, 10 Dec 2019 23:27:43 +0000 (00:27 +0100)
commit8bafe996a9c836820dd2379edc5a8d4763aa2533
tree9185a7841c50da7ecaa35fdfa3991ee8c95db4d5
parentd0de0842e00e69300812be69efda53e8f73cafe6
Revert "locking/mutex: Complain upon mutex API misuse in IRQ contexts"

This ended up causing some noise in places such as rxrpc running in softirq.

The warning is misleading in this case as the mutex trylock and unlock
operations are done within the same context; and therefore we need not
worry about the PI-boosting issues that comes along with no single-owner
lock guarantees.

While we don't want to support this in mutexes, there is no way out of
this yet; so lets get rid of the WARNs for now, as it is only fair to
code that has historically relied on non-preemptible softirq guarantees.
In addition, changing the lock type is also unviable: exclusive rwsems
have the same issue (just not the WARN_ON) and counting semaphores
would introduce a performance hit as mutexes are a lot more optimized.

This reverts:

    ffa070fa9b93: ("locking/mutex: Complain upon mutex API misuse in IRQ contexts")

Fixes: ffa070fa9b93: ("locking/mutex: Complain upon mutex API misuse in IRQ contexts")
Signed-off-by: Davidlohr Bueso <dbueso@suse.de>
Tested-by: David Howells <dhowells@redhat.com>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: linux-afs@lists.infradead.org
Cc: linux-fsdevel@vger.kernel.org
Cc: will@kernel.org
Link: https://lkml.kernel.org/r/20191210220523.28540-1-dave@stgolabs.net
Signed-off-by: Ingo Molnar <mingo@kernel.org>
kernel/locking/mutex.c