nfsd: don't create nfsv4recoverydir in nfsdfs when not used.
authorNeilBrown <neilb@suse.de>
Tue, 16 Apr 2024 21:23:02 +0000 (07:23 +1000)
committerChuck Lever <chuck.lever@oracle.com>
Mon, 6 May 2024 13:07:19 +0000 (09:07 -0400)
commit0770249b90f9d9f69714b76adc36cf6c895bc1f9
tree657fb346a476a0686d98b9da6ff45bb7cf1429af
parentd43113fbbf79c00fe3ecd637f8cadebc4df9a63c
nfsd: don't create nfsv4recoverydir in nfsdfs when not used.

When CONFIG_NFSD_LEGACY_CLIENT_TRACKING is not set, the virtual file
  /proc/fs/nfsd/nfsv4recoverydir
is created but responds EINVAL to any access.
This is not useful, is somewhat surprising, and it causes ltp to
complain.

The only known user of this file is in nfs-utils, which handles
non-existence and read-failure equally well.  So there is nothing to
gain from leaving the file present but inaccessible.

So this patch removes the file when its content is not available - i.e.
when that config option is not selected.

Also remove the #ifdef which hides some of the enum values when
CONFIG_NFSD_V$ not selection.  simple_fill_super() quietly ignores array
entries that are not present, so having slots in the array that don't
get used is perfectly acceptable.  So there is no value in this #ifdef.

Reported-by: Petr Vorel <pvorel@suse.cz>
Reviewed-by: Jeff Layton <jlayton@kernel.org>
Fixes: 74fd48739d04 ("nfsd: new Kconfig option for legacy client tracking")
Signed-off-by: NeilBrown <neilb@suse.de>
Reviewed-by: Petr Vorel <pvorel@suse.cz>
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
fs/nfsd/nfsctl.c