Messages in this thread | | | Date | Wed, 24 Oct 2007 10:32:50 -0400 | From | Jeff Garzik <> | Subject | Re: [GIT PATCH] final SCSI pieces for the merge window |
| |
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/
| |