On Thu, May 07, 2015 at 09:53:02AM +0200, Christoph Hellwig wrote: > On Wed, May 06, 2015 at 12:12:13PM +0200, Andreas Grünbacher wrote: > > 2015-05-06 8:56 GMT+02:00 Christoph Hellwig <hch@xxxxxx>: > > > How about moving the workaround into the NFSv2 specific code? > > > > Not trivially, we would have to fh_verify() the file handle in > > nfsd_proc_setattr() first. Is that preferable? > > Sounds better to me than making this workaround even more invasive > in the core. Bruce, what do you think? I haven't looked into it, but I certainly don't see a reason why an extra call to nfsd_proc_setattr would be a problem. And I agree that it would be nice to have this in NFSv2-specific code. --b. -- 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