From: Erik Andersen <andersen@codepoet.org>
To: James Bottomley <James.Bottomley@steeleye.com>,
linux-scsi@vger.kernel.org
Subject: Re: [PATCH] SCSI hotplug support
Date: Tue, 15 Oct 2002 13:32:16 -0600 [thread overview]
Message-ID: <20021015193216.GE15778@codepoet.org> (raw)
In-Reply-To: <20021015191330.GC4391@redhat.com>
On Tue Oct 15, 2002 at 03:13:30PM -0400, Doug Ledford wrote:
> On Tue, Oct 15, 2002 at 12:45:35PM -0600, Erik Andersen wrote:
> > On Tue Oct 15, 2002 at 02:22:47PM -0400, Doug Ledford wrote:
> > > On Mon, Oct 14, 2002 at 11:25:22PM -0600, Erik Andersen wrote:
> > > >
> > > > It is called from within the insmod's context.
> > >
> > > So it's called by what module then? And what happens if we have a user
> > > add a hard drive and it gets the module loaded and then attaches the
> > > drive, then we attach another drive sometime later, and this time hotplug
> > > doesn't do anything because the right module is already loaded? How does
> > > the drive get added then? Am I missing something?
> >
> > Oops. I spoke wrongly.... It is called from within
> > /sbin/hotplug's context.
>
> Then what the sam hell has all this crap been for!?!? We've been saying
> all along "Add the device from hotplug instead of kernel module", and now
> you tell us that it's already happening from hotplug's context? If it's
> already happening from hotplug's context, then why not just do the
> fprintf(); that I put in one of my emails and be done with it? And what
The main reason why I'd prefer to put this stuff into the sbp2
driver is to avoid 1394 layering violations. For ieee1394
devices, the 1394 nodemgr makes the hotplug calls. But it does
the hotplug thing for all 1394 devices, and has no knowledge of
whether a device is an SBP-2 device (one that need to be
connected to the SCSI subsystem). I suspect the 1394 maintainers
would reject patches that infect the nodemgr with SBP-2 and/or
SCSI specific stuff. Sigh. We seem to be at an impasse. We
either screw up SCSI or we screw up 1394...
> about the scenario I asked about in my last email regarding a second disc
> and the situation where no module needs loaded? How is that handled?
Same deal. The 1394 nodemgr invokes /sbin/hotplug.
-Erik
--
Erik B. Andersen http://codepoet-consulting.com/
--This message was written using 73% post-consumer electrons--
next prev parent reply other threads:[~2002-10-15 19:32 UTC|newest]
Thread overview: 42+ messages / expand[flat|nested] mbox.gz Atom feed top
2002-10-14 5:40 [PATCH] SCSI hotplug support Erik Andersen
2002-10-14 7:06 ` Matthew Dharm
2002-10-14 7:12 ` Erik Andersen
2002-10-14 15:47 ` Kurt Garloff
2002-10-14 16:19 ` Mike Anderson
2002-10-14 20:41 ` Erik Andersen
2002-10-14 22:10 ` Mike Anderson
2002-10-14 20:46 ` Erik Andersen
2002-10-14 15:57 ` James Bottomley
2002-10-14 17:22 ` Mike Anderson
2002-10-14 17:30 ` Matthew Dharm
2002-10-14 17:39 ` James Bottomley
2002-10-14 19:11 ` Oliver Xymoron
2002-10-15 0:42 ` Kurt Garloff
2002-10-14 20:37 ` Erik Andersen
2002-10-14 21:07 ` James Bottomley
2002-10-14 21:54 ` Erik Andersen
2002-10-14 22:25 ` Doug Ledford
2002-10-15 5:25 ` Erik Andersen
2002-10-15 15:33 ` Doug Ledford
2002-10-15 18:18 ` Erik Andersen
2002-10-15 18:22 ` Doug Ledford
2002-10-15 18:45 ` Erik Andersen
2002-10-15 19:13 ` Doug Ledford
2002-10-15 19:32 ` Erik Andersen [this message]
2002-10-15 19:45 ` James Bottomley
2002-10-15 19:50 ` Scott Merritt
2002-10-15 19:55 ` Doug Ledford
2002-10-15 22:07 ` Erik Andersen
2002-10-16 2:40 ` Doug Ledford
2002-10-18 11:28 ` Erik Andersen
2002-10-15 21:43 ` Oliver Neukum
2002-10-15 22:07 ` Erik Andersen
2002-10-14 22:19 ` Oliver Neukum
2002-10-15 0:22 ` Doug Ledford
2002-10-15 7:53 ` Oliver Neukum
2002-10-15 14:35 ` Doug Ledford
2002-10-15 15:19 ` Oliver Neukum
2002-10-15 15:40 ` James Bottomley
2002-10-15 17:47 ` Erik Andersen
2002-10-15 18:34 ` Doug Ledford
2002-10-15 18:22 ` Scott Merritt
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=20021015193216.GE15778@codepoet.org \
--to=andersen@codepoet.org \
--cc=James.Bottomley@steeleye.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