All of lore.kernel.org
 help / color / mirror / Atom feed
From: Rusty Russell <rusty@rustcorp.com.au>
To: Andi Kleen <andi@firstfloor.org>
Cc: Johannes Berg <johannes@sipsolutions.net>,
	Christoph Hellwig <hch@infradead.org>,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH -next 00/11] remove CONFIG_KMOD
Date: Mon, 14 Jul 2008 10:38:49 +1000	[thread overview]
Message-ID: <200807141038.49652.rusty@rustcorp.com.au> (raw)
In-Reply-To: <87zlolzv15.fsf@basil.nowhere.org>

On Monday 14 July 2008 01:03:02 Andi Kleen wrote:
> Johannes Berg <johannes@sipsolutions.net> writes:
> > I've rebased this patch series against -next-20080708 (one conflict
> > against ftrace, another against BKL pushdown) and fixed the things
> > Adrian Bunk pointed out.
>
> You forgot to say why you want to remove that?
>
> I personally like to not set CONFIG_KMOD because the auto loader
> tends to load way too many modules. e.g. currently when someone requests
> aes on x86 you get three different aes modules (including VIA PadLock
> aes and an unoptimized C version), and keeping those around
> is just a waste.

Well, if you're building your own kernel anyway, don't build modules you don't 
want.

> Also if I didn't load a driver I prefer if it's not loaded at all.
> This is also useful for power saving for example where it's better
> to keep modules not loaded that you don't need and it's hard to
> control who accesses the respective devices.
>
> Please keep the option.

"echo /bin/true > /proc/sys/kernel/modprobe".

Given that, I don't think the config option is useful any more.

Rusty.

  reply	other threads:[~2008-07-14  1:47 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-07-09  8:28 [PATCH -next 00/11] remove CONFIG_KMOD Johannes Berg
2008-07-09  8:28 ` [PATCH -next 01/11] make CONFIG_KMOD invisible Johannes Berg
2008-07-09  8:28 ` [PATCH -next 02/11] remove CONFIG_KMOD from core kernel code Johannes Berg
2008-07-09  8:28 ` [PATCH -next 03/11] rework try_then_request_module to do less in non-modular kernels Johannes Berg
2008-07-09  8:28 ` [PATCH -next 04/11] remove CONFIG_KMOD from drivers Johannes Berg
2008-07-09  8:28   ` Johannes Berg
2008-07-09  8:28 ` [PATCH -next 05/11] remove CONFIG_KMOD from sparc64 Johannes Berg
2008-07-09  8:28 ` [PATCH -next 06/11] remove CONFIG_KMOD from fs Johannes Berg
2008-07-09  8:28 ` [PATCH -next 07/11] remove CONFIG_KMOD from sound Johannes Berg
2008-07-09 17:33   ` Takashi Iwai
2008-07-09  8:28 ` [PATCH -next 08/11] remove CONFIG_KMOD from net Johannes Berg
2008-07-09  8:28 ` [PATCH -next 09/11] remove CONFIG_KMOD from lib Johannes Berg
2008-07-09  8:28 ` [PATCH -next 10/11] remove mention of CONFIG_KMOD from documentation Johannes Berg
2008-07-09  8:28 ` [PATCH -next 11/11] remove CONFIG_KMOD Johannes Berg
2008-07-13 15:03 ` [PATCH -next 00/11] " Andi Kleen
2008-07-14  0:38   ` Rusty Russell [this message]
2008-07-14  7:52   ` Johannes Berg

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=200807141038.49652.rusty@rustcorp.com.au \
    --to=rusty@rustcorp.com.au \
    --cc=andi@firstfloor.org \
    --cc=hch@infradead.org \
    --cc=johannes@sipsolutions.net \
    --cc=linux-kernel@vger.kernel.org \
    /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.