Commit d4e5076c authored by xuyehan's avatar xuyehan Committed by Peter Zijlstra
Browse files

locking/rwsem: Remove an unused parameter of rwsem_wake()



The 2nd parameter 'count' is not used in this function.
The places where the function is called are also modified.

Signed-off-by: default avatarxuyehan <xuyehan@xiaomi.com>
Signed-off-by: default avatarPeter Zijlstra (Intel) <peterz@infradead.org>
Acked-by: default avatarWaiman Long <longman@redhat.com>
Link: https://lore.kernel.org/r/1625547043-28103-1-git-send-email-yehanxu1@gmail.com
parent d1bbfd0c
Loading
Loading
Loading
Loading
+3 −3
Original line number Original line Diff line number Diff line
@@ -1165,7 +1165,7 @@ rwsem_down_write_slowpath(struct rw_semaphore *sem, int state)
 * handle waking up a waiter on the semaphore
 * handle waking up a waiter on the semaphore
 * - up_read/up_write has decremented the active part of count if we come here
 * - up_read/up_write has decremented the active part of count if we come here
 */
 */
static struct rw_semaphore *rwsem_wake(struct rw_semaphore *sem, long count)
static struct rw_semaphore *rwsem_wake(struct rw_semaphore *sem)
{
{
	unsigned long flags;
	unsigned long flags;
	DEFINE_WAKE_Q(wake_q);
	DEFINE_WAKE_Q(wake_q);
@@ -1297,7 +1297,7 @@ static inline void __up_read(struct rw_semaphore *sem)
	if (unlikely((tmp & (RWSEM_LOCK_MASK|RWSEM_FLAG_WAITERS)) ==
	if (unlikely((tmp & (RWSEM_LOCK_MASK|RWSEM_FLAG_WAITERS)) ==
		      RWSEM_FLAG_WAITERS)) {
		      RWSEM_FLAG_WAITERS)) {
		clear_nonspinnable(sem);
		clear_nonspinnable(sem);
		rwsem_wake(sem, tmp);
		rwsem_wake(sem);
	}
	}
}
}


@@ -1319,7 +1319,7 @@ static inline void __up_write(struct rw_semaphore *sem)
	rwsem_clear_owner(sem);
	rwsem_clear_owner(sem);
	tmp = atomic_long_fetch_add_release(-RWSEM_WRITER_LOCKED, &sem->count);
	tmp = atomic_long_fetch_add_release(-RWSEM_WRITER_LOCKED, &sem->count);
	if (unlikely(tmp & RWSEM_FLAG_WAITERS))
	if (unlikely(tmp & RWSEM_FLAG_WAITERS))
		rwsem_wake(sem, tmp);
		rwsem_wake(sem);
}
}


/*
/*