NFS: callback per-net usage counting introduced
authorStanislav Kinsbursky <skinsbursky@parallels.com>
Mon, 20 Aug 2012 14:00:46 +0000 (18:00 +0400)
committerTrond Myklebust <Trond.Myklebust@netapp.com>
Mon, 1 Oct 2012 22:25:57 +0000 (15:25 -0700)
This patch also introduces refcount-aware nfs_callback_down_net() wrapper for
svc_shutdown_net().

Signed-off-by: Stanislav Kinsbursky <skinsbursky@parallels.com>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
fs/nfs/callback.c
fs/nfs/netns.h

index baafa0f1e555e9a95822536db4bc67b32a11fceb..6dfdc8311f27a01ee3b8f71d4d50e6c38a4a8886 100644 (file)
@@ -260,10 +260,25 @@ static int nfs_callback_start_svc(int minorversion, struct rpc_xprt *xprt,
        return 0;
 }
 
+static void nfs_callback_down_net(u32 minorversion, struct svc_serv *serv, struct net *net)
+{
+       struct nfs_net *nn = net_generic(net, nfs_net_id);
+
+       if (--nn->cb_users[minorversion])
+               return;
+
+       dprintk("NFS: destroy per-net callback data; net=%p\n", net);
+       svc_shutdown_net(serv, net);
+}
+
 static int nfs_callback_up_net(int minorversion, struct svc_serv *serv, struct net *net)
 {
+       struct nfs_net *nn = net_generic(net, nfs_net_id);
        int ret;
 
+       if (nn->cb_users[minorversion]++)
+               return 0;
+
        dprintk("NFS: create per-net callback data; net=%p\n", net);
 
        ret = svc_bind(serv, net);
@@ -378,7 +393,7 @@ err_create:
        return ret;
 
 err_start:
-       svc_shutdown_net(serv, net);
+       nfs_callback_down_net(minorversion, serv, net);
        dprintk("NFS: Couldn't create server thread; err = %d\n", ret);
        goto err_net;
 }
@@ -391,10 +406,10 @@ void nfs_callback_down(int minorversion, struct net *net)
        struct nfs_callback_data *cb_info = &nfs_callback_info[minorversion];
 
        mutex_lock(&nfs_callback_mutex);
+       nfs_callback_down_net(minorversion, cb_info->serv, net);
        cb_info->users--;
        if (cb_info->users == 0 && cb_info->task != NULL) {
                kthread_stop(cb_info->task);
-               svc_shutdown_net(cb_info->serv, net);
                svc_exit_thread(cb_info->rqst);
                cb_info->serv = NULL;
                cb_info->rqst = NULL;
index 137238b012fb76a2274c93de1e73a42db9ed7430..b9c7f9b1f91b11230cf3f6033fc4a86c77a94f7c 100644 (file)
@@ -24,6 +24,7 @@ struct nfs_net {
        struct idr cb_ident_idr; /* Protected by nfs_client_lock */
        unsigned short nfs_callback_tcpport;
        unsigned short nfs_callback_tcpport6;
+       int cb_users[NFS4_MAX_MINOR_VERSION + 1];
 #endif
        spinlock_t nfs_client_lock;
        struct timespec boot_time;