Date: Wed, 9 Mar 2016 11:55:07 -0500
From: Matthew Wilcox <willy@linux.intel.com>
To: Ingo Molnar <mingo@kernel.org>
Cc: akpm@linux-foundation.org, hpa@zytor.com, mingo@elte.hu,
        tglx@linutronix.de, linux-kernel@vger.kernel.org,
        Andrea Arcangeli <aarcange@redhat.com>
Subject: Re: +
 x86-add-support-for-pud-sized-transparent-hugepages-checkpatch-fixes.patch
 added to -mm tree
Message-ID: <20160309165507.GA2464@linux.intel.com>
References: <56b13381.v0wS03ZQEKxwivVW%akpm@linux-foundation.org>
 <20160203074835.GB32652@gmail.com>
 <20160304203018.GC5530@linux.intel.com>
 <20160309120807.GA3161@gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20160309120807.GA3161@gmail.com>
User-Agent: Mutt/1.5.24 (2015-08-30)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1280
Lines: 26

On Wed, Mar 09, 2016 at 01:08:08PM +0100, Ingo Molnar wrote:
> * Matthew Wilcox <willy@linux.intel.com> wrote:
> > 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).

The "it" in my sentence referred to the comment.  I have no idea what
the comment is supposed to mean.  I am the worst person to figure out
what the comment is supposed to mean as I have the least experience with
the code here.

The PUD and PMD code should be as similar as possible, down to the
comments and the spacing.  If you want the original fixed, that's fine,
and I'm willing to include it as part of this patch set.  But it's not
my responsibility to fix up the comments that you don't like.

> > 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 ...

I'm not complicating it.  I'm duplicating it.