nfsd: skip subsequent UMH "create" operations after the first one for v4.0 clients
authorJeff Layton <jlayton@primarydata.com>
Fri, 12 Sep 2014 20:40:22 +0000 (16:40 -0400)
committerJ. Bruce Fields <bfields@redhat.com>
Wed, 17 Sep 2014 20:33:17 +0000 (16:33 -0400)
In the case of v4.0 clients, we may call into the "create" client
tracking operation multiple times (once for each openowner). Upcalling
for each one of those is wasteful and slow however. We can skip doing
further "create" operations after the first one if we know that one has
already been done.

v4.1+ clients generally only call into this function once (on
RECLAIM_COMPLETE), and we can't skip upcalling on the create even if the
STABLE bit is set. Doing so would make it impossible for nfsdcltrack to
lift the grace period early since the timestamp has a different meaning
in the case where the client is expected to issue a RECLAIM_COMPLETE.

Signed-off-by: Jeff Layton <jlayton@primarydata.com>
fs/nfsd/nfs4recover.c

index d0fc003d130ac21b43d880b97ccffa8c1246f4c7..ea95a2bc21b59542a517b70e2086f63430b252e9 100644 (file)
@@ -1286,6 +1286,22 @@ nfsd4_umh_cltrack_create(struct nfs4_client *clp)
        char *hexid, *has_session, *grace_start;
        struct nfsd_net *nn = net_generic(clp->net, nfsd_net_id);
 
+       /*
+        * With v4.0 clients, there's little difference in outcome between a
+        * create and check operation, and we can end up calling into this
+        * function multiple times per client (once for each openowner). So,
+        * for v4.0 clients skip upcalling once the client has been recorded
+        * on stable storage.
+        *
+        * For v4.1+ clients, the outcome of the two operations is different,
+        * so we must ensure that we upcall for the create operation. v4.1+
+        * clients call this on RECLAIM_COMPLETE though, so we should only end
+        * up doing a single create upcall per client.
+        */
+       if (clp->cl_minorversion == 0 &&
+           test_bit(NFSD4_CLIENT_STABLE, &clp->cl_flags))
+               return;
+
        hexid = bin_to_hex_dup(clp->cl_name.data, clp->cl_name.len);
        if (!hexid) {
                dprintk("%s: can't allocate memory for upcall!\n", __func__);