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

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

On Fri, May 20, 2011 at 02:04:21PM -0700, Ethan Jackson wrote:
> > I'd be more comfortable with the memory management if the cfm object
> > had its own copy of the name, instead of relying on the client to keep
> > it valid.
> A future patch moves to this style of managing 'name'.  I probably
> should pull it back into this one though.  I'll go ahead and do that.

I noticed that later in the series.

For that reason, it doesn't matter to me.

> > Any reason not to use an shash or hmap instead of a list?
> 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.

More information about the dev mailing list