Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932836Ab3FQNA0 (ORCPT ); Mon, 17 Jun 2013 09:00:26 -0400 Received: from merlin.infradead.org ([205.233.59.134]:43060 "EHLO merlin.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756150Ab3FQNAZ (ORCPT ); Mon, 17 Jun 2013 09:00:25 -0400 Date: Mon, 17 Jun 2013 15:00:04 +0200 From: Peter Zijlstra To: Paul Turner Cc: Alex Shi , Ingo Molnar , Thomas Gleixner , Andrew Morton , Borislav Petkov , Namhyung Kim , Mike Galbraith , Morten Rasmussen , Vincent Guittot , Preeti U Murthy , Viresh Kumar , LKML , Mel Gorman , Rik van Riel , Michael Wang , Jason Low , Changlong Xie , sgruszka@redhat.com, =?iso-8859-1?Q?Fr=E9d=E9ric?= Weisbecker Subject: Re: [patch v8 3/9] sched: set initial value of runnable avg for new forked task Message-ID: <20130617130004.GV3204@twins.programming.kicks-ass.net> References: <1370589652-24549-1-git-send-email-alex.shi@intel.com> <1370589652-24549-4-git-send-email-alex.shi@intel.com> <51BB25BC.9070500@intel.com> <20130617092146.GM3204@twins.programming.kicks-ass.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2012-12-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2251 Lines: 55 On Mon, Jun 17, 2013 at 02:39:53AM -0700, Paul Turner wrote: > I actually did read it before, and still wasn't sure of the right tag to use. > > "13) When to use Acked-by: and Cc: > > The Signed-off-by: tag indicates that the signer was involved in the > development of the patch, or that he/she was in the patch's delivery path. > > If a person was not directly involved in the preparation or handling of a > patch but wishes to signify and record their approval of it then they can > arrange to have an Acked-by: line added to the patch's changelog." > https://www.kernel.org/doc/Documentation/SubmittingPatches > > Acked-By seemed to fail the direct involvement test. > The definition of "delivery path" is not clear; is this strictly by > inputs to Linus' tree or recipients of the original patch? The way I interpret the delivery path is when its part of a bigger grouping. The author is always the first SOB, if the patch series isn't by the same author then he who compiles the aggregate work adds his SOB. Similar for (sub)maintainers passing it onwards towards Linus. I compile batches of patch series and feed them to Ingo, Ingo stuffs the lot into -tip and feeds them to Linus. So supposing you wrote a patch, gave it to Alex who composed the bigger series, which I picked up and handed to Ingo we'd get something like: SoB: PJT SoB: Alex SoB: PeterZ SoB: Mingo and then when Linus pulls the lot he could add his SOB too, although looking at git history he typically doesn't add his sob to each and everything he pulls. But yes, I see where the confusion stems from. > Is Reviewed-By always more appropriate here? Yes, or even Acked would work. The difference between reviewed and acked is the level of involvement. An ack is for something you glanced over and generally agree with, a review is for something you looked at in minute detail. Then there's some people (and I tend to lean towards this) who can't be bothered with that distinction too much and simply always use ack. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/