-
Notifications
You must be signed in to change notification settings - Fork 102
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
Crash when cursor rapidly changes #1347
Comments
Another, similar trace:
|
wmww
added a commit
that referenced
this issue
Apr 2, 2020
1348: Safely show and hide software cursor r=AlanGriffiths a=wmww Fixes #1347 1355: Enable `--display-config` for Mir-on-X r=wmww a=AlanGriffiths Enable `--display-config` for Mir-on-X. (Fixes: #1326) Note: if you test, using "--display config clone" with this exposes the race that #1354 addresses. Co-authored-by: William Wold <wm@wmww.sh> Co-authored-by: Alan Griffiths <alan@octopull.co.uk>
wmww
added a commit
that referenced
this issue
Apr 3, 2020
1348: Safely show and hide software cursor r=AlanGriffiths a=wmww Fixes #1347 1355: Enable `--display-config` for Mir-on-X r=wmww a=AlanGriffiths Enable `--display-config` for Mir-on-X. (Fixes: #1326) Note: if you test, using "--display config clone" with this exposes the race that #1354 addresses. Co-authored-by: William Wold <wm@wmww.sh> Co-authored-by: Alan Griffiths <alan@octopull.co.uk>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
To reproduce (convoluted, I know):
params->server_side_decorated = true;
toWindowWlSurfaceRole::create_scene_surface()
to force server side decorations on Wayland appsweston-transformed
(which has no cursor)Expected: it works
Actual: it crashes after a while with
I suspect the problem has to do with insuficient locking in
mg::SoftwareCursor::hide()
Stack trace:
The text was updated successfully, but these errors were encountered: