diff options
| author | Brian Behlendorf <[email protected]> | 2018-06-19 21:52:45 -0700 |
|---|---|---|
| committer | Tony Hutter <[email protected]> | 2018-07-06 02:46:51 -0700 |
| commit | b5ee3df7765cd3b46c28f2b1ff3f8d9895f4155c (patch) | |
| tree | b91ab1cdfef772d30a12855bb4c654c3e2a002e1 /config | |
| parent | 17cd9a8e0c7d9ebdc01f24c7ec8409ccde7a41fb (diff) | |
Linux 4.14 compat: blk_queue_stackable()
The blk_queue_stackable() function was replaced in the 4.14 kernel
by queue_is_rq_based(), commit torvalds/linux@5fdee212. This change
resulted in the default elevator being used which can negatively
impact performance.
Rather than adding additional compatibility code to detect the
new interface unconditionally attempt to set the elevator. Since
we expect this to fail for block devices without an elevator the
error message has been moved in to zfs_dbgmsg().
Finally, it was observed that the elevator_change() was removed
from the 4.12 kernel, commit torvalds/linux@c033269. Update the
comment to clearly specify which are expected to export the
elevator_change() symbol.
Reviewed-by: Matthew Ahrens <[email protected]>
Reviewed-by: Tony Hutter <[email protected]>
Signed-off-by: Brian Behlendorf <[email protected]>
Closes #7645
Diffstat (limited to 'config')
| -rw-r--r-- | config/kernel-elevator-change.m4 | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/config/kernel-elevator-change.m4 b/config/kernel-elevator-change.m4 index ace5aa82d..eba252579 100644 --- a/config/kernel-elevator-change.m4 +++ b/config/kernel-elevator-change.m4 @@ -1,6 +1,6 @@ dnl # -dnl # 2.6.36 API change -dnl # Verify the elevator_change() symbol is available. +dnl # 2.6.36 API, exported elevator_change() symbol +dnl # 4.12 API, removed elevator_change() symbol dnl # AC_DEFUN([ZFS_AC_KERNEL_ELEVATOR_CHANGE], [ AC_MSG_CHECKING([whether elevator_change() is available]) |