From e89bd69775d5379f6835dade1e14005aa17f4903 Mon Sep 17 00:00:00 2001 From: Isaac Huang Date: Sat, 21 Mar 2015 23:32:54 -0600 Subject: [PATCH] zio_injection_enabled should not be a module option The zio_inject.c keeps zio_injection_enabled as a counter of fault handlers, so it should not be exported to user space as a module option. Several EXPORT_SYMBOLs are moved from zio.c to zio_inject.c, where the symbols are defined. Signed-off-by: Isaac Huang Signed-off-by: Brian Behlendorf Closes #3199 --- man/man5/zfs-module-parameters.5 | 11 ----------- module/zfs/zio.c | 8 -------- module/zfs/zio_inject.c | 9 +++++++-- 3 files changed, 7 insertions(+), 21 deletions(-) diff --git a/man/man5/zfs-module-parameters.5 b/man/man5/zfs-module-parameters.5 index fe31e292a..9a3e2149a 100644 --- a/man/man5/zfs-module-parameters.5 +++ b/man/man5/zfs-module-parameters.5 @@ -1413,17 +1413,6 @@ Max zio millisec delay before posting event Default value: \fB30,000\fR. .RE -.sp -.ne 2 -.na -\fBzio_injection_enabled\fR (int) -.ad -.RS 12n -Enable fault injection -.sp -Use \fB1\fR for yes and \fB0\fR for no (default). -.RE - .sp .ne 2 .na diff --git a/module/zfs/zio.c b/module/zfs/zio.c index 49e2d93b6..ad0064443 100644 --- a/module/zfs/zio.c +++ b/module/zfs/zio.c @@ -3331,14 +3331,6 @@ zbookmark_is_before(const dnode_phys_t *dnp, const zbookmark_phys_t *zb1, } #if defined(_KERNEL) && defined(HAVE_SPL) -/* Fault injection */ -EXPORT_SYMBOL(zio_injection_enabled); -EXPORT_SYMBOL(zio_inject_fault); -EXPORT_SYMBOL(zio_inject_list_next); -EXPORT_SYMBOL(zio_clear_fault); -EXPORT_SYMBOL(zio_handle_fault_injection); -EXPORT_SYMBOL(zio_handle_device_injection); -EXPORT_SYMBOL(zio_handle_label_injection); EXPORT_SYMBOL(zio_type_name); EXPORT_SYMBOL(zio_buf_alloc); EXPORT_SYMBOL(zio_data_buf_alloc); diff --git a/module/zfs/zio_inject.c b/module/zfs/zio_inject.c index c168f3b47..5afb23c59 100644 --- a/module/zfs/zio_inject.c +++ b/module/zfs/zio_inject.c @@ -525,6 +525,11 @@ zio_inject_fini(void) } #if defined(_KERNEL) && defined(HAVE_SPL) -module_param(zio_injection_enabled, int, 0644); -MODULE_PARM_DESC(zio_injection_enabled, "Enable fault injection"); +EXPORT_SYMBOL(zio_injection_enabled); +EXPORT_SYMBOL(zio_inject_fault); +EXPORT_SYMBOL(zio_inject_list_next); +EXPORT_SYMBOL(zio_clear_fault); +EXPORT_SYMBOL(zio_handle_fault_injection); +EXPORT_SYMBOL(zio_handle_device_injection); +EXPORT_SYMBOL(zio_handle_label_injection); #endif