All of lore.kernel.org
 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.