net: openvswitch: use hlist_for_each_entry_rcu instead of hlist_for_each_entry

The struct sw_flow is protected by RCU, when traversing them,
use hlist_for_each_entry_rcu.

Signed-off-by: Tonghao Zhang <xiangxia.m.yue@gmail.com>
Tested-by: Greg Rose <gvrose8192@gmail.com>
Reviewed-by: Greg Rose <gvrose8192@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

diff --git a/net/openvswitch/flow_table.c b/net/openvswitch/flow_table.c
index fd8a01c..2398d72 100644 (file)
--- a/net/openvswitch/flow_table.c
+++ b/net/openvswitch/flow_table.c
@@ -462,12 +462,14 @@ static void flow_table_copy_flows(struct table_instance *old,
                struct hlist_head *head = &old->buckets[i];
 
                if (ufid)
-                       hlist_for_each_entry(flow, head,
-                                            ufid_table.node[old_ver])
+                       hlist_for_each_entry_rcu(flow, head,
+                                                ufid_table.node[old_ver],
+                                                lockdep_ovsl_is_held())
                                ufid_table_instance_insert(new, flow);
                else
-                       hlist_for_each_entry(flow, head,
-                                            flow_table.node[old_ver])
+                       hlist_for_each_entry_rcu(flow, head,
+                                                flow_table.node[old_ver],
+                                                lockdep_ovsl_is_held())
                                table_instance_insert(new, flow);
        }