All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stanislav Kinsbursky <skinsbursky@parallels.com>
To: Trond.Myklebust@netapp.com
Cc: linux-nfs@vger.kernel.org, xemul@parallels.com, neilb@suse.de,
	netdev@vger.kernel.org, linux-kernel@vger.kernel.org,
	bfields@fieldses.org, davem@davemloft.net
Subject: [PATCH v4 8/8] SUNRPC: remove rpcbind clients destruction on module cleanup
Date: Tue, 20 Sep 2011 14:14:34 +0400	[thread overview]
Message-ID: <20110920101434.9861.57007.stgit@localhost6.localdomain6> (raw)
In-Reply-To: <20110920101031.9861.18444.stgit@localhost6.localdomain6>

Rpcbind clients destruction during SUNRPC module removing is obsolete since now
those clients are destroying during last RPC service shutdown.

Signed-off-by: Stanislav Kinsbursky <skinsbursky@parallels.com>

---
 net/sunrpc/rpcb_clnt.c   |   12 ------------
 net/sunrpc/sunrpc_syms.c |    3 ---
 2 files changed, 0 insertions(+), 15 deletions(-)

diff --git a/net/sunrpc/rpcb_clnt.c b/net/sunrpc/rpcb_clnt.c
index 9327305..80ddf55 100644
--- a/net/sunrpc/rpcb_clnt.c
+++ b/net/sunrpc/rpcb_clnt.c
@@ -1097,15 +1097,3 @@ static struct rpc_program rpcb_program = {
 	.version	= rpcb_version,
 	.stats		= &rpcb_stats,
 };
-
-/**
- * cleanup_rpcb_clnt - remove xprtsock's sysctls, unregister
- *
- */
-void cleanup_rpcb_clnt(void)
-{
-	if (rpcb_local_clnt4)
-		rpc_shutdown_client(rpcb_local_clnt4);
-	if (rpcb_local_clnt)
-		rpc_shutdown_client(rpcb_local_clnt);
-}
diff --git a/net/sunrpc/sunrpc_syms.c b/net/sunrpc/sunrpc_syms.c
index 9d08091..8ec9778 100644
--- a/net/sunrpc/sunrpc_syms.c
+++ b/net/sunrpc/sunrpc_syms.c
@@ -61,8 +61,6 @@ static struct pernet_operations sunrpc_net_ops = {
 
 extern struct cache_detail unix_gid_cache;
 
-extern void cleanup_rpcb_clnt(void);
-
 static int __init
 init_sunrpc(void)
 {
@@ -102,7 +100,6 @@ out:
 static void __exit
 cleanup_sunrpc(void)
 {
-	cleanup_rpcb_clnt();
 	rpcauth_remove_module();
 	cleanup_socket_xprt();
 	svc_cleanup_xprt_sock();


  parent reply	other threads:[~2011-09-20 10:14 UTC|newest]

Thread overview: 49+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-09-20 10:13 [PATCH v4 0/8] SUNRPC: make rpcbind clients allocated and destroyed dynamically Stanislav Kinsbursky
2011-09-20 10:13 ` [PATCH v4 1/8] SUNRPC: introduce helpers for reference counted rpcbind clients Stanislav Kinsbursky
2011-09-20 13:05   ` Bryan Schumaker
2011-09-20 13:15     ` Myklebust, Trond
2011-09-20 13:15       ` Myklebust, Trond
2011-09-20 13:15       ` Myklebust, Trond
2011-09-20 13:34       ` Stanislav Kinsbursky
2011-09-20 14:14         ` Myklebust, Trond
2011-09-20 14:14           ` Myklebust, Trond
2011-09-20 14:14           ` Myklebust, Trond
2011-09-20 14:35           ` Stanislav Kinsbursky
2011-09-20 14:38             ` Myklebust, Trond
2011-09-20 14:38               ` Myklebust, Trond
2011-09-20 14:38               ` Myklebust, Trond
2011-09-20 15:03               ` Stanislav Kinsbursky
2011-09-20 16:20               ` Stanislav Kinsbursky
2011-09-20 17:13                 ` Myklebust, Trond
2011-09-20 17:13                   ` Myklebust, Trond
2011-09-20 17:13                   ` Myklebust, Trond
2011-09-20 17:26                   ` Stanislav Kinsbursky
2011-09-20 13:49   ` [PATCH v5 " Stanislav Kinsbursky
2011-09-20 14:24     ` Jeff Layton
2011-09-20 14:41       ` Myklebust, Trond
2011-09-20 15:58         ` Stanislav Kinsbursky
2011-09-20 14:43       ` Stanislav Kinsbursky
2011-09-20 14:43         ` Stanislav Kinsbursky
2011-09-20 14:58         ` Bryan Schumaker
2011-09-20 15:38           ` Stanislav Kinsbursky
2011-09-20 16:06             ` Bryan Schumaker
2011-09-20 16:06               ` Bryan Schumaker
2011-09-20 15:11         ` Jeff Layton
2011-09-20 16:20           ` Stanislav Kinsbursky
2011-09-21  9:07     ` [PATCH v6 " Stanislav Kinsbursky
2011-09-23 14:41       ` Stanislav Kinsbursky
2011-09-23 17:26         ` Myklebust, Trond
2011-09-23 17:26           ` Myklebust, Trond
2011-09-23 17:26           ` Myklebust, Trond
2011-09-20 10:13 ` [PATCH v4 2/8] SUNRPC: use rpcbind reference counting helpers Stanislav Kinsbursky
2011-09-20 10:13 ` [PATCH v4 3/8] SUNRPC: introduce svc helpers for prepairing rpcbind infrastructure Stanislav Kinsbursky
2011-09-20 10:14 ` [PATCH v4 4/8] SUNRPC: setup rpcbind clients if service requires it Stanislav Kinsbursky
2011-09-20 11:22   ` Jeff Layton
2011-09-20 11:22     ` Jeff Layton
2011-09-20 10:14 ` [PATCH v4 5/8] SUNRPC: cleanup service destruction Stanislav Kinsbursky
2011-09-20 10:14   ` Stanislav Kinsbursky
2011-09-20 10:14 ` [PATCH v4 6/8] NFSd: call svc rpcbind cleanup explicitly Stanislav Kinsbursky
2011-09-20 10:14 ` [PATCH v4 7/8] SUNRPC: remove rpcbind clients creation during service registering Stanislav Kinsbursky
2011-09-20 10:14 ` Stanislav Kinsbursky [this message]
2011-09-20 11:24 ` [PATCH v4 0/8] SUNRPC: make rpcbind clients allocated and destroyed dynamically Jeff Layton
2011-09-20 11:24   ` Jeff Layton

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20110920101434.9861.57007.stgit@localhost6.localdomain6 \
    --to=skinsbursky@parallels.com \
    --cc=Trond.Myklebust@netapp.com \
    --cc=bfields@fieldses.org \
    --cc=davem@davemloft.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-nfs@vger.kernel.org \
    --cc=neilb@suse.de \
    --cc=netdev@vger.kernel.org \
    --cc=xemul@parallels.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.