]> granicus.if.org Git - zfs/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)
committerBrian Behlendorf <behlendorf1@llnl.gov>
Thu, 2 Aug 2018 22:51:45 +0000 (15:51 -0700)
commit62840030a7dceaee013ddbcc1eebcfc7922edf7c
tree793c26621902f7094be3120aee5821e447e38b97
parent499b5497cbc62a4db35227bc98c6244454dfab4c
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