On Jun. 17, 2010, 11:15 -0400, Fred Isaman <iisaman@xxxxxxxxx> wrote: > On Thu, Jun 17, 2010 at 10:49 AM, Benny Halevy <bhalevy@xxxxxxxxxxx> wrote: >> Do not renew the data server lease. >> >> Signed-off-by: Benny Halevy <bhalevy@xxxxxxxxxxx> >> --- >> fs/nfs/nfs4proc.c | 2 +- >> 1 files changed, 1 insertions(+), 1 deletions(-) >> >> diff --git a/fs/nfs/nfs4proc.c b/fs/nfs/nfs4proc.c >> index 77b7182..e8b0f33 100644 >> --- a/fs/nfs/nfs4proc.c >> +++ b/fs/nfs/nfs4proc.c >> @@ -3164,7 +3164,7 @@ static int nfs4_read_done(struct rpc_task *task, struct nfs_read_data *data) >> } >> >> nfs_invalidate_atime(data->inode); >> - if (task->tk_status > 0) >> + if (task->tk_status > 0 && client == server->nfs_client) > > Will this test work if there is a dual role MDS/DS? > The filehandle determines the "DS personality" as per 13.6. Operations Sent to NFSv4.1 Data Servers so the dual role server should act no different than any other DS with this respect. Benny > Fred > >> renew_lease(server, data->timestamp); >> return 0; >> } >> -- >> 1.6.4.4 >> >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-nfs" in >> the body of a message to majordomo@xxxxxxxxxxxxxxx >> More majordomo info at http://vger.kernel.org/majordomo-info.html >> -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
