sched/wait: Standardize 'struct wait_bit_queue' wait-queue entry field name
authorIngo Molnar <mingo@kernel.org>
Sun, 5 Mar 2017 10:25:39 +0000 (11:25 +0100)
committerIngo Molnar <mingo@kernel.org>
Tue, 20 Jun 2017 10:18:28 +0000 (12:18 +0200)
Rename 'struct wait_bit_queue::wait' to ::wq_entry, to more clearly
name it as a wait-queue entry.

Propagate it to a couple of usage sites where the wait-bit-queue internals
are exposed.

Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Ingo Molnar <mingo@kernel.org>
fs/inode.c
fs/jbd2/journal.c
fs/xfs/xfs_icache.c
fs/xfs/xfs_inode.c
include/linux/wait.h
kernel/sched/wait.c

index db5914783a7130d77725502cb4182c05ff7775c2..70761d6cafcd7ef97e172b2e84fb9c9412c809a6 100644 (file)
@@ -1891,11 +1891,11 @@ static void __wait_on_freeing_inode(struct inode *inode)
        wait_queue_head_t *wq;
        DEFINE_WAIT_BIT(wait, &inode->i_state, __I_NEW);
        wq = bit_waitqueue(&inode->i_state, __I_NEW);
-       prepare_to_wait(wq, &wait.wait, TASK_UNINTERRUPTIBLE);
+       prepare_to_wait(wq, &wait.wq_entry, TASK_UNINTERRUPTIBLE);
        spin_unlock(&inode->i_lock);
        spin_unlock(&inode_hash_lock);
        schedule();
-       finish_wait(wq, &wait.wait);
+       finish_wait(wq, &wait.wq_entry);
        spin_lock(&inode_hash_lock);
 }
 
@@ -2038,11 +2038,11 @@ static void __inode_dio_wait(struct inode *inode)
        DEFINE_WAIT_BIT(q, &inode->i_state, __I_DIO_WAKEUP);
 
        do {
-               prepare_to_wait(wq, &q.wait, TASK_UNINTERRUPTIBLE);
+               prepare_to_wait(wq, &q.wq_entry, TASK_UNINTERRUPTIBLE);
                if (atomic_read(&inode->i_dio_count))
                        schedule();
        } while (atomic_read(&inode->i_dio_count));
