public inbox for linux-scsi@vger.kernel.org
 help / color / mirror / Atom feed
From: James Bottomley <James.Bottomley@SteelEye.com>
To: "Gibbons, Terry" <Terry.Gibbons@lsil.com>
Cc: "'arjanv@redhat.com'" <arjanv@redhat.com>,
	Eric Dean Moore <Emoore@lsil.com>,
	"Stephens, Larry" <larry.stephens@lsil.com>,
	"Shirron, Stephen" <Stephen.Shirron@lsil.com>,
	Matthew Wilcox <matthew@wil.cx>, Christoph Hellwig <hch@lst.de>,
	SCSI Mailing List <linux-scsi@vger.kernel.org>
Subject: RE: [PATCH] fusion: streamline ->queuecommand
Date: 06 Oct 2004 10:47:30 -0500	[thread overview]
Message-ID: <1097077662.1949.37.camel@mulgrave> (raw)
In-Reply-To: <BC3CE0375D29D51199A400508B66CA6C0862AEFE@exc-snake.co.lsil.com>

On Wed, 2004-10-06 at 09:23, Gibbons, Terry wrote:
> Maybe we should change the focus. What are your suggestions for making sure
> that every major server vendor isn't negatively impacted by changes to DV? 

Well, since every statment from you and your engineers is that you
haven't looked through the generic DV code, I suggest you do so if you
want to participate meaningfully in the discussion.  Saying point blank
you won't use any replacement whatever it looks like is hardly conducive
to a measured debate.

After you look through it, any technical criticisms or requests for
missing features would be welcome.

You appear to be assuming that you can't suggest changes to the generic
DV code, which isn't the case.  The object of making the code generic is
to avoid all the pitfalls that the other drivers fell into trying to
roll their own while creating the best code for all of them.

As has been said before no-one gets a veto on open source code.  Your
choice is either to participate in the evolution of the SCSI subsystem
or not to.  However, the evolution will continue, whatever you decide.

The thing I know it's hardest to appreciate, coming from the computing
industry myself, is that maintaining a driver doesn't just make you
responsible for that driver alone but also for improving the subsystem
it sits in and the rest of the kernel that surrounds it.  If you find a
bug, it's always tempting to apply a workaround to your driver when what
is really needed is a fix for the subsystem itself.  However, in
exchange for thinking of the kernel and the subsystem as well as just
your driver you get a genuine franchise in how the kernel and the
subsystem evolve ... most industry maintainers find that to be more than
worth the extra effort.

James



  parent reply	other threads:[~2004-10-06 15:48 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-10-06 14:23 [PATCH] fusion: streamline ->queuecommand Gibbons, Terry
2004-10-06 14:30 ` Christoph Hellwig
2004-10-06 15:47 ` James Bottomley [this message]
  -- strict thread matches above, loose matches on Subject: below --
2004-10-21 14:58 Moore, Eric Dean
2004-10-06 14:46 Gibbons, Terry
2004-10-06 14:58 ` Matthew Wilcox
2004-10-06 14:00 Gibbons, Terry
2004-10-06 14:13 ` Arjan van de Ven
2004-10-06 14:25 ` Matthew Wilcox
2004-10-05 22:38 Moore, Eric Dean
2004-10-05 23:11 ` James Bottomley
2004-10-05 23:37   ` Patrick Mansfield
2004-10-06  0:48     ` James Bottomley
2004-10-04 21:33 Moore, Eric Dean
2004-10-04 21:57 ` James Bottomley
2004-10-06 15:41 ` Christoph Hellwig
2004-10-02  8:13 Christoph Hellwig
2004-10-02 13:39 ` Matthew Wilcox
2004-10-02 14:49   ` Christoph Hellwig
2004-10-21  9:21 ` Christoph Hellwig

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=1097077662.1949.37.camel@mulgrave \
    --to=james.bottomley@steeleye.com \
    --cc=Emoore@lsil.com \
    --cc=Stephen.Shirron@lsil.com \
    --cc=Terry.Gibbons@lsil.com \
    --cc=arjanv@redhat.com \
    --cc=hch@lst.de \
    --cc=larry.stephens@lsil.com \
    --cc=linux-scsi@vger.kernel.org \
    --cc=matthew@wil.cx \
    /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