From 4901f6ceea24e7964b65aa7b1db32385af3ec836 Mon Sep 17 00:00:00 2001
From: =?utf8?q?Kristian=20H=C3=B8gsberg?= <krh@bitplanet.net>
Date: Sat, 14 Jul 2012 01:24:58 -0400
Subject: [PATCH] compositor-drm: Allocate hw cursor like other planes

We no longer reserve the hw cursor for just the pointer sprite.  Any
surface that satisfies the requirements for the hw cursor can be used.

Of course that's more or less always the pointer sprite, but at least now
we don't hard-code the references to compositor->seat anymore.
---
 src/compositor-drm.c | 45 +++++++++++++++++++--------------------------
 1 file changed, 19 insertions(+), 26 deletions(-)

diff --git a/src/compositor-drm.c b/src/compositor-drm.c
index 4f94733..e4afe5f 100644
--- a/src/compositor-drm.c
+++ b/src/compositor-drm.c
@@ -108,7 +108,7 @@ struct drm_output {
 
 	struct gbm_surface *surface;
 	struct gbm_bo *cursor_bo[2];
-	int current_cursor;
+	int current_cursor, cursor_free;
 	EGLSurface egl_surface;
 	struct drm_fb *current, *next;
 	struct backlight *backlight;
@@ -666,7 +666,7 @@ drm_output_prepare_overlay_surface(struct weston_output *output_base,
 
 static void
 drm_output_set_cursor(struct weston_output *output_base,
-		      struct weston_seat *seat)
+		      struct weston_surface *es)
 {
 	struct drm_output *output = (struct drm_output *) output_base;
 	struct drm_compositor *c =
@@ -677,23 +677,21 @@ drm_output_set_cursor(struct weston_output *output_base,
 	unsigned char *s;
 	int i;
 
-	if (seat->sprite->output_mask != (1u << output_base->id))
+	if (!output->cursor_free)
 		return;
-
-	if (seat->sprite->buffer == NULL ||
-	    !wl_buffer_is_shm(seat->sprite->buffer) ||
-	    seat->sprite->geometry.width > 64 ||
-	    seat->sprite->geometry.height > 64)
+	if (es->output_mask != (1u << output_base->id))
+		return;
+	if (es->buffer == NULL || !wl_buffer_is_shm(es->buffer) ||
+	    es->geometry.width > 64 || es->geometry.height > 64)
 		return;
 
 	output->current_cursor ^= 1;
 	bo = output->cursor_bo[output->current_cursor];
 	memset(buf, 0, sizeof buf);
-	stride = wl_shm_buffer_get_stride(seat->sprite->buffer);
-	s = wl_shm_buffer_get_data(seat->sprite->buffer);
-	for (i = 0; i < seat->sprite->geometry.height; i++)
-		memcpy(buf + i * 64, s + i * stride,
-		       seat->sprite->geometry.width * 4);
+	stride = wl_shm_buffer_get_stride(es->buffer);
+	s = wl_shm_buffer_get_data(es->buffer);
+	for (i = 0; i < es->geometry.height; i++)
+		memcpy(buf + i * 64, s + i * stride, es->geometry.width * 4);
 
 	if (gbm_bo_write(bo, buf, sizeof buf) < 0)
 		return;
@@ -705,13 +703,14 @@ drm_output_set_cursor(struct weston_output *output_base,
 	}
 
 	if (drmModeMoveCursor(c->drm.fd, output->crtc_id,
-			      seat->sprite->geometry.x - output->base.x,
-			      seat->sprite->geometry.y - output->base.y)) {
+			      es->geometry.x - output->base.x,
+			      es->geometry.y - output->base.y)) {
 		weston_log("failed to move cursor: %m\n");
 		return;
 	}
 
-	seat->sprite->plane = WESTON_PLANE_DRM_CURSOR;
+	es->plane = WESTON_PLANE_DRM_CURSOR;
+	output->cursor_free = 0;
 }
 
 static void
@@ -722,7 +721,6 @@ drm_assign_planes(struct weston_output *output)
 	struct drm_output *drm_output = (struct drm_output *) output;
 	struct weston_surface *es, *next;
 	pixman_region32_t overlap, surface_overlap;
-	struct weston_seat *seat;
 	int prev_plane;
 
 	/*
@@ -738,13 +736,9 @@ drm_assign_planes(struct weston_output *output)
 	 * the client buffer can be used directly for the sprite surface
 	 * as we do for flipping full screen surfaces.
 	 */
-	seat = (struct weston_seat *) c->base.seat;
 	pixman_region32_init(&overlap);
+	drm_output->cursor_free = 1;
 	wl_list_for_each_safe(es, next, &c->base.surface_list, link) {
-		/*
-		 * FIXME: try to assign hw cursors here too, they're just
-		 * special overlays
-		 */
 		pixman_region32_init(&surface_overlap);
 		pixman_region32_intersect(&surface_overlap, &overlap,
 					  &es->transform.boundingbox);
@@ -754,9 +748,8 @@ drm_assign_planes(struct weston_output *output)
 		if (pixman_region32_not_empty(&surface_overlap))
 			goto bail;
 
-		if (es == seat->sprite)
-			drm_output_set_cursor(output, seat);
-
+		if (es->plane == WESTON_PLANE_PRIMARY)
+			drm_output_set_cursor(output, es);
 		if (es->plane == WESTON_PLANE_PRIMARY)
 			drm_output_prepare_scanout_surface(output, es);
 		if (es->plane == WESTON_PLANE_PRIMARY)
@@ -778,7 +771,7 @@ drm_assign_planes(struct weston_output *output)
 	}
 	pixman_region32_fini(&overlap);
 
-	if (!seat->sprite || seat->sprite->plane == WESTON_PLANE_PRIMARY)
+	if (drm_output->cursor_free)
 		drmModeSetCursor(c->drm.fd, drm_output->crtc_id, 0, 0, 0);
 
 	drm_disable_unused_sprites(output);
-- 
2.7.4