Summary: | Crash when enabling DPMS | ||
---|---|---|---|
Product: | [Plasma] kwin | Reporter: | Martin Flöser <mgraesslin> |
Component: | libinput | Assignee: | KWin default assignee <kwin-bugs-null> |
Status: | RESOLVED FIXED | ||
Severity: | crash | CC: | subdiff |
Priority: | VHI | Flags: | mgraesslin:
Wayland+
|
Version: | git master | ||
Target Milestone: | --- | ||
Platform: | Other | ||
OS: | Linux | ||
Latest Commit: | https://commits.kde.org/kwin/b868df2715d2b81a9d7f556ef57de5fb8aa99baa | Version Fixed In: | |
Sentry Crash Report: |
Description
Martin Flöser
2017-01-04 18:08:01 UTC
Since you thought a few days ago the org.kde.KWin.InputDeviceManager interface is the problem because of the backtrace, something else: When issuing kwin_wayland --xwayland from terminal, it gives a segfault. Removing the registerObject of the interface in connection.cpp solves this. So there is indeed a problem with the current implementation, which seems to be related to the multiple threads in the Connection class. Atleast there are correlated warning messages spamed to the log. What if we move the register into the doInit method which is called in the connection thread? Currently the register is done from ctor which is main thread. Didn't work for me. But try this out instead: https://phabricator.kde.org/D4027 Git commit b868df2715d2b81a9d7f556ef57de5fb8aa99baa by Roman Gilg. Committed on 09/01/2017 at 08:07. Pushed by romangilg into branch 'master'. [libinput] Fix threaded Connection DBus object registering On registering object use the object in the right thread. Otherwise there are multiple errors. Differential Revision: https://phabricator.kde.org/D4027 M +1 -1 libinput/connection.cpp https://commits.kde.org/kwin/b868df2715d2b81a9d7f556ef57de5fb8aa99baa Just tested with latest neon and I can turn the screen off again! Thanks a lot! |