OSDN Git Service

NFSD: Rename boot verifier functions
authorChuck Lever <chuck.lever@oracle.com>
Thu, 30 Dec 2021 15:22:05 +0000 (10:22 -0500)
committerChuck Lever <chuck.lever@oracle.com>
Sat, 8 Jan 2022 19:42:02 +0000 (14:42 -0500)
Clean up: These functions handle what the specs call a write
verifier, which in the Linux NFS server implementation is now
divorced from the server's boot instance

Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
fs/nfsd/filecache.c
fs/nfsd/netns.h
fs/nfsd/nfs4proc.c
fs/nfsd/nfssvc.c
fs/nfsd/vfs.c

index e290454..8bc807c 100644 (file)
@@ -243,7 +243,7 @@ nfsd_file_do_unhash(struct nfsd_file *nf)
        trace_nfsd_file_unhash(nf);
 
        if (nfsd_file_check_write_error(nf))
-               nfsd_reset_boot_verifier(net_generic(nf->nf_net, nfsd_net_id));
+               nfsd_reset_write_verifier(net_generic(nf->nf_net, nfsd_net_id));
        --nfsd_file_hashtbl[nf->nf_hashval].nfb_count;
        hlist_del_rcu(&nf->nf_node);
        atomic_long_dec(&nfsd_filecache_count);
index a6ed300..1b1a962 100644 (file)
@@ -198,6 +198,6 @@ extern void nfsd_netns_free_versions(struct nfsd_net *nn);
 
 extern unsigned int nfsd_net_id;
 
-void nfsd_copy_boot_verifier(__be32 verf[2], struct nfsd_net *nn);
-void nfsd_reset_boot_verifier(struct nfsd_net *nn);
+void nfsd_copy_write_verifier(__be32 verf[2], struct nfsd_net *nn);
+void nfsd_reset_write_verifier(struct nfsd_net *nn);
 #endif /* __NFSD_NETNS_H__ */
index 4305708..6f53eb9 100644 (file)
@@ -598,7 +598,7 @@ static void gen_boot_verifier(nfs4_verifier *verifier, struct net *net)
 
        BUILD_BUG_ON(2*sizeof(*verf) != sizeof(verifier->data));
 
-       nfsd_copy_boot_verifier(verf, net_generic(net, nfsd_net_id));
+       nfsd_copy_write_verifier(verf, net_generic(net, nfsd_net_id));
 }
 
 static __be32
index 81d4704..0719359 100644 (file)
@@ -346,14 +346,14 @@ static bool nfsd_needs_lockd(struct nfsd_net *nn)
 }
 
 /**
- * nfsd_copy_boot_verifier - Atomically copy a write verifier
+ * nfsd_copy_write_verifier - Atomically copy a write verifier
  * @verf: buffer in which to receive the verifier cookie
  * @nn: NFS net namespace
  *
  * This function provides a wait-free mechanism for copying the
- * namespace's boot verifier without tearing it.
+ * namespace's write verifier without tearing it.
  */
-void nfsd_copy_boot_verifier(__be32 verf[2], struct nfsd_net *nn)
+void nfsd_copy_write_verifier(__be32 verf[2], struct nfsd_net *nn)
 {
        int seq = 0;
 
@@ -364,7 +364,7 @@ void nfsd_copy_boot_verifier(__be32 verf[2], struct nfsd_net *nn)
        done_seqretry(&nn->writeverf_lock, seq);
 }
 
-static void nfsd_reset_boot_verifier_locked(struct nfsd_net *nn)
+static void nfsd_reset_write_verifier_locked(struct nfsd_net *nn)
 {
        struct timespec64 now;
        u64 verf;
@@ -379,7 +379,7 @@ static void nfsd_reset_boot_verifier_locked(struct nfsd_net *nn)
 }
 
 /**
- * nfsd_reset_boot_verifier - Generate a new boot verifier
+ * nfsd_reset_write_verifier - Generate a new write verifier
  * @nn: NFS net namespace
  *
  * This function updates the ->writeverf field of @nn. This field
@@ -391,10 +391,10 @@ static void nfsd_reset_boot_verifier_locked(struct nfsd_net *nn)
  * server and MUST be unique between instances of the NFSv4.1
  * server."
  */
-void nfsd_reset_boot_verifier(struct nfsd_net *nn)
+void nfsd_reset_write_verifier(struct nfsd_net *nn)
 {
        write_seqlock(&nn->writeverf_lock);
-       nfsd_reset_boot_verifier_locked(nn);
+       nfsd_reset_write_verifier_locked(nn);
        write_sequnlock(&nn->writeverf_lock);
 }
 
@@ -683,7 +683,7 @@ int nfsd_create_serv(struct net *net)
                register_inet6addr_notifier(&nfsd_inet6addr_notifier);
 #endif
        }
-       nfsd_reset_boot_verifier(nn);
+       nfsd_reset_write_verifier(nn);
        return 0;
 }
 
index 70ea7e0..4956445 100644 (file)
@@ -559,8 +559,8 @@ __be32 nfsd4_clone_file_range(struct svc_rqst *rqstp,
                                        &nfsd4_get_cstate(rqstp)->current_fh,
                                        dst_pos,
                                        count, status);
-                       nfsd_reset_boot_verifier(net_generic(nf_dst->nf_net,
-                                                nfsd_net_id));
+                       nfsd_reset_write_verifier(net_generic(nf_dst->nf_net,
+                                                 nfsd_net_id));
                        ret = nfserrno(status);
                }
        }
@@ -1013,10 +1013,10 @@ nfsd_vfs_write(struct svc_rqst *rqstp, struct svc_fh *fhp, struct nfsd_file *nf,
        iov_iter_kvec(&iter, WRITE, vec, vlen, *cnt);
        since = READ_ONCE(file->f_wb_err);
        if (verf)
-               nfsd_copy_boot_verifier(verf, nn);
+               nfsd_copy_write_verifier(verf, nn);
        host_err = vfs_iter_write(file, &iter, &pos, flags);
        if (host_err < 0) {
-               nfsd_reset_boot_verifier(nn);
+               nfsd_reset_write_verifier(nn);
                goto out_nfserr;
        }
        *cnt = host_err;
@@ -1029,7 +1029,7 @@ nfsd_vfs_write(struct svc_rqst *rqstp, struct svc_fh *fhp, struct nfsd_file *nf,
        if (stable && use_wgather) {
                host_err = wait_for_concurrent_writes(file);
                if (host_err < 0)
-                       nfsd_reset_boot_verifier(nn);
+                       nfsd_reset_write_verifier(nn);
        }
 
 out_nfserr:
@@ -1142,7 +1142,7 @@ nfsd_commit(struct svc_rqst *rqstp, struct svc_fh *fhp,
                err2 = vfs_fsync_range(nf->nf_file, offset, end, 0);
                switch (err2) {
                case 0:
-                       nfsd_copy_boot_verifier(verf, nn);
+                       nfsd_copy_write_verifier(verf, nn);
                        err2 = filemap_check_wb_err(nf->nf_file->f_mapping,
                                                    since);
                        break;
@@ -1150,11 +1150,11 @@ nfsd_commit(struct svc_rqst *rqstp, struct svc_fh *fhp,
                        err = nfserr_notsupp;
                        break;
                default:
-                       nfsd_reset_boot_verifier(nn);
+                       nfsd_reset_write_verifier(nn);
                }
                err = nfserrno(err2);
        } else
-               nfsd_copy_boot_verifier(verf, nn);
+               nfsd_copy_write_verifier(verf, nn);
 
        nfsd_file_put(nf);
 out: