On Fri, Apr 12, 2024 at 02:45:17AM +0000, Ihar Hrachyshka wrote: > Before the patch, the size of the backlog depended on the type of socket > (UNIX vs INET) as well as on the language (C vs Python), specifically: > > - python used backlog size = 10 for all sockets; > - C used 64 for UNIX sockets but 10 for INET sockets. > > This consolidates the values across the board. It effectively bumps the > number of simultaneous connections to python unixctl servers to 64. Also > for INET C servers too. > > The rationale to do it, on top of consistency, is as follows: > > - fmt_pkt in ovn testsuite is limited by python server listen backlog, > and as was found out when adopting the tool, it is sometimes useful to > run lots of parallel calls to fmt_pkt unixctl server in some tests. > (See [1] for example.) > > - there is a recent report [2] on discuss@ ML where the reporter noticed > significant listen queue overflows in some scenarios (large openstack > deployments; happens during leader transition when hundreds of neutron > nodes - with dozens of neutron api workers each - simultaneously > reconnect to the same northbound leader.) Note: While there is no > clear indication that this backlog size bump would resolve the > reported issues, it would probably help somewhat. > > [1] > https://github.com/ovn-org/ovn/commit/0baca3e519756cbe98a32526ccc637bb73468743 > [2] https://mail.openvswitch.org/pipermail/ovs-discuss/2024-April/053049.html > > Signed-off-by: Ihar Hrachyshka <[email protected]> > Acked-by: Eelco Chaudron <[email protected]>
Acked-by: Simon Horman <[email protected]> _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
