On Thu, Aug 08, 2013 at 02:04:25PM -0700, Ethan Jackson wrote: > > There is some gratuitous whitespace change (tabs vs. spaces?) in > > xlate_fin_timeout(). > > > > There's something about this change that makes me uncomfortable but I > > can't quite put my finger on what it is. > > I agree, I don't like it. Ideally we'd just get rid of the expirable > list, but I don't know the code well enough to know whether or not > that's feasible.
Simon presented numbers that showed it to be a valuable optimization in some cases, otherwise I'd just say get rid of it. > At any rate, I think this is fine for now. I agree. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev