summaryrefslogtreecommitdiffstats
path: root/sys/dev/nvme/nvme_sysctl.c
diff options
context:
space:
mode:
authormav <mav@FreeBSD.org>2016-03-10 17:13:10 +0000
committermav <mav@FreeBSD.org>2016-03-10 17:13:10 +0000
commit5c07241f1aa6ca3073298cff41130646463432ca (patch)
treeb93d0ef7e1a3ff13efee7b78ef033c7869c810fd /sys/dev/nvme/nvme_sysctl.c
parent9a3b845122f98fdbb9f4a721718ae9ac9fb920c0 (diff)
downloadFreeBSD-src-5c07241f1aa6ca3073298cff41130646463432ca.zip
FreeBSD-src-5c07241f1aa6ca3073298cff41130646463432ca.tar.gz
Revert r292074 (by smh): Limit stripesize reported from nvd(4) to 4K
I believe that this patch handled the problem from the wrong side. Instead of making ZFS properly handle large stripe sizes, it made unrelated driver to lie in reported parameters to workaround that. Alternative solution for this problem from ZFS side was committed at r296615. Discussed with: smh
Diffstat (limited to 'sys/dev/nvme/nvme_sysctl.c')
-rw-r--r--sys/dev/nvme/nvme_sysctl.c16
1 files changed, 0 insertions, 16 deletions
diff --git a/sys/dev/nvme/nvme_sysctl.c b/sys/dev/nvme/nvme_sysctl.c
index 08cd15e..44b0ab7 100644
--- a/sys/dev/nvme/nvme_sysctl.c
+++ b/sys/dev/nvme/nvme_sysctl.c
@@ -33,22 +33,6 @@ __FBSDID("$FreeBSD$");
#include "nvme_private.h"
-SYSCTL_NODE(_kern, OID_AUTO, nvme, CTLFLAG_RD, 0, "NVM Express");
-/*
- * Intel NVMe controllers have a slow path for I/Os that span a 128KB
- * stripe boundary but ZFS limits ashift, which is derived from
- * d_stripesize, to 13 (8KB) so we limit the stripesize reported to
- * geom(8) to 4KB by default.
- *
- * This may result in a small number of additional I/Os to require
- * splitting in nvme(4), however the NVMe I/O path is very efficient
- * so these additional I/Os will cause very minimal (if any) difference
- * in performance or CPU utilisation.
- */
-int nvme_max_optimal_sectorsize = 1<<12;
-SYSCTL_INT(_kern_nvme, OID_AUTO, max_optimal_sectorsize, CTLFLAG_RWTUN,
- &nvme_max_optimal_sectorsize, 0, "The maximum optimal sectorsize reported");
-
/*
* CTLTYPE_S64 and sysctl_handle_64 were added in r217616. Define these
* explicitly here for older kernels that don't include the r217616
OpenPOWER on IntegriCloud