]> CyberLeo.Net >> Repos - FreeBSD/FreeBSD.git/commit
Add warning for zfs_vdev_elevator option removal
authorBrian Behlendorf <behlendorf1@llnl.gov>
Wed, 25 Sep 2019 16:23:29 +0000 (09:23 -0700)
committerGitHub <noreply@github.com>
Wed, 25 Sep 2019 16:23:29 +0000 (09:23 -0700)
commitf81d5ef686e8198c38caa8622905647667165622
tree7bdd3223686e4656a28d2fc65c05f0a14d37d422
parent5df7e9d85ced424292e8312a7848b6092a29c930
Add warning for zfs_vdev_elevator option removal

Originally the zfs_vdev_elevator module option was added as a
convenience so the requested elevator would be automatically set
on the underlying block devices.  At the time this was simple
because the kernel provided an API function which did exactly this.

This API was then removed in the Linux 4.12 kernel which prompted
us to add compatibly code to set the elevator via a usermodehelper.
While well intentioned this introduced a bug which could cause a
system hang, that issue was subsequently fixed by commit 2a0d4188.

In order to avoid future bugs in this area, and to simplify the code,
this functionality is being deprecated.  A console warning has been
added to notify any existing consumers and the documentation updated
accordingly.  This option will remain for the lifetime of the 0.8.x
series for compatibility but if planned to be phased out of master.

Reviewed-by: Richard Laager <rlaager@wiktel.com>
Reviewed-by: loli10K <ezomori.nozomu@gmail.com>
Reviewed-by: Tony Hutter <hutter2@llnl.gov>
Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
Issue #8664
Closes #9317
man/man5/zfs-module-parameters.5
module/os/linux/zfs/vdev_disk.c