On Fri, Jul 15, 2011 at 01:50:02PM -0700, Ethan Jackson wrote: > This simplifies the implementation of future patches. This seems OK. An alternative would be to add a helper function that takes an ofport_dpif and returns whether it should be enabled. I don't have a good reason to prefer either choice.