Merge: nfsd: don't destroy global nfs4_file table in per-net shutdown

MR: https://gitlab.com/redhat/centos-stream/src/kernel/centos-stream-9/-/merge_requests/2058

Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=2169017

        The nfs4_file table is global, so shutting it down when a containerized
        nfsd is shut down is wrong and can lead to double-frees. Tear down the
        nfs4_file_rhltable in nfs4_state_shutdown instead of
        nfs4_state_shutdown_net.
    
        Fixes: d47b295 ("NFSD: Use rhashtable for managing nfs4_file objects")
        Link: https://bugzilla.redhat.com/show_bug.cgi?id=2169017
        Reported-by: JianHong Yin <jiyin@redhat.com>
        Signed-off-by: Jeff Layton <jlayton@kernel.org>
        Signed-off-by: Chuck Lever <chuck.lever@oracle.com>

Signed-off-by: Jeffrey Layton <jlayton@redhat.com>

Approved-by: Scott Mayhew <smayhew@redhat.com>
Approved-by: Benjamin Coddington <bcodding@redhat.com>

Signed-off-by: Herton R. Krzesinski <herton@redhat.com>

Author: Herton R. Krzesinski

fs/nfsd/nfs4state.c

@@ -7760,7 +7760,6 @@ nfs4_state_shutdown_net(struct net *net)
 
 	nfsd4_client_tracking_exit(net);
 	nfs4_state_destroy_net(net);
-	rhltable_destroy(&nfs4_file_rhltable);
 #ifdef CONFIG_NFSD_V4_2_INTER_SSC
 	nfsd4_ssc_shutdown_umount(nn);
 #endif
@@ -7770,6 +7769,7 @@ void
 nfs4_state_shutdown(void)
 {
 	nfsd4_destroy_callback_queue();
+	rhltable_destroy(&nfs4_file_rhltable);
 }
 
 static void