svn commit: r316921 - vendor-sys/illumos/dist/uts/common/fs/zfs

[Andriy Gapon]

Author: avg
Date: Fri Apr 14 18:29:35 2017
New Revision: 316921
URL: https://svnweb.freebsd.org/changeset/base/316921

Log:
  8027 tighten up dsl_pool_dirty_delta
  
  illumos/illumos-gate at 313ae1e182df6e6a04b56c4b73ded33e11b75666
  https://github.com/illumos/illumos-gate/commit/313ae1e182df6e6a04b56c4b73ded33e11b75666
  
  https://www.illumos.org/issues/8027
    dsl_pool_dirty_delta() should not wake up waiters when dp->dp_dirty_total ==
    zfs_dirty_data_max, because they wait for dp_dirty_total to fall strictly below
    the threshold.
    It's probably very rare for that condition to occur, but it's better to have
    more accurate code.
  
  Reviewed by: Matt Ahrens <mahrens at delphix.com>
  Reviewed by: Serapheim Dimitropoulos <serapheim at delphix.com>
  Reviewed by: Paul Dagnelie <pcd at delphix.com>
  Approved by: Dan McDonald <danmcd at omniti.com>
  Author: Andriy Gapon <avg at FreeBSD.org>

Modified:
  vendor-sys/illumos/dist/uts/common/fs/zfs/dsl_pool.c

Modified: vendor-sys/illumos/dist/uts/common/fs/zfs/dsl_pool.c
==============================================================================
--- vendor-sys/illumos/dist/uts/common/fs/zfs/dsl_pool.c	Fri Apr 14 18:29:13 2017	(r316920)
+++ vendor-sys/illumos/dist/uts/common/fs/zfs/dsl_pool.c	Fri Apr 14 18:29:35 2017	(r316921)
@@ -459,7 +459,7 @@ dsl_pool_dirty_delta(dsl_pool_t *dp, int
 	 * Note: we signal even when increasing dp_dirty_total.
 	 * This ensures forward progress -- each thread wakes the next waiter.
 	 */
-	if (dp->dp_dirty_total <= zfs_dirty_data_max)
+	if (dp->dp_dirty_total < zfs_dirty_data_max)
 		cv_signal(&dp->dp_spaceavail_cv);
 }