Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 7 Sep 2016 20:05:10 +0000 (UTC)
From:      Alexander Motin <mav@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r305561 - head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs
Message-ID:  <201609072005.u87K5ADp052196@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: mav
Date: Wed Sep  7 20:05:10 2016
New Revision: 305561
URL: https://svnweb.freebsd.org/changeset/base/305561

Log:
  MFV r305560: 7278 tuning zfs_arc_max does not impact arc_c_min
  
  When changing zfs_arc_max (e.g. as zdb does), it may be set to less
  than the default arc_c_min. arc_c_min should decrease to not be more than
  arc_c_max, but it doesn't; therefore tuning of arc_c_max is ineffective.
  
  Reviewed by: Dan Kimmel <dan.kimmel@delphix.com>
  Reviewed by: Paul Dagnelie <paul.dagnelie@delphix.com>
  Reviewed by: Prakash Surya <prakash.surya@delphix.com>
  Reviewed by: Igor Kozhukhov <ikozhukhov@gmail.com>
  Author: Matthew Ahrens <mahrens@delphix.com>
  
  openzfs/openzfs@608764beadaf4bb71c5d8fe1818e8392ac66a61b

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	Wed Sep  7 20:00:22 2016	(r305560)
+++ head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/arc.c	Wed Sep  7 20:05:10 2016	(r305561)
@@ -6062,8 +6062,10 @@ arc_init(void)
 	 * Allow the tunables to override our calculations if they are
 	 * reasonable.
 	 */
-	if (zfs_arc_max > arc_abs_min && zfs_arc_max < kmem_size())
+	if (zfs_arc_max > arc_abs_min && zfs_arc_max < kmem_size()) {
 		arc_c_max = zfs_arc_max;
+		arc_c_min = MIN(arc_c_min, arc_c_max);
+	}
 	if (zfs_arc_min > arc_abs_min && zfs_arc_min <= arc_c_max)
 		arc_c_min = zfs_arc_min;
 #endif



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201609072005.u87K5ADp052196>