diff options
| author | Etienne Dechamps <etienne.dechamps@ovh.net> | 2011-09-05 13:15:45 +0000 |
|---|---|---|
| committer | Brian Behlendorf <behlendorf1@llnl.gov> | 2012-02-08 00:23:06 +0000 |
| commit | 34037afe24e0bff97cf5262f8f1a76f5e0815dc1 (patch) | |
| tree | 563a33de02991aeade1393ef1c61f4b46d733474 /lib/libshare | |
| parent | b18019d2d810585185493c62e9567fa85e51692c (diff) | |
| download | src-34037afe24e0bff97cf5262f8f1a76f5e0815dc1.tar.gz src-34037afe24e0bff97cf5262f8f1a76f5e0815dc1.zip | |
Improve ZVOL queue behavior.
The Linux block device queue subsystem exposes a number of configurable
settings described in Linux block/blk-settings.c. The defaults for these
settings are tuned for hard drives, and are not optimized for ZVOLs. Proper
configuration of these options would allow upper layers (I/O scheduler) to
take better decisions about write merging and ordering.
Detailed rationale:
- max_hw_sectors is set to unlimited (UINT_MAX). zvol_write() is able to
handle writes of any size, so there's no reason to impose a limit. Let the
upper layer decide.
- max_segments and max_segment_size are set to unlimited. zvol_write() will
copy the requests' contents into a dbuf anyway, so the number and size of
the segments are irrelevant. Let the upper layer decide.
- physical_block_size and io_opt are set to the ZVOL's block size. This
has the potential to somewhat alleviate issue #361 for ZVOLs, by warning
the upper layers that writes smaller than the volume's block size will be
slow.
- The NONROT flag is set to indicate this isn't a rotational device.
Although the backing zpool might be composed of rotational devices, the
resulting ZVOL often doesn't exhibit the same behavior due to the COW
mechanisms used by ZFS. Setting this flag will prevent upper layers from
making useless decisions (such as reordering writes) based on incorrect
assumptions about the behavior of the ZVOL.
Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
Diffstat (limited to 'lib/libshare')
| -rw-r--r-- | lib/libshare/Makefile.in | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/lib/libshare/Makefile.in b/lib/libshare/Makefile.in index 23550232f1a0..8951784ad256 100644 --- a/lib/libshare/Makefile.in +++ b/lib/libshare/Makefile.in @@ -52,6 +52,11 @@ am__aclocal_m4_deps = \ $(top_srcdir)/config/kernel-blk-end-request.m4 \ $(top_srcdir)/config/kernel-blk-fetch-request.m4 \ $(top_srcdir)/config/kernel-blk-queue-flush.m4 \ + $(top_srcdir)/config/kernel-blk-queue-io-opt.m4 \ + $(top_srcdir)/config/kernel-blk-queue-max-hw-sectors.m4 \ + $(top_srcdir)/config/kernel-blk-queue-max-segments.m4 \ + $(top_srcdir)/config/kernel-blk-queue-nonrot.m4 \ + $(top_srcdir)/config/kernel-blk-queue-physical-block-size.m4 \ $(top_srcdir)/config/kernel-blk-requeue-request.m4 \ $(top_srcdir)/config/kernel-blk-rq-bytes.m4 \ $(top_srcdir)/config/kernel-blk-rq-pos.m4 \ |