diff options
| author | Richard Yao <[email protected]> | 2014-07-11 14:35:58 -0400 |
|---|---|---|
| committer | Brian Behlendorf <[email protected]> | 2017-05-04 18:00:27 -0400 |
| commit | bc17f1047a83cc8c4065e0ef84333a0d9b9d73aa (patch) | |
| tree | 240277a2f851d5bcfe4839dcb7e720dd58662945 /include | |
| parent | 5731140eaf4aaf2526a8bfdbfe250195842e79eb (diff) | |
Enable Linux read-ahead for a single page on ZVOLs
Linux has read-ahead logic designed to accelerate sequential workloads.
ZFS has its own read-ahead logic called zprefetch that operates on both
ZVOLs and datasets. Having two prefetchers active at the same time can
cause overprefetching, which unnecessarily reduces IOPS performance on
CoW filesystems like ZFS.
Testing shows that entirely disabling the Linux prefetch results in
a significant performance penalty for reads while commensurate benefits
are seen in random writes. It appears that read-ahead benefits are
inversely proportional to random write benefits, and so a single page
of Linux-layer read-ahead appears to offer the middle ground for both
workloads.
Reviewed-by: Chunwei Chen <[email protected]>
Reviewed-by: Brian Behlendorf <[email protected]>
Signed-off-by: Richard Yao <[email protected]>
Issue #5902
Diffstat (limited to 'include')
| -rw-r--r-- | include/linux/blkdev_compat.h | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/include/linux/blkdev_compat.h b/include/linux/blkdev_compat.h index 15824c0b8..822e964a7 100644 --- a/include/linux/blkdev_compat.h +++ b/include/linux/blkdev_compat.h @@ -31,6 +31,7 @@ #include <linux/blkdev.h> #include <linux/elevator.h> +#include <linux/backing-dev.h> #ifndef HAVE_FMODE_T typedef unsigned __bitwise__ fmode_t; @@ -128,6 +129,16 @@ __blk_queue_max_segments(struct request_queue *q, unsigned short max_segments) } #endif +static inline void +blk_queue_set_read_ahead(struct request_queue *q, unsigned long ra_pages) +{ +#ifdef HAVE_BLK_QUEUE_BDI_DYNAMIC + q->backing_dev_info->ra_pages = ra_pages; +#else + q->backing_dev_info.ra_pages = ra_pages; +#endif +} + #ifndef HAVE_GET_DISK_RO static inline int get_disk_ro(struct gendisk *disk) |