diff options
author | Ying Xue <ying.xue@windriver.com> | 2017-03-28 12:28:27 +0200 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2017-03-29 03:03:32 +0200 |
commit | 139bb36f754adbf6d3c836db09d6459e25167b38 (patch) | |
tree | 67aaf23ea8b620ffeba8fd79273d480811b8cf5c /net/tipc/name_table.c | |
parent | stmmac: use netif_set_real_num_{rx,tx}_queues (diff) | |
download | linux-139bb36f754adbf6d3c836db09d6459e25167b38.tar.xz linux-139bb36f754adbf6d3c836db09d6459e25167b38.zip |
tipc: advance the time of deleting subscription from subscriber->subscrp_list
After a subscription object is created, it's inserted into its
subscriber subscrp_list list under subscriber lock protection,
similarly, before it's destroyed, it should be first removed from
its subscriber->subscrp_list. Since the subscription list is
accessed with subscriber lock, all the subscriptions are valid
during the lock duration. Hence in tipc_subscrb_subscrp_delete(), we
remove subscription get/put and the extra subscriber unlock/lock.
After this change, the subscriptions refcount cleanup is very simple
and does not access any lock.
Acked-by: Jon Maloy <jon.maloy@ericsson.com>
Signed-off-by: Ying Xue <ying.xue@windriver.com>
Signed-off-by: Parthasarathy Bhuvaragan <parthasarathy.bhuvaragan@ericsson.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/tipc/name_table.c')
0 files changed, 0 insertions, 0 deletions