From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932273AbcCIMIV (ORCPT ); Wed, 9 Mar 2016 07:08:21 -0500 Received: from mail-wm0-f67.google.com ([74.125.82.67]:35471 "EHLO mail-wm0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752797AbcCIMIM (ORCPT ); Wed, 9 Mar 2016 07:08:12 -0500 Date: Wed, 9 Mar 2016 13:08:08 +0100 From: Ingo Molnar To: Matthew Wilcox Cc: akpm@linux-foundation.org, hpa@zytor.com, mingo@elte.hu, tglx@linutronix.de, linux-kernel@vger.kernel.org, Andrea Arcangeli Subject: Re: + x86-add-support-for-pud-sized-transparent-hugepages-checkpatch-fixes.patch added to -mm tree Message-ID: <20160309120807.GA3161@gmail.com> References: <56b13381.v0wS03ZQEKxwivVW%akpm@linux-foundation.org> <20160203074835.GB32652@gmail.com> <20160304203018.GC5530@linux.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20160304203018.GC5530@linux.intel.com> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org * Matthew Wilcox wrote: > On Wed, Feb 03, 2016 at 08:48:35AM +0100, Ingo Molnar wrote: > > > @@ -111,8 +111,10 @@ static inline pud_t native_pudp_get_and_ > > > #ifdef CONFIG_SMP > > > return native_make_pud(xchg(&pudp->pud, 0)); > > > #else > > > - /* native_local_pudp_get_and_clear, > > > - but duplicated because of cyclic dependency */ > > > + /* > > > + * native_local_pudp_get_and_clear, but duplicated because of cyclic > > > + * dependency > > > + */ > > > pud_t ret = *pudp; > > > native_pud_clear(pudp); > > > return ret; > > > > When referring to functions in comments (or changelogs) please use () to make it > > clear on sight what is being referred to. > > > > Also, please try to construct proper English sentences with verbs and such! > > > > I.e. something like this would work for me: > > > > > + /* > > > + * This is a duplicate of native_local_pudp_get_and_clear(), > > > + * because we cannot use the original due to a cyclic header > > > + * file dependency: > > > + */ > > > > (Assuming I managed to decode the shorthand form properly.) > > I have no idea what it means. This is copy-and-change of the pmd version, > which was originally commit db3eb96f4e6281b84dd33c8980dacc27f2efe177 by > Andrea. It means that we don't want to copy-and-change a crappy comment that slipped through 5 years ago, we want to copy-and-improve. I even suggested the comment improvement (which needs to be checked though). > It seems unfair to ask me to do better than what is there right now. It's absolutely fair for maintainers to require the improvement of existing code you want to modify, especially when you are complicating existing code ... Thanks, Ingo