lkml.org 
[lkml]   [2012]   [Jan]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] kconfig: untangle EXPERT and EMBEDDED
On Thu, Jan 12, 2012 at 09:18:52AM +0000, Arnd Bergmann wrote:
> On Wednesday 11 January 2012, David Rientjes wrote:
> > We discussed this when the patch was made to change EMBEDDED to EXPERT and
> > we knew that things like CONFIG_SLOB exist that would only make sense on a
> > platform with a very small memory footprint. So what criteria are you
> > using to determine what makes sense for EMBEDDED or not and what would be
> > generally useful for platforms with very small memory footprints?
> >
> > I would think that something like CONFIG_SMALLMEM would identify those
> > candidates (for things like SLOB) and then you could separate the rest
> > based on the platform they exist for since you're essentially introducing
> > a new config symbol here that existed in the past but are now using it
> > with different semantics.

No, I'm putting EMBEDDED back to work doing the same thing it did before,
i.e. the semantics for it are the same as they were before 6a108a14fa35.
This is not a revert of that patch though. I'm keeping EXPERT, but
ensuring it doesn't do everything EMBEDDED used to do. I'm ensuring it
only does what it advertises it does, which is to give a useful configmenu
cleanup symbol to the whole kernel.

>
> The CONFIG_SMALLMEM would also require people to know what they are doing,
> beyond what we expect the average person building a kernel to, which is
> just what CONFIG_EXPERT means, and the same reason why we ended up renaming
> CONFIG_EMBEDDED to CONFIG_EXPERT. People have a very different understanding
> of what "embedded" actually means, and not all of those require small
> memory footprint or even a custom kernel. Let's just stay with CONFIG_EXPERT.

For the SLOB example, and everything else that is 'if EXPERT' after
6a108a14fa35, then don't worry, they are still 'if EXPERT'. See the patch.

I chose EMBEDDED to handle the side effects, simply because it used to
anyway. I think it's best that I leave any further customizing, such as
replacing EMBEDDED with SMALLMEM for particular options, to those that are
more familiar with those options. I can conceive of people wanting EXPERT
-> SMALLMEM -> SLOB type menus, but that's not in the scope of this patch.

Drew


\
 
 \ /
  Last update: 2012-01-12 12:31    [W:1.646 / U:1.908 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site