linux-scsi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Jeff Mahoney <jeffm@suse.de>
To: James Bottomley <James.Bottomley@HansenPartnership.com>
Cc: linux-scsi <linux-scsi@vger.kernel.org>,
	Dave Jones <davej@redhat.com>,
	Ben Hutchings <ben@decadent.org.uk>
Subject: Re: Remove scsi_wait_scan module
Date: Sun, 27 May 2012 10:48:31 -0400	[thread overview]
Message-ID: <4FC23EBF.7050709@suse.de> (raw)
In-Reply-To: <1338110026.2957.5.camel@dabdike.int.hansenpartnership.com>

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 05/27/2012 05:13 AM, James Bottomley wrote:
> scsi_wait_scan was introduced with asynchronous host scanning as a
> hack for distributions that weren't using proper udev based wait
> for root to appear in their initramfs scripts.  In 2.6.30 Commit

AFIACT we don't use it.

- -Jeff

> c751085943362143f84346d274e0011419c84202 Author: Rafael J. Wysocki
> <rjw@sisk.pl> Date:   Sun Apr 12 20:06:56 2009 +0200
> 
> PM/Hibernate: Wait for SCSI devices scan to complete during resume
> 
> Actually broke scsi_wait_scan because it renders 
> scsi_complete_async_scans() a nop for modular SCSI if you include 
> scsi_scans.h (which this module does).
> 
> The lack of bug reports is sufficient proof that this module is no 
> longer used.
> 
> James
> 
> ---
> 
> diff --git a/drivers/scsi/Kconfig b/drivers/scsi/Kconfig index
> bea04e5..ac6ea28 100644 --- a/drivers/scsi/Kconfig +++
> b/drivers/scsi/Kconfig @@ -263,23 +263,6 @@ config SCSI_SCAN_ASYNC 
> You can override this choice by specifying "scsi_mod.scan=sync" or
> async on the kernel's command line.
> 
> -config SCSI_WAIT_SCAN -	tristate  # No prompt here, this is an
> invisible symbol. -	default m -	depends on SCSI -	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. - menu "SCSI
> Transports" depends on SCSI
> 
> diff --git a/drivers/scsi/Makefile b/drivers/scsi/Makefile index
> 8deedea..f188509 100644 --- a/drivers/scsi/Makefile +++
> b/drivers/scsi/Makefile @@ -161,8 +161,6 @@
> obj-$(CONFIG_SCSI_OSD_INITIATOR) += osd/ # This goes last, so that
> "real" scsi devices probe earlier obj-$(CONFIG_SCSI_DEBUG)	+=
> scsi_debug.o
> 
> -obj-$(CONFIG_SCSI_WAIT_SCAN)	+= scsi_wait_scan.o - scsi_mod-y			+=
> scsi.o hosts.o scsi_ioctl.o constants.o \ scsicam.o scsi_error.o
> scsi_lib.o scsi_mod-$(CONFIG_SCSI_DMA)	+= scsi_lib_dma.o diff --git
> a/drivers/scsi/scsi_wait_scan.c b/drivers/scsi/scsi_wait_scan.c 
> deleted file mode 100644 index 74708fc..0000000 ---
> a/drivers/scsi/scsi_wait_scan.c +++ /dev/null @@ -1,42 +0,0 @@ -/* 
> - * scsi_wait_scan.c - * - * Copyright (C) 2006 James Bottomley
> <James.Bottomley@SteelEye.com> - * - * This is a simple module to
> wait 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 busses before - * allowing
> the boot to proceed - */ - -#include <linux/module.h> -#include
> <linux/device.h> -#include <scsi/scsi_scan.h> - -static int __init
> wait_scan_init(void) -{ -	/* -	 * First we need to wait for device
> probing to finish; -	 * the drivers we just loaded might just still
> be probing -	 * and might not yet have reached the scsi async
> scanning -	 */ -	wait_for_device_probe(); -	/* -	 * and then we
> wait for the actual asynchronous scsi scan -	 * to finish. -	 */ -
> scsi_complete_async_scans(); -	return 0; -} - -static void __exit
> wait_scan_exit(void) -{ -} - -MODULE_DESCRIPTION("SCSI wait for
> scans"); -MODULE_AUTHOR("James Bottomley"); 
> -MODULE_LICENSE("GPL"); - -late_initcall(wait_scan_init); 
> -module_exit(wait_scan_exit);
> 
> 


- -- 
Jeff Mahoney
SUSE Labs
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.19 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iQIcBAEBAgAGBQJPwj6+AAoJEB57S2MheeWyueEP/RgGFBw4hOnW0gRt6360Ary3
qFIdtOzAwqhU01xGiSENAJuDpc75TpAfmzsR1gASQpel8iIFq8/GjHR+tnhZZfC8
D7B5mEKew3KFHBwssmUjdpkXHUJ8UjSdapSKADAp6isTzd8EzJ+U7UbRRhObgpis
wsu/IbYDS1zxTAoO9FnAqN7wkmWHIK0IzOBdu41SsV7OfFUx2DY1cpVC3XvkuYPq
5gP3Fd0Y53omGXVvB/9ZOTQP9D7fBu2vTxhC/eNhIEBW7Ce3eQXdCQeVOHoHAwGZ
ftvDqZYVLvENPjs6jVrG8Lp+2dx5kDoqjlE7vFmTBeu/LAvaLuOM53W3+Dq9AdJ0
jqF7Dmp7fj2Qh3OIs9ivU/Qdlif4SCYXjVSyYbfA0rMif1U1uh1DW8PQNXwSbysF
X+QXw/hgioQGHUBcN7qOw+Dfd2Xv0wK1clxxEYAkYkRHqazSIIbq0tbKcYYY+9Xh
aaMqaQJC5ZHwLJCD99mVo8I//KY0BaGC+v7LAcVqAFAUj5yp4sQ0DeAsyXoGa1Ac
GK9MCxevdqCO8d24tFeZYyL6bZTrblDJCvux/+xI06dBhA+EGibtg8CGyO9/ef7n
gVYugO1bhiitHozQeAlufdAcfpQ135iLM8wRl4i6JegNTqMr/cZuEhD6VEwFwDn/
J+wOc/yUuvd8j5070mWg
=vnGS
-----END PGP SIGNATURE-----

  reply	other threads:[~2012-05-27 14:48 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-05-27  9:13 Remove scsi_wait_scan module James Bottomley
2012-05-27 14:48 ` Jeff Mahoney [this message]
     [not found] ` <1338110026.2957.5.camel-sFMDBYUN5F8GjUHQrlYNx2Wm91YjaHnnhRte9Li2A+AAvxtiuMwx3w@public.gmane.org>
2012-05-28 10:00   ` maximilian attems
     [not found]     ` <20120528100015.GB10036-VJknIhvjf2Ov8OlOgJ4AIV6hYfS7NtTn@public.gmane.org>
2012-05-28 12:07       ` James Bottomley
2012-05-30 10:03         ` maximilian attems
2012-05-30 18:26         ` Dan Williams
     [not found]           ` <CAA9_cmfh9ZtepX+hjLVRM95u_pAe90kZTkBeVmFPWm8AMaqJ5g-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-05-30 23:32             ` James Bottomley
2012-05-31  2:34               ` Dan Williams
     [not found]                 ` <CAA9_cmfZFm5+9ARB7tNVwNzwxJh3bPD=h7gMWhByjy00-qZvgg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-05-31  8:15                   ` Harald Hoyer
     [not found]                     ` <4FC7288C.302-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2012-05-31 16:45                       ` Dan Williams
2012-05-31 17:44                         ` James Bottomley
2012-05-31 17:54                     ` Dave Jones
2012-05-31  8:21                 ` James Bottomley
     [not found]                   ` <1338452505.3073.4.camel-sFMDBYUN5F8GjUHQrlYNx2Wm91YjaHnnhRte9Li2A+AAvxtiuMwx3w@public.gmane.org>
2012-05-31 16:40                     ` Dan Williams
     [not found]                       ` <CAA9_cmet6tMi6rNNv_ktRwHh3_N4rOSd+_jBW0P_gomHLpOytw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-05-31 17:42                         ` James Bottomley
2012-05-31 18:49                           ` Dan Williams

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4FC23EBF.7050709@suse.de \
    --to=jeffm@suse.de \
    --cc=James.Bottomley@HansenPartnership.com \
    --cc=ben@decadent.org.uk \
    --cc=davej@redhat.com \
    --cc=linux-scsi@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).