Commit e46b498c authored by J. Bruce Fields's avatar J. Bruce Fields
Browse files

nfsd4: simplify lease/grace interaction 



The original code here assumed we'd allow the user to change the lease
any time, but only allow the change to take effect on restart.  Since
then we modified the code to allow setting the lease on when the server
is down.  Update the rest of the code to reflect that fact, clarify
variable names, and add document.

Also, the code insisted that the grace period always be the longer of
the old and new lease periods, but that's overly conservative--as long
as it lasts at least the old lease period, old clients should still know
to recover in time.

Signed-off-by: default avatarJ. Bruce Fields <bfields@citi.umich.edu>
parent cf07d2ea

fs/nfsd/nfs4state.c

+11 −21
Original line number Original line Diff line number Diff line
@@ -45,7 +45,7 @@ 




/* Globals */

/* Globals */

time_t nfsd4_lease = 90;     /* default lease time */

time_t nfsd4_lease = 90;     /* default lease time */

static time_t user_lease_time = 90;

static time_t nfsd4_grace = 90;

static time_t boot_time;

static time_t boot_time;

static u32 current_ownerid = 1;

static u32 current_ownerid = 1;

static u32 current_fileid = 1;

static u32 current_fileid = 1;
@@ -2551,6 +2551,12 @@ nfsd4_end_grace(void) 
	dprintk("NFSD: end of grace period\n");

	dprintk("NFSD: end of grace period\n");

	nfsd4_recdir_purge_old();

	nfsd4_recdir_purge_old();

	locks_end_grace(&nfsd4_manager);

	locks_end_grace(&nfsd4_manager);

	/*

	 * Now that every NFSv4 client has had the chance to recover and

	 * to see the (possibly new, possibly shorter) lease time, we

	 * can safely set the next grace time to the current lease time:

	 */

	nfsd4_grace = nfsd4_lease;

}

}





static time_t

static time_t
@@ -3973,12 +3979,6 @@ nfsd4_load_reboot_recovery_data(void) 
		printk("NFSD: Failure reading reboot recovery data\n");

		printk("NFSD: Failure reading reboot recovery data\n");

}

}





unsigned long

get_nfs4_grace_period(void)

{

	return max(user_lease_time, nfsd4_lease) * HZ;

}



/*

/*

 * Since the lifetime of a delegation isn't limited to that of an open, a

 * Since the lifetime of a delegation isn't limited to that of an open, a

 * client may quite reasonably hang on to a delegation as long as it has

 * client may quite reasonably hang on to a delegation as long as it has
@@ -4005,18 +4005,14 @@ set_max_delegations(void) 
static int

static int

__nfs4_state_start(void)

__nfs4_state_start(void)

{

{

	unsigned long grace_time;



	boot_time = get_seconds();

	boot_time = get_seconds();

	grace_time = get_nfs4_grace_period();

	nfsd4_lease = user_lease_time;

	locks_start_grace(&nfsd4_manager);

	locks_start_grace(&nfsd4_manager);

	printk(KERN_INFO "NFSD: starting %ld-second grace period\n",

	printk(KERN_INFO "NFSD: starting %ld-second grace period\n",

	       grace_time/HZ);

	       nfsd4_grace);

	laundry_wq = create_singlethread_workqueue("nfsd4");

	laundry_wq = create_singlethread_workqueue("nfsd4");

	if (laundry_wq == NULL)

	if (laundry_wq == NULL)

		return -ENOMEM;

		return -ENOMEM;

	queue_delayed_work(laundry_wq, &laundromat_work, grace_time);

	queue_delayed_work(laundry_wq, &laundromat_work, nfsd4_grace * HZ);

	set_max_delegations();

	set_max_delegations();

	return set_callback_cred();

	return set_callback_cred();

}

}
@@ -4123,17 +4119,11 @@ nfs4_recoverydir(void) 
/*

/*

 * Called when leasetime is changed.

 * Called when leasetime is changed.

 *

 *

 * The only way the protocol gives us to handle on-the-fly lease changes is to

 * nfsd4_lease is protected by nfsd_mutex since it's only really accessed

 * simulate a reboot.  Instead of doing that, we just wait till the next time

 * we start to register any changes in lease time.  If the administrator

 * really wants to change the lease time *now*, they can go ahead and bring

 * nfsd down and then back up again after changing the lease time.

 *

 * user_lease_time is protected by nfsd_mutex since it's only really accessed

 * when nfsd is starting

 * when nfsd is starting

 */

 */

void

void

nfs4_reset_lease(time_t leasetime)

nfs4_reset_lease(time_t leasetime)

{

{

	user_lease_time = leasetime;

	nfsd4_lease = leasetime;

}
 
}

CRA Git | Maintained and supported by SUSTech CRA and CCSE