[PATCH 0/2] sysvinit: adjust the boot sequence and fix missing of boot log

[PATCH 0/2] sysvinit: adjust the boot sequence and fix missing of boot log

[Qi.Chen]

From: Chen Qi <Qi.Chen@windriver.com>

The following changes since commit f3541226b8b1187e79dec0f6f9f3c58cedf9ac9b:

  bitbake: hob: do not display the "Package list may be incomplete!" dialog (2013-11-01 17:59:31 +0000)

are available in the git repository at:

  git://git.pokylinux.org/poky-contrib ChenQi/sysvinit-bootlogd
  http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=ChenQi/sysvinit-bootlogd

Chen Qi (2):
  bootlogd: create log file if not present
  sysvinit: adjust boot sequence and remove hack from udev

 meta/recipes-core/initscripts/initscripts_1.0.bb   |    2 +-
 meta/recipes-core/sysvinit/sysvinit/bootlogd.init  |    4 ++--
 meta/recipes-core/udev/udev.inc                    |    2 +-
 meta/recipes-core/udev/udev/init                   |    4 +++-
 .../modutils-initscripts/modutils-initscripts.bb   |    2 +-
 5 files changed, 8 insertions(+), 6 deletions(-)

-- 
1.7.9.5

[PATCH 1/2] bootlogd: create log file if not present

[Qi.Chen]

From: Chen Qi <qi.chen@windriver.com>

Previously, our system had no boot log even if the bootlogd daemon was
started correctly. The root cause is that the log file doesn't exist
when starting the bootlogd.

Add '-c' option to bootlogd so that it will create the boot log if
it doesn't exist.

[YOCTO #5273]

Signed-off-by: Chen Qi <Qi.Chen@windriver.com>
---
 meta/recipes-core/sysvinit/sysvinit/bootlogd.init |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-core/sysvinit/sysvinit/bootlogd.init b/meta/recipes-core/sysvinit/sysvinit/bootlogd.init
index 7b87827..b181357 100755
--- a/meta/recipes-core/sysvinit/sysvinit/bootlogd.init
+++ b/meta/recipes-core/sysvinit/sysvinit/bootlogd.init
@@ -47,9 +47,9 @@ case "$ACTION" in
 		then
 			umask 027
 			start-stop-daemon --start --quiet \
-				--exec $DAEMON -- -r
+				--exec $DAEMON -- -r -c
 		else
-			$DAEMON -r
+			$DAEMON -r -c
 		fi
 		echo "$NAME."
 		;;
-- 
1.7.9.5

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

[Qi.Chen]

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.

[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
 
-- 
1.7.9.5

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

[Mark Hatle]

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
>
>