netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: James Morris <jmorris@namei.org>
To: Al Viro <viro@ZenIV.linux.org.uk>
Cc: Andrew Morton <akpm@linux-foundation.org>,
	Willy Tarreau <w@1wt.eu>,
	david@lang.hm, Stephen Clark <sclark46@earthlink.net>,
	Evgeniy Polyakov <johnpol@2ka.mipt.ru>,
	"Rafael J. Wysocki" <rjw@sisk.pl>,
	Tilman Schmidt <tilman@imap.cc>,
	Valdis.Kletnieks@vt.edu, Mark Lord <lkml@rtr.ca>,
	David Miller <davem@davemloft.net>,
	jesper.juhl@gmail.com, yoshfuji@linux-ipv6.org, jeff@garzik.org,
	linux-kernel <linux-kernel@vger.kernel.org>,
	git@vger.kernel.org, netdev@vger.kernel.org
Subject: Re: Reporting bugs and bisection
Date: Tue, 15 Apr 2008 01:54:00 +1000 (EST)	[thread overview]
Message-ID: <Xine.LNX.4.64.0804150131300.4160@us.intercode.com.au> (raw)
In-Reply-To: <20080414072328.GW9785@ZenIV.linux.org.uk>

On Mon, 14 Apr 2008, Al Viro wrote:

> Real review of code in tree and patches getting into the tree.

There is currently little incentive for developers to perform review.  

It's difficult work, and is generally not rewarded or recognized, except 
in often quite negative ways.  There is a small handful of people who do a 
lot of review, but they are exceptional in various ways.

OTOH, writing code is relatively simple, and is much more highly rewarded:

- People tend to get paid to write kernel code, but not so much to review 
  it.

- Things like "who made the kernel" statistics and related articles ignore 
  code review.

- Creating new features is perceived as the highest form of contribution 
  for general developers, and likely important as career currency 
  (similar to the publish or perish model in the academic world).

I don't know how to solve this, but suspect that encouraging the use of 
reviewed-by and also including it in things like analysis of who is 
contributing, selection for kernel summit invitations etc. would be a 
start.  At least, better than nothing.


- James 
-- 
James Morris
<jmorris@namei.org>

  parent reply	other threads:[~2008-04-14 15:55 UTC|newest]

Thread overview: 122+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20080409.182228.193699767.davem@davemloft.net>
     [not found] ` <47FE3020.1070502@imap.cc>
     [not found]   ` <9a8748490804101509l5d043ff8w565dc44dfeaf0072@mail.gmail.com>
     [not found]     ` <20080410.154651.101700010.davem@davemloft.net>
2008-04-11  0:16       ` 2.6.25-rc8: FTP transfer errors Mark Lord
2008-04-11  0:24         ` David Miller
2008-04-11  0:27           ` Mark Lord
2008-04-11  0:39             ` David Miller
2008-04-11  1:23               ` Mark Lord
2008-04-11  6:40                 ` Ilpo Järvinen
2008-04-11 13:19                   ` Mark Lord
2008-04-11 14:35                     ` Evgeniy Polyakov
2008-04-11 14:59                       ` Mark Lord
2008-04-11 15:18                         ` Evgeniy Polyakov
2008-04-11 18:07                           ` David Miller
2008-04-11 21:29                             ` Evgeniy Polyakov
2008-04-12  8:44                             ` Willy Tarreau
2008-04-12  9:49                               ` David Miller
2008-04-13 18:15                                 ` Rafael J. Wysocki
2008-04-13 18:51                                   ` Sergio Luis
2008-04-13 19:24                                     ` Rafael J. Wysocki
2008-04-11 19:58                         ` Valdis.Kletnieks
2008-04-11 22:16                       ` Tilman Schmidt
2008-04-11 22:25                         ` Evgeniy Polyakov
2008-04-11 22:27                           ` David Miller
2008-04-11 23:23                           ` Tilman Schmidt
2008-04-12  5:37                             ` Evgeniy Polyakov
2008-04-12  7:06                             ` Ilpo Järvinen
2008-04-11 22:26                         ` David Miller
2008-04-11 19:58                 ` Valdis.Kletnieks
2008-04-11 22:27                   ` Tilman Schmidt
2008-04-13 18:40                     ` Reporting bugs and bisection (was: Re: 2.6.25-rc8: FTP transfer errors) Rafael J. Wysocki
2008-04-13 18:47                       ` Willy Tarreau
2008-04-13 19:18                         ` Andrew Morton
2008-04-13 19:27                           ` Rafael J. Wysocki
2008-04-13 19:47                           ` Reporting bugs and bisection David Miller
2008-04-13 20:21                           ` Reporting bugs and bisection (was: Re: 2.6.25-rc8: FTP transfer errors) Evgeniy Polyakov
2008-04-13 20:33                             ` Rafael J. Wysocki
2008-04-13 20:54                               ` Evgeniy Polyakov
2008-04-13 22:24                                 ` Reporting bugs and bisection Stephen Clark
2008-04-13 22:41                                   ` Rafael J. Wysocki
2008-04-13 23:51                                   ` david
2008-04-14  0:36                                     ` Jakub Narebski
2008-04-14  4:39                                     ` Willy Tarreau
2008-04-14  5:39                                       ` Al Viro
2008-04-14  6:24                                         ` Andrew Morton
2008-04-14  6:39                                           ` David Miller
2008-04-14  6:43                                             ` David Miller
2008-04-14  7:23                                           ` Al Viro
2008-04-14  7:43                                             ` Al Viro
2008-04-14  8:04                                             ` Andrew Morton
2008-04-14  8:30                                               ` David Miller
2008-04-14  9:06                                                 ` Christoph Hellwig
2008-04-14  9:46                                                 ` Andi Kleen
2008-04-15  5:25                                                   ` Bill Fink
2008-04-14 10:15                                                 ` Andrew Morton
2008-04-14 10:41                                                   ` David Miller
2008-04-14 17:35                                                     ` Roman Shaposhnik
2008-04-14 12:08                                               ` Adrian Bunk
2008-04-14 14:43                                               ` Arjan van de Ven
2008-04-14 17:51                                                 ` Andrew Morton
2008-04-14 18:24                                                   ` Arjan van de Ven
2008-04-14 19:30                                                   ` Ilpo Järvinen
2008-04-14 15:54                                             ` James Morris [this message]
2008-04-14 22:01                                               ` David Miller
2008-04-14 23:05                                                 ` Andrew Morton
2008-04-15  4:55                                                   ` Willy Tarreau
2008-04-15 13:18                                                     ` Work WAS(Re: " jamal
2008-04-15  9:33                                               ` David Newall
2008-04-15  9:54                                                 ` Michael Kerrisk
2008-04-15 14:04                                                   ` David Newall
2008-04-15 20:51                                                     ` Rafael J. Wysocki
2008-04-16  2:34                                                       ` David Newall
2008-04-16  3:53                                                         ` david
2008-04-16  9:06                                                           ` David Newall
2008-04-16 11:02                                                             ` Andi Kleen
2008-04-16 12:41                                                           ` Stephen Clark
2008-04-16  4:29                                                         ` Willy Tarreau
2008-04-16 12:13                                                           ` Rafael J. Wysocki
2008-04-16 12:15                                                 ` Sverre Rabbelier
2008-04-16 13:26                                                   ` Adrian Bunk
2008-04-16 19:02                                                     ` Andrew Morton
2008-04-16 19:43                                                       ` Sverre Rabbelier
2008-04-16 19:55                                                       ` Adrian Bunk
2008-04-17 13:50                                                         ` J. Bruce Fields
2008-04-17 15:26                                                           ` Adrian Bunk
2008-04-16 19:58                                                       ` Alexey Dobriyan
2008-04-16 20:01                                                       ` Arjan van de Ven
2008-04-16 19:39                                                     ` Sverre Rabbelier
2008-04-16 20:16                                                       ` Adrian Bunk
2008-04-16 20:53                                                         ` Adrian Bunk
2008-04-16 21:05                                                           ` Sverre Rabbelier
2008-04-16 21:25                                                             ` Adrian Bunk
2008-04-16 20:04                                                     ` Willy Tarreau
2008-04-16 20:55                                                       ` Jakub Narebski
2008-04-16 21:17                                                   ` Jesper Juhl
2008-04-17 17:04                                                     ` David Newall
2008-04-17 19:09                                                       ` Rafael J. Wysocki
2008-04-17 19:35                                                         ` Ray Lee
2008-04-17 19:57                                                           ` Sverre Rabbelier
2008-04-17 20:16                                                           ` Al Viro
2008-04-17 20:38                                                             ` Ray Lee
2008-04-17 20:53                                                               ` Al Viro
2008-04-17 21:01                                                                 ` Ray Lee
2008-04-14 19:13                                           ` Rene Herman
2008-04-14 20:38                                             ` Andrew Morton
2008-04-14 22:18                                               ` Rene Herman
2008-04-14  9:26                                   ` Andi Kleen
2008-04-13 20:35                             ` David Miller
2008-04-14 10:18                           ` Reporting bugs and bisection (was: Re: 2.6.25-rc8: FTP transfer errors) Ingo Molnar
2008-04-14 10:29                             ` Reporting bugs and bisection Andi Kleen
2008-04-13 20:10                         ` Reporting bugs and bisection (was: Re: 2.6.25-rc8: FTP transfer errors) Adrian Bunk
2008-04-14  9:58                         ` Reporting bugs and bisection Andi Kleen
2008-04-14 10:00                           ` Willy Tarreau
2008-04-14 10:16                             ` Andi Kleen
2008-04-15 21:53               ` about bisections (was: Re: 2.6.25-rc8: FTP transfer errors) Ingo Molnar
2008-04-15 22:30                 ` about bisections David Miller
2008-04-15 22:48                   ` Ingo Molnar
2008-04-11  0:56           ` 2.6.25-rc8: FTP transfer errors Tilman Schmidt
2008-04-11  1:08             ` David Miller
2008-04-11  0:26         ` David Miller
2008-04-11  0:29           ` Mark Lord
2008-04-11  2:59             ` YOSHIFUJI Hideaki / 吉藤英明
2008-04-11  3:18               ` [PATCH 2.6.25] net sockets: fix timewait namespace regression Mark Lord
2008-04-11  3:51                 ` David Miller
2008-04-11  7:50               ` 2.6.25-rc8: FTP transfer errors Pavel Emelyanov

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=Xine.LNX.4.64.0804150131300.4160@us.intercode.com.au \
    --to=jmorris@namei.org \
    --cc=Valdis.Kletnieks@vt.edu \
    --cc=akpm@linux-foundation.org \
    --cc=davem@davemloft.net \
    --cc=david@lang.hm \
    --cc=git@vger.kernel.org \
    --cc=jeff@garzik.org \
    --cc=jesper.juhl@gmail.com \
    --cc=johnpol@2ka.mipt.ru \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lkml@rtr.ca \
    --cc=netdev@vger.kernel.org \
    --cc=rjw@sisk.pl \
    --cc=sclark46@earthlink.net \
    --cc=tilman@imap.cc \
    --cc=viro@ZenIV.linux.org.uk \
    --cc=w@1wt.eu \
    --cc=yoshfuji@linux-ipv6.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).