Re: [PATCH 2/3] locking/percpu-rwsem: Replace bulky wait-queues with swait

2016-11-21 Thread Davidlohr Bueso
On Mon, 21 Nov 2016, Oleg Nesterov wrote: On 11/18, Davidlohr Bueso wrote: @@ -12,7 +12,7 @@ struct percpu_rw_semaphore { struct rcu_sync rss; unsigned int __percpu *read_count; struct rw_semaphore rw_sem; - wait_queue_head_t writer; + st

Re: [PATCH 2/3] locking/percpu-rwsem: Replace bulky wait-queues with swait

2016-11-21 Thread Oleg Nesterov
On 11/18, Davidlohr Bueso wrote: > > @@ -12,7 +12,7 @@ struct percpu_rw_semaphore { > struct rcu_sync rss; > unsigned int __percpu *read_count; > struct rw_semaphore rw_sem; > - wait_queue_head_t writer; > + struct swait_queue_head writer; I won't argu

[PATCH 2/3] locking/percpu-rwsem: Replace bulky wait-queues with swait

2016-11-18 Thread Davidlohr Bueso
In the case of the percpu-rwsem, they don't need any of the fancy/bulky features, such as custom callbacks or fine grained wakeups. Users that can convert to simple wait-queues are encouraged to do so for the various rt and (indirect) performance benefits. Signed-off-by: Davidlohr Bueso --- inc