From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ug-out-1314.google.com (ug-out-1314.google.com [66.249.92.175]) by ozlabs.org (Postfix) with ESMTP id 17966DDDFE for ; Mon, 8 Jan 2007 17:31:16 +1100 (EST) Received: by ug-out-1314.google.com with SMTP id k3so2756541ugf for ; Sun, 07 Jan 2007 22:31:14 -0800 (PST) Message-ID: Date: Sun, 7 Jan 2007 22:31:14 -0800 From: "Tim Pepper" Sender: tpepper@gmail.com To: "Dave Hansen" Subject: Re: [PATCH] Fix sparsemem on Cell (take 3) In-Reply-To: <1168160307.6740.9.camel@localhost.localdomain> MIME-Version: 1.0 Content-Type: multipart/alternative; boundary="----=_Part_59152_16376959.1168237874232" References: <20061215165335.61D9F775@localhost.localdomain> <200612182354.47685.arnd@arndb.de> <1166483780.8648.26.camel@localhost.localdomain> <200612190959.47344.arnd@arndb.de> <1168045803.8945.14.camel@localhost.localdomain> <1168059162.23226.1.camel@sinatra.austin.ibm.com> <1168160307.6740.9.camel@localhost.localdomain> Cc: Andrew Morton , mkravetz@us.ibm.com, Arnd Bergmann , linux-mm@kvack.org, lkml , hch@infradead.org, External List , Paul Mackerras , kmannth@us.ibm.com, gone@us.ibm.com, KAMEZAWA Hiroyuki List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , ------=_Part_59152_16376959.1168237874232 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Content-Disposition: inline On 1/7/07, Dave Hansen wrote: > > On Fri, 2007-01-05 at 22:52 -0600, John Rose wrote: > > > Could this break ia64, given that it uses memmap_init_zone()? > > You are right, I think it does. > > Here's an updated patch to replace the earlier one. I had to move the > enum definition over to a different header because ia64 evidently has a > different include order. Boot tested OK on ia64 with this latest version of the patch. Tim ------=_Part_59152_16376959.1168237874232 Content-Type: text/html; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Content-Disposition: inline On 1/7/07, Dave Hansen <haveblue@us.ibm.com> wrote:
On Fri, 2007-01-05 at 22:52 -0600, John Rose wrote:

> Could this break ia64, given that it uses memmap_init_zone()?

You are right, I think it does.

Here's an updated patch to replace the earlier one.  I had to move the
enum definition over to a different header because ia64 evidently has a
different include order.

Boot tested OK on ia64 with this latest version of the patch.

Tim

------=_Part_59152_16376959.1168237874232--