Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753599Ab3I0W0F (ORCPT ); Fri, 27 Sep 2013 18:26:05 -0400 Received: from g6t0186.atlanta.hp.com ([15.193.32.63]:16837 "EHLO g6t0186.atlanta.hp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751009Ab3I0W0C (ORCPT ); Fri, 27 Sep 2013 18:26:02 -0400 Message-ID: <1380320637.14046.44.camel@misato.fc.hp.com> Subject: Re: [PATCH v5 1/6] memblock: Factor out of top-down allocation From: Toshi Kani To: Zhang Yanfei Cc: "Rafael J . Wysocki" , lenb@kernel.org, Thomas Gleixner , mingo@elte.hu, "H. Peter Anvin" , Andrew Morton , Tejun Heo , Wanpeng Li , Thomas Renninger , Yinghai Lu , Jiang Liu , Wen Congyang , Lai Jiangshan , isimatu.yasuaki@jp.fujitsu.com, izumi.taku@jp.fujitsu.com, Mel Gorman , Minchan Kim , mina86@mina86.com, gong.chen@linux.intel.com, vasilis.liaskovitis@profitbricks.com, lwoodman@redhat.com, Rik van Riel , jweiner@redhat.com, prarit@redhat.com, "x86@kernel.org" , linux-doc@vger.kernel.org, "linux-kernel@vger.kernel.org" , Linux MM , linux-acpi@vger.kernel.org, imtangchen@gmail.com, Zhang Yanfei Date: Fri, 27 Sep 2013 16:23:57 -0600 In-Reply-To: <5241D90D.6030203@gmail.com> References: <5241D897.1090905@gmail.com> <5241D90D.6030203@gmail.com> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.6.4 (3.6.4-3.fc18) Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2270 Lines: 60 On Wed, 2013-09-25 at 02:25 +0800, Zhang Yanfei wrote: > From: Tang Chen > > This patch creates a new function __memblock_find_range_rev > to factor out of top-down allocation from memblock_find_in_range_node. > This is a preparation because we will introduce a new bottom-up > allocation mode in the following patch. > > Acked-by: Tejun Heo > Signed-off-by: Tang Chen > Signed-off-by: Zhang Yanfei Acked-by: Toshi Kani A minor comment below... > --- > mm/memblock.c | 47 ++++++++++++++++++++++++++++++++++------------- > 1 files changed, 34 insertions(+), 13 deletions(-) > > diff --git a/mm/memblock.c b/mm/memblock.c > index 0ac412a..3d80c74 100644 > --- a/mm/memblock.c > +++ b/mm/memblock.c > @@ -83,33 +83,25 @@ static long __init_memblock memblock_overlaps_region(struct memblock_type *type, > } > > /** > - * memblock_find_in_range_node - find free area in given range and node > + * __memblock_find_range_rev - find free area utility, in reverse order > * @start: start of candidate range > * @end: end of candidate range, can be %MEMBLOCK_ALLOC_{ANYWHERE|ACCESSIBLE} > * @size: size of free area to find > * @align: alignment of free area to find > * @nid: nid of the free area to find, %MAX_NUMNODES for any node > * > - * Find @size free area aligned to @align in the specified range and node. > + * Utility called from memblock_find_in_range_node(), find free area top-down. > * > * RETURNS: > * Found address on success, %0 on failure. > */ > -phys_addr_t __init_memblock memblock_find_in_range_node(phys_addr_t start, > - phys_addr_t end, phys_addr_t size, > - phys_addr_t align, int nid) > +static phys_addr_t __init_memblock > +__memblock_find_range_rev(phys_addr_t start, phys_addr_t end, Since we are now using the terms "top down" and "bottom up" consistently, how about name this function as __memblock_find_range_top_down()? Thanks, -Toshi -- 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/