On Wed, Jan 08, 2014 at 09:48:45AM -0800, Pravin Shelar wrote:
> I found atomic_init without destroy for following var.
> clock->slow_path
That one doesn't have a call to atomic_destroy() because the timeval
module doesn't ever get uninitialized.
> rule->ref_count
Thanks. I folded this in:
diff --git a/ofproto/ofproto.c b/ofproto/ofproto.c
index 028b07a..f651374 100644
--- a/ofproto/ofproto.c
+++ b/ofproto/ofproto.c
@@ -2578,6 +2578,7 @@ ofproto_rule_destroy__(struct rule *rule)
cls_rule_destroy(CONST_CAST(struct cls_rule *, &rule->cr));
rule_actions_unref(rule->actions);
ovs_mutex_destroy(&rule->mutex);
+ atomic_destroy(&rule->ref_count);
rule->ofproto->ofproto_class->rule_dealloc(rule);
}
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev
