Use pixman regions to reduce repainting
For now, we just use glScissor and clip to the extent of the damage region,
but we can do even better by clipping the repaint to the region rectangles.
diff --git a/compositor/compositor.c b/compositor/compositor.c
index 535b3f1..ec0d727 100644
--- a/compositor/compositor.c
+++ b/compositor/compositor.c
@@ -163,6 +163,27 @@
return surface;
}
+void
+wlsc_surface_damage_rectangle(struct wlsc_surface *surface,
+ int32_t x, int32_t y,
+ int32_t width, int32_t height)
+{
+ struct wlsc_compositor *compositor = surface->compositor;
+
+ pixman_region32_union_rect(&compositor->damage_region,
+ &compositor->damage_region,
+ surface->x + x, surface->y + y,
+ width, height);
+ wlsc_compositor_schedule_repaint(compositor);
+}
+
+void
+wlsc_surface_damage(struct wlsc_surface *surface)
+{
+ wlsc_surface_damage_rectangle(surface, 0, 0,
+ surface->width, surface->height);
+}
+
uint32_t
get_time(void)
{
@@ -178,10 +199,11 @@
{
struct wlsc_surface *surface =
container_of(resource, struct wlsc_surface, surface.resource);
- struct wlsc_compositor *compositor = surface->compositor;
struct wl_listener *l, *next;
uint32_t time;
+ wlsc_surface_damage(surface);
+
wl_list_remove(&surface->link);
glDeleteTextures(1, &surface->texture);
@@ -191,8 +213,6 @@
l->func(l, &surface->surface, time);
free(surface);
-
- wlsc_compositor_schedule_repaint(compositor);
}
uint32_t *
@@ -407,9 +427,28 @@
struct wlsc_compositor *ec = output->compositor;
struct wlsc_surface *es;
struct wlsc_input_device *eid;
+ pixman_region32_t new_damage, total_damage;
+ pixman_box32_t *extents;
glViewport(0, 0, output->width, output->height);
+ pixman_region32_init(&new_damage);
+ pixman_region32_init(&total_damage);
+ pixman_region32_intersect_rect(&new_damage,
+ &ec->damage_region,
+ output->x, output->y,
+ output->width, output->height);
+ pixman_region32_subtract(&ec->damage_region,
+ &ec->damage_region, &new_damage);
+ pixman_region32_union(&total_damage, &new_damage,
+ &output->previous_damage_region);
+ pixman_region32_copy(&output->previous_damage_region, &new_damage);
+
+ extents = pixman_region32_extents(&total_damage);
+ glEnable(GL_SCISSOR_TEST);
+ glScissor(extents->x1, extents->y1,
+ extents->x2 - extents->x1, extents->y2 - extents->y1);
+
es = container_of(ec->surface_list.next, struct wlsc_surface, link);
if (es->map_type == WLSC_SURFACE_MAP_FULLSCREEN &&
es->fullscreen_output == output) {
@@ -483,6 +522,12 @@
{
struct wlsc_surface *es = (struct wlsc_surface *) surface;
+ /* FIXME: This damages the entire old surface, but we should
+ * really just damage the part that's no longer covered by the
+ * surface. Anything covered by the new surface will be
+ * damaged by the client. */
+ wlsc_surface_damage(es);
+
buffer->attach(buffer, surface);
es->buffer = buffer;
es->x += x;
@@ -517,7 +562,7 @@
break;
}
- wlsc_compositor_schedule_repaint(es->compositor);
+ wlsc_surface_damage(es);
es->map_type = WLSC_SURFACE_MAP_TOPLEVEL;
}
@@ -544,7 +589,7 @@
es->y = pes->y + y;
wlsc_surface_update_matrix(es);
- wlsc_compositor_schedule_repaint(es->compositor);
+ wlsc_surface_damage(es);
es->map_type = WLSC_SURFACE_MAP_TRANSIENT;
}
@@ -577,7 +622,7 @@
es->y = (output->height - es->height) / 2;
es->fullscreen_output = output;
wlsc_surface_update_matrix(es);
- wlsc_compositor_schedule_repaint(es->compositor);
+ wlsc_surface_damage(es);
es->map_type = WLSC_SURFACE_MAP_FULLSCREEN;
}
@@ -589,7 +634,8 @@
struct wlsc_surface *es = (struct wlsc_surface *) surface;
es->buffer->damage(es->buffer, surface, x, y, width, height);
- wlsc_compositor_schedule_repaint(es->compositor);
+
+ wlsc_surface_damage_rectangle(es, x, y, width, height);
}
const static struct wl_surface_interface surface_interface = {
@@ -605,8 +651,7 @@
wlsc_input_device_attach(struct wlsc_input_device *device,
struct wl_buffer *buffer, int x, int y)
{
- struct wlsc_compositor *ec =
- (struct wlsc_compositor *) device->input_device.compositor;
+ wlsc_surface_damage(device->sprite);
buffer->attach(buffer, &device->sprite->surface);
device->hotspot_x = x;
@@ -618,7 +663,7 @@
device->sprite->height = buffer->height;
wlsc_surface_update_matrix(device->sprite);
- wlsc_compositor_schedule_repaint(ec);
+ wlsc_surface_damage(device->sprite);
}
@@ -770,11 +815,13 @@
time, x, y, sx, sy);
}
+ wlsc_surface_damage(wd->sprite);
+
wd->sprite->x = device->x - wd->hotspot_x;
wd->sprite->y = device->y - wd->hotspot_y;
wlsc_surface_update_matrix(wd->sprite);
- wlsc_compositor_schedule_repaint(ec);
+ wlsc_surface_damage(wd->sprite);
}
static void
@@ -854,6 +901,7 @@
{
struct wl_list *l;
+ wlsc_surface_damage(switcher->current);
l = switcher->current->link.next;
if (l == &switcher->compositor->surface_list)
l = switcher->compositor->surface_list.next;
@@ -861,7 +909,7 @@
wl_list_remove(&switcher->listener.link);
wl_list_insert(switcher->current->surface.destroy_listener_list.prev,
&switcher->listener.link);
- wlsc_compositor_schedule_repaint(switcher->compositor);
+ wlsc_surface_damage(switcher->current);
}
static void
@@ -1006,7 +1054,7 @@
compositor->focus = 0;
}
- wlsc_compositor_schedule_repaint(compositor);
+ wlsc_surface_damage(wd->sprite);
}
void
@@ -1213,6 +1261,8 @@
output->width = width;
output->height = height;
+ pixman_region32_init(&output->previous_damage_region);
+
output->background =
background_create(output, option_background);
@@ -1227,6 +1277,10 @@
wl_display_add_object(c->wl_display, &output->object);
wl_display_add_global(c->wl_display, &output->object,
wlsc_output_post_geometry);
+
+ pixman_region32_union_rect(&c->damage_region,
+ &c->damage_region,
+ x, y, width, height);
}
int
@@ -1266,6 +1320,7 @@
loop = wl_display_get_event_loop(ec->wl_display);
ec->timer_source = wl_event_loop_add_timer(loop, repaint, ec);
+ pixman_region32_init(&ec->damage_region);
wlsc_compositor_schedule_repaint(ec);
return 0;