Author: mav
Date: Tue Feb 20 04:14:12 2018
New Revision: 329623
URL: https://svnweb.freebsd.org/changeset/base/329623

Log:
  MFV r302649: 7016 arc_available_memory is not 32-bit safe
  
  illumos/illumos-gate@0dd053d7d890618ea1fc697b07de364e69eb4190
  
https://github.com/illumos/illumos-gate/commit/0dd053d7d890618ea1fc697b07de364e69eb4190
  
  https://www.illumos.org/issues/7016
    upstream DLPX-39446 arc_available_memory is not 32-bit safe
    https://github.com/delphix/delphix-os/commit/
    6b353ea3b8a1610be22e71e657d051743c64190b
    related to this upstream:
    DLPX-38547 delphix engine hang
    https://github.com/delphix/delphix-os/commit/
    3183a567b3e8c62a74a65885ca60c86f3d693783
    DLPX-38547 delphix engine hang (fix static global)
    https://github.com/delphix/delphix-os/commit/
    22ac551d8ef085ad66cc8f65e51ac372b12993b9
    DLPX-38882 system hung waiting on free segment
    https://github.com/delphix/delphix-os/commit/
    cdd6beef7548cd3b12f0fc0328eeb3af540079c2
  
  Reviewed by: Igor Kozhukhov <ikozhuk...@gmail.com>
  Reviewed by: Matthew Ahrens <mahr...@delphix.com>
  Reviewed by: Paul Dagnelie <p...@delphix.com>
  Reviewed by: George Wilson <george.wil...@delphix.com>
  Approved by: Gordon Ross <gordon.r...@nexenta.com>
  Author: Prakash Surya <prakash.su...@delphix.com>

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

Modified: head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/arc.c
==============================================================================
--- head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/arc.c   Tue Feb 20 
04:11:48 2018        (r329622)
+++ head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/arc.c   Tue Feb 20 
04:14:12 2018        (r329623)
@@ -350,6 +350,11 @@ extern boolean_t zfs_prefetch_disable;
 static boolean_t arc_warm;
 
 /*
+ * log2 fraction of the zio arena to keep free.
+ */
+int arc_zio_arena_free_shift = 2;
+
+/*
  * These tunables are for performance analysis.
  */
 uint64_t zfs_arc_max;
@@ -4321,15 +4326,16 @@ arc_available_memory(void)
        /*
         * If zio data pages are being allocated out of a separate heap segment,
         * then enforce that the size of available vmem for this arena remains
-        * above about 1/16th free.
+        * above about 1/4th (1/(2^arc_zio_arena_free_shift)) free.
         *
-        * Note: The 1/16th arena free requirement was put in place
-        * to aggressively evict memory from the arc in order to avoid
-        * memory fragmentation issues.
+        * Note that reducing the arc_zio_arena_free_shift keeps more virtual
+        * memory (in the zio_arena) free, which can avoid memory
+        * fragmentation issues.
         */
        if (zio_arena != NULL) {
                n = (int64_t)vmem_size(zio_arena, VMEM_FREE) -
-                   (vmem_size(zio_arena, VMEM_ALLOC) >> 4);
+                   (vmem_size(zio_arena, VMEM_ALLOC) >>
+                   arc_zio_arena_free_shift);
                if (n < lowest) {
                        lowest = n;
                        r = FMR_ZIO_ARENA;
_______________________________________________
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"

Reply via email to