On Fri, Jul 06, 2018 at 05:22:55PM +0000, Vishal Deep Ajmera wrote: > > > > At first glance it looks like you're correct, but this code was added in > > 2013 with the following commit and hasn't changed since, so I wonder whether > > we're missing something important. Ethan, you wrote this code, do you have > > any thoughts? > > > > commit e79a6c833e0d72370951d6f8841098103cbb0b2d > > Author: Ethan J. Jackson <e...@eecs.berkeley.edu> Tue Sep 24 13:39:56 > > 2013 > > Committer: Ethan J. Jackson <e...@eecs.berkeley.edu> Thu Dec 19 > > 13:27:23 > > 2013 > > > > ofproto: Handle flow installation and eviction in upcall. > > > > This patch moves flow installation and eviction from ofproto-dpif and > > the main thread, into ofproto-dpif-upcall. This performs > > significantly better (approximately 2x TCP_CRR improvement), and > > allows ovs-vswitchd to maintain significantly larger datapath flow > > tables. On top of that, it significantly simplifies the code, > > retiring "struct facet" and friends. > > > > Signed-off-by: Ethan Jackson <et...@nicira.com> > > Acked-by: Ben Pfaff <b...@nicira.com> > > Thanks Ben for looking into this issue. Should I send a formal patch > for review by removing this check (n_flows > 2000) ?
Let's see what Ethan has to say first. _______________________________________________ discuss mailing list disc...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-discuss