public inbox for linux-scsi@vger.kernel.org
 help / color / mirror / Atom feed
From: James Bottomley <James.Bottomley@SteelEye.com>
To: Jeff Garzik <jeff@garzik.org>
Cc: akpm@linux-foundation.org, linux-scsi@vger.kernel.org, "Salyzyn,
	Mark" <mark_salyzyn@adaptec.com>,
	Andrew Vasquez <andrew.vasquez@qlogic.com>
Subject: Re: [patch 14/25] SCSI: use irq_handler_t where appropriate
Date: Thu, 24 May 2007 09:04:06 -0500	[thread overview]
Message-ID: <1180015447.3692.6.camel@mulgrave.il.steeleye.com> (raw)
In-Reply-To: <465527D7.6030909@garzik.org>

On Thu, 2007-05-24 at 01:51 -0400, Jeff Garzik wrote:
> James Bottomley wrote:
> > It's not a bug fix or even an enhancement.  Historically, it is quite
> > difficult to get maintainers to ack these ... particularly if you don't
> > cc them.
> 
> If neither you nor the maintainers are reading and responding to patches 
> sent to linux-scsi, I don't think the problem is sitting in my chair.

Oh come off it ... You've been around long enough to know that
maintainers are not always watching everything ... it would be nice if
they were, but to give a patch the best shot at review, you try to
attract their attention.  Specifically, in this case, you should cc the
maintainers and you should have a subject line explaining that you are
modifying their driver.  It is very easy to ignore a patch that's simply
waved at the SCSI list with a generic subject line.

> If others have SCSI patches that have been sitting in limbo for weeks or 
> months, send them to me, and I'll queue them in misc-2.6.git#scsi.

James



  reply	other threads:[~2007-05-24 14:04 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-05-23 21:41 [patch 14/25] SCSI: use irq_handler_t where appropriate akpm
2007-05-23 22:07 ` Jeff Garzik
2007-05-23 22:16   ` Andrew Morton
2007-05-23 22:55   ` James Bottomley
2007-05-24  5:51     ` Jeff Garzik
2007-05-24 14:04       ` James Bottomley [this message]
2007-05-24 14:28         ` Salyzyn, Mark
2007-05-24 15:21           ` James Bottomley
2007-05-26 20:45             ` James Bottomley
2007-05-24 15:17         ` Randy Dunlap
2007-05-24 15:28           ` James Bottomley
2007-05-24 20:49             ` Jeff Garzik
2007-05-25  9:32               ` Stefan Richter
2007-05-25  9:06           ` Stefan Richter
2007-05-24 20:52         ` Jeff Garzik
2007-05-24 13:32     ` Salyzyn, Mark
2007-05-23 23:00   ` David Miller
2007-05-24 11:31 ` Andrew Vasquez

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=1180015447.3692.6.camel@mulgrave.il.steeleye.com \
    --to=james.bottomley@steeleye.com \
    --cc=akpm@linux-foundation.org \
    --cc=andrew.vasquez@qlogic.com \
    --cc=jeff@garzik.org \
    --cc=linux-scsi@vger.kernel.org \
    --cc=mark_salyzyn@adaptec.com \
    /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