From: Kay Sievers <kay.sievers@vrfy.org>
To: Andreas Robinson <andr345@gmail.com>
Cc: Rusty Russell <rusty@rustcorp.com.au>,
sam@ravnborg.org, linux-kernel@vger.kernel.org
Subject: Re: [RFC PATCH 0/6] module, kbuild: Faster boot with custom kernel.
Date: Mon, 2 Mar 2009 16:59:17 +0100 [thread overview]
Message-ID: <ac3eb2510903020759j100adbdekf2067d142adbfbbe@mail.gmail.com> (raw)
In-Reply-To: <1236004353.10055.49.camel@andreas-laptop>
On Mon, Mar 2, 2009 at 15:32, Andreas Robinson <andr345@gmail.com> wrote:
> I have finished testing parallel module loading. It looks like a small
> kernel with a minimal initramfs and running several instances of insmod
> or modprobe in parallel has the best complexity to performance ratio.
>
> Testing shows that a megamodule is slightly slower than parallel
> insmods, so it's not really an option anymore.
That's what I expected, that we would need more changes, to make a
mega-module working, because we need parallel code execution here.
> A monolithic kernel with parallelized initcalls is better - about 200 ms
> faster than parallel insmods on my test system. However, it comes with a
> fairly large set of changes:
>
> * First, you need a 200-line patch in init/main.c (do_initcalls() and
> friends)
>
> * Then the built-in module dependencies must be calculated properly, eg
> with a modified depmod, and added to the build process.
>
> * Finally "soft" dependencies, i.e dependencies that are not implied by
> symbol use, have to be formalized and moved into the kernel somehow.
> Right now they're only defined in "install" commands in modprobe.conf.
>
> So, what do you think, should I keep going? IMHO, the slower userspace
> implementation is acceptable since it's so much simpler.
I think so too, and think we are fine using the current standard
tools. Also the recent modprobe changes make modprobe very fast by
using a binary index to resolve aliases. I think, we can leave the
stuff as it is.
Seems with the queued stop-machine()-removal patch, and possibly with
the minimized-mutex patch Rusty did, we are fine for now -- at least
with the current numbers, and from a distro's point of view.
Most of the "module load delay" distros see, are likely caused by I/O
loading the module from disk into the kernel, at least that's what my
profiles show. I did all the recent tests (which is a real world udev
coldplug) with a ramfs mount containing a copy of the module
directory, before running the module autoloader. I can link 40 modules
in 0.26 seconds into the kernel with the recent kernel patches.
Thanks for all your testing, it was really helpful, and your patches
have been the reason we've found the stop-machine problem. I'm glad
that we have real numbers now to point people to. It seems, that we
don't need to start fiddling around with hacks in userspace or
initramfs tools.
Thanks a lot,
Kay
next prev parent reply other threads:[~2009-03-02 15:59 UTC|newest]
Thread overview: 46+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-02-15 18:20 Andreas Robinson
2009-02-15 18:20 ` [RFC PATCH 1/6] New option: Static linking of external modules Andreas Robinson
2009-02-15 18:20 ` [RFC PATCH 2/6] module: add module ELF section with module_init() pointer Andreas Robinson
2009-02-15 18:20 ` [RFC PATCH 3/6] module: always prefix module parameters with the module name Andreas Robinson
2009-02-15 18:20 ` [RFC PATCH 4/6] kbuild: allow linking of an external object into vmlinux Andreas Robinson
2009-02-15 18:20 ` [RFC PATCH 5/6] scripts: new module preprocessor for static linking Andreas Robinson
2009-02-15 18:20 ` [RFC PATCH 6/6] kbuild: enable relinking of vmlinux without full kernel tree Andreas Robinson
2009-02-16 22:51 ` [RFC PATCH 0/6] module, kbuild: Faster boot with custom kernel Rusty Russell
2009-02-17 10:42 ` Andreas Robinson
2009-02-17 11:53 ` Kay Sievers
2009-02-18 4:58 ` Rusty Russell
2009-02-18 9:15 ` Kay Sievers
2009-02-18 10:25 ` Andreas Robinson
2009-02-20 0:37 ` Andreas Robinson
2009-02-20 1:55 ` Kay Sievers
2009-02-21 11:43 ` Andreas Robinson
2009-03-02 14:32 ` Andreas Robinson
2009-03-02 15:59 ` Kay Sievers [this message]
2009-03-02 16:20 ` Arjan van de Ven
2009-03-02 16:29 ` Kay Sievers
2009-03-02 18:27 ` Arjan van de Ven
2009-03-02 21:41 ` Andreas Robinson
2009-03-04 18:47 ` Andreas Robinson
2009-03-06 0:18 ` Arjan van de Ven
2009-03-06 15:15 ` Andreas Robinson
2009-03-06 15:45 ` Arjan van de Ven
2009-03-08 10:47 ` Andreas Robinson
2009-03-08 16:01 ` Arjan van de Ven
2009-03-08 20:13 ` [PATCH] sata_nv: add a module parameter to enable async scanning Andreas Robinson
2009-03-09 17:12 ` [RFC PATCH 0/6] module, kbuild: Faster boot with custom kernel Will Newton
2009-03-06 7:05 ` fastboot kernel parameter Sitsofe Wheeler
2009-03-06 11:23 ` Arjan van de Ven
2009-02-24 1:27 ` [RFC PATCH 0/6] module, kbuild: Faster boot with custom kernel Rusty Russell
2009-02-18 11:57 ` Rusty Russell
2009-02-18 13:57 ` Kay Sievers
2009-02-19 11:15 ` Rusty Russell
2009-02-19 11:41 ` Kay Sievers
2009-02-19 20:48 ` Kay Sievers
2009-02-19 21:59 ` Kay Sievers
2009-02-20 0:58 ` Rusty Russell
2009-02-20 1:33 ` Kay Sievers
2009-02-24 1:39 ` Rusty Russell
2009-02-20 11:32 ` Rusty Russell
2009-02-23 16:42 ` Kay Sievers
2009-02-25 7:03 ` Rusty Russell
2009-02-25 18:12 ` Kay Sievers
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=ac3eb2510903020759j100adbdekf2067d142adbfbbe@mail.gmail.com \
--to=kay.sievers@vrfy.org \
--cc=andr345@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=rusty@rustcorp.com.au \
--cc=sam@ravnborg.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®