svn commit: r358835 - stable/12/sys/cddl/contrib/opensolaris/uts/common/fs/zfs

Alexander Motin mav at FreeBSD.org
Tue Mar 10 15:18:54 UTC 2020


Author: mav
Date: Tue Mar 10 15:18:53 2020
New Revision: 358835
URL: https://svnweb.freebsd.org/changeset/base/358835

Log:
  MFC r358580: Increase number of write completion threads, matching ZoL.
  
  Our iSCSI benchmarks on a large 80-core system show that previous limit
  of 8 threads can be a bottleneck.  At some points this change increases
  write IOPS by as much as 50%.  I am still not sure that so many threads
  is really required, but we tested lower amounts and got no significant
  benefits, while latencies were a bit worse, so decided to not diverge.

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

Modified: stable/12/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/spa.c
==============================================================================
--- stable/12/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/spa.c	Tue Mar 10 13:39:05 2020	(r358834)
+++ stable/12/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/spa.c	Tue Mar 10 15:18:53 2020	(r358835)
@@ -155,7 +155,7 @@ const zio_taskq_info_t zio_taskqs[ZIO_TYPES][ZIO_TASKQ
 	/* ISSUE	ISSUE_HIGH	INTR		INTR_HIGH */
 	{ ZTI_ONE,	ZTI_NULL,	ZTI_ONE,	ZTI_NULL }, /* NULL */
 	{ ZTI_N(8),	ZTI_NULL,	ZTI_P(12, 8),	ZTI_NULL }, /* READ */
-	{ ZTI_BATCH,	ZTI_N(5),	ZTI_N(8),	ZTI_N(5) }, /* WRITE */
+	{ ZTI_BATCH,	ZTI_N(5),	ZTI_P(12, 8),	ZTI_N(5) }, /* WRITE */
 	{ ZTI_P(12, 8),	ZTI_NULL,	ZTI_ONE,	ZTI_NULL }, /* FREE */
 	{ ZTI_ONE,	ZTI_NULL,	ZTI_ONE,	ZTI_NULL }, /* CLAIM */
 	{ ZTI_ONE,	ZTI_NULL,	ZTI_ONE,	ZTI_NULL }, /* IOCTL */


More information about the svn-src-all mailing list