Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753679Ab0GUUDc (ORCPT ); Wed, 21 Jul 2010 16:03:32 -0400 Received: from wolverine02.qualcomm.com ([199.106.114.251]:3265 "EHLO wolverine02.qualcomm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752681Ab0GUUDa (ORCPT ); Wed, 21 Jul 2010 16:03:30 -0400 X-IronPort-AV: E=McAfee;i="5400,1158,6050"; a="48047744" Subject: Re: [PATCH 2/4] mm: cma: Contiguous Memory Allocator added From: Daniel Walker To: =?UTF-8?Q?Micha=C5=82?= Nazarewicz 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 In-Reply-To: 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> Content-Type: text/plain; charset="UTF-8" Date: Wed, 21 Jul 2010 13:03:24 -0700 Message-ID: <1279742604.31376.40.camel@c-dwalke-linux.qualcomm.com> Mime-Version: 1.0 X-Mailer: Evolution 2.28.3 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2580 Lines: 61 On Wed, 2010-07-21 at 21:53 +0200, Michał 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 idea > > what they should be. What do you do then? > > Ask Google? Exactly, that's why they need to be in the kernel .. > I have a better question for you though: assume the "mem" parameter is > lost and you have no idea what it should be? There are many parameters > 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 hardware 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 possibilities > -- especially convenient during development -- but I would expect platform > defaults in a final product so you may well not need to worry about it. I honestly don't thing the "development" angle flies either , but if you keep this there's no way it should be enabled for anything but debug. > > Well, I like my kernel minus bloat so that's a good reason. I don't see > > a good reason to keep the interface in a production situation .. Maybe > > during development , but really I don't see even a developer needing to > > make the kind of changes your suggesting very often. > > As I've said, removing the command line parameters would not benefit the > kernel that much. It's like 1% of the code or less. On the other hand, > it would add complexity to the CMA framework which is a good reason not > to do that. If we allowed everyone to add there little tiny bit of bloat where would we be? > Would you also argue about removing all the other kernel parameters as > 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 .. Daniel -- Sent by an consultant of the Qualcomm Innovation Center, Inc. The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum. -- 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/