Re: [ovs-dev] [thread-safety 03/11] cfm: Make the CFM module thread safe.

2013-07-29 Thread Ben Pfaff
On Fri, Jul 26, 2013 at 06:07:04PM -0700, Ethan Jackson wrote: > Signed-off-by: Ethan Jackson Why does cfm_generate_maid() need the mutex? It doesn't seem to use any static data and when it is called the new struct cfm isn't visible outside the running thread. Can all_cfms be a const pointer, e

[ovs-dev] [thread-safety 03/11] cfm: Make the CFM module thread safe.

2013-07-26 Thread Ethan Jackson
Signed-off-by: Ethan Jackson --- lib/cfm.c | 168 +++-- 1 file changed, 120 insertions(+), 48 deletions(-) diff --git a/lib/cfm.c b/lib/cfm.c index a76a3ec..fc58a81 100644 --- a/lib/cfm.c +++ b/lib/cfm.c @@ -91,8 +91,6 @@ struct cfm {