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

Session closed on external monitor plug #54

Closed
davc0n opened this issue Aug 19, 2024 · 16 comments
Closed

Session closed on external monitor plug #54

davc0n opened this issue Aug 19, 2024 · 16 comments

Comments

@davc0n
Copy link

davc0n commented Aug 19, 2024

Hello,

With version 0.3.2 the first time an external monitor is plugged the session gets closed:

Aug 19 09:39:08 thinkpad login[569]: pam_unix(login:session): session closed for user davide
Aug 19 09:39:08 thinkpad login[569]: pam_systemd(login:session): New sd-bus connection (system-bus-pam-systemd-569) opened.
Aug 19 09:39:08 thinkpad systemd[1]: [email protected]: Deactivated successfully.
Aug 19 09:39:08 thinkpad systemd[1]: run-credentials-getty\x40tty1.service.mount: Deactivated successfully.
Aug 19 09:39:08 thinkpad systemd[1]: session-1.scope: Deactivated successfully.
Aug 19 09:39:08 thinkpad systemd[1]: session-1.scope: Consumed 2.190s CPU time, 341.1M memory peak.
Aug 19 09:39:08 thinkpad systemd-logind[477]: Session 1 logged out. Waiting for processes to exit.
Aug 19 09:39:08 thinkpad systemd[1]: [email protected]: Scheduled restart job, restart counter is at 1.
Aug 19 09:39:08 thinkpad systemd[1]: Started Getty on tty1.
Aug 19 09:39:08 thinkpad systemd-logind[477]: Removed session 1.
Aug 19 09:39:08 thinkpad login[815]: pam_systemd_home(login:auth): New sd-bus connection (system-bus-pam-systemd-home-815) opened.

Issue is not present in 0.3.1.

@vaxerski
Copy link
Member

can you bisect

@sp0nks
Copy link

sp0nks commented Aug 19, 2024

I hope this is it:

5893461 is the first bad commit
commit 5893461 (HEAD)
Author: Vaxry [email protected]
Date: Thu Aug 15 16:54:05 2024 +0100

DRM: Allow checking no-crtc connectors (#49)

include/aquamarine/backend/DRM.hpp | 2 +-
src/backend/drm/DRM.cpp | 16 ++++++----------
2 files changed, 7 insertions(+), 11 deletions(-)

@vaxerski
Copy link
Member

and what does "session gets closed" mean? crash?

@sp0nks
Copy link

sp0nks commented Aug 19, 2024

Yes.

I start Hyprland with only one screen, then a while Hyprland is running I plug in my external screen and Hyprland dies.

@vaxerski
Copy link
Member

can you attach a crash report from latest aquamarine-git with latest hyprland-git?

@sp0nks
Copy link

sp0nks commented Aug 19, 2024

Attached is what I could get.

The log and the crash weren't from the same run.

hyprland.log
hyprlandCrashReport657129.txt

@vaxerski
Copy link
Member

this crash report looks outdated. That issue was fixed in 0.3.2 iirc

@vaxerski
Copy link
Member

vaxerski commented Aug 19, 2024

and I don't see anything wrong in the log? (Except possibly one monitor being a bit wonk)

@sp0nks
Copy link

sp0nks commented Aug 19, 2024

Apologies. Here is a fresh crash log.
hyprlandCrashReport783249.txt

@vaxerski
Copy link
Member

can you build aquamarine in debug and get the crash report again

@sp0nks
Copy link

sp0nks commented Aug 19, 2024

I am going to try, will be back soon...

@sp0nks
Copy link

sp0nks commented Aug 19, 2024

I'm not sure if I did it right, but here is the crash log
hyprlandCrashReport824122.txt

And I also got this:
coredump.txt

@davc0n
Copy link
Author

davc0n commented Aug 19, 2024

and what does "session gets closed" mean? crash?

Logout (e.g. loginctl terminate-session 1).
Hyprland does not crash.

can you bisect

Maybe next weekend.

vaxerski added a commit that referenced this issue Aug 20, 2024
@vaxerski
Copy link
Member

should be fixed

@vaxerski
Copy link
Member

pls verify

@davc0n
Copy link
Author

davc0n commented Aug 21, 2024

Fixed, thanks.

@davc0n davc0n closed this as completed Aug 21, 2024
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

No branches or pull requests

3 participants