>>>>> "Hannes" == Hannes Reinecke <h...@suse.de> writes:

> diff --git a/drivers/scsi/scsi_scan.c b/drivers/scsi/scsi_scan.c
> index 97074c9..929204b 100644
> --- a/drivers/scsi/scsi_scan.c
> +++ b/drivers/scsi/scsi_scan.c
> @@ -96,10 +96,11 @@ MODULE_PARM_DESC(max_luns,
>  #define SCSI_SCAN_TYPE_DEFAULT "sync"
>  #endif
>  
> -char scsi_scan_type[6] = SCSI_SCAN_TYPE_DEFAULT;
> +char scsi_scan_type[9] = SCSI_SCAN_TYPE_DEFAULT;
>  
> -module_param_string(scan, scsi_scan_type, sizeof(scsi_scan_type), S_IRUGO);
> -MODULE_PARM_DESC(scan, "sync, async or none");
> +module_param_string(scan, scsi_scan_type, sizeof(scsi_scan_type),
> +                 S_IRUGO|S_IWUSR);
> +MODULE_PARM_DESC(scan, "sync, async, disabled, or none");

So "disabled" is "sometimes enabled" and "none" is "really disabled"?
 
>  static unsigned int scsi_inq_timeout = SCSI_TIMEOUT/HZ + 18;
>  
> @@ -1604,7 +1605,9 @@ static void __scsi_scan_target(struct device *parent, 
> unsigned int channel,
>   * @channel: channel to scan
>   * @id:              target id to scan
>   * @lun:     Specific LUN to scan or SCAN_WILD_CARD
> - * @rescan:  passed to LUN scanning routines
> + * @rescan:  passed to LUN scanning routines; 0 for no rescan, 1 to rescan
> + *              existing LUNs, and 2 to override any 'scan_disabled' setting
> + *              from the host.

enum SCSI_SCAN_FOO_BAR instead of magic values, please.

What's this "scan_disabled" setting from the host?

-- 
Martin K. Petersen      Oracle Linux Engineering
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to