From: Matthew Wilcox <matthew@wil.cx>
To: Boaz Harrosh <bharrosh@panasas.com>
Cc: James Bottomley <James.Bottomley@SteelEye.com>,
"David S. Miller" <davem@davemloft.net>,
Andrew Morton <akpm@linux-foundation.org>,
linux-scsi <linux-scsi@vger.kernel.org>,
FUJITA Tomonori <tomof@acm.org>
Subject: Re: [PATCH 2/3] pluto/fc - Enable compilation for all ARCHs
Date: Mon, 15 Oct 2007 12:39:28 -0600 [thread overview]
Message-ID: <20071015183928.GF25488@parisc-linux.org> (raw)
In-Reply-To: <4713AEF0.5070706@panasas.com>
On Mon, Oct 15, 2007 at 08:18:24PM +0200, Boaz Harrosh wrote:
> This is all new territory for me. But CONFIG_SCSI_PLUTO is dependent on
> SCSI and fc.c is not the real driver just the needed bits from the sparc
> side. So the code mess calls for a Kconfig mess, I guess.
I've spent a lot of time looking at pluto/fc4 recently. And I'm very
much still in a 'wtf' mode. Pluto creates a new scsi *host* for every
Sun Storage Array proessor *device* on the SCSI bus. I have no clue
why it does this.
I'm sure there's a sane sensible driver or three trying to get out from
inside the pluto/soc/socal/fc4 maze, but I'm not quite sure what form
it should take.
Dave, can you explain what's going on? I really want to help this
driver fit the current scsi model better, but I don't understand
what it's trying to do.
--
Intel are signing my paycheques ... these opinions are still mine
"Bill, look, we understand that you're interested in selling us this
operating system, but compare it to ours. We can't possibly take such
a retrograde step."
next prev parent reply other threads:[~2007-10-15 18:39 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-10-15 17:20 [PATCHSET 0/3] pluto/fc - some fixes and cleanups Boaz Harrosh
2007-10-15 17:25 ` [PATCH 1/3] pluto/fc - Remove uses of the scsi_cmnd->done Boaz Harrosh
2007-10-15 17:56 ` Matthew Wilcox
2007-10-15 17:26 ` [PATCH 2/3] pluto/fc - Enable compilation for all ARCHs Boaz Harrosh
2007-10-15 18:09 ` Matthew Wilcox
2007-10-15 18:18 ` Boaz Harrosh
2007-10-15 18:39 ` Matthew Wilcox [this message]
2007-10-15 20:15 ` [PATCH 1/2] fc4/pluto " Randy Dunlap
2007-10-16 7:43 ` Boaz Harrosh
2007-10-15 20:16 ` [PATCH 2/2] pluto fix disable/enable irq Randy Dunlap
2007-10-15 17:34 ` [PATCH 3/3] pluto/fc - fix INQUIRY still using !use_sg commands Boaz Harrosh
2007-10-15 18:00 ` [PATCH 3/3 ver2] " Boaz Harrosh
2007-10-15 18:12 ` Matthew Wilcox
2007-10-15 18:32 ` [PATCH 3/3 ver3] " Boaz Harrosh
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=20071015183928.GF25488@parisc-linux.org \
--to=matthew@wil.cx \
--cc=James.Bottomley@SteelEye.com \
--cc=akpm@linux-foundation.org \
--cc=bharrosh@panasas.com \
--cc=davem@davemloft.net \
--cc=linux-scsi@vger.kernel.org \
--cc=tomof@acm.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).