Author: mav
Date: Thu Apr 25 17:22:41 2019
New Revision: 346684
URL: https://svnweb.freebsd.org/changeset/base/346684

Log:
  MFC r340311: Do not ignore arc_adjust() return value.
  
  This covers scenario when ARC may not shrink as fast as it could:
  1. arc_size < arc_c and arc_adjust() does not evict anything, returning
     zero to arc_reclaim_thread();
  2. arc_available_memory() reports memory pressure, which can not be
     satisfied by arc_kmem_reap_now();
  3. arc_shrink() reduces arc_c and calls arc_adjust(), return of which is
     ignored;
  4. even if the last arc_adjust() could not satisfy arc_size < arc_c,
     arc_reclaim_thread() will still go to sleep, since the first one
     returned zero.

Modified:
  stable/11/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/arc.c
Directory Properties:
  stable/11/   (props changed)

Modified: stable/11/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/arc.c
==============================================================================
--- stable/11/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/arc.c      Thu Apr 
25 16:47:15 2019        (r346683)
+++ stable/11/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/arc.c      Thu Apr 
25 17:22:41 2019        (r346684)
@@ -4251,7 +4251,7 @@ arc_flush(spa_t *spa, boolean_t retry)
        (void) arc_flush_state(arc_mfu_ghost, guid, ARC_BUFC_METADATA, retry);
 }
 
-void
+uint64_t
 arc_shrink(int64_t to_free)
 {
        uint64_t asize = aggsum_value(&arc_size);
@@ -4279,8 +4279,9 @@ arc_shrink(int64_t to_free)
        if (asize > arc_c) {
                DTRACE_PROBE2(arc__shrink_adjust, uint64_t, asize,
                        uint64_t, arc_c);
-               (void) arc_adjust();
+               return (arc_adjust());
        }
+       return (0);
 }
 
 typedef enum free_memory_reason_t {
@@ -4622,7 +4623,7 @@ arc_reclaim_thread(void *unused __unused)
                                to_free = MAX(to_free, ptob(needfree));
 #endif
 #endif
-                               arc_shrink(to_free);
+                               evicted += arc_shrink(to_free);
                        }
                } else if (free_memory < arc_c >> arc_no_grow_shift) {
                        arc_no_grow = B_TRUE;
_______________________________________________
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to