Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755546Ab2JDSuB (ORCPT ); Thu, 4 Oct 2012 14:50:01 -0400 Received: from mail-wg0-f44.google.com ([74.125.82.44]:58386 "EHLO mail-wg0-f44.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754255Ab2JDSt7 (ORCPT ); Thu, 4 Oct 2012 14:49:59 -0400 MIME-Version: 1.0 In-Reply-To: <20121004.142335.1467206545795435493.davem@davemloft.net> References: <20121004074442.180d8f01@nehalam.linuxnetplumber.net> <20121004.142335.1467206545795435493.davem@davemloft.net> Date: Thu, 4 Oct 2012 20:49:57 +0200 Message-ID: Subject: Re: [PATCH 19/20] drivers/net/ethernet/marvell/skge.c: fix error return code From: Peter Senna Tschudin To: David Miller Cc: shemminger@vyatta.com, mlindner@marvell.com, kernel-janitors@vger.kernel.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1609 Lines: 49 On Thu, Oct 4, 2012 at 8:23 PM, David Miller wrote: > From: Peter Senna Tschudin > Date: Thu, 4 Oct 2012 19:32:12 +0200 > >> I can't understand the advantages of describing each patch as you are >> asking. "For me" the generic commit message together with the patch >> makes sense. Can you please help me on that? > > Stop being so dense. > > We want to know the implications of the bug being fixed. > > Does it potentially cause an OOPS? Bad reference counting and thus > potential leaks or early frees? > > You have to analyze the implications and ramifications of the bug > being fixed. We need that information. > > Your commit messages are in fact robotic, they don't describe the > salient details of what kinds of problems the bug being fixed might > cause. > > It's just "bad error code, this is the script that fixed it, kthx, > bye" which is pretty much useless for anaylsis. Thank you David. What about this as commit message? --- // -- This patch fixes bug(s) related to return value of function(s). In some error cases, the function is returning non-negative SUCCESS value, when the correct would be negative ERROR value. When on error, returning non negatives values, or SUCCESS, breaks error propagation, producing unpredictable behavior that are very difficult to debug. --- // -- -- Peter -- 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/