-       finish_wait(wq, &q.wait);
+       finish_wait(wq, &q.wq_entry);
 }
 
 /**
index ebad34266bcfbbf4fb5d491b2f2699583f7116e1..7d5ef3bf3f3e06410199e1240243a506b32b50b3 100644 (file)
@@ -2579,10 +2579,10 @@ restart:
                wait_queue_head_t *wq;
                DEFINE_WAIT_BIT(wait, &jinode->i_flags, __JI_COMMIT_RUNNING);
                wq = bit_waitqueue(&jinode->i_flags, __JI_COMMIT_RUNNING);
-               prepare_to_wait(wq, &wait.wait, TASK_UNINTERRUPTIBLE);
+               prepare_to_wait(wq, &wait.wq_entry, TASK_UNINTERRUPTIBLE);
                spin_unlock(&journal->j_list_lock);
                schedule();
-               finish_wait(wq, &wait.wait);
+               finish_wait(wq, &wait.wq_entry);
                goto restart;
        }
 
index 990210fcb9c326f923c0c25a86557a0aba3bb40f..b9c12e1cc23a5d39e64362203b6b1852d7659b35 100644 (file)
@@ -269,12 +269,12 @@ xfs_inew_wait(
        DEFINE_WAIT_BIT(wait, &ip->i_flags, __XFS_INEW_BIT);
 
        do {
-               prepare_to_wait(wq, &wait.wait, TASK_UNINTERRUPTIBLE);
+               prepare_to_wait(wq, &wait.wq_entry, TASK_UNINTERRUPTIBLE);
                if (!xfs_iflags_test(ip, XFS_INEW))
                        break;
                schedule();
        } while (true);
-       finish_wait(wq, &wait.wait);
+       finish_wait(wq, &wait.wq_entry);
 }
 
 /*
index ec9826c565009e5244682beae93c4efc5261b487..c0a1e840a5884e57ce736324eee6d0467b0b0ebd 100644 (file)
@@ -622,12 +622,12 @@ __xfs_iflock(
        DEFINE_WAIT_BIT(wait, &ip->i_flags, __XFS_IFLOCK_BIT);
 
        do {
-               prepare_to_wait_exclusive(wq, &wait.wait, TASK_UNINTERRUPTIBLE);
+               prepare_to_wait_exclusive(wq, &wait.wq_entry, TASK_UNINTERRUPTIBLE);
                if (xfs_isiflocked(ip))
                        io_schedule();
        } while (!xfs_iflock_nowait(ip));
 
-       finish_wait(wq, &wait.wait);
+       finish_wait(wq, &wait.wq_entry);
 }
 
 STATIC uint
@@ -2486,11 +2486,11 @@ __xfs_iunpin_wait(
        xfs_iunpin(ip);
 
        do {
-               prepare_to_wait(wq, &wait.wait, TASK_UNINTERRUPTIBLE);
+               prepare_to_wait(wq, &wait.wq_entry, TASK_UNINTERRUPTIBLE);
                if (xfs_ipincount(ip))
                        io_schedule();
        } while (xfs_ipincount(ip));
-       finish_wait(wq, &wait.wait);
+       finish_wait(wq, &wait.wq_entry);
 }
 
 void
index c3d1cefc7853e25256a88b81d11c55232410c87a..1c8add685f2251b136da08aad31490de0f57fcc9 100644 (file)
@@ -38,7 +38,7 @@ struct wait_bit_key {
 
 struct wait_bit_queue {
        struct wait_bit_key     key;
-       struct wait_queue_entry wait;
+       struct wait_queue_entry wq_entry;
 };
 
 struct wait_queue_head {
@@ -991,11 +991,11 @@ int wake_bit_function(struct wait_queue_entry *wq_entry, unsigned mode, int sync
 #define DEFINE_WAIT_BIT(name, word, bit)                               \
        struct wait_bit_queue name = {                                  \
                .key = __WAIT_BIT_KEY_INITIALIZER(word, bit),           \
-               .wait   = {                                             \
+               .wq_entry = {                                           \
                        .private        = current,                      \
                        .func           = wake_bit_function,            \
                        .task_list      =                               \
-                               LIST_HEAD_INIT((name).wait.task_list),  \
+                               LIST_HEAD_INIT((name).wq_entry.task_list), \
                },                                                      \
        }
 
index 203aeea96f1678e02a002a45e8c896b2d67e84b0..f1ba0625b8bef8e177e42a5f3c2cf880ce63fc0e 100644 (file)
@@ -395,7 +395,7 @@ int wake_bit_function(struct wait_queue_entry *wq_entry, unsigned mode, int sync
 {
        struct wait_bit_key *key = arg;
        struct wait_bit_queue *wait_bit
-               = container_of(wq_entry, struct wait_bit_queue, wait);
+               = container_of(wq_entry, struct wait_bit_queue, wq_entry);
 
        if (wait_bit->key.flags != key->flags ||
                        wait_bit->key.bit_nr != key->bit_nr ||
@@ -418,11 +418,11 @@ __wait_on_bit(struct wait_queue_head *wq_head, struct wait_bit_queue *q,
        int ret = 0;
 
        do {
-               prepare_to_wait(wq_head, &q->wait, mode);
+               prepare_to_wait(wq_head, &q->wq_entry, mode);
                if (test_bit(q->key.bit_nr, q->key.flags))
                        ret = (*action)(&q->key, mode);
        } while (test_bit(q->key.bit_nr, q->key.flags) && !ret);
-       finish_wait(wq_head, &q->wait);
+       finish_wait(wq_head, &q->wq_entry);
        return ret;
 }
 EXPORT_SYMBOL(__wait_on_bit);
@@ -431,9 +431,9 @@ int __sched out_of_line_wait_on_bit(void *word, int bit,
                                    wait_bit_action_f *action, unsigned mode)
 {
        struct wait_queue_head *wq_head = bit_waitqueue(word, bit);
-       DEFINE_WAIT_BIT(wait, word, bit);
+       DEFINE_WAIT_BIT(wq_entry, word, bit);
 
-       return __wait_on_bit(wq_head, &wait, action, mode);
+       return __wait_on_bit(wq_head, &wq_entry, action, mode);
 }
 EXPORT_SYMBOL(out_of_line_wait_on_bit);
 
@@ -442,10 +442,10 @@ int __sched out_of_line_wait_on_bit_timeout(
        unsigned mode, unsigned long timeout)
 {
        struct wait_queue_head *wq_head = bit_waitqueue(word, bit);
-       DEFINE_WAIT_BIT(wait, word, bit);
+       DEFINE_WAIT_BIT(wq_entry, word, bit);
 
-       wait.key.timeout = jiffies + timeout;
-       return __wait_on_bit(wq_head, &wait, action, mode);
+       wq_entry.key.timeout = jiffies + timeout;
+       return __wait_on_bit(wq_head, &wq_entry, action, mode);
 }
 EXPORT_SYMBOL_GPL(out_of_line_wait_on_bit_timeout);
 
@@ -456,7 +456,7 @@ __wait_on_bit_lock(struct wait_queue_head *wq_head, struct wait_bit_queue *q,
        int ret = 0;
 
        for (;;) {
-               prepare_to_wait_exclusive(wq_head, &q->wait, mode);
+               prepare_to_wait_exclusive(wq_head, &q->wq_entry, mode);
                if (test_bit(q->key.bit_nr, q->key.flags)) {
                        ret = action(&q->key, mode);
                        /*
@@ -466,11 +466,11 @@ __wait_on_bit_lock(struct wait_queue_head *wq_head, struct wait_bit_queue *q,
                         * smp_mb__after_atomic() before wake_up_page().
                         */
                        if (ret)
-                               finish_wait(wq_head, &q->wait);
+                               finish_wait(wq_head, &q->wq_entry);
                }
                if (!test_and_set_bit(q->key.bit_nr, q->key.flags)) {
                        if (!ret)
-                               finish_wait(wq_head, &q->wait);
+                               finish_wait(wq_head, &q->wq_entry);
                        return 0;
                } else if (ret) {
                        return ret;
@@ -483,9 +483,9 @@ int __sched out_of_line_wait_on_bit_lock(void *word, int bit,
                                         wait_bit_action_f *action, unsigned mode)
 {
        struct wait_queue_head *wq_head = bit_waitqueue(word, bit);
-       DEFINE_WAIT_BIT(wait, word, bit);
+       DEFINE_WAIT_BIT(wq_entry, word, bit);
 
-       return __wait_on_bit_lock(wq_head, &wait, action, mode);
+       return __wait_on_bit_lock(wq_head, &wq_entry, action, mode);
 }
 EXPORT_SYMBOL(out_of_line_wait_on_bit_lock);
 
@@ -538,8 +538,7 @@ static int wake_atomic_t_function(struct wait_queue_entry *wq_entry, unsigned mo
                                  void *arg)
 {
        struct wait_bit_key *key = arg;
-       struct wait_bit_queue *wait_bit
-               = container_of(wq_entry, struct wait_bit_queue, wait);
+       struct wait_bit_queue *wait_bit = container_of(wq_entry, struct wait_bit_queue, wq_entry);
        atomic_t *val = key->flags;
 
        if (wait_bit->key.flags != key->flags ||
@@ -562,24 +561,24 @@ int __wait_on_atomic_t(struct wait_queue_head *wq_head, struct wait_bit_queue *q
        int ret = 0;
 
        do {
-               prepare_to_wait(wq_head, &q->wait, mode);
+               prepare_to_wait(wq_head, &q->wq_entry, mode);
                val = q->key.flags;
                if (atomic_read(val) == 0)
                        break;
                ret = (*action)(val);
        } while (!ret && atomic_read(val) != 0);
-       finish_wait(wq_head, &q->wait);
+       finish_wait(wq_head, &q->wq_entry);
        return ret;
 }
 
 #define DEFINE_WAIT_ATOMIC_T(name, p)                                  \
        struct wait_bit_queue name = {                                  \
                .key = __WAIT_ATOMIC_T_KEY_INITIALIZER(p),              \
-               .wait   = {                                             \
+               .wq_entry = {                                           \
                        .private        = current,                      \
                        .func           = wake_atomic_t_function,       \
                        .task_list      =                               \
-                               LIST_HEAD_INIT((name).wait.task_list),  \
+                               LIST_HEAD_INIT((name).wq_entry.task_list), \
                },                                                      \
        }
 
@@ -587,9 +586,9 @@ __sched int out_of_line_wait_on_atomic_t(atomic_t *p, int (*action)(atomic_t *),
                                         unsigned mode)
 {
        struct wait_queue_head *wq_head = atomic_t_waitqueue(p);
-       DEFINE_WAIT_ATOMIC_T(wait, p);
+       DEFINE_WAIT_ATOMIC_T(wq_entry, p);
 
-       return __wait_on_atomic_t(wq_head, &wait, action, mode);
+       return __wait_on_atomic_t(wq_head, &wq_entry, action, mode);
 }
 EXPORT_SYMBOL(out_of_line_wait_on_atomic_t);