mm: cma: Contiguous Memory Allocator added
in [Kernel]
|
From: Michał Nazarewicz on 21 Jul 2010 16:30 On Wed, 21 Jul 2010 22:03:24 +0200, Daniel Walker <dwalker(a)codeaurora.org> wrote: > On Wed, 2010-07-21 at 21:53 +0200, Michał Nazarewicz wrote: >> On Wed, 21 Jul 2010 21:37:09 +0200, Daniel Walker <dwalker(a)codeaurora.org> 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 .. Right..... Please show me a place where I've written that it won't be in the kernel? I keep repeating command line is only one of the possibilities. I would imagine that in final product defaults from platform would be used and bootloader would be left alone. >> 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. 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 “foo” then “bar” is not needed. >> 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 .. 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 command line is has a minimal impact on the code. -- Best regards, _ _ | Humble Liege of Serenely Enlightened Majesty of o' \,=./ `o | Computer Science, Michał "mina86" Nazarewicz (o o) +----[mina86*mina86.com]---[mina86*jabber.org]----ooO--(_)--Ooo-- -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo(a)vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
From: Daniel Walker on 21 Jul 2010 16:40 On Wed, 2010-07-21 at 22:22 +0200, Michał Nazarewicz wrote: > On Wed, 21 Jul 2010 22:03:24 +0200, Daniel Walker <dwalker(a)codeaurora.org> wrote: > > > On Wed, 2010-07-21 at 21:53 +0200, Michał Nazarewicz wrote: > >> On Wed, 21 Jul 2010 21:37:09 +0200, Daniel Walker <dwalker(a)codeaurora.org> 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 .. > > Right..... Please show me a place where I've written that it won't be in > the kernel? I keep repeating command line is only one of the possibilities. > I would imagine that in final product defaults from platform would be used > and bootloader would be left alone. It should never be anyplace else. 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(a)vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
From: Daniel Walker on 21 Jul 2010 16:50 On Wed, 2010-07-21 at 22:43 +0200, Michał Nazarewicz wrote: > > On Wed, 2010-07-21 at 22:22 +0200, Michał Nazarewicz wrote: > >> Right..... Please show me a place where I've written that it won't be in > >> the kernel? I keep repeating command line is only one of the possibilities. > >> I would imagine that in final product defaults from platform would be used > >> and bootloader would be left alone. > > On Wed, 21 Jul 2010 22:34:32 +0200, Daniel Walker <dwalker(a)codeaurora.org> wrote: > > It should never be anyplace else. > > I disagree. There are countless “dubug_level” kernel parameters or even > some “printk” related parameters. Those are completely hardware-independent. > There are also parameters that are hardware dependent but most users won't > care to set them. That's how the things are: there are some defaults but > you can override them by command line parameters. Your not hearing the issues.. IT'S TOO COMPLEX! Please remove it. 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(a)vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
From: Michał Nazarewicz on 21 Jul 2010 16:50 > On Wed, 2010-07-21 at 22:22 +0200, Michał Nazarewicz wrote: >> Right..... Please show me a place where I've written that it won't be in >> the kernel? I keep repeating command line is only one of the possibilities. >> I would imagine that in final product defaults from platform would be used >> and bootloader would be left alone. On Wed, 21 Jul 2010 22:34:32 +0200, Daniel Walker <dwalker(a)codeaurora.org> wrote: > It should never be anyplace else. I disagree. There are countless “dubug_level” kernel parameters or even some “printk” related parameters. Those are completely hardware-independent. There are also parameters that are hardware dependent but most users won't care to set them. That's how the things are: there are some defaults but you can override them by command line parameters. -- Best regards, _ _ | Humble Liege of Serenely Enlightened Majesty of o' \,=./ `o | Computer Science, Michał "mina86" Nazarewicz (o o) +----[mina86*mina86.com]---[mina86*jabber.org]----ooO--(_)--Ooo-- -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo(a)vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
From: Michał Nazarewicz on 21 Jul 2010 17:00
On Wed, 21 Jul 2010 22:45:43 +0200, Daniel Walker <dwalker(a)codeaurora.org> wrote: > Your not hearing the issues.. IT'S TOO COMPLEX! Please remove it. Remove what exactly? The command line parameter? It's like 50 lines of code, so I don't see any benefits. The possibility to specify the configuration? It would defy the whole purpose of CMA, so I won't do that. The complexity has to be there one way or the other and even though I am aware that less complex code is the better and am trying to remove unnecessary complexity but saying “remove it” won't make any good unless you provide me with a better alternative. At this point, I am still convinced that the string parameters are the best option. -- Best regards, _ _ | Humble Liege of Serenely Enlightened Majesty of o' \,=./ `o | Computer Science, Michał "mina86" Nazarewicz (o o) +----[mina86*mina86.com]---[mina86*jabber.org]----ooO--(_)--Ooo-- -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo(a)vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/ |