From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754676Ab1GBOP0 (ORCPT ); Sat, 2 Jul 2011 10:15:26 -0400 Received: from gate.crashing.org ([63.228.1.57]:42438 "EHLO gate.crashing.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752676Ab1GBOPZ (ORCPT ); Sat, 2 Jul 2011 10:15:25 -0400 Subject: Re: [PATCH 2/2] powerpc/mm: Fix memory_block_size_bytes() for non-pseries From: Benjamin Herrenschmidt To: Ingo Molnar Cc: "linux-kernel@vger.kernel.org" , Thomas Gleixner , "linux-mm@kvack.org" , linuxppc-dev In-Reply-To: <20110702102333.GC17482@elte.hu> References: <1308013071.2874.785.camel@pasglop> <20110701121516.GD28008@elte.hu> <1309562112.14501.257.camel@pasglop> <20110702102333.GC17482@elte.hu> Content-Type: text/plain; charset="UTF-8" Date: Sun, 03 Jul 2011 00:15:01 +1000 Message-ID: <1309616101.14501.262.camel@pasglop> Mime-Version: 1.0 X-Mailer: Evolution 2.30.3 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sat, 2011-07-02 at 12:23 +0200, Ingo Molnar wrote: > It's certainly not a hard rule - but note that the file in question > (arch/powerpc/platforms/pseries/hotplug-memory.c) has a rather > inconsistent comment style, sometimes even within the same function: > > /* > * Remove htab bolted mappings for this section of memory > */ > ... > > /* Ensure all vmalloc mappings are flushed in case they also > * hit that section of memory > */ > > That kind of inconsistency within the same .c file and within the > same function is not defensible with a "style is a matter of taste" > argument. Right, that's a matter of different people with different taste mucking around with the same file I suppose. Most of this probably predates my involvement as a maintainer but even if not (and I really can't be bothered digging into the history), it might very well be something I didn't pay attention to while reviewing. Seriously, it's so low on the scale of what matters ... I'm sure we both have more valuable stuff to spend our time and energy on :-) Cheers, Ben.