From d4ffa5b7a6d751ae8161cc1d301efd66342c2975 Mon Sep 17 00:00:00 2001 From: emersion Date: Sat, 19 Jan 2019 10:14:01 +0100 Subject: [PATCH] backend/drm: fix state for outputs loosing their CRTC When there aren't enough CRTCs for all outputs, we try to move a CRTC from a disabled output to an enabled one. When this happens, the old output's state wasn't changed, so the compositor thought it was still enabled and rendering. This commit marks the old output as WLR_DRM_CONN_NEEDS_MODESET and sets its current mode to NULL. --- backend/drm/drm.c | 17 ++++++++++++++--- types/wlr_output.c | 8 ++++++-- 2 files changed, 20 insertions(+), 5 deletions(-) diff --git a/backend/drm/drm.c b/backend/drm/drm.c index 735b7c29..28ee41a5 100644 --- a/backend/drm/drm.c +++ b/backend/drm/drm.c @@ -391,6 +391,8 @@ static void attempt_enable_needs_modeset(struct wlr_drm_backend *drm) { if (conn->state == WLR_DRM_CONN_NEEDS_MODESET && conn->crtc != NULL && conn->desired_mode != NULL && conn->desired_enabled) { + wlr_log(WLR_DEBUG, "Output %s has a desired mode and a CRTC, " + "attempting a modeset", conn->output.name); drm_connector_set_mode(&conn->output, conn->desired_mode); } } @@ -637,7 +639,7 @@ static bool drm_connector_set_cursor(struct wlr_output *output, plane->surf.height, output->transform); struct wlr_box hotspot = { .x = hotspot_x, .y = hotspot_y }; - wlr_box_transform(&hotspot, &hotspot, + wlr_box_transform(&hotspot, &hotspot, wlr_output_transform_invert(output->transform), plane->surf.width, plane->surf.height); @@ -981,8 +983,17 @@ static void realloc_crtcs(struct wlr_drm_backend *drm, bool *changed_outputs) { i++; struct wlr_output_mode *mode = conn->output.current_mode; - if (conn->state != WLR_DRM_CONN_CONNECTED || !changed_outputs[i] - || conn->crtc == NULL) { + if (conn->state != WLR_DRM_CONN_CONNECTED || !changed_outputs[i]) { + continue; + } + + if (conn->crtc == NULL) { + wlr_log(WLR_DEBUG, "Output has %s lost its CRTC", + conn->output.name); + conn->state = WLR_DRM_CONN_NEEDS_MODESET; + wlr_output_update_enabled(&conn->output, false); + conn->desired_mode = conn->output.current_mode; + wlr_output_update_mode(&conn->output, NULL); continue; } diff --git a/types/wlr_output.c b/types/wlr_output.c index 9a21196e..7cfbb085 100644 --- a/types/wlr_output.c +++ b/types/wlr_output.c @@ -173,8 +173,12 @@ bool wlr_output_set_custom_mode(struct wlr_output *output, int32_t width, void wlr_output_update_mode(struct wlr_output *output, struct wlr_output_mode *mode) { output->current_mode = mode; - wlr_output_update_custom_mode(output, mode->width, mode->height, - mode->refresh); + if (mode != NULL) { + wlr_output_update_custom_mode(output, mode->width, mode->height, + mode->refresh); + } else { + wlr_output_update_custom_mode(output, 0, 0, 0); + } } void wlr_output_update_custom_mode(struct wlr_output *output, int32_t width,