All of lore.kernel.org
 help / color / mirror / Atom feed
From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: Laurentiu Palcu <laurentiu.palcu@intel.com>
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [PATCH 0/6] Remove obsolete update-modules
Date: Fri, 18 Jan 2013 13:10:13 +0000	[thread overview]
Message-ID: <1358514613.27799.25.camel@ted> (raw)
In-Reply-To: <cover.1358432402.git.laurentiu.palcu@intel.com>

On Thu, 2013-01-17 at 16:58 +0200, Laurentiu Palcu wrote:
> All,
> 
> Working on the postinstall improvements I stumbled over update-modules
> script which postponed postinstalls for all the kernel or kernel module
> packages for first boot. However, after some investigations I saw that this
> script is pretty much obsolete. So, this patchset will remove it from all
> recipes and bbclasses it was referenced.
> 
> Here's why it's useless now:
> 
> update-modules was used mainly to create the /etc/modules.conf file, out of files
> in /etc/modutils, and /etc/modules file from /etc/modules-load.d/*.conf files.
> Then it just ran depmod -A.
> 
> * depmod can be run in the postinst/postrm on its own, no need to run through update-modules;
> * /etc/modules.conf is not used anymore by modprobe. modprobe now looks in
>   /etc/modprobe.d/ directory for *.conf files;
> * /etc/modules was used by /etc/init.d/modutils.sh to automatically load
>   modules at boot. However, I improved the script in order to also look into
>   /etc/modules-load.d/ directory and load all the modules listed there that were not
>   already loaded (in case /etc/modules existed);
> 
> I know this is a major change and will affect a lot of people. But, in order
> to support RO rootfs and run all the postinstalls on host, this change was
> kind of necessary. Note, though, that the update-modules recipe was not removed
> yet. It will be removed eventually but, for now, I decided to leave it in place
> in case some people are nostalgic and want to still use it for a while.

I've merged these, I'll also take a patch to remove update-modules
itself.

Cheers,

Richard




      parent reply	other threads:[~2013-01-18 13:25 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-01-17 14:58 [PATCH 0/6] Remove obsolete update-modules Laurentiu Palcu
2013-01-17 14:58 ` [PATCH 1/6] modutils-initscripts: improve modutils.sh Laurentiu Palcu
2013-01-17 14:58 ` [PATCH 2/6] orinoco-conf: remove dependencies of update-modules Laurentiu Palcu
2013-01-17 14:58 ` [PATCH 3/6] hostap-conf: " Laurentiu Palcu
2013-01-17 14:58 ` [PATCH 4/6] module.bbclass: do not use update-modules anymore Laurentiu Palcu
2013-01-17 14:58 ` [PATCH 5/6] kernel.bbclass: remove references to update-modules Laurentiu Palcu
2013-01-17 15:26   ` Bruce Ashfield
2013-01-22 20:48   ` Darren Hart
2013-01-23  8:22     ` Laurentiu Palcu
2013-01-23 16:24       ` Darren Hart
2013-01-17 14:58 ` [PATCH 6/6] bitbake.conf: remove update-modules from DISTRO_FEATURES Laurentiu Palcu
2013-01-18 13:10 ` Richard Purdie [this message]

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=1358514613.27799.25.camel@ted \
    --to=richard.purdie@linuxfoundation.org \
    --cc=laurentiu.palcu@intel.com \
    --cc=openembedded-core@lists.openembedded.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.