lkml.org 
[lkml]   [2008]   [Apr]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: Slow DOWN, please!!!
On Wed, Apr 30, 2008 at 03:41:24PM -0700, Andrew Morton wrote:
> On Wed, 30 Apr 2008 15:31:22 -0700 (PDT)
> Linus Torvalds <torvalds@linux-foundation.org> wrote:
>
> > Any suggestions on how to convince people that their code is not worth
> > merging?
>
> Raise the quality. Then the volume will automatically decrease.

100% ACK to "Raise the quality" (no matter whether it influences the volume).

> Which leads us to... the volume isn't a problem per-se. The problem is
> quality. It's the fact that they vary inversely which makes us say "slow
> down".
>
> So David's Subject: should have been "Do Better, please". Slowing down is
> just a side-effect. And, we expect, a tool.
>
>
> We should be discussing how to raise the quality of our work.

One big problem I see is Linus wanting to merge all drivers regardless
of the quality.

Linus said in [1]:
"I'd really rather have the driver merged, and then *other* people can
send patches!"

The problem is that such "other people" do not exist (except perhaps Al)
for non-trivial stuff.

My favorite gem from this driver we merged in 2.6.25 is:
grep -C4 volatile drivers/infiniband/hw/nes/nes_nic.c

Fixing such stuff aren't "janitorial kind of things", and people are
actually more motivated to fix their code for getting it into the kernel
than to fix their code after it went into the kernel.

I am not saying we shouldn't merge such a driver at all or set
unrealistic high quality goals - I'm for merging all code of good
quality that provides functionality not yet into the kernel.

But we need some minimum quality level.

cu
Adrian

[1] http://lkml.org/lkml/2008/2/21/334

--

"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed



\
 
 \ /
  Last update: 2008-05-01 03:01    [from the cache]
©2003-2014 Jasper Spaans. Advertise on this site