]> granicus.if.org Git - zfs/commitdiff
Linux 4.6 compat: Fall back to d_prune_aliases() if necessary
authorTim Chase <tim@chase2k.com>
Thu, 16 Jun 2016 15:19:32 +0000 (10:19 -0500)
committerBrian Behlendorf <behlendorf1@llnl.gov>
Fri, 17 Jun 2016 20:33:49 +0000 (13:33 -0700)
As of 4.6, the icache and dcache LRUs are memcg aware insofar as the
kernel's per-superblock shrinker is concerned.  The effect is that dcache
or icache entries added by a task in a non-root memcg won't be scanned
by the shrinker in the context of the root (or NULL) memcg.  This defeats
the attempts by zfs_sb_prune() to unpin buffers and can allow metadata to
grow uncontrollably.  This patch reverts to the d_prune_aliaes() method
in case the kernel's per-superblock shrinker is not able to free anything.

Signed-off-by: Tim Chase <tim@chase2k.com>
Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
Signed-off-by: Chunwei Chen <tuxoko@gmail.com>
Closes: #4726
module/zfs/zfs_vfsops.c

index 7696071f1fe7be18ef58e7e506cc1616fd2d3e56..a72841c15e6c217a27d59c3a50c45d24948b65e8 100644 (file)
@@ -1050,8 +1050,7 @@ zfs_root(zfs_sb_t *zsb, struct inode **ipp)
 }
 EXPORT_SYMBOL(zfs_root);
 
-#if !defined(HAVE_SPLIT_SHRINKER_CALLBACK) && !defined(HAVE_SHRINK) && \
-       defined(HAVE_D_PRUNE_ALIASES)
+#ifdef HAVE_D_PRUNE_ALIASES
 /*
  * Linux kernels older than 3.1 do not support a per-filesystem shrinker.
  * To accommodate this we must improvise and manually walk the list of znodes
@@ -1141,15 +1140,29 @@ zfs_sb_prune(struct super_block *sb, unsigned long nr_to_scan, int *objects)
        } else {
                        *objects = (*shrinker->scan_objects)(shrinker, &sc);
        }
+
 #elif defined(HAVE_SPLIT_SHRINKER_CALLBACK)
        *objects = (*shrinker->scan_objects)(shrinker, &sc);
 #elif defined(HAVE_SHRINK)
        *objects = (*shrinker->shrink)(shrinker, &sc);
 #elif defined(HAVE_D_PRUNE_ALIASES)
+#define        D_PRUNE_ALIASES_IS_DEFAULT
        *objects = zfs_sb_prune_aliases(zsb, nr_to_scan);
 #else
 #error "No available dentry and inode cache pruning mechanism."
 #endif
+
+#if defined(HAVE_D_PRUNE_ALIASES) && !defined(D_PRUNE_ALIASES_IS_DEFAULT)
+#undef D_PRUNE_ALIASES_IS_DEFAULT
+       /*
+        * Fall back to zfs_sb_prune_aliases if the kernel's per-superblock
+        * shrinker couldn't free anything, possibly due to the inodes being
+        * allocated in a different memcg.
+        */
+       if (*objects == 0)
+               *objects = zfs_sb_prune_aliases(zsb, nr_to_scan);
+#endif
+
        ZFS_EXIT(zsb);
 
        dprintf_ds(zsb->z_os->os_dsl_dataset,