aboutsummaryrefslogtreecommitdiffstats
path: root/net/openvswitch
diff options
context:
space:
mode:
authorAndreea-Cristina Bernat <bernat.ada@gmail.com>2014-08-17 09:29:43 -0400
committerDavid S. Miller <davem@davemloft.net>2014-08-22 15:23:10 -0400
commit8c6b00c816191ded80d1ccd5164b53168255ec15 (patch)
tree8e5e9af329e34d0c26c76d870a49229914679d42 /net/openvswitch
parente6b688838e25b22e10dd273b48581cb2f62ec16e (diff)
net/openvswitch/flow.c: Replace rcu_dereference() with rcu_access_pointer()
The "rcu_dereference()" call is used directly in a condition. Since its return value is never dereferenced it is recommended to use "rcu_access_pointer()" instead of "rcu_dereference()". Therefore, this patch makes the replacement. The following Coccinelle semantic patch was used: @@ @@ ( if( (<+... - rcu_dereference + rcu_access_pointer (...) ...+>)) {...} | while( (<+... - rcu_dereference + rcu_access_pointer (...) ...+>)) {...} ) Signed-off-by: Andreea-Cristina Bernat <bernat.ada@gmail.com> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/openvswitch')
-rw-r--r--net/openvswitch/flow.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/net/openvswitch/flow.c b/net/openvswitch/flow.c
index d07ab538fc9d..7064da92f420 100644
--- a/net/openvswitch/flow.c
+++ b/net/openvswitch/flow.c
@@ -89,7 +89,7 @@ void ovs_flow_stats_update(struct sw_flow *flow, __be16 tcp_flags,
89 * allocated stats as we have already locked them. 89 * allocated stats as we have already locked them.
90 */ 90 */
91 if (likely(flow->stats_last_writer != NUMA_NO_NODE) 91 if (likely(flow->stats_last_writer != NUMA_NO_NODE)
92 && likely(!rcu_dereference(flow->stats[node]))) { 92 && likely(!rcu_access_pointer(flow->stats[node]))) {
93 /* Try to allocate node-specific stats. */ 93 /* Try to allocate node-specific stats. */
94 struct flow_stats *new_stats; 94 struct flow_stats *new_stats;
95 95