linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
* howto get a patch merged (WAS: Re: -mm merge plans for 2.6.23)
@ 2007-07-25 15:30 Kacper Wysocki
  2007-07-25 16:01 ` Rene Herman
  2007-07-25 16:07 ` Ingo Molnar
  0 siblings, 2 replies; 27+ messages in thread
From: Kacper Wysocki @ 2007-07-25 15:30 UTC (permalink / raw)
  To: Ingo Molnar
  Cc: Rene Herman, david, Nick Piggin, Valdis.Kletnieks, Ray Lee,
	linux-kernel, ck list, linux-mm, Paul Jackson, Andrew Morton,
	Jesper Juhl

On 7/25/07, Ingo Molnar <mingo@elte.hu> wrote:
>
> * Rene Herman <rene.herman@gmail.com> wrote:
>
> > Nick Piggin is the person to convince it seems and if I've read things
> > right (I only stepped into this thing at the updatedb mention, so
> > maybe I haven't) his main question is _why_ the hell it helps
> > updatedb. [...]
[snip howto get a patch merged]
> But a "here is a solution, take it or leave it" approach, before having
> communicated the problem to the maintainer and before having debugged
> the problem is the wrong way around. It might still work out fine if the
> solution is correct (especially if the patch is small and obvious), but
> if there are any non-trivial tradeoffs involved, or if nontrivial amount
> of code is involved, you might see your patch at the end of a really
> long (and constantly growing) waiting list of patches.

Is that what happened with swap prefetch these two years? The approach
has been wrong?

In this particular instance, there are lots of people who have looked
at, tried, tested, reported on, advocated, and most importantly -been
using for several years- swap prefetch. That's a lot of people, it's
hard to coordinate a unified approach, eh?

You are the gatekeepers. The dudes with the keys. The guys who must
claim technical and moral superiority over the rabble, the rest of us.

I always believed that the linux development model consisted of
collaboration with technical merit as the prime goal in mind. But the
sheer volume of patches and the small number of keyholders precludes
such a scenario. So the gatekeepers have trusted advisors, and it's
quicker to look at numbers than to try out all the random (and often
crappy) patches that flow in.

It's easier to ask for more testing, more feedback, more "unbiased",
easier to just drop it rather than to look at the new code itself.
It's easier and less risky to try and understand a problem yourself,
and write your own code. After all, you trust your own code, right?

Life is so much simpler without any alien objects around.
However, cool things usually come from outside such a stable environment.

The problem isn't debugging the wrong way around. The problem is no
matter how many people read it, test it, use it, brag about it,
measure it, it doesn't matter at all unless they can convince one of
these few dudes that he should really turn that key.

0K

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

^ permalink raw reply	[flat|nested] 27+ messages in thread

end of thread, other threads:[~2007-07-27 13:32 UTC | newest]

Thread overview: 27+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-07-25 15:30 howto get a patch merged (WAS: Re: -mm merge plans for 2.6.23) Kacper Wysocki
2007-07-25 16:01 ` Rene Herman
2007-07-25 17:15   ` Robert Deaton
2007-07-26  3:59     ` updatedb Rene Herman
2007-07-26  6:23       ` updatedb Andika Triwidada
2007-07-26  7:49         ` updatedb Rene Herman
2007-07-26  9:37           ` updatedb Andika Triwidada
2007-07-27  0:46         ` updatedb Jesper Juhl
2007-07-27  6:00           ` updatedb Rene Herman
2007-07-27  7:54             ` updatedb Mike Galbraith
2007-07-27  8:28               ` updatedb Rene Herman
2007-07-27  9:26                 ` updatedb Mike Galbraith
2007-07-27 11:09                   ` updatedb Rene Herman
2007-07-27 11:48                     ` updatedb Mike Galbraith
2007-07-27 12:28                       ` updatedb Rene Herman
2007-07-27 13:32                         ` updatedb Tilman Schmidt
2007-07-26  6:39       ` updatedb Bongani Hlope
2007-07-26  6:56         ` updatedb Rene Herman
2007-07-26  7:08           ` updatedb Bongani Hlope
2007-07-26  8:01             ` updatedb Rene Herman
2007-07-26 21:25               ` updatedb Bongani Hlope
2007-07-26  9:58           ` updatedb Björn Steinbrink
2007-07-26 10:23             ` updatedb Björn Steinbrink
2007-07-26 11:00             ` updatedb Rene Herman
2007-07-26 13:54     ` Re: howto get a patch merged (WAS: Re: -mm merge plans for 2.6.23) Jos Poortvliet
2007-07-25 16:07 ` Ingo Molnar
2007-07-25 16:40   ` [ck] " Michael Chang

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox