From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from na01-bn1-obe.outbound.protection.outlook.com (mail-bn1bn0102.outbound.protection.outlook.com [157.56.110.102]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-SHA384 (256/256 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id B92731A097F for ; Thu, 30 Jul 2015 02:19:37 +1000 (AEST) Message-ID: <1438186761.2993.337.camel@freescale.com> Subject: Re: [RFC v2] genalloc:add an gen_pool_first_fit_align algo to genalloc From: Scott Wood To: Zhao Qiang-B45475 CC: "lauraa@codeaurora.org" , "linux-kernel@vger.kernel.org" , "linuxppc-dev@lists.ozlabs.org" , "akpm@linux-foundation.org" , "olof@lixom.net" , "catalin.marinas@arm.com" , "Xie Xiaobo-R63061" Date: Wed, 29 Jul 2015 11:19:21 -0500 In-Reply-To: References: <1437991074-35377-1-git-send-email-qiang.zhao@freescale.com> <1438032043.2993.311.camel@freescale.com> Content-Type: text/plain; charset="UTF-8" MIME-Version: 1.0 List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Tue, 2015-07-28 at 00:32 -0500, Zhao Qiang-B45475 wrote: > On Tue, 2015-07-28 at 5:21, Scott Wood wrote: > > -----Original Message----- > > From: Wood Scott-B07421 > > Sent: Tuesday, July 28, 2015 5:21 AM > > To: Zhao Qiang-B45475 > > Cc: lauraa@codeaurora.org; linux-kernel@vger.kernel.org; linuxppc- > > dev@lists.ozlabs.org; akpm@linux-foundation.org; olof@lixom.net; > > catalin.marinas@arm.com; Xie Xiaobo-R63061 > > Subject: Re: [RFC v2] genalloc:add an gen_pool_first_fit_align algo to > > genalloc > > > > On Mon, 2015-07-27 at 17:57 +0800, Zhao Qiang wrote: > > > > Where's the part that adds the ability to pass in data to each allocation > > call, as per the previous discussion? > > You means to use gen_pool_alloc_data()? Yes. > Previously you said that the format of data is algorithm-specific, > So I think it is better to handle data in algorithm function. It is a channel for communication from the API caller to the algorithm. > If you still prefer gen_pool_alloc_data(), I will modify it. > But there still are details I want to confirm with you. > 1. If use gen_pool_alloc_data(), should I pass data as a parameter? Yes. > 2. Should I count align_mask in gen_pool_alloc_data(), meanwhile, add > a align_mask to data as a member? gen_pool_alloc_data() should just pass data to the algorithm. The algorithm should calculate align_mask based on align. I don't think exposing align_mask to API users would be very friendly. > 3. where to define the data, in genalloc.h or caller layer? Same place as where the algorithm function is declared. -Scott