diff options
author | Paul E. McKenney | 2013-12-05 03:42:03 +0100 |
---|---|---|
committer | Paul E. McKenney | 2014-02-18 00:01:42 +0100 |
commit | cb1e78cfa267453bb19e7edafd214c03834b664c (patch) | |
tree | 1211ed872c631b1cd71052cbd20e0ca89221cb51 /kernel/rcu/torture.c | |
parent | rcu: Stop tracking FSF's postal address (diff) | |
download | kernel-qcow2-linux-cb1e78cfa267453bb19e7edafd214c03834b664c.tar.gz kernel-qcow2-linux-cb1e78cfa267453bb19e7edafd214c03834b664c.tar.xz kernel-qcow2-linux-cb1e78cfa267453bb19e7edafd214c03834b664c.zip |
rcu: Remove ACCESS_ONCE() from jiffies
Because jiffies is one of a very few variables marked "volatile", there
is no need to use ACCESS_ONCE() when accessing it. This commit therefore
removes the redundant ACCESS_ONCE() wrappers.
Reported by: Eric Dumazet <eric.dumazet@gmail.com>
Signed-off-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
Reviewed-by: Josh Triplett <josh@joshtriplett.org>
Diffstat (limited to 'kernel/rcu/torture.c')
-rw-r--r-- | kernel/rcu/torture.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/kernel/rcu/torture.c b/kernel/rcu/torture.c index ab7dd192a50b..022c5312b725 100644 --- a/kernel/rcu/torture.c +++ b/kernel/rcu/torture.c @@ -1352,7 +1352,7 @@ rcu_torture_shutdown(void *arg) unsigned long jiffies_snap; VERBOSE_PRINTK_STRING("rcu_torture_shutdown task started"); - jiffies_snap = ACCESS_ONCE(jiffies); + jiffies_snap = jiffies; while (ULONG_CMP_LT(jiffies_snap, shutdown_time) && !kthread_should_stop()) { delta = shutdown_time - jiffies_snap; @@ -1361,7 +1361,7 @@ rcu_torture_shutdown(void *arg) "rcu_torture_shutdown task: %lu jiffies remaining\n", torture_type, delta); schedule_timeout_interruptible(delta); - jiffies_snap = ACCESS_ONCE(jiffies); + jiffies_snap = jiffies; } if (kthread_should_stop()) { VERBOSE_PRINTK_STRING("rcu_torture_shutdown task stopping"); |