Re-Sending: James, The current feature request is to allow the user to change "cmd_per_lun" and "max_sectors" only during the driver loading time. I agree with you it is very good feature to provide option to modify these during run time. In future this may be done as required/requested for. But as you may see, there are more changes included in the patch submission. We really need this submission be accepted and then we can start to submit the next set of changes. Thanks & Regards, Bo Yang From: Yang, Bo Sent: Thursday, November 01, 2007 3:17 PM To: James Bottomley Cc: linux-scsi@xxxxxxxxxxxxxxx; akpm@xxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; Patro, Sumant; Kolli, Neela Subject: RE: [PATCH 1/8] scsi: megaraid_sas - add hibernation support James, The current feature request is to allow the user to change "cmd_per_lun" and "max_sectors" only during the driver loading time. I agree with you it is very good feature to provide option to modify these during run time. In future this may be done as required/requested for. But as you may see, there are more changes included in the patch submission. We really need this submission be accepted and then we can start to submit the next set of changes. Thanks & Regards, Bo Yang From: James Bottomley [mailto:James.Bottomley@xxxxxxxxxxxx] Sent: Wed 10/31/2007 11:12 AM To: Yang, Bo Cc: linux-scsi@xxxxxxxxxxxxxxx; akpm@xxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; Patro, Sumant; Kolli, Neela Subject: RE: [PATCH 1/8] scsi: megaraid_sas - add hibernation support On Wed, 2007-10-31 at 07:25 -0600, Yang, Bo wrote: > James, > I did respond to Randy earlier. Sorry, I couldn't find it in the archive when I looked ... although the scsi reflector seems to be missing quite a few of your emails for some reason. > About the module params being added : > > The fast_load parameter is for the user to decide at driver load time > if (s)he wants to skip scan of devices in PD channels. > > After driver is loaded the user cannot be permitted to modify this > value. If the user needs to see the devices in the PD channels, (s)he > may initiate a scan via sysfs/proc based on the kernel being used. > Once the user has done the scan, the fast_load value does not have any > significance and thus not exposed for reading. > > cmd_per_lun & max_sectors are also intended to be provided by user > only at driver load time. In the current implementation both these do > appear as read-only values under host# in sysfs. The current design is > not to allow these values to be modified on the fly. fastload sounds reasonable. cmd_per_lun and max_sectors could reasonably be altered at runtime ... just execute and unbind and rebind operation to make them take effect. James - To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
