From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932241AbcEKJiw (ORCPT ); Wed, 11 May 2016 05:38:52 -0400 Received: from eddie.linux-mips.org ([148.251.95.138]:41618 "EHLO cvs.linux-mips.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932168AbcEKJiu (ORCPT ); Wed, 11 May 2016 05:38:50 -0400 Date: Wed, 11 May 2016 11:38:45 +0200 From: Ralf Baechle To: James Hogan Cc: Paul Burton , linux-mips@linux-mips.org, "Maciej W. Rozycki" , linux-kernel@vger.kernel.org, Markos Chandras , Alex Smith , "Kirill A. Shutemov" Subject: Re: [PATCH 04/12] MIPS: Use enums to make asm/pgtable-bits.h readable Message-ID: <20160511093845.GJ16402@linux-mips.org> References: <1460716620-13382-1-git-send-email-paul.burton@imgtec.com> <1460716620-13382-5-git-send-email-paul.burton@imgtec.com> <20160415202906.GF7859@jhogan-linux.le.imgtec.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20160415202906.GF7859@jhogan-linux.le.imgtec.org> User-Agent: Mutt/1.6.0 (2016-04-01) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Apr 15, 2016 at 09:29:06PM +0100, James Hogan wrote: > Having had to work my way through some of this file to manually walk > page tables only this week, I really do think this is an excellent > cleanup (if nothing else, look at that diffstat :-D ). I agree. Lots of history in this file. It uses #define because well, i386 was using #define back in the 90's when Elvis was still alive. Much of the page table code has been rewritten for simplicity, performance and ease of maintenance but somehow this has escaped so far. And I'm wondering if eventually the rewrite should be taken even further making things fully dynamic. Ralf