Messages in this thread | | | Date | Wed, 3 Oct 2007 14:00:38 -0700 | From | Randy Dunlap <> | Subject | Re: [PATCH 3/8] scsi: megaraid_sas - add module param max_sectors, cmd_per_lun |
| |
On Mon, 01 Oct 2007 11:51:48 -0400 bo yang wrote:
> Adding module parameters to configure max sectors per request & # of cmds per lun. > > Signed-off-by: Bo Yang <bo.yang@lsi.com> > > --- > drivers/scsi/megaraid/megaraid_sas.c | 68 ++++++++++++++++++++++++- > drivers/scsi/megaraid/megaraid_sas.h | 2 > 2 files changed, 68 insertions(+), 2 deletions(-) > > diff -uprN linux-2.6.22_orig/drivers/scsi/megaraid/megaraid_sas.c linux-2.6.22_new/drivers/scsi/megaraid/megaraid_sas.c > --- linux-2.6.22_orig/drivers/scsi/megaraid/megaraid_sas.c 2007-10-01 00:14:29.000000000 -0700 > +++ linux-2.6.22_new/drivers/scsi/megaraid/megaraid_sas.c 2007-10-01 02:15:16.000000000 -0700 > @@ -62,6 +62,23 @@ MODULE_PARM_DESC(fast_load, > "megasas: Faster loading of the driver, skips physical devices! "\ > "(default = 0)"); > > +/* > + * Number of sectors per IO command will be set in megasas_init_mfi > + * if user does not provide > + */ > +static unsigned int max_sectors; > +module_param_named(max_sectors, max_sectors, int, 0); > +MODULE_PARM_DESC(max_sectors, > + "Maximum number of sectors per IO command");
Are you sure that you want these parameters hidden (permission = 0) instead of readable via sysfs?
(same applies to the fast_load parameter patch also)
> +/* > + * Number of cmds per logical unit > + */ > +static unsigned int cmd_per_lun = MEGASAS_DEFAULT_CMD_PER_LUN; > +module_param_named(cmd_per_lun, cmd_per_lun, int, 0); > +MODULE_PARM_DESC(cmd_per_lun, > + "Maximum number of commands per logical unit (default=128)"); > + > MODULE_LICENSE("GPL"); > MODULE_VERSION(MEGASAS_VERSION); > MODULE_AUTHOR("megaraidlinux@lsi.com");
--- ~Randy - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |