Re: [PATCH 2/2] sysvinit: adjust boot sequence and remove hack from udev

[Mark Hatle <mark.hatle@windriver.com>]

On 11/2/13, 12:22 AM, Qi.Chen@windriver.com wrote:
> From: Chen Qi <qi.chen@windriver.com>
>
> Adjust the boot sequence in sysvinit based systems.
>
> The mountall.sh (mounting the local file system) needs to be started
> before udev and bootlogd.
>
> This patch makes mountall.sh start before udev and removes the hack of
> mounting tmpfs in the udev init script.
>
> This patch also adds some comments to the udev init script to make it
> clear why we create the '/var/volatile/tmp' directory.

Someone else might be able to comment on this.  Can you move mountall before 
udev?  I thought udev may be required in order to setup the hardware, or at 
least the device nodes to permit the mounting.

Mounting before bootlogd I can see.  Since w/o the mounts there may be nowhere 
to write to.

--Mark

> [YOCTO #5273]
>
> Signed-off-by: Chen Qi <Qi.Chen@windriver.com>
> ---
>   meta/recipes-core/initscripts/initscripts_1.0.bb   |    2 +-
>   meta/recipes-core/udev/udev.inc                    |    2 +-
>   meta/recipes-core/udev/udev/init                   |    4 +++-
>   .../modutils-initscripts/modutils-initscripts.bb   |    2 +-
>   4 files changed, 6 insertions(+), 4 deletions(-)
>
> diff --git a/meta/recipes-core/initscripts/initscripts_1.0.bb b/meta/recipes-core/initscripts/initscripts_1.0.bb
> index c2998c3..2d58266 100644
> --- a/meta/recipes-core/initscripts/initscripts_1.0.bb
> +++ b/meta/recipes-core/initscripts/initscripts_1.0.bb
> @@ -115,7 +115,7 @@ do_install () {
>   	update-rc.d -r ${D} save-rtc.sh start 25 0 6 .
>   	update-rc.d -r ${D} banner.sh start 02 S .
>   	update-rc.d -r ${D} checkroot.sh start 06 S .
> -	update-rc.d -r ${D} mountall.sh start 35 S .
> +	update-rc.d -r ${D} mountall.sh start 03 S .
>   	update-rc.d -r ${D} hostname.sh start 39 S .
>   	update-rc.d -r ${D} mountnfs.sh start 45 S .
>   	update-rc.d -r ${D} bootmisc.sh start 55 S .
> diff --git a/meta/recipes-core/udev/udev.inc b/meta/recipes-core/udev/udev.inc
> index 2ff8f00..02cab3b 100644
> --- a/meta/recipes-core/udev/udev.inc
> +++ b/meta/recipes-core/udev/udev.inc
> @@ -50,7 +50,7 @@ PACKAGES =+ "libgudev libgudev-dev libgudev-dbg"
>
>   INITSCRIPT_PACKAGES = "udev udev-cache"
>   INITSCRIPT_NAME_udev = "udev"
> -INITSCRIPT_PARAMS_udev = "start 03 S ."
> +INITSCRIPT_PARAMS_udev = "start 04 S ."
>   INITSCRIPT_NAME_udev-cache = "udev-cache"
>   INITSCRIPT_PARAMS_udev-cache = "start 36 S ."
>
> diff --git a/meta/recipes-core/udev/udev/init b/meta/recipes-core/udev/udev/init
> index e048a17..410a650 100644
> --- a/meta/recipes-core/udev/udev/init
> +++ b/meta/recipes-core/udev/udev/init
> @@ -54,7 +54,9 @@ case "$1" in
>       }
>       [ -e /dev/pts ] || mkdir -m 0755 /dev/pts
>       [ -e /dev/shm ] || mkdir -m 1777 /dev/shm
> -    mount -a -t tmpfs 2>/dev/null
> +    # the automount rule for udev needs /tmp directory available, as /tmp is a symlink
> +    # to /var/tmp which in turn is a symlink to /var/volatile/tmp, we need to make sure
> +    # /var/volatile/tmp directory to be available.
>       mkdir -p /var/volatile/tmp
>
>       # Cache handling.
> diff --git a/meta/recipes-kernel/modutils-initscripts/modutils-initscripts.bb b/meta/recipes-kernel/modutils-initscripts/modutils-initscripts.bb
> index 89d38cf..594e554 100644
> --- a/meta/recipes-kernel/modutils-initscripts/modutils-initscripts.bb
> +++ b/meta/recipes-kernel/modutils-initscripts/modutils-initscripts.bb
> @@ -7,7 +7,7 @@ SRC_URI = "file://modutils.sh \
>   PR = "r6"
>
>   INITSCRIPT_NAME = "modutils.sh"
> -INITSCRIPT_PARAMS = "start 4 S ."
> +INITSCRIPT_PARAMS = "start 05 S ."
>
>   inherit update-rc.d
>
>