linux-kernel - Re: Possible bug in SCSI Kconfig

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <1224702340.6851.20.camel@localhost.localdomain>
Date:	Wed, 22 Oct 2008 13:05:40 -0600
From:	James Bottomley <James.Bottomley@...senPartnership.com>
To:	Fabio Comolli <fabio.comolli@...il.com>
Cc:	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
	sam@...nborg.org, linux-scsi <linux-scsi@...r.kernel.org>
Subject: Re: Possible bug in SCSI Kconfig

On Wed, 2008-10-22 at 18:29 +0200, Fabio Comolli wrote:
> Hi.
> In kernel 2.6.27.2 - drivers/scsi/Kconfig we have:
> 
> config SCSI_WAIT_SCAN
>         tristate
>         default m
>         depends on SCSI
>         depends on MODULES
> 
> The tristate field is empty. This has the effect that this option is
> not visible in menuconfig and so it's always selected. The default is
> "m" for all architectures and so this module is always compiled if
> SCSI and MODULES are both enabled.
> 
> I'm using a patch like this one:
> 
>  config SCSI_WAIT_SCAN
> -	tristate
> +	tristate "Wait until all the async scans are complete"
>  	default m
>  	depends on SCSI
>  	depends on MODULES
> 
> to get rid of that module.
> 
> Of course, I have no idea if this is correct or the current behavior
> is the expected one.

The point of all of this is that if you enable async scanning, you need
a method of waiting for the scans to complete, which is all this module
does (you insert it and it doesn't come back from the insertion until
all the pending async scans are complete).  It's tristate because really
it only makes sense to be M or N.  The consensus is that, given async
scanning is always an option, this function should always be built as a
module *if* modules and SCSI are enabled ... because you might have a
HBA module you enabled async scanning for and you need to wait.

There's really not much point giving the user the choice, since this
module is part of the initrd sequence ... it's not really anything a
user would want to use stand alone, hence there's no point giving a
choice about it.

James

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/