public inbox for openembedded-core@lists.openembedded.org
 help / color / mirror / Atom feed
* [PATCH 1/1] initramfs-live-boot: Make sure we kill udev before switching root when live booting
       [not found] <cover.1469570866.git.alejandro.hernandez@linux.intel.com>
@ 2016-07-26 22:10 ` Alejandro Hernandez
  2016-08-15 23:57   ` Saul Wold
  0 siblings, 1 reply; 2+ messages in thread
From: Alejandro Hernandez @ 2016-07-26 22:10 UTC (permalink / raw)
  To: openembedded-core

When live booting, we need to make sure the running udev processes are killed
to avoid unexepected behavior, we do this just before switching root,
once we do, a new udev process will be spawned from init and will take care
of whatever work was still missing

[YOCTO #9520]

Signed-off-by: Alejandro Hernandez <alejandro.hernandez@linux.intel.com>
---
 meta/recipes-core/initrdscripts/files/init-live.sh | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-core/initrdscripts/files/init-live.sh b/meta/recipes-core/initrdscripts/files/init-live.sh
index 09fb479..441b41c 100644
--- a/meta/recipes-core/initrdscripts/files/init-live.sh
+++ b/meta/recipes-core/initrdscripts/files/init-live.sh
@@ -80,7 +80,9 @@ read_args() {
 boot_live_root() {
     # Watches the udev event queue, and exits if all current events are handled
     udevadm settle --timeout=3 --quiet
-    killall "${_UDEV_DAEMON##*/}" 2>/dev/null
+    # Kills the current udev running processes, which survived after
+    # device node creation events were handled, to avoid unexpected behavior
+    killall -9 "${_UDEV_DAEMON##*/}" 2>/dev/null
 
     # Allow for identification of the real root even after boot
     mkdir -p  ${ROOT_MOUNT}/media/realroot
-- 
2.6.6



^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH 1/1] initramfs-live-boot: Make sure we kill udev before switching root when live booting
  2016-07-26 22:10 ` [PATCH 1/1] initramfs-live-boot: Make sure we kill udev before switching root when live booting Alejandro Hernandez
@ 2016-08-15 23:57   ` Saul Wold
  0 siblings, 0 replies; 2+ messages in thread
From: Saul Wold @ 2016-08-15 23:57 UTC (permalink / raw)
  To: Alejandro Hernandez, openembedded-core

On Tue, 2016-07-26 at 22:10 +0000, Alejandro Hernandez wrote:
> When live booting, we need to make sure the running udev processes
> are killed
> to avoid unexepected behavior, we do this just before switching root,
> once we do, a new udev process will be spawned from init and will
> take care
> of whatever work was still missing
> 
> [YOCTO #9520]
> 
> Signed-off-by: Alejandro Hernandez
> <alejandro.hernandez@linux.intel.com>
Reviewed-by: Saul Wold <sgw@linux.intel.com>

> ---
>  meta/recipes-core/initrdscripts/files/init-live.sh | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/meta/recipes-core/initrdscripts/files/init-live.sh
> b/meta/recipes-core/initrdscripts/files/init-live.sh
> index 09fb479..441b41c 100644
> --- a/meta/recipes-core/initrdscripts/files/init-live.sh
> +++ b/meta/recipes-core/initrdscripts/files/init-live.sh
> @@ -80,7 +80,9 @@ read_args() {
>  boot_live_root() {
>      # Watches the udev event queue, and exits if all current events
> are handled
>      udevadm settle --timeout=3 --quiet
> -    killall "${_UDEV_DAEMON##*/}" 2>/dev/null
> +    # Kills the current udev running processes, which survived after
> +    # device node creation events were handled, to avoid unexpected
> behavior
> +    killall -9 "${_UDEV_DAEMON##*/}" 2>/dev/null
>  
>      # Allow for identification of the real root even after boot
>      mkdir -p  ${ROOT_MOUNT}/media/realroot
> -- 
> 2.6.6
> 


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2016-08-15 23:57 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <cover.1469570866.git.alejandro.hernandez@linux.intel.com>
2016-07-26 22:10 ` [PATCH 1/1] initramfs-live-boot: Make sure we kill udev before switching root when live booting Alejandro Hernandez
2016-08-15 23:57   ` Saul Wold

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox