locking/rwsem: Return void in __rwsem_mark_wake()
authorDavidlohr Bueso <dave@stgolabs.net>
Fri, 5 Aug 2016 08:04:43 +0000 (01:04 -0700)
committerIngo Molnar <mingo@kernel.org>
Thu, 18 Aug 2016 13:37:03 +0000 (15:37 +0200)
commit84b23f9b58687a11ced66cc4be9b0219e8ecab84
tree55d064b1c09c98b9ddc234a057c2db52ef348615
parent3942a9bd7b5842a924e99ee6ec1350b8006c94ec
locking/rwsem: Return void in __rwsem_mark_wake()

We currently return a rw_semaphore structure, which is the
same lock we passed to the function's argument in the first
place. While there are several functions that choose this
return value, the callers use it, for example, for things
like ERR_PTR. This is not the case for __rwsem_mark_wake(),
and in addition this function is really about the lock
waiters (which we know there are at this point), so its
somewhat odd to be returning the sem structure.

Signed-off-by: Davidlohr Bueso <dbueso@suse.de>
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Cc: Andrew Morton <akpm@linux-foundation.org>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: Waiman.Long@hp.com
Cc: dave@stgolabs.net
Cc: jason.low2@hpe.com
Cc: wanpeng.li@hotmail.com
Link: http://lkml.kernel.org/r/1470384285-32163-2-git-send-email-dave@stgolabs.net
Signed-off-by: Ingo Molnar <mingo@kernel.org>
kernel/locking/rwsem-xadd.c