nfsd: make NFSv4 grace time per net
authorStanislav Kinsbursky <skinsbursky@parallels.com>
Tue, 27 Nov 2012 11:11:49 +0000 (14:11 +0300)
committerJ. Bruce Fields <bfields@redhat.com>
Wed, 28 Nov 2012 15:39:47 +0000 (10:39 -0500)
Grace time is a part of NFSv4 state engine, which is constructed per network
namespace.

Signed-off-by: Stanislav Kinsbursky <skinsbursky@parallels.com>
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
fs/nfsd/netns.h
fs/nfsd/nfs4state.c
fs/nfsd/nfsctl.c
fs/nfsd/nfsd.h

index 0c20be82cb01fa21245cfd9f2211586da52b3312..2c4b2e2896ddbb8ad5841a91d5615fe5b4207892 100644 (file)
@@ -89,6 +89,7 @@ struct nfsd_net {
        bool in_grace;
 
        time_t nfsd4_lease;
+       time_t nfsd4_grace;
 };
 
 extern int nfsd_net_id;
index 932b2ca6f203ec6ae805510bf14e6c2dd5a2ea42..3db7617e6d39e724d86c4b87e4e5a9df34c39d7f 100644 (file)
@@ -50,9 +50,6 @@
 
 #define NFSDDBG_FACILITY                NFSDDBG_PROC
 
-/* Globals */
-time_t nfsd4_grace = 90;
-
 #define all_ones {{~0,~0},~0}
 static const stateid_t one_stateid = {
        .si_generation = ~0,
@@ -3183,7 +3180,7 @@ nfsd4_end_grace(struct nfsd_net *nn)
         * to see the (possibly new, possibly shorter) lease time, we
         * can safely set the next grace time to the current lease time:
         */
-       nfsd4_grace = nn->nfsd4_lease;
+       nn->nfsd4_grace = nn->nfsd4_lease;
 }
 
 static time_t
@@ -4884,8 +4881,8 @@ nfs4_state_start_net(struct net *net)
        locks_start_grace(net, &nn->nfsd4_manager);
        nn->grace_ended = false;
        printk(KERN_INFO "NFSD: starting %ld-second grace period (net %p)\n",
-              nfsd4_grace, net);
-       queue_delayed_work(laundry_wq, &nn->laundromat_work, nfsd4_grace * HZ);
+              nn->nfsd4_grace, net);
+       queue_delayed_work(laundry_wq, &nn->laundromat_work, nn->nfsd4_grace * HZ);
        return 0;
 }
 
index 09d909a42ececdf4f8fbb7a8dc65face91048b4e..d902f83681e72187a08d47332cceb7578c1fe8c1 100644 (file)
@@ -925,7 +925,8 @@ static ssize_t write_leasetime(struct file *file, char *buf, size_t size)
  */
 static ssize_t write_gracetime(struct file *file, char *buf, size_t size)
 {
-       return nfsd4_write_time(file, buf, size, &nfsd4_grace);
+       struct nfsd_net *nn = net_generic(&init_net, nfsd_net_id);
+       return nfsd4_write_time(file, buf, size, &nn->nfsd4_grace);
 }
 
 static ssize_t __write_recoverydir(struct file *file, char *buf, size_t size)
@@ -1070,6 +1071,7 @@ static __net_init int nfsd_init_net(struct net *net)
        if (retval)
                goto out_idmap_error;
        nn->nfsd4_lease = 90;   /* default lease time */
+       nn->nfsd4_grace = 90;
        return 0;
 
 out_idmap_error:
index a8f7325a9124aa0f9ffaeafc3c0f83a1e5c6bd2b..5eea0f5021fd22b8736a533007d4a330e00476fd 100644 (file)
@@ -276,8 +276,6 @@ extern struct timeval       nfssvc_boot;
 
 #ifdef CONFIG_NFSD_V4
 
-extern time_t nfsd4_grace;
-
 /* before processing a COMPOUND operation, we have to check that there
  * is enough space in the buffer for XDR encode to succeed.  otherwise,
  * we might process an operation with side effects, and be unable to