Re: add_wait_queue() (unintentional?) behavior change in v4.13

From: Fubo Chen
Date: Tue Dec 05 2017 - 21:35:20 EST


On Wed, Nov 29, 2017 at 4:58 PM, Omar Sandoval <osandov@xxxxxxxxxxx> wrote:
> diff --git a/kernel/sched/wait.c b/kernel/sched/wait.c
> index 98feab7933c7..929ecb7d6b78 100644
> --- a/kernel/sched/wait.c
> +++ b/kernel/sched/wait.c
> @@ -27,7 +27,7 @@ void add_wait_queue(struct wait_queue_head *wq_head, struct wait_queue_entry *wq
>
> wq_entry->flags &= ~WQ_FLAG_EXCLUSIVE;
> spin_lock_irqsave(&wq_head->lock, flags);
> - __add_wait_queue_entry_tail(wq_head, wq_entry);
> + __add_wait_queue(wq_head, wq_entry);
> spin_unlock_irqrestore(&wq_head->lock, flags);
> }
> EXPORT_SYMBOL(add_wait_queue);

Reviewed-by: Fubo Chen <fubo.chen@xxxxxxxxx>