[ovs-dev] [RFC flow tunnels 7/8] lib: Switch to flow based tunneling.

Ben Pfaff blp at nicira.com
Sat Jan 26 23:48:15 UTC 2013


On Sat, Jan 26, 2013 at 03:16:24PM -0800, Ethan Jackson wrote:
> > Did you consider reference-counting the backer ports instead of doing a
> > search per-port-destruction?
> 
> I did.  I'm not sure it's necessary though.  This isn't in the fast
> path, so the performance benefit doesn't really help.  I think it's
> less complex to put all of this logic into a single function, instead
> of having to main a data structure as a cache.  Mostly a personal
> preference though, I'd be happy to change it.

You're right that it's probably not important.  Let's leave it for now.

> I've addressed the rest of your comments in the version I'll be
> sending out soon.

Thanks.



More information about the dev mailing list