[ovs-dev] [cfm 1/7] cfm: Migrate cfm/show unixctl command to CFM module.

Ben Pfaff blp at nicira.com
Fri May 20 21:21:53 UTC 2011


On Fri, May 20, 2011 at 02:18:58PM -0700, Ethan Jackson wrote:
> >> The main reason is that the 'name' parameter of a cfm object is not
> >> necessarily fixed. ?It can change arbitrarily at any time for any
> >> reason by the client. ? I think an obvious solution to this problem is
> >> to require 'name' in cfm_create and remove it from the globally
> >> visible struct cfm object. ?Thus it won't be mutable. ?I'll write up
> >> an incremental.
> >
> > Seems reasonable to me. ?Thanks.
> 
> Actually if you think it's fine how it is. I'll hold off on changing
> it for now.  I've been thinking about how merging cfm_create and
> cfm_configure so that it's not possible to have un-configured cfm
> objects floating around.  I'll change it as part of that at the end of
> the series.

That's fine too.



More information about the dev mailing list