All of lore.kernel.org
 help / color / mirror / Atom feed
From: Andrew Morton <akpm@osdl.org>
To: Matt Mackall <mpm@selenic.com>
Cc: linux-kernel@vger.kernel.org
Subject: Re: [PATCH 0/8] core-small: Introduce CONFIG_CORE_SMALL from -tiny
Date: Sun, 23 Jan 2005 13:05:14 -0800	[thread overview]
Message-ID: <20050123130514.0a9656bb.akpm@osdl.org> (raw)
In-Reply-To: <20050123175204.GV12076@waste.org>

Matt Mackall <mpm@selenic.com> 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.

  reply	other threads:[~2005-01-23 21:05 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-01-22  0:09 [PATCH 0/8] core-small: Introduce CONFIG_CORE_SMALL from -tiny Matt Mackall
2005-01-22  0:09 ` [PATCH 1/8] core-small: Add option to embedded menu Matt Mackall
2005-01-22  0:09   ` [PATCH 2/8] core-small: Collapse major names hash Matt Mackall
2005-01-22  0:09     ` [PATCH 3/8] core-small: Collapse chrdevs hash Matt Mackall
2005-01-22  0:09       ` [PATCH 4/8] core-small: Shrink PID lookup tables Matt Mackall
2005-01-22  0:09         ` [PATCH 5/8] core-small: Shrink uid hash Matt Mackall
2005-01-22  0:09           ` [PATCH 6/8] core-small: Shrink futex queue hash Matt Mackall
2005-01-22  0:09             ` [PATCH 7/8] core-small: Shrink timer lists Matt Mackall
2005-01-22  0:09               ` [PATCH 8/8] core-small: Shrink console buffer Matt Mackall
2005-01-23  8:40 ` [PATCH 0/8] core-small: Introduce CONFIG_CORE_SMALL from -tiny Andrew Morton
2005-01-23 17:52   ` Matt Mackall
2005-01-23 21:05     ` Andrew Morton [this message]
2005-01-24  2:50       ` Matt Mackall
2005-01-24 20:00     ` Alan Cox
2005-01-24 22:47       ` Tim Bird

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20050123130514.0a9656bb.akpm@osdl.org \
    --to=akpm@osdl.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mpm@selenic.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.