output: trigger frame/present events on all commits on enabled output

Up until now, frame/present events were only triggered when the
user submitted a buffer. Change the wlr_output API so that these
events are triggered when any commit is applied on an enabled
output.

Closes: https://gitlab.freedesktop.org/wlroots/wlroots/-/issues/3708

4 files changed, 11 insertions, 7 deletions

diff --git a/backend/drm/drm.c b/backend/drm/drm.c
index 85240448..a012e678 100644
--- a/backend/drm/drm.c
+++ b/backend/drm/drm.c
@@ -715,12 +715,10 @@ bool drm_connector_commit_state(struct wlr_drm_connector *conn,
 		}
 	}
 
-	uint32_t flags = 0;
 	if (pending.base->committed & WLR_OUTPUT_STATE_BUFFER) {
 		if (!drm_connector_state_update_primary_fb(conn, &pending)) {
 			goto out;
 		}
-		flags |= DRM_MODE_PAGE_FLIP_EVENT;
 
 		// wlr_drm_interface.crtc_commit will perform either a non-blocking
 		// page-flip, either a blocking modeset. When performing a blocking modeset
@@ -732,9 +730,6 @@ bool drm_connector_commit_state(struct wlr_drm_connector *conn,
 			goto out;
 		}
 	}
-	if (pending.modeset && pending.active) {
-		flags |= DRM_MODE_PAGE_FLIP_EVENT;
-	}
 	if (pending.base->committed & WLR_OUTPUT_STATE_LAYERS) {
 		if (!drm_connector_set_pending_layer_fbs(conn, pending.base)) {
 			return false;
@@ -751,6 +746,8 @@ bool drm_connector_commit_state(struct wlr_drm_connector *conn,
 		}
 	}
 
+	uint32_t flags = pending.active ? DRM_MODE_PAGE_FLIP_EVENT : 0;
+
 	ok = drm_crtc_commit(conn, &pending, flags, false);
 	if (!ok) {
 		goto out;
diff --git a/backend/headless/output.c b/backend/headless/output.c
index 7dc7f0bc..f4719a3c 100644
--- a/backend/headless/output.c
+++ b/backend/headless/output.c
@@ -5,6 +5,7 @@
 #include <wlr/types/wlr_output_layer.h>
 #include <wlr/util/log.h>
 #include "backend/headless.h"
+#include "types/wlr_output.h"
 
 static const uint32_t SUPPORTED_OUTPUT_STATE =
 	WLR_OUTPUT_STATE_BACKEND_OPTIONAL |
@@ -65,7 +66,7 @@ static bool output_commit(struct wlr_output *wlr_output,
 		output_update_refresh(output, state->custom_mode.refresh);
 	}
 
-	if (state->committed & WLR_OUTPUT_STATE_BUFFER) {
+	if (output_pending_enabled(wlr_output, state)) {
 		struct wlr_output_event_present present_event = {
 			.commit_seq = wlr_output->commit_seq + 1,
 			.presented = true,
diff --git a/backend/wayland/output.c b/backend/wayland/output.c
index 7854d476..e51be4c5 100644
--- a/backend/wayland/output.c
+++ b/backend/wayland/output.c
@@ -19,6 +19,7 @@
 #include "backend/wayland.h"
 #include "render/pixel_format.h"
 #include "render/wlr_renderer.h"
+#include "types/wlr_output.h"
 
 #include "linux-dmabuf-unstable-v1-client-protocol.h"
 #include "presentation-time-client-protocol.h"
@@ -555,7 +556,7 @@ static bool output_commit(struct wlr_output *wlr_output,
 		return false;
 	}
 
-	if (state->committed & (WLR_OUTPUT_STATE_BUFFER | WLR_OUTPUT_STATE_LAYERS)) {
+	if (output_pending_enabled(wlr_output, state)) {
 		if (output->frame_callback != NULL) {
 			wl_callback_destroy(output->frame_callback);
 		}
diff --git a/backend/x11/output.c b/backend/x11/output.c
index d95f0767..553f56a8 100644
--- a/backend/x11/output.c
+++ b/backend/x11/output.c
@@ -22,6 +22,7 @@
 
 #include "backend/x11.h"
 #include "util/time.h"
+#include "types/wlr_output.h"
 
 static const uint32_t SUPPORTED_OUTPUT_STATE =
 	WLR_OUTPUT_STATE_BACKEND_OPTIONAL |
@@ -375,6 +376,10 @@ static bool output_commit(struct wlr_output *wlr_output,
 		if (!output_commit_buffer(output, state)) {
 			return false;
 		}
+	} else if (output_pending_enabled(wlr_output, state)) {
+		uint32_t serial = output->wlr_output.commit_seq;
+		uint64_t target_msc = output->last_msc ? output->last_msc + 1 : 0;
+		xcb_present_notify_msc(x11->xcb, output->win, serial, target_msc, 0, 0);
 	}
 
 	xcb_flush(x11->xcb);