From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp1.linuxfoundation.org (smtp1.linux-foundation.org [172.17.192.35]) by mail.linuxfoundation.org (Postfix) with ESMTPS id 9E7E2BC4 for ; Wed, 8 Jul 2015 02:03:05 +0000 (UTC) Received: from mail-qk0-f177.google.com (mail-qk0-f177.google.com [209.85.220.177]) by smtp1.linuxfoundation.org (Postfix) with ESMTPS id E3CB2175 for ; Wed, 8 Jul 2015 02:03:04 +0000 (UTC) Received: by qkeo142 with SMTP id o142so153812425qke.1 for ; Tue, 07 Jul 2015 19:03:04 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: References: <201507080121.41463.PeterHuewe@gmx.de> <1481488.5WJFbB0Dlm@vostro.rjw.lan> Date: Wed, 8 Jul 2015 11:03:04 +0900 Message-ID: From: =?UTF-8?Q?Krzysztof_Koz=C5=82owski?= To: Josh Boyer Content-Type: text/plain; charset=UTF-8 Cc: Jason Cooper , ksummit-discuss@lists.linuxfoundation.org Subject: Re: [Ksummit-discuss] [CORE TOPIC] Recruitment (Reviewers, Testers, Maintainers, Hobbyists) List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , 2015-07-08 10:14 GMT+09:00 Josh Boyer : > On Tue, Jul 7, 2015 at 9:29 PM, Rafael J. Wysocki wrote: >> On Wednesday, July 08, 2015 01:21:40 AM Peter Huewe wrote: >>> Hi, >>> >>> In order to continue our traditions I would like to propose again the topic of >>> recruitment, but this time not only limiting to the hobbyists market. >>> >>> We are definitely short on reviewers and thus have mostly overloaded >>> maintainers. >>> For testers it's usually even worse - how many patches are actually tested? >>> Judging from what I read on LKML not that many. >>> >>> So we should definitely discuss: >>> - how can we encourage hobbyists to become regular contributors >>> -- how to keep people interested, the drop-out rates are huge. >>> - encourage regular contributors to become reviewers and testers >>> - reviewers to become co-maintainers and finally maintainers (once the >>> original maintainer is used up or moves up/on) >> >> Good topic. >> >> Unfortunately, there are not too many incentives for people to become >> code reviewers or testers, or at least to spend more time reviewing patches. >> >> Most of the time there's a little to no recognition for doing that work and, >> quite frankly, writing code is more rewarding than that for the majority of >> people anyway. >> >> The only way to address this problem I can see is to recognize reviewers >> *much* more than we tend to do and not just "encourage" them, because that's >> way insufficient. > > You could make a Reviewed-by tag required before a patch can be > included in a submaintainer's tree. At least some maintainers seem to > (arbitrarily?) require this at times. However, if you do that then it > would likely slow down development quite a bit. Then Greg might cry > because he wouldn't get to show pretty graphs at conferences about how > fast the rate of change is in the kernel. Enforcing reviewed-by or tested-by tags won't be enough if no one actually will do the review and testing. The patch can wait indefinitely with maintainer's response "I expect an independent review". This goes back to first question - will such enforcement boost number of reviews or tests? Before doing some work there is always a cause, an answer to "why I am doing this"? Employer may pay for my commits but would he pay for reviewing time? That is his decision and it would be difficult to change policies inside companies. Other reason for doing open source work may be the fame. Being recognizable, getting better job offers, doing tasks which are sensible and meaningful for someone. Currently probably most of the fame goes to authors and maintainers. For example in the form of `git log --author/committer=` or LWN articles about statistics. How to get more reviews from such people (when employer does not pay for it)? Give them fame! :) Best regards, Krzysztof