]> git.proxmox.com Git - mirror_zfs.git/commit
Reduce taskq and context-switch cost of zio pipe
authorMatthew Ahrens <mahrens@delphix.com>
Thu, 2 Aug 2018 22:51:45 +0000 (15:51 -0700)
committerTony Hutter <hutter2@llnl.gov>
Thu, 8 Nov 2018 22:38:28 +0000 (14:38 -0800)
commit45579c951587c5856ea3db7e429be4866c4dfe85
treef88dfd41678159086ca0c7fb6156ea101d04380f
parentb32f1279d4a09a932dcbfa79e125ce60cd8d75bf
Reduce taskq and context-switch cost of zio pipe

When doing a read from disk, ZFS creates 3 ZIO's: a zio_null(), the
logical zio_read(), and then a physical zio. Currently, each of these
results in a separate taskq_dispatch(zio_execute).

On high-read-iops workloads, this causes a significant performance
impact. By processing all 3 ZIO's in a single taskq entry, we reduce the
overhead on taskq locking and context switching.  We accomplish this by
allowing zio_done() to return a "next zio to execute" to zio_execute().

This results in a ~12% performance increase for random reads, from
96,000 iops to 108,000 iops (with recordsize=8k, on SSD's).

Reviewed by: Pavel Zakharov <pavel.zakharov@delphix.com>
Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Reviewed by: George Wilson <george.wilson@delphix.com>
Signed-off-by: Matthew Ahrens <mahrens@delphix.com>
External-issue: DLPX-59292
Closes #7736
include/sys/zio.h
module/zfs/zio.c