]> git.proxmox.com Git - mirror_zfs.git/commit - module/zfs/vdev_queue.c
Add separate aggregation limit for non-rotating media
authorAlexander Motin <mav@FreeBSD.org>
Wed, 13 Mar 2019 19:00:10 +0000 (15:00 -0400)
committerBrian Behlendorf <behlendorf1@llnl.gov>
Wed, 13 Mar 2019 19:00:10 +0000 (12:00 -0700)
commit1af240f3b51c080376bb6ae1efc13d62b087b65d
tree0f602dc3baa2d9d9a9f1543d102204a3d6327990
parent12a935ee9c2e2aa92309e7afb9d73e0757f61cd7
Add separate aggregation limit for non-rotating media

Before sequential scrub patches ZFS never aggregated I/Os above 128KB.
Sequential scrub bumped that to 1MB, supposedly to reduce number of
head seeks for spinning disks.  But for SSDs it makes little to no
sense, especially on FreeBSD, where due to MAXPHYS limitation device
will likely still see bunch of 128KB I/Os instead of one large.
Having more strict aggregation limit for SSDs allows to avoid
allocation of large memory buffer and copy to/from it, that is a
serious problem when throughput reaches gigabytes per second.

Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Reviewed-by: Richard Elling <Richard.Elling@RichardElling.com>
Signed-off-by: Alexander Motin <mav@FreeBSD.org>
Closes #8494
man/man5/zfs-module-parameters.5
module/zfs/vdev_queue.c