Skip to content

Fix kmscon not restarting the login shell#96

Merged
Aetf merged 1 commit intoAetf:developfrom
kdj0c:sigchld_fix
Jan 9, 2025
Merged

Fix kmscon not restarting the login shell#96
Aetf merged 1 commit intoAetf:developfrom
kdj0c:sigchld_fix

Conversation

@kdj0c
Copy link
Collaborator

@kdj0c kdj0c commented Jan 6, 2025

Sometime the SIGCHLD handler is not called, but kmscon still receives a HUP on the pty file descriptor. In this case, reap the child, and spawn another login process, to avoid hanging for ever on a defunct child.

Fix #82
Fix #80

@kdj0c kdj0c mentioned this pull request Jan 6, 2025
jflanglois added a commit to jflanglois/debug-kmscon that referenced this pull request Jan 6, 2025
Copy link
Owner

@Aetf Aetf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR!

Sometime the SIGCHLD handler is not called, but kmscon still receives
a HUP on the pty file descriptor. In this case, reap the child, and
spawn another login process, to avoid hanging for ever on a defunct
child.

Signed-off-by: Jocelyn Falempe <[email protected]>
@Aetf Aetf merged commit a81941f into Aetf:develop Jan 9, 2025
Mop-u added a commit to Mop-u/dotfiles that referenced this pull request Jan 10, 2025
@Vogtinator
Copy link

Sometime the SIGCHLD handler is not called

The reason for that is that SIGCHLD is not blocked before pango creates its own glib thread. I made a RFC fix here: #106

@kdj0c kdj0c deleted the sigchld_fix branch October 10, 2025 09:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Freeze issue on logout / CTRL+d kmscon enters zombie state

3 participants