Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755456Ab0KNNUI (ORCPT ); Sun, 14 Nov 2010 08:20:08 -0500 Received: from caramon.arm.linux.org.uk ([78.32.30.218]:54098 "EHLO caramon.arm.linux.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751957Ab0KNNUF (ORCPT ); Sun, 14 Nov 2010 08:20:05 -0500 Date: Sun, 14 Nov 2010 13:19:41 +0000 From: Russell King - ARM Linux To: Catalin Marinas Cc: linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Will Deacon Subject: Re: [PATCH v2 03/20] ARM: LPAE: use u32 instead of unsigned long for 32-bit ptes Message-ID: <20101114131941.GA10871@n2100.arm.linux.org.uk> References: <1289584840-18097-1-git-send-email-catalin.marinas@arm.com> <1289584840-18097-4-git-send-email-catalin.marinas@arm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1289584840-18097-4-git-send-email-catalin.marinas@arm.com> User-Agent: Mutt/1.5.19 (2009-01-05) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 985 Lines: 22 On Fri, Nov 12, 2010 at 06:00:23PM +0000, Catalin Marinas wrote: > From: Will Deacon > > When using 2-level paging, pte_t and pmd_t are typedefs for > unsigned long but phys_addr_t is a typedef for u32. > > This patch uses u32 for the page table entry types when > phys_addr_t is not 64-bit, allowing the same conversion > specifier to be used for physical addresses and page table > entries regardless of LPAE. However, code which prints the value of page table entries assumes that they are unsigned long, and places where we store the raw pte value also uses 'unsigned long'. If we're going to make this change, we need to change more places than this patch covers. grep for pte_val to help find those places. -- 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/