From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752889AbbDGMHD (ORCPT ); Tue, 7 Apr 2015 08:07:03 -0400 Received: from v094114.home.net.pl ([79.96.170.134]:63207 "HELO v094114.home.net.pl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S1750993AbbDGMHB (ORCPT ); Tue, 7 Apr 2015 08:07:01 -0400 From: "Rafael J. Wysocki" To: Peter Zijlstra Cc: Richard Weinberger , Greg Kroah-Hartman , Ingo Molnar , Joe Perches , LKML , Ingo Molnar , Linus Torvalds , Thomas Gleixner , "H. Peter Anvin" Subject: Re: about the flood of trivial patches and the Code of Conduct (was: Re: [PATCH 19/25] sched: Use bool function return values of true/false not 1/0) Date: Tue, 07 Apr 2015 14:31:23 +0200 Message-ID: <1835148.WK9y2EKkR9@vostro.rjw.lan> User-Agent: KMail/4.11.5 (Linux/3.19.0+; KDE/4.11.5; x86_64; ; ) In-Reply-To: <20150407113212.GM21418@twins.programming.kicks-ass.net> References: <20150407113212.GM21418@twins.programming.kicks-ass.net> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="utf-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tuesday, April 07, 2015 01:32:12 PM Peter Zijlstra wrote: > On Tue, Apr 07, 2015 at 01:28:27PM +0200, Richard Weinberger wrote: > > Can't we send all these kind of patches through the trivial tree? > > Don't get me wrong, if you are fine with these patches that's you decision. > > But other maintainers might think they have to take these patches and > > get overloaded. I'm thinking of drivers maintainers that can only work > > one or two hours per week on Linux. > > Not everyone works full time on it like you. > > > > I propose to send all this stuff though the trivial tree such that maintainers > > of other subsystems have less workload and newbies (which are supposed > > to send such patches) know which tree they have to work against. > > Let's have to well defined and ordered. :-) > > As per the other branch of this tree; an emphatic NO to that. The > trivial tree is not a backdoor to bypass maintainers. Actual code > changes do not get to go through any tree but the maintainer tree unless > explicitly ACKed. Well, practically speaking, that would make changes like the recent clockevents_notify() removal very difficult to carry out. Also there is some natural cross-talk between certain subsystems. Different matter is the real value of tree-wide cleanup changes. If code is old enough it often is better to leave it alone, even though it may be doing things that we don't usually do nowadays. Or things that new patches are not supposed to do, for that matter, so I generally don't like the "checkpatch.pl error fix" changes in the old code. Rafael