Date: Tue, 22 Jul 2008 10:36:26 +0200
From: Ingo Molnar <mingo@elte.hu>
To: Jeremy Fitzhardinge <jeremy@goop.org>
Cc: Rusty Russell <rusty@rustcorp.com.au>, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 1/2] x86: rename PTE_MASK to PTE_PFN_MASK
Message-ID: <20080722083626.GC2065@elte.hu>
References: <200807221431.58991.rusty@rustcorp.com.au> <488566EA.5040805@goop.org> <200807221540.47161.rusty@rustcorp.com.au> <4885774E.3000909@goop.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <4885774E.3000909@goop.org>
User-Agent: Mutt/1.5.18 (2008-05-17)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1160
Lines: 27


* Jeremy Fitzhardinge <jeremy@goop.org> wrote:

> Rusty, in his peevish way, complained that macros defining constants 
> should have a name which somewhat accurately reflects the actual 
> purpose of the constant.
>
> Aside from the fact that PTE_MASK gives no clue as to what's actually 
> being masked, and is misleadingly similar to the functionally entirely 
> different PMD_MASK, PUD_MASK and PGD_MASK, I don't really see what the 
> problem is.

Has Rusty ever heard about the economy of the healthy flow of incoming 
regressions? What will we do without obscure names and hard to find 
bugs? First he writes a simple and readable hypervisor (ruining a whole 
industry based on obscurity!) and now that. It's _so_ unamerican and 
unaustralian. I'm worried.

Applied to tip/x86/cleanups anyway. Rusty will find out himself how bad 
this whole concept of clean and understandable code is, soon enough!

	Ingo
--
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/