From d3fa34ca298e734d4e7f14a7023848ff13360109 Mon Sep 17 00:00:00 2001 From: Kristian Høgsberg Date: Sun, 2 Nov 2008 15:28:22 -0500 Subject: Don't use poll timeout for timing animation. Receiving events breaks out of the poll and resets the time out. Effectively, this blocks the animation when the client receives events. As a quick fix, use timerfd for timing the animation. --- Makefile | 2 +- flower.c | 32 +++++++++++++++++++++++++++----- 2 files changed, 28 insertions(+), 6 deletions(-) diff --git a/Makefile b/Makefile index 84b65aa..fa7ac49 100644 --- a/Makefile +++ b/Makefile @@ -21,7 +21,7 @@ flower_objs = flower.o pointer_objs = pointer.o $(clients) : CFLAGS += $(shell pkg-config --cflags cairo) -$(clients) : LDLIBS += $(shell pkg-config --libs cairo) +$(clients) : LDLIBS += $(shell pkg-config --libs cairo) -lrt define client_template $(1): $$($(1)_objs) libwayland.so diff --git a/flower.c b/flower.c index 4654de0..46deb05 100644 --- a/flower.c +++ b/flower.c @@ -5,6 +5,8 @@ #include #include #include +#include +#include #include #include #include @@ -151,9 +153,10 @@ int main(int argc, char *argv[]) int fd, i, ret; uint32_t name, mask; cairo_surface_t *s; - struct pollfd p[1]; - - srandom(time(NULL)); + struct pollfd p[2]; + struct timespec ts; + struct itimerspec its; + uint64_t expires; fd = open(gem_device, O_RDWR); if (fd < 0) { @@ -171,6 +174,24 @@ int main(int argc, char *argv[]) surface = wl_display_create_surface(display); + p[1].fd = timerfd_create(CLOCK_MONOTONIC, 0); + if (p[1].fd < 0) { + fprintf(stderr, "could not create timerfd\n: %m"); + return -1; + } + + p[1].events = POLLIN; + clock_gettime(CLOCK_MONOTONIC, &ts); + its.it_value = ts; + its.it_interval.tv_sec = 0; + its.it_interval.tv_nsec = 20 * 1000000; + if (timerfd_settime(p[1].fd, TFD_TIMER_ABSTIME, &its, NULL) < 0) { + fprintf(stderr, "could not set timerfd\n: %m"); + return -1; + } + + srandom(ts.tv_nsec); + s = draw_stuff(width, height); name = name_cairo_surface(fd, s); @@ -178,8 +199,9 @@ int main(int argc, char *argv[]) cairo_image_surface_get_stride(s)); i = 0; - while (ret = poll(p, 1, 20), ret >= 0) { - if (ret == 0) { + while (ret = poll(p, 2, -1), ret >= 0) { + if (p[1].revents & POLLIN) { + read(p[1].fd, &expires, sizeof expires); wl_surface_map(surface, x + cos(i / 30.0) * 200, y + sin(i / 31.0) * 200, -- cgit