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--
| Linus Torvalds | Linux 2.6.21 |
| Greg Kroah-Hartman | [PATCH 002/196] Chinese: rephrase English introduction in HOWTO |
| Con Kolivas | Re: [PATCH][RSDL-mm 0/7] RSDL cpu scheduler for 2.6.21-rc3-mm2 |
| Andrew Morton | echo mem > /sys/power/state |
git: | |
| Jarek Poplawski | [PATCH] pkt_sched: Destroy gen estimators under rtnl_lock(). |
| Gerrit Renker | [PATCH 27/37] dccp: Integration of dynamic feature activation - part 2 (server side) |
| David Miller | Re: [GIT]: Networking |
| Michael S. Tsirkin | Re: [RFC PATCH v2 03/19] vbus: add connection-client helper infrastructure |
