Hi Ricardo,

I tested your patch and it's ok another improvement to it will be add a sleep 1 before chvt because killproc xinit don't take a defined time to shutdown and chvt didn't work because
X it's running.

I'll modify a little your patch and send v2 with your signature.

Regards,
    alimon

On 20/05/15 03:34, Ricardo Ribalda Delgado wrote:
ping?

On Mon, May 11, 2015 at 5:23 PM, Ricardo Ribalda Delgado
<[email protected]> wrote:
If reboot was called from inside the Xserver there could happen a race
condition where chvt would never end, and therefore the whole system was
stalled.

The user could not recover the system by ssh the machine or using the
keyboard.

Running chvt in background fixes the issue.

Signed-off-by: Ricardo Ribalda Delgado <[email protected]>
---
  meta/recipes-graphics/x11-common/xserver-nodm-init/xserver-nodm | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-graphics/x11-common/xserver-nodm-init/xserver-nodm 
b/meta/recipes-graphics/x11-common/xserver-nodm-init/xserver-nodm
index 2e3e09077792..31f545f79199 100755
--- a/meta/recipes-graphics/x11-common/xserver-nodm-init/xserver-nodm
+++ b/meta/recipes-graphics/x11-common/xserver-nodm-init/xserver-nodm
@@ -48,7 +48,7 @@ case "$1" in
    stop)
          echo "Stopping XServer"
          killproc xinit
-        chvt 1
+        chvt 1 &
    ;;

    restart)
--
2.1.4




--
_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Reply via email to