On Wed, Jan 8, 2020 at 6:34 AM Ben Pfaff <[email protected]> wrote: > > 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. v2 is sent, pls help me review this patch: http://patchwork.ozlabs.org/patch/1219494/
Thanks. > Thanks, > > Ben. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
