Re: [GIT PULL] SCSI fixes for 2.6.32-rc3

[James Bottomley]

On Thu, 2009-10-08 at 07:39 -0700, Linus Torvalds wrote:
> 
> On Thu, 8 Oct 2009, James Bottomley wrote:
> > 
> > So what do you want to do about this? 
> 
> I'm taking it (and the parisc one I was also unhappy with), but I'm a bit 
> grumpy as usual. The parisc pull came totally outside the merge window, 
> and the SCSI fix pull is technically perfectly fine, but what makes me 
> grumpy is that I get the strong feeling that people aren't even _trying_ 
> to hit the merge window with new drivers, because they decide that they 
> instead can just push them any time.

OK, so we still have a bit of a mismatch here.  I *do* tell people who
come to SCSI with new drivers that for the *first* submission they don't
need to worry about the merge window because of the new driver
exception.  This allows us to clean them up ready for going in without
the submitter feeling huge pressure to hit the merge window rather than
concentrating on code quality and what we need to make a technically
correct submission.

I think this is the correct thing to do for new drivers (which often
come with new writers) because training people to hit the merge window
is often long an painful (when I say not yet to their important driver
enhancements) ... starting people off with the carrot is better than the
stick.

> So I don't think I necessarily want to change the "new driver" policy per 
> se, but I want people to see the merge window as the _primary_ time you 
> get any new code in. The "yes, we'll take new drivers" thing should be the 
> exception rather than the rule. It doesn't seem to be an exception.

Like I said, this will be my fourth new driver under the exception in
the whole year it's been going.  On the other hand, I do have pm8001
still going ... it just got resubmitted after another round of fixes.
If it's ready for the next rc submissions round, it will be my fifth ...

James


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