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? 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