wlseeds

History

Simon Ser cc8bc0db20 backend/drm: stop restoring CRTCs on exit This is the cause of the spurious "drmHandleEvent failed" messages at exit. restore_drm_outputs calls handle_drm_event in a loop without checking whether the FD is readable, so drmHandleEvent ends up with a short read (0 bytes) and returns an error. The loop's goal is to wait for all queued page-flip events to complete, to allow drmModeSetCrtc calls to succeed without EBUSY. The drmModeSetCrtc calls are supposed to restore whatever KMS state we were started with. But it's not clear from my PoV that restoring the KMS state on exit is desirable. KMS clients are supposed to save and restore the (full) KMS state on VT switch, but not on exit. Leaving our KMS state on exit avoids unnecessary modesets and allows flicker-free transitions between clients. See [1] for more details, and note that with Pekka we've concluded that a new flag to reset some KMS props to their default value on compositor start-up is the best way forward. As a side note, Weston doesn't restore the CRTC by does disable the cursor plane on exit (see drm_output_deinit_planes, I still think disabling the cursor plane shouldn't be necessary on exit). Additionally, restore_drm_outputs only a subset of the KMS state. Gamma and other atomic properties aren't accounted for. If the previous KMS client had some outputs disabled, restore_drm_outputs would restore a garbage mode. [1]: https://blog.ffwll.ch/2016/01/vt-switching-with-atomic-modeset.html		3 years ago
..
backend	backend/drm: stop restoring CRTCs on exit	3 years ago
render	render/gles2: rename wlr_egl.exts to better match Khronos	3 years ago
types	buffer: re-use wlr_shm_client_buffer	3 years ago
util	util/array: add array_remove_at	3 years ago
wlr	input/pointer: send axis source once per frame	3 years ago
xcursor	xcursor: make cursor data and metadata const	4 years ago
xwayland	xwm: implement _NET_CLIENT_LIST_STACKING	4 years ago
meson.build	render: make GLES2 renderer optional	4 years ago