Re: [PATCH] scsi: disable automatic target scan

2016-03-15 Thread Hannes Reinecke
On 03/15/2016 07:48 AM, Christoph Hellwig wrote: > On Tue, Mar 15, 2016 at 07:47:34AM +0100, Hannes Reinecke wrote: >>> So "disabled" is "sometimes enabled" and "none" is "really disabled"? >>> >> 'disabled' means: >> disable automatic scanning, but allow manual scanning via the 'scan' >>

Re: [PATCH] scsi: disable automatic target scan

2016-03-15 Thread Christoph Hellwig
On Tue, Mar 15, 2016 at 07:47:34AM +0100, Hannes Reinecke wrote: > > So "disabled" is "sometimes enabled" and "none" is "really disabled"? > > > 'disabled' means: > disable automatic scanning, but allow manual scanning via the 'scan' > attribute. maybe rename disabled to manual? And improve

Re: [PATCH] scsi: disable automatic target scan

2016-03-15 Thread Hannes Reinecke
On 03/15/2016 12:57 AM, Martin K. Petersen wrote: >> "Hannes" == Hannes Reinecke 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

Re: [PATCH] scsi: disable automatic target scan

2016-03-14 Thread Martin K. Petersen
> "Hannes" == Hannes Reinecke 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

Re: [PATCH] scsi: disable automatic target scan

2016-03-14 Thread Johannes Thumshirn
On Fri, Mar 11, 2016 at 04:33:40PM +0100, Hannes Reinecke wrote: > On larger installations it is useful to disable automatic LUN > scanning, and only add the required LUNs via udev rules. > This can speed up bootup dramatically. > > This patch introduces a scan_type 'disabled', which work like >

[PATCH] scsi: disable automatic target scan

2016-03-11 Thread Hannes Reinecke
On larger installations it is useful to disable automatic LUN scanning, and only add the required LUNs via udev rules. This can speed up bootup dramatically. This patch introduces a scan_type 'disabled', which work like 'none', but can be overriden by setting the 'rescan' value from