nfsd: note inadequate stats locking
authorJ. Bruce Fields <bfields@redhat.com>
Fri, 17 May 2019 20:22:18 +0000 (16:22 -0400)
committerJ. Bruce Fields <bfields@redhat.com>
Wed, 3 Jul 2019 21:52:08 +0000 (17:52 -0400)
After 89a26b3d295d "nfsd: split DRC global spinlock into per-bucket
locks", there is no longer a single global spinlock to protect these
stats.

So, really we need to fix that.  For now, at least fix the comment.

Signed-off-by: J. Bruce Fields <bfields@redhat.com>
fs/nfsd/netns.h

index 46240c4..ceb413f 100644 (file)
@@ -142,8 +142,11 @@ struct nfsd_net {
        unsigned int             drc_hashsize;
 
        /*
-        * Stats and other tracking of on the duplicate reply cache. All of these and
-        * the "rc" fields in nfsdstats are protected by the cache_lock
+        * Stats and other tracking of on the duplicate reply cache.
+        * These fields and the "rc" fields in nfsdstats are modified
+        * with only the per-bucket cache lock, which isn't really safe
+        * and should be fixed if we want the statistics to be
+        * completely accurate.
         */
 
        /* total number of entries */