From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758601AbYHVVL2 (ORCPT ); Fri, 22 Aug 2008 17:11:28 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753137AbYHVVLT (ORCPT ); Fri, 22 Aug 2008 17:11:19 -0400 Received: from gw.goop.org ([64.81.55.164]:45917 "EHLO mail.goop.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753123AbYHVVLT (ORCPT ); Fri, 22 Aug 2008 17:11:19 -0400 Message-ID: <48AF2B74.5030605@goop.org> Date: Fri, 22 Aug 2008 14:11:16 -0700 From: Jeremy Fitzhardinge User-Agent: Thunderbird 2.0.0.16 (X11/20080723) MIME-Version: 1.0 To: Andrew Morton CC: ehabkost@redhat.com, benh@kernel.crashing.org, paulus@samba.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 1 of 3] add phys_addr_t for holding physical addresses References: <73196c9129c7298f2428.1219176170@localhost> <20080822130235.af69ccb5.akpm@linux-foundation.org> In-Reply-To: <20080822130235.af69ccb5.akpm@linux-foundation.org> X-Enigmail-Version: 0.95.7 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Andrew Morton wrote: > On Tue, 19 Aug 2008 13:02:50 -0700 > Jeremy Fitzhardinge wrote: > > >> Add a kernel-wide "phys_addr_t" which is guaranteed to be able to hold >> any physical address. By default it equals the word size of the >> architecture, but a 32-bit architecture can set ARCH_PHYS_ADDR_T_64BIT >> if it needs a 64-bit phys_addr_t. >> >> > > You say this is a bugfix but you don't describe the bug. This makes it > rather hard to make the 2.6.2[5678] decisions. > > Ditto on [patch 2/3]. > 1/3 is not a bugfix in itself, but a pre-requisite for 2/3. 2/3 replaces an ad-hoc Xen fix with a general fix to prevent address truncation when using PFN_PHYS() on any PFN above the 4G mark. The Xen crash is the only bug I know of that's directly attributable to this, and it was already addressed in older kernels with the casts in the Xen code that this patch removes. So I don't think there's any strong need to push this to earlier kernels. J