From: Tonghao Zhang <xiangxia.m....@gmail.com>

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

Signed-off-by: Tonghao Zhang <xiangxia.m....@gmail.com>
---
 net/openvswitch/flow_table.c | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/net/openvswitch/flow_table.c b/net/openvswitch/flow_table.c
index fd8a01c..b7b5744 100644
--- 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);
        }
 
-- 
1.8.3.1

_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to