On Thu, Sep 07, 2017 at 10:43:29AM +1000, NeilBrown wrote: > > >> + host_err = nfsd_conflicting_leases(dentry, rqstp); > >> + host_err = host_err ?: notify_change(dentry, &size_attr, nfsd_deleg_owner, NULL); > > > > And then you recall nfsd delegations and delegations held by > > (hypothetical) non-nfsd users separately, OK (also ignoring how). > > > > There are no such users currently, so nfsd could just pass NULL. > > I don't think so. If we pass NULL (as the owner), when VFS will recall > the one nfsd delegation that we want to preserve. ??? Oops, right.--b.