On Fri, Jul 15, 2022 at 8:35 AM wangchuanlei <[email protected]> wrote: > > Hi, > when start service ovn-controller, the br-int may be not connected, in > case, swconn->version = -1, > on my enviroment, i have loadlalancer, in the process of sending health > check packet, ofp_proto is 0, which will leads controller serivice to abort > in ofputil_encode_packet_out. > So this patch is to avoid this problem. > > Signed-off-by: wangchuanlei <[email protected]>
Thanks for the patch. I applied this patch to the main branch and backported to branch-22.06. I will be backporting to branch-22.03 soon (running tests). Before applying I modified the commit message a bit and also added your name to the AUTHORS.rst file. Num > --- > controller/pinctrl.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/controller/pinctrl.c b/controller/pinctrl.c > index f954362b7..2fcf91bc9 100644 > --- a/controller/pinctrl.c > +++ b/controller/pinctrl.c > @@ -3454,11 +3454,11 @@ pinctrl_handler(void *arg_) > > ip_mcast_querier_run(swconn, &send_mcast_query_time); > } > - } > > - ovs_mutex_lock(&pinctrl_mutex); > - svc_monitors_run(swconn, &svc_monitors_next_run_time); > - ovs_mutex_unlock(&pinctrl_mutex); > + ovs_mutex_lock(&pinctrl_mutex); > + svc_monitors_run(swconn, &svc_monitors_next_run_time); > + ovs_mutex_unlock(&pinctrl_mutex); > + } > > rconn_run_wait(swconn); > rconn_recv_wait(swconn); > -- > 2.27.0 > > _______________________________________________ > dev mailing list > [email protected] > https://mail.openvswitch.org/mailman/listinfo/ovs-dev > _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
