Messages in this thread |  | | From | Trond Myklebust <> | Subject | Re: [PATCH] NFSv4: nfs4_do_fsinfo() should not do implicit lease renewals | Date | Mon, 16 Dec 2019 18:58:19 +0000 |
| |
-On Mon, 2019-12-16 at 18:43 +0000, Robert Milkowski wrote: > > From: Trond Myklebust <trondmy@hammerspace.com> > ... > > NACK. The above argument only applies to legacy minor version 0 > > setups, and does not apply to NFSv4.1 or newer. > > Correct. However many sites still use v4.0. >
That's not a good reason to break code that works just fine for NFSv4.1.
It would be better to move the initialisation of clp->cl_last_renewal into nfs4_init_clientid() and nfs41_init_clientid() (after the calls to nfs4_proc_setclientid_confirm() and nfs4_proc_create_session() respectively).
-- Trond Myklebust Linux NFS client maintainer, Hammerspace trond.myklebust@hammerspace.com
|  |