diff options
| author | Jarno Rajahalme <jrajahalme@nicira.com> | 2014-07-07 13:18:46 -0700 |
|---|---|---|
| committer | Jarno Rajahalme <jrajahalme@nicira.com> | 2014-07-07 13:22:57 -0700 |
| commit | 24f8381214966e90819bf4a9ecabf076cbfc1b08 (patch) | |
| tree | 21563e7ce162f4769d191c08d1e78cafd3a83759 /lib/lacp.c | |
| parent | 6969766b7557b33e7588abe5956c21c23450110c (diff) | |
| download | openvswitch-24f8381214966e90819bf4a9ecabf076cbfc1b08.tar.gz | |
Use ovs_refcount_unref_relaxed.
After a quick analysis, in most cases the access to refcounted objects
is clearly protected either with an explicit lock/mutex, or RCU. there
are only a few places where I left a call to ovs_refcount_unref().
Upon closer analysis it may well be that those could also use the
relaxed form.
Signed-off-by: Jarno Rajahalme <jrajahalme@nicira.com>
Acked-by: Ben Pfaff <blp@nicira.com>
Diffstat (limited to 'lib/lacp.c')
| -rw-r--r-- | lib/lacp.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/lacp.c b/lib/lacp.c index 0d30e518b..3b50d46fe 100644 --- a/lib/lacp.c +++ b/lib/lacp.c @@ -251,7 +251,7 @@ lacp_ref(const struct lacp *lacp_) void lacp_unref(struct lacp *lacp) OVS_EXCLUDED(mutex) { - if (lacp && ovs_refcount_unref(&lacp->ref_cnt) == 1) { + if (lacp && ovs_refcount_unref_relaxed(&lacp->ref_cnt) == 1) { struct slave *slave, *next; lacp_lock(); |