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

Reply via email to