From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail144.messagelabs.com (mail144.messagelabs.com [216.82.254.51]) by kanga.kvack.org (Postfix) with SMTP id 35A926B024D for ; Wed, 21 Jul 2010 16:20:58 -0400 (EDT) Received: from eu_spt1 (mailout1.w1.samsung.com [210.118.77.11]) by mailout1.w1.samsung.com (iPlanet Messaging Server 5.2 Patch 2 (built Jul 14 2004)) with ESMTP id <0L5X00I64CIVPJ@mailout1.w1.samsung.com> for linux-mm@kvack.org; Wed, 21 Jul 2010 21:20:55 +0100 (BST) Received: from linux.samsung.com ([106.116.38.10]) by spt1.w1.samsung.com (iPlanet Messaging Server 5.2 Patch 2 (built Jul 14 2004)) with ESMTPA id <0L5X00FI8CIULH@spt1.w1.samsung.com> for linux-mm@kvack.org; Wed, 21 Jul 2010 21:20:54 +0100 (BST) Date: Wed, 21 Jul 2010 22:22:19 +0200 From: =?utf-8?B?TWljaGHFgiBOYXphcmV3aWN6?= Subject: Re: [PATCH 2/4] mm: cma: Contiguous Memory Allocator added In-reply-to: <1279742604.31376.40.camel@c-dwalke-linux.qualcomm.com> Message-id: MIME-version: 1.0 Content-type: text/plain; charset=utf-8; format=flowed; delsp=yes Content-transfer-encoding: Quoted-Printable References: <1279649724.26765.23.camel@c-dwalke-linux.qualcomm.com> <1279654698.26765.31.camel@c-dwalke-linux.qualcomm.com> <1279733750.31376.14.camel@c-dwalke-linux.qualcomm.com> <1279736348.31376.20.camel@c-dwalke-linux.qualcomm.com> <1279738688.31376.24.camel@c-dwalke-linux.qualcomm.com> <1279741029.31376.33.camel@c-dwalke-linux.qualcomm.com> <1279742604.31376.40.camel@c-dwalke-linux.qualcomm.com> Sender: owner-linux-mm@kvack.org To: Daniel Walker Cc: linux-mm@kvack.org, Marek Szyprowski , Pawel Osciak , Xiaolin Zhang , Hiremath Vaibhav , Robert Fekete , Marcus Lorentzon , linux-kernel@vger.kernel.org, Kyungmin Park , linux-arm-msm@vger.kernel.org List-ID: On Wed, 21 Jul 2010 22:03:24 +0200, Daniel Walker wrote: > On Wed, 2010-07-21 at 21:53 +0200, Micha=C5=82 Nazarewicz wrote: >> On Wed, 21 Jul 2010 21:37:09 +0200, Daniel Walker wrote: >> > What makes you assume that the bootloader would have these strings?= >> > Do your devices have these strings? Maybe mine don't have them. >> >> I don't assume. I only state it as one of the possibilities. >> >> > Assume the strings are gone and you can't find them, or have no ide= a >> > what they should be. What do you do then? >> >> Ask Google? > > Exactly, that's why they need to be in the kernel .. Right..... Please show me a place where I've written that it won't be i= n the kernel? I keep repeating command line is only one of the possibiliti= es. I would imagine that in final product defaults from platform would be us= ed and bootloader would be left alone. >> I have a better question for you though: assume the "mem" parameter i= s >> lost and you have no idea what it should be? There are many paramete= rs >> passed to kernel by bootloader and you could ask about all of them. > > That's hardware based tho. Of course you need info on what your hardwa= re > is. What your doing isn't based on hardware specifics, it's based on > optimizations. > >> Passing cma configuration via command line is one of the possibilitie= s >> -- especially convenient during development -- but I would expect pla= tform >> defaults in a final product so you may well not need to worry about i= t. > > I honestly don't thing the "development" angle flies either , but if y= ou > keep this there's no way it should be enabled for anything but debug. If you are developing the whole platform and optimising the allocators, etc. it's very convenient. If you develop something else then it's not needed but then again it's usually the case that if you develop =E2=80=9C= foo=E2=80=9D then =E2=80=9Cbar=E2=80=9D is not needed. >> Would you also argue about removing all the other kernel parameters a= s >> well? I bet you don't use most of them. Still they are there because= >> removing them would add too much complexity to the code (conditional >> compilation, etc.). > > Your is at a different level of complexity .. Which most of will remain even if the command line parameters were to be removed. One needs to specify this configuration somehow and no matter how you do it it will be complex in one way or another. In my code the complexity is parsing of the strings, in a different approach it would be complex in a different way. At the same time, the fact that the parameters can be provided via comma= nd line is has a minimal impact on the code. -- = Best regards, _ _ | Humble Liege of Serenely Enlightened Majesty of o' \,=3D./ `o | Computer Science, Micha=C5=82 "mina86" Nazarewicz (o o) +----[mina86*mina86.com]---[mina86*jabber.org]----ooO--(_)--Ooo-- -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org