Re: [PATCH 0/8] core-small: Introduce CONFIG_CORE_SMALL from -tiny

From: Andrew Morton
Date: Sun Jan 23 2005 - 16:06:44 EST


Matt Mackall <mpm@xxxxxxxxxxx> wrote:
>
> > I wish it didn't have "core" in the name. A little misleading.
>
> Well I've got another set called NET_SMALL. BASE?

BASE works, I guess.

> > #define PID_MAX_DEFAULT (CONFIG_CORE_SMALL ? 0x1000 : 0x8000)
> > #define UIDHASH_BITS (CONFIG_CORE_SMALL ? 3 : 8)
> > #define FUTEX_HASHBITS (CONFIG_CORE_SMALL ? 4 : 8)
> > etc.
>
> Hmm. I think we'd want a hidden config variable for this and I'm not
> sure how well the config language allows setting an int from a bool.

config AKPM_BOOL
bool "akpm"

config AKPM_INT
int
default 1 if AKPM_BOOL
default 0 if !AKPM_BOOL

seems to do everything which it should.

> And then it would need another name. On the whole, seems more complex
> than what I've done.

No, it's quite simple and avoids lots of ifdeffing.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/