On Mon, Dec 23, 2019 at 07:17:34PM +0800, [email protected] wrote: > From: Tonghao Zhang <[email protected]> > > There may be too many flows (> MAX_FLOWS 65536) on > dpif-netdev at same time. For this case, we support > the ovs-appctl command to change the flow max number. > > Signed-off-by: Tonghao Zhang <[email protected]>
Thanks for the patch. I think that netdev_max_flows should be an atomic_uint32_t because it is accessed from multiple threads. Usually the replies to appctl commands are very terse, so that a successful reply to change a value would just be empty and the reply for getting a value would just be the raw value, and usually we don't include a new-line at the end. Thanks, Ben. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
