* [PATCH 1/1] ipr: Expose debug and fastfail parameters [not found] <20090130162631.970634030@linux.vnet.ibm.com> @ 2009-01-30 16:47 ` Wayne Boyer 2009-02-24 19:36 ` [PATCH 1/1 (resend) ] " Wayne Boyer 1 sibling, 0 replies; 3+ messages in thread From: Wayne Boyer @ 2009-01-30 16:47 UTC (permalink / raw) To: linux-scsi Expose the debug and fastfail parameters to /sys/module/ipr/parameters such that they can be enabled/disabled at run time. Signed-off-by: Wayne Boyer <wayneb@linux.vnet.ibm.com> --- drivers/scsi/ipr.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) Index: linux-2.6/drivers/scsi/ipr.c =================================================================== --- linux-2.6.orig/drivers/scsi/ipr.c 2009-01-30 08:14:51.000000000 -0800 +++ linux-2.6/drivers/scsi/ipr.c 2009-01-30 08:14:41.000000000 -0800 @@ -152,13 +152,13 @@ MODULE_PARM_DESC(log_level, "Set to 0 - 4 for increasing verbosity of device driver"); module_param_named(testmode, ipr_testmode, int, 0); MODULE_PARM_DESC(testmode, "DANGEROUS!!! Allows unsupported configurations"); -module_param_named(fastfail, ipr_fastfail, int, 0); +module_param_named(fastfail, ipr_fastfail, int, S_IRUGO | S_IWUSR); MODULE_PARM_DESC(fastfail, "Reduce timeouts and retries"); module_param_named(transop_timeout, ipr_transop_timeout, int, 0); MODULE_PARM_DESC(transop_timeout, "Time in seconds to wait for adapter to come operational (default: 300)"); module_param_named(enable_cache, ipr_enable_cache, int, 0); MODULE_PARM_DESC(enable_cache, "Enable adapter's non-volatile write cache (default: 1)"); -module_param_named(debug, ipr_debug, int, 0); +module_param_named(debug, ipr_debug, int, S_IRUGO | S_IWUSR); MODULE_PARM_DESC(debug, "Enable device driver debugging logging. Set to 1 to enable. (default: 0)"); module_param_named(dual_ioa_raid, ipr_dual_ioa_raid, int, 0); MODULE_PARM_DESC(dual_ioa_raid, "Enable dual adapter RAID support. Set to 1 to enable. (default: 1)"); ^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH 1/1 (resend) ] ipr: Expose debug and fastfail parameters [not found] <20090130162631.970634030@linux.vnet.ibm.com> 2009-01-30 16:47 ` [PATCH 1/1] ipr: Expose debug and fastfail parameters Wayne Boyer @ 2009-02-24 19:36 ` Wayne Boyer 2009-02-24 20:23 ` Brian King 1 sibling, 1 reply; 3+ messages in thread From: Wayne Boyer @ 2009-02-24 19:36 UTC (permalink / raw) To: linux-scsi, Brian King Resending this patch that was originally sent on 1/30. Brian, the driver version number was bumped with the MSI changes. So, there's no need to bump it here. Wayne --- Expose the debug and fastfail parameters to /sys/module/ipr/parameters such that they can be enabled/disabled at run time. Signed-off-by: Wayne Boyer <wayneb@linux.vnet.ibm.com> --- drivers/scsi/ipr.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) Index: linux-2.6/drivers/scsi/ipr.c =================================================================== --- linux-2.6.orig/drivers/scsi/ipr.c 2009-01-30 08:14:51.000000000 -0800 +++ linux-2.6/drivers/scsi/ipr.c 2009-01-30 08:14:41.000000000 -0800 @@ -152,13 +152,13 @@ MODULE_PARM_DESC(log_level, "Set to 0 - 4 for increasing verbosity of device driver"); module_param_named(testmode, ipr_testmode, int, 0); MODULE_PARM_DESC(testmode, "DANGEROUS!!! Allows unsupported configurations"); -module_param_named(fastfail, ipr_fastfail, int, 0); +module_param_named(fastfail, ipr_fastfail, int, S_IRUGO | S_IWUSR); MODULE_PARM_DESC(fastfail, "Reduce timeouts and retries"); module_param_named(transop_timeout, ipr_transop_timeout, int, 0); MODULE_PARM_DESC(transop_timeout, "Time in seconds to wait for adapter to come operational (default: 300)"); module_param_named(enable_cache, ipr_enable_cache, int, 0); MODULE_PARM_DESC(enable_cache, "Enable adapter's non-volatile write cache (default: 1)"); -module_param_named(debug, ipr_debug, int, 0); +module_param_named(debug, ipr_debug, int, S_IRUGO | S_IWUSR); MODULE_PARM_DESC(debug, "Enable device driver debugging logging. Set to 1 to enable. (default: 0)"); module_param_named(dual_ioa_raid, ipr_dual_ioa_raid, int, 0); MODULE_PARM_DESC(dual_ioa_raid, "Enable dual adapter RAID support. Set to 1 to enable. (default: 1)"); -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 1/1 (resend) ] ipr: Expose debug and fastfail parameters 2009-02-24 19:36 ` [PATCH 1/1 (resend) ] " Wayne Boyer @ 2009-02-24 20:23 ` Brian King 0 siblings, 0 replies; 3+ messages in thread From: Brian King @ 2009-02-24 20:23 UTC (permalink / raw) To: Wayne Boyer; +Cc: linux-scsi Acked by: Brian King <brking@linux.vnet.ibm.com> Wayne Boyer wrote: > Resending this patch that was originally sent on 1/30. > > Brian, the driver version number was bumped with the MSI changes. So, > there's no need to bump it here. > > Wayne > > --- > > Expose the debug and fastfail parameters to /sys/module/ipr/parameters such > that they can be enabled/disabled at run time. > > Signed-off-by: Wayne Boyer <wayneb@linux.vnet.ibm.com> > --- > drivers/scsi/ipr.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > Index: linux-2.6/drivers/scsi/ipr.c > =================================================================== > --- linux-2.6.orig/drivers/scsi/ipr.c 2009-01-30 08:14:51.000000000 -0800 > +++ linux-2.6/drivers/scsi/ipr.c 2009-01-30 08:14:41.000000000 -0800 > @@ -152,13 +152,13 @@ > MODULE_PARM_DESC(log_level, "Set to 0 - 4 for increasing verbosity of device driver"); > module_param_named(testmode, ipr_testmode, int, 0); > MODULE_PARM_DESC(testmode, "DANGEROUS!!! Allows unsupported configurations"); > -module_param_named(fastfail, ipr_fastfail, int, 0); > +module_param_named(fastfail, ipr_fastfail, int, S_IRUGO | S_IWUSR); > MODULE_PARM_DESC(fastfail, "Reduce timeouts and retries"); > module_param_named(transop_timeout, ipr_transop_timeout, int, 0); > MODULE_PARM_DESC(transop_timeout, "Time in seconds to wait for adapter to come operational (default: 300)"); > module_param_named(enable_cache, ipr_enable_cache, int, 0); > MODULE_PARM_DESC(enable_cache, "Enable adapter's non-volatile write cache (default: 1)"); > -module_param_named(debug, ipr_debug, int, 0); > +module_param_named(debug, ipr_debug, int, S_IRUGO | S_IWUSR); > MODULE_PARM_DESC(debug, "Enable device driver debugging logging. Set to 1 to enable. (default: 0)"); > module_param_named(dual_ioa_raid, ipr_dual_ioa_raid, int, 0); > MODULE_PARM_DESC(dual_ioa_raid, "Enable dual adapter RAID support. Set to 1 to enable. (default: 1)"); > > -- > To unsubscribe from this list: send the line "unsubscribe linux-scsi" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- > To unsubscribe from this list: send the line "unsubscribe linux-scsi" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- Brian King Linux on Power Virtualization IBM Linux Technology Center ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2009-02-24 20:25 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20090130162631.970634030@linux.vnet.ibm.com>
2009-01-30 16:47 ` [PATCH 1/1] ipr: Expose debug and fastfail parameters Wayne Boyer
2009-02-24 19:36 ` [PATCH 1/1 (resend) ] " Wayne Boyer
2009-02-24 20:23 ` Brian King
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).