Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
From:   "NeilBrown" <neilb@suse.de>
To:     "Steve Dickson" <steved@redhat.com>
Cc:     "Linux NFS Mailing list" <linux-nfs@vger.kernel.org>
Subject: Re: [PATCH RFC 0/3] Remove NFS v2 support from the client and server
In-reply-to: <20211129192731.783466-1-steved@redhat.com>
References: <20211129192731.783466-1-steved@redhat.com>
Date:   Tue, 30 Nov 2021 07:38:10 +1100
Message-id: <163821829080.26075.9477835708436642432@noble.neil.brown.name>
Precedence: bulk

On Tue, 30 Nov 2021, Steve Dickson wrote:
> These patches will remove the all references and 
> support of NFS v2 in both the server and client.

What is the motivation for this?
I don't necessarily disagree, but I'm curious as to what you hope to
gain.

Thanks,
NeilBrown

> 
> 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.
> 
> So I just decided to make all V2 mounts fail with
> EOPNOTSUPP, with no way of turn them back on.
> 
> 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     |  3 +++
>  utils/nfsd/nfsd.c         |  2 --
>  utils/nfsd/nfsd.man       |  4 ++--
>  9 files changed, 13 insertions(+), 27 deletions(-)
> 
> -- 
> 2.31.1
> 
>