Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Avoid race condition between graphics driver and lightdm #237

Merged
merged 1 commit into from
Jun 29, 2022

Conversation

gsilvan
Copy link
Contributor

@gsilvan gsilvan commented Mar 9, 2022

There is a possible race condition where lightdm fails to a black screen/tty, if lightdm loads faster than the system graphics driver. There is already the setting logind-check-graphical but it is unset per default.
As more and more people run into this pitfail, this PR sets logind-check-graphical=True.

References
https://wiki.archlinux.org/title/LightDM#LightDM_does_not_appear_or_monitor_only_displays_TTY_output
https://www.reddit.com/r/linuxmasterrace/comments/p9ebi0/logindcheckgraphicaltrue/
https://askubuntu.com/questions/1360285/login-screen-issue-first-lightdm-needs-to-be-killed-to-get-x-session-to-work
#165

@github-actions
Copy link

github-actions bot commented Mar 9, 2022

Everyone contributing to this PR have now signed the CLA. Thanks!

@gsilvan
Copy link
Contributor Author

gsilvan commented Mar 9, 2022

signed

@qwertychouskie
Copy link
Contributor

CI needs re-trigger but the change itself checks out.

Without `logind-check-graphical=True` beeing set, lightdm is prone to a race condition, where lightdm has started before the graphics driver was loaded.
@robert-ancell
Copy link
Collaborator

Looks like in 2014 this feature was unreliable. I'd hope that is no longer the case!

@robert-ancell
Copy link
Collaborator

Tests passed locally, so this should be good to land, thanks @gsilvan!

@robert-ancell robert-ancell merged commit 77a7c6b into canonical:main Jun 29, 2022
@tangruize
Copy link

tangruize commented Jul 30, 2022

It seems that logind-check-graphical=true causes LightDM in LXC container (Fedora 36) failed to start. I created an issue #263 .

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.

4 participants