The function is called from main thread, and wait_* subprocedures access data structures that are managed by pinctrl thread, so make sure the access to them is guarded.
Signed-off-by: Ihar Hrachyshka <[email protected]> --- controller/pinctrl.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/controller/pinctrl.c b/controller/pinctrl.c index 428863293..9a1a0faa1 100644 --- a/controller/pinctrl.c +++ b/controller/pinctrl.c @@ -4030,12 +4030,14 @@ prepare_ipv6_ras(const struct shash *local_active_ports_ras, void pinctrl_wait(struct ovsdb_idl_txn *ovnsb_idl_txn) { + ovs_mutex_lock(&pinctrl_mutex); wait_put_mac_bindings(ovnsb_idl_txn); wait_controller_event(ovnsb_idl_txn); wait_put_vport_bindings(ovnsb_idl_txn); int64_t new_seq = seq_read(pinctrl_main_seq); seq_wait(pinctrl_main_seq, new_seq); wait_put_fdbs(ovnsb_idl_txn); + ovs_mutex_unlock(&pinctrl_mutex); } /* Called by ovn-controller. */ -- 2.34.1 _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
