diff options
author | Allan Stephens <Allan.Stephens@windriver.com> | 2011-01-18 13:58:27 -0500 |
---|---|---|
committer | Paul Gortmaker <paul.gortmaker@windriver.com> | 2011-02-23 18:05:13 -0500 |
commit | 9f54b545bd62a42ec354727d90eacadc5846406b (patch) | |
tree | 275c8a726f962ce90df0acf10dc619ff468781f2 | |
parent | 01d83eddc55c138cbb24a5917d5271c0b24956a1 (diff) |
tipc: Eliminate unnecessary locking when starting topology service
Modifies the initialization code for TIPC's topology service to
avoid taking the spinlock protecting the subscriber list, since
there is no need to do this.
Signed-off-by: Allan Stephens <Allan.Stephens@windriver.com>
Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com>
-rw-r--r-- | net/tipc/subscr.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/net/tipc/subscr.c b/net/tipc/subscr.c index 98ee50b332ae..138737242739 100644 --- a/net/tipc/subscr.c +++ b/net/tipc/subscr.c | |||
@@ -542,7 +542,6 @@ int tipc_subscr_start(void) | |||
542 | spin_lock_init(&topsrv.lock); | 542 | spin_lock_init(&topsrv.lock); |
543 | INIT_LIST_HEAD(&topsrv.subscriber_list); | 543 | INIT_LIST_HEAD(&topsrv.subscriber_list); |
544 | 544 | ||
545 | spin_lock_bh(&topsrv.lock); | ||
546 | res = tipc_createport(NULL, | 545 | res = tipc_createport(NULL, |
547 | TIPC_CRITICAL_IMPORTANCE, | 546 | TIPC_CRITICAL_IMPORTANCE, |
548 | NULL, | 547 | NULL, |
@@ -563,12 +562,10 @@ int tipc_subscr_start(void) | |||
563 | goto failed; | 562 | goto failed; |
564 | } | 563 | } |
565 | 564 | ||
566 | spin_unlock_bh(&topsrv.lock); | ||
567 | return 0; | 565 | return 0; |
568 | 566 | ||
569 | failed: | 567 | failed: |
570 | err("Failed to create subscription service\n"); | 568 | err("Failed to create subscription service\n"); |
571 | spin_unlock_bh(&topsrv.lock); | ||
572 | return res; | 569 | return res; |
573 | } | 570 | } |
574 | 571 | ||