mbox series

[v2,0/3] Remove NFS v2 support from the client and server

Message ID 20211208163057.954500-1-steved@redhat.com (mailing list archive)
Headers show
Series Remove NFS v2 support from the client and server | expand

Message

Steve Dickson Dec. 8, 2021, 4:30 p.m. UTC
These patches will remove the all references and 
support of NFS v2 in both the server and client.

On server side the support has been off, by default, 
since 2013 (6b4e4965a6b). With this server patch the
ability to enable v2 will be remove.

Currently even with CONFIG_NFS_V2 not set
v2 mounts are still tied (over-the-wire). I looked at creating 
a kernel parameter module so support could re-enabled 
but that got ugly quick.

On the client, the -o v2 option is still a 
valid option because unknown mount options 
are passed to the kernel which will cause an 
actual mount to happen. 

But the option has been removed from the man
pages will cause the mount to fail with
"NFS version is not supported"

I guess the only question left is does there
need some type compilation flag or config flag
that would re-enable the support. I'm thinking not.

Steve Dickson (3):
  nfsd: Remove the ability to enable NFS v2.
  nfs.man: Remove references to NFS v2 from the man pages
  mount: Remove NFS v2 support from mount.nfs

 nfs.conf                  |  1 -
 utils/mount/configfile.c  |  2 +-
 utils/mount/mount.nfs.man |  2 +-
 utils/mount/network.c     |  4 ++--
 utils/mount/nfs.man       | 20 +++-----------------
 utils/mount/nfsmount.conf |  2 +-
 utils/mount/stropts.c     | 10 +++++++++-
 utils/nfsd/nfsd.c         |  2 --
 utils/nfsd/nfsd.man       |  4 ++--
 9 files changed, 19 insertions(+), 28 deletions(-)

Comments

Steve Dickson Jan. 10, 2022, 3:06 p.m. UTC | #1
On 12/8/21 11:30, Steve Dickson wrote:
> These patches will remove the all references and
> support of NFS v2 in both the server and client.
> 
> On server side the support has been off, by default,
> since 2013 (6b4e4965a6b). With this server patch the
> ability to enable v2 will be remove.
> 
> Currently even with CONFIG_NFS_V2 not set
> v2 mounts are still tied (over-the-wire). I looked at creating
> a kernel parameter module so support could re-enabled
> but that got ugly quick.
> 
> On the client, the -o v2 option is still a
> valid option because unknown mount options
> are passed to the kernel which will cause an
> actual mount to happen.
> 
> But the option has been removed from the man
> pages will cause the mount to fail with
> "NFS version is not supported"
> 
> I guess the only question left is does there
> need some type compilation flag or config flag
> that would re-enable the support. I'm thinking not.
> 
> Steve Dickson (3):
>    nfsd: Remove the ability to enable NFS v2.
>    nfs.man: Remove references to NFS v2 from the man pages
>    mount: Remove NFS v2 support from mount.nfs
> 
>   nfs.conf                  |  1 -
>   utils/mount/configfile.c  |  2 +-
>   utils/mount/mount.nfs.man |  2 +-
>   utils/mount/network.c     |  4 ++--
>   utils/mount/nfs.man       | 20 +++-----------------
>   utils/mount/nfsmount.conf |  2 +-
>   utils/mount/stropts.c     | 10 +++++++++-
>   utils/nfsd/nfsd.c         |  2 --
>   utils/nfsd/nfsd.man       |  4 ++--
>   9 files changed, 19 insertions(+), 28 deletions(-)
> 
Committed.... (tag: nfs-utils-2-5-5-rc5)

steved.