[PATCH 1/2] Revert "add option --ignore-kernel-modules"

[PATCH 1/2] Revert "add option --ignore-kernel-modules"

[Amadeusz Żołnowski]

This reverts commit 0ca3a5ee849614e85241274d24380f3eabd73b78
and removes later changes related to this commit.

Conflicts:

	dracut
	dracut.8
	modules.d/99base/install
---
 NEWS                     |    1 +
 dracut                   |   10 +---------
 dracut.8.xml             |    9 ---------
 modules.d/99base/install |   12 ++----------
 4 files changed, 4 insertions(+), 28 deletions(-)

diff --git a/NEWS b/NEWS
index 766867d..a2a7c7b 100644
--- a/NEWS
+++ b/NEWS
@@ -1,5 +1,6 @@
 dracut-008
 ==========
+- removed --ignore-kernel-modules option (no longer necessary)
 
 dracut-007
 ==========
diff --git a/dracut b/dracut
index 9c1223a..303d817 100755
--- a/dracut
+++ b/dracut
@@ -49,10 +49,6 @@ Creates initial ramdisk images for preloading modules
                         firmwares, separated by :
   --kernel-only         Only install kernel drivers and firmware files
   --no-kernel           Do not install kernel drivers and firmware files
-  --ignore-kernel-modules
-                        Don't try to load modules. It automatically implies
-                         '--no-kernel'. It's assumed that everything needed
-                         is built into kernel.
   --strip               Strip binaries in the initramfs
   --nostrip             Do not strip binaries in the initramfs (default)
   --mdadmconf           Include local /etc/mdadm.conf
@@ -147,10 +143,6 @@ while (($# > 0)); do
             type pigz > /dev/null 2>&1 && compress="pigz -9" || \
                 compress="gzip -9"
             fi;;
-        --ignore-kernel-modules) kernel_only="no"; no_kernel="yes"
-            ignore_kmodules="yes"
-            omit_dracutmodules_l+=\ kernel-modules
-            ;;
         --list-modules)
             do_list="yes";
             ;;
@@ -304,7 +296,7 @@ chmod 755 "$initdir"
 
 export initdir hookdirs dracutbasedir dracutmodules drivers \
     fw_dir drivers_dir debug beverbose no_kernel kernel_only \
-    add_drivers mdadmconf lvmconf filesystems ignore_kmodules \
+    add_drivers mdadmconf lvmconf filesystems \
     use_fstab libdir usrlibdir
 
 if [[ $kernel_only != yes ]]; then
diff --git a/dracut.8.xml b/dracut.8.xml
index a27c6e9..0931f48 100644
--- a/dracut.8.xml
+++ b/dracut.8.xml
@@ -204,15 +204,6 @@ include in the generic initramfs. This parameter can be specified multiple times
         </varlistentry>
         <varlistentry>
           <term>
-            <option>--ignore-kernel-modules</option>
-          </term>
-          <listitem>
-            <para>don&apos;t try to load modules; it automatically implies <option>--no-kernel</option>; it&apos;s
-assumed that everything needed is built into kernel</para>
-          </listitem>
-        </varlistentry>
-        <varlistentry>
-          <term>
             <option>--mdadmconf</option>
           </term>
           <listitem>
diff --git a/modules.d/99base/install b/modules.d/99base/install
index 6fac851..aa897e8 100755
--- a/modules.d/99base/install
+++ b/modules.d/99base/install
@@ -1,15 +1,8 @@
 # -*- mode: shell-script; indent-tabs-mode: nil; sh-basic-offset: 4; -*-
 # ex: ts=8 sw=4 sts=4 et filetype=sh
 
-dracut_install mount mknod mkdir pidof sleep chroot sed ls flock cp mv dmesg \
-    rm ln mkfifo less 
-if [[ ${ignore_kmodules} = yes ]]; then
-    inst /bin/true
-    ln -s /bin/true "${initdir}"/sbin/modprobe
-    ln -s /bin/true "${initdir}"/sbin/rmmod
-else
-    dracut_install modprobe rmmod
-fi
+dracut_install mount mknod mkdir modprobe pidof sleep chroot \
+    sed ls flock cp mv dmesg rm ln rmmod mkfifo less 
 if [ ! -e "${initdir}/bin/sh" ]; then
     dracut_install bash
     (ln -s bash "${initdir}/bin/sh" || :)
@@ -33,4 +26,3 @@ inst "$moddir/dracut-lib.sh" "/lib/dracut-lib.sh"
 inst_hook cmdline 10 "$moddir/parse-root-opts.sh"
 inst_hook cmdline 20 "$moddir/parse-blacklist.sh"
 mkdir -p "${initdir}/var/run"
-
-- 
1.7.3.1

[PATCH 2/2] suppress modprobe errors on builtins (credits to Kay Sievers)

[Amadeusz Żołnowski]

Install /lib/modules/$kv/modules.builtin.bin to suppress modprobe error
messages saying module was not found, while it's built-in.

Credits go to Kay Sievers who enlighten us about meaning of this cool
file.
---
 modules.d/90kernel-modules/install |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/modules.d/90kernel-modules/install b/modules.d/90kernel-modules/install
index 9810d12..863ffac 100755
--- a/modules.d/90kernel-modules/install
+++ b/modules.d/90kernel-modules/install
@@ -4,3 +4,4 @@
 [ -f /etc/modprobe.conf ] && dracut_install /etc/modprobe.conf
 dracut_install $(find /etc/modprobe.d/ -type f -name '*.conf')
 inst_hook cmdline 01 "$moddir/parse-kernel.sh"
+inst "$srcmods/modules.builtin.bin"
-- 
1.7.3.1

Re: [PATCH 1/2] Revert "add option --ignore-kernel-modules"

[Harald Hoyer]

On 10/13/2010 07:39 PM, Amadeusz Żołnowski wrote:
> This reverts commit 0ca3a5ee849614e85241274d24380f3eabd73b78
> and removes later changes related to this commit.
>
> Conflicts:
>
> 	dracut
> 	dracut.8
> 	modules.d/99base/install
> ---
>   NEWS                     |    1 +
>   dracut                   |   10 +---------
>   dracut.8.xml             |    9 ---------
>   modules.d/99base/install |   12 ++----------
>   4 files changed, 4 insertions(+), 28 deletions(-)
>
> diff --git a/NEWS b/NEWS
> index 766867d..a2a7c7b 100644
> --- a/NEWS
> +++ b/NEWS
> @@ -1,5 +1,6 @@
>   dracut-008
>   ==========
> +- removed --ignore-kernel-modules option (no longer necessary)
>
>   dracut-007
>   ==========
> diff --git a/dracut b/dracut
> index 9c1223a..303d817 100755
> --- a/dracut
> +++ b/dracut
> @@ -49,10 +49,6 @@ Creates initial ramdisk images for preloading modules
>                           firmwares, separated by :
>     --kernel-only         Only install kernel drivers and firmware files
>     --no-kernel           Do not install kernel drivers and firmware files
> -  --ignore-kernel-modules
> -                        Don't try to load modules. It automatically implies
> -                         '--no-kernel'. It's assumed that everything needed
> -                         is built into kernel.
>     --strip               Strip binaries in the initramfs
>     --nostrip             Do not strip binaries in the initramfs (default)
>     --mdadmconf           Include local /etc/mdadm.conf
> @@ -147,10 +143,6 @@ while (($#>  0)); do
>               type pigz>  /dev/null 2>&1&&  compress="pigz -9" || \
>                   compress="gzip -9"
>               fi;;
> -        --ignore-kernel-modules) kernel_only="no"; no_kernel="yes"
> -            ignore_kmodules="yes"
> -            omit_dracutmodules_l+=\ kernel-modules
> -            ;;
>           --list-modules)
>               do_list="yes";
>               ;;
> @@ -304,7 +296,7 @@ chmod 755 "$initdir"
>
>   export initdir hookdirs dracutbasedir dracutmodules drivers \
>       fw_dir drivers_dir debug beverbose no_kernel kernel_only \
> -    add_drivers mdadmconf lvmconf filesystems ignore_kmodules \
> +    add_drivers mdadmconf lvmconf filesystems \
>       use_fstab libdir usrlibdir
>
>   if [[ $kernel_only != yes ]]; then
> diff --git a/dracut.8.xml b/dracut.8.xml
> index a27c6e9..0931f48 100644
> --- a/dracut.8.xml
> +++ b/dracut.8.xml
> @@ -204,15 +204,6 @@ include in the generic initramfs. This parameter can be specified multiple times
>           </varlistentry>
>           <varlistentry>
>             <term>
> -<option>--ignore-kernel-modules</option>
> -</term>
> -<listitem>
> -<para>don&apos;t try to load modules; it automatically implies<option>--no-kernel</option>; it&apos;s
> -assumed that everything needed is built into kernel</para>
> -</listitem>
> -</varlistentry>
> -<varlistentry>
> -<term>
>               <option>--mdadmconf</option>
>             </term>
>             <listitem>
> diff --git a/modules.d/99base/install b/modules.d/99base/install
> index 6fac851..aa897e8 100755
> --- a/modules.d/99base/install
> +++ b/modules.d/99base/install
> @@ -1,15 +1,8 @@
>   # -*- mode: shell-script; indent-tabs-mode: nil; sh-basic-offset: 4; -*-
>   # ex: ts=8 sw=4 sts=4 et filetype=sh
>
> -dracut_install mount mknod mkdir pidof sleep chroot sed ls flock cp mv dmesg \
> -    rm ln mkfifo less
> -if [[ ${ignore_kmodules} = yes ]]; then
> -    inst /bin/true
> -    ln -s /bin/true "${initdir}"/sbin/modprobe
> -    ln -s /bin/true "${initdir}"/sbin/rmmod
> -else
> -    dracut_install modprobe rmmod
> -fi
> +dracut_install mount mknod mkdir modprobe pidof sleep chroot \
> +    sed ls flock cp mv dmesg rm ln rmmod mkfifo less
>   if [ ! -e "${initdir}/bin/sh" ]; then
>       dracut_install bash
>       (ln -s bash "${initdir}/bin/sh" || :)
> @@ -33,4 +26,3 @@ inst "$moddir/dracut-lib.sh" "/lib/dracut-lib.sh"
>   inst_hook cmdline 10 "$moddir/parse-root-opts.sh"
>   inst_hook cmdline 20 "$moddir/parse-blacklist.sh"
>   mkdir -p "${initdir}/var/run"
> -

pushed

Re: [PATCH 2/2] suppress modprobe errors on builtins (credits to Kay Sievers)

[Harald Hoyer]

On 10/13/2010 07:39 PM, Amadeusz Żołnowski wrote:
> Install /lib/modules/$kv/modules.builtin.bin to suppress modprobe error
> messages saying module was not found, while it's built-in.
>
> Credits go to Kay Sievers who enlighten us about meaning of this cool
> file.
> ---
>   modules.d/90kernel-modules/install |    1 +
>   1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/modules.d/90kernel-modules/install b/modules.d/90kernel-modules/install
> index 9810d12..863ffac 100755
> --- a/modules.d/90kernel-modules/install
> +++ b/modules.d/90kernel-modules/install
> @@ -4,3 +4,4 @@
>   [ -f /etc/modprobe.conf ]&&  dracut_install /etc/modprobe.conf
>   dracut_install $(find /etc/modprobe.d/ -type f -name '*.conf')
>   inst_hook cmdline 01 "$moddir/parse-kernel.sh"
> +inst "$srcmods/modules.builtin.bin"

pushed