Re: [GIT PATCH] final SCSI pieces for the merge window

[Jeff Garzik <jeff@garzik.org>]

Douglas Gilbert wrote:
> Jeff Garzik wrote:
> > James Bottomley wrote:
> > > On Tue, 2007-10-23 at 17:09 -0400, Jeff Garzik wrote:
> > > > James Bottomley wrote:
> > > > > This should be the final SCSI updates; it's mainly just a few accessor
> > > > > completion updates and two driver merges (sym2 and qla2xxx) we also
> > > > > secured DaveM's agreement to remove fcal/fc4, which explains the high
> > > > > removal line count.
> > > > >
> > > > > The patch is available here:
> > > > >
> > > > > master.kernel.org:/pub/scm/linux/kernel/git/jejb/scsi-misc-2.6.git
> > > > I guess I have the go-ahead to merge the end-CDROM-polling async
> > > > notification work you've been repeatedly ignoring?
> > > I haven't been ignoring it ... it just needs quite a bit of work; the
> > > best way to accelerate it seems to be simply to do it (add the
> > > supported/trigger event bitmasks and expand the infrastructure).  I just
> > > haven't had the time within the merge window.
> > James, things cannot get bottlenecked like this.  You have had MONTHS to
> > say something like this.  The code was ready BEFORE the merge window.
> >
> > I really think you have the knowledge to be SCSI maintainer, but not the
> > time.
>
> Jeff,
> If only I had a dollar (Canadian unit please) for
> each day some of my libata patches were queued
> up to you before you accepted them.
>
> Remember MODE SELECT ...

You're darned right, I've screwed up in the past.  Sat on stuff "until I 
get time to rewrite it" and that sort of thing.  My colleagues give me 
lumps for it too :)

We talked about this issue at the Kernel Summit -- collectively we need 
to stop holding on to useful, working stuff for months on end.  It 
serves nobody.

We have to rediscover our roots:  "release early, release often"

	Jeff



-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/