Re: [Patch]: Enable SCSI Wait Scan only if BLK_DEV_INITRD is set.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



René Bolldorf wrote:
On 02/23/2010 05:08 PM, Stefan Richter wrote:
René Bolldorf wrote:

(I'm not sure whether the maintainers are going to accept that change
this time around, but anyway:)

diff --git a/drivers/scsi/Kconfig b/drivers/scsi/Kconfig
index 9191d1e..b785703 100644
--- a/drivers/scsi/Kconfig
+++ b/drivers/scsi/Kconfig
@@ -258,21 +258,22 @@ config SCSI_SCAN_ASYNC
         or async on the kernel's command line.

   config SCSI_WAIT_SCAN
-    tristate  # No prompt here, this is an invisible symbol.
+    tristate "SCSI Wait Scan"
       default m
       depends on SCSI
+    depends on SCSI_SCAN_ASYNC
       depends on MODULES
[...]
+    depends on BLK_DEV_INITRD

There needs to be an additional "depends on m" now that you made it
visible, so that it cannot be set to y.

+    help
+ scsi_wait_scan is a loadable module which waits until all the async scans are + complete. The idea is to use it in initrd/ initramfs scripts. You modprobe + it after all the modprobes of the root SCSI drivers and it will wait until + they have all finished scanning their buses before allowing the boot to
+      proceed.
[...]
+      People that use a initramfs should say Y or M.

They "...should say M." actually.

Ok, updated. thx
I hope someone pick it up, because it's damn that the module will be built even with no initramfs support.

==================================================================
diff --git a/drivers/scsi/Kconfig b/drivers/scsi/Kconfig
index 9191d1e..1cb07da 100644
--- a/drivers/scsi/Kconfig
+++ b/drivers/scsi/Kconfig
@@ -258,21 +258,23 @@ config SCSI_SCAN_ASYNC
       or async on the kernel's command line.

 config SCSI_WAIT_SCAN
-    tristate  # No prompt here, this is an invisible symbol.
+    tristate "SCSI Wait Scan"
     default m
+    depends on m

Seems wrong to me (not a Kconfig expert).  Is there a CONFIG_m symbol?
It already has 'depends on MODULES'.

     depends on SCSI
+    depends on SCSI_SCAN_ASYNC
     depends on MODULES
-# scsi_wait_scan is a loadable module which waits until all the async scans are -# complete. The idea is to use it in initrd/ initramfs scripts. You modprobe -# it after all the modprobes of the root SCSI drivers and it will wait until
-# they have all finished scanning their buses before allowing the boot to
-# proceed. (This method is not applicable if targets boot independently in -# parallel with the initiator, or with transports with non-deterministic target -# discovery schemes, or if a transport driver does not support scsi_wait_scan.)
-#
-# This symbol is not exposed as a prompt because little is to be gained by
-# disabling it, whereas people who accidentally switch it off may wonder why
-# their mkinitrd gets into trouble.
+    depends on BLK_DEV_INITRD
+    help
+ scsi_wait_scan is a loadable module which waits until all the async scans are + complete. The idea is to use it in initrd/ initramfs scripts. You modprobe + it after all the modprobes of the root SCSI drivers and it will wait until + they have all finished scanning their buses before allowing the boot to + proceed. (This method is not applicable if targets boot independently in + parallel with the initiator, or with transports with non-deterministic target + discovery schemes, or if a transport driver does not support scsi_wait_scan.)
+
+      People that use a initramfs should say M here.

 menu "SCSI Transports"
     depends on SCSI
--
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

--
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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]
  Powered by Linux