Re: [PATCH] fs: ext4: inode->i_generation not assigned 0.
From: Jeff Layton
Date: Fri Jul 07 2017 - 06:51:55 EST
On Wed, 2017-07-05 at 16:27 -0400, Theodore Ts'o wrote:
> On Wed, Jul 05, 2017 at 12:19:33PM -0700, Darrick J. Wong wrote:
> > So, what's the probability that there are clients out there that started
> > talking to a 2.2-based knfsd and will now want to talk to a modern 4.13
> > kernel seventeen years later? (Do nfs handles persist across client
> > restarts/remounts?)
> It's whether or not nfs handles persist across server restarts which
> would be the more interesting question. So if you had a NAS box that
> was using a Linux 2.2 kernel, and you had clients access the box, and
> then that box gets upgraded to use a 4.13 kernel, what happens?
> In the ideal world, the client wouldn't notice, and its 2.2-based file
> handles that it obtained while the 2.2 kernel was running would
> continue to work after the box came back up running the new 4.13
Right. That's the case today if we don't remove support for old
filehandles. If we were to remove them, the clients would get back
-ESTALE there if they tried to use the old 2.2-style fh's that they saw
before the upgrade.
The main takeaway here is that NFS filehandle lifetime is really only
bounded by the boot time of the oldest clients.
> To be honest, I'm not sure I care that much, but I don't use NFS much
> if at all these days myself. And in reality, what's the chance that
> an NAS box vendor would continue to support a box that is 17 years old
> and provide an upgrade for it? (OK, everyone can stop laughing now. :-)
Agreed. I think we're safe enough to remove this support.
If we were really concerned about it, we could move support for it under
a Kconfig option that defaults to "off" for now, and then we could plan
to just drop it in a couple of years.
Vendors that thought they might need it could enable it and speak up
with their use case if they wanted to keep it in for future releases.
Most likely though, no-one will care.
Jeff Layton <jlayton@xxxxxxxxxx>