From mboxrd@z Thu Jan 1 00:00:00 1970 From: Benjamin Herrenschmidt Subject: Re: [PATCH 05/35] x86, lmb: Add lmb_find_area_size() Date: Fri, 14 May 2010 18:13:29 +1000 Message-ID: <1273824809.21352.584.camel@pasglop> References: <1273796396-29649-1-git-send-email-yinghai@kernel.org> <1273796396-29649-6-git-send-email-yinghai@kernel.org> <1273803621.21352.368.camel@pasglop> <4BECEDAA.9070009@oracle.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: Received: from gate.crashing.org ([63.228.1.57]:34583 "EHLO gate.crashing.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754271Ab0ENIO4 (ORCPT ); Fri, 14 May 2010 04:14:56 -0400 In-Reply-To: <4BECEDAA.9070009@oracle.com> Sender: linux-arch-owner@vger.kernel.org List-ID: To: Yinghai Cc: Ingo Molnar , Thomas Gleixner , "H. Peter Anvin" , Andrew Morton , David Miller , Linus Torvalds , Johannes Weiner , linux-kernel@vger.kernel.org, linux-arch@vger.kernel.org On Thu, 2010-05-13 at 23:28 -0700, Yinghai wrote: > > this just line by line translation from early_res version to lmb > changes > > please focus on lmb core at this point. Well, the problem is that you dig into the LMB core with those functions which means I -will- break your stuff if/when I change it, for example to use linked lists. Besides, the code lacks comments and explanation in the changeset. So please provide that, I'm sure Thomas and Peter also want to understand what's going on in the x86 side of things. Cheers, Ben.