X-Git-Url: http://git.ozlabs.org/?p=petitboot;a=blobdiff_plain;f=lib%2Fwaiter%2Fwaiter.c;h=d684348f4fc9566be5bc6ab9023793b169e5580f;hp=1d9a7d8945e1cd992ebf458bcb797a36b016a9e0;hb=5b3dd56d982629349e612b9e984e65dec09b438f;hpb=7f4057edddbcb1b18293ebb96475ed37551edc73

diff --git a/lib/waiter/waiter.c b/lib/waiter/waiter.c
index 1d9a7d8..d684348 100644
--- a/lib/waiter/waiter.c
+++ b/lib/waiter/waiter.c
@@ -3,6 +3,7 @@
 #include <stdbool.h>
 #include <string.h>
 #include <assert.h>
+#include <errno.h>
 #include <sys/time.h>
 
 #include <talloc/talloc.h>
@@ -57,16 +58,11 @@ struct waitset *waitset_create(void *ctx)
 	return set;
 }
 
-void waitset_destroy(struct waitset *set)
-{
-	talloc_free(set);
-}
-
 static struct waiter *waiter_new(struct waitset *set)
 {
 	struct waiter **waiters, *waiter;
 
-	waiter = talloc(set->waiters, struct waiter);
+	waiter = talloc(set, struct waiter);
 	if (!waiter)
 		return NULL;
 
@@ -138,7 +134,7 @@ void waiter_remove(struct waiter *waiter)
 	memmove(&set->waiters[i], &set->waiters[i+1],
 		(set->n_waiters - i) * sizeof(set->waiters[0]));
 
-	set->waiters = talloc_realloc(set->waiters, set->waiters,
+	set->waiters = talloc_realloc(set, set->waiters,
 			struct waiter *, set->n_waiters);
 	set->waiters_changed = true;
 
@@ -176,34 +172,32 @@ static void update_waiters(struct waitset *set)
 		set->n_time_waiters = n_time;
 	}
 
-	/* IO waiters: copy to io_waiters, populate pollfds */
-	for (i = i_io = 0; i < set->n_waiters; i++) {
-		struct waiter *waiter = set->waiters[i];
-
-		if (waiter->type != WAITER_IO)
-			continue;
-
-		set->pollfds[i_io].fd = waiter->io.fd;
-		set->pollfds[i_io].events = waiter->io.events;
-		set->io_waiters[i_io] = waiter;
-		i_io++;
-	}
+	i_io = 0;
+	i_time = 0;
 
-	/* time waiters: copy to time_waiters, calculate next expiry */
 	timerclear(&set->next_timeout);
-	for (i = i_time = 0; i < set->n_waiters; i++) {
-		struct waiter *waiter = set->waiters[i];
-
-		if (waiter->type != WAITER_TIME)
-			continue;
 
-		if (!timerisset(&set->next_timeout) ||
-				timercmp(&waiter->timeout,
-					&set->next_timeout, <))
-			set->next_timeout = waiter->timeout;
+	for (i = 0; i < set->n_waiters; i++) {
+		struct waiter *waiter = set->waiters[i];
 
-		set->time_waiters[i_time] = waiter;
-		i_time++;
+		/* IO waiters: copy to io_waiters, populate pollfds */
+		if (waiter->type == WAITER_IO) {
+			set->pollfds[i_io].fd = waiter->io.fd;
+			set->pollfds[i_io].events = waiter->io.events;
+			set->io_waiters[i_io] = waiter;
+			i_io++;
+		}
+
+		/* time waiters: copy to time_waiters, calculate next expiry */
+		if (waiter->type == WAITER_TIME) {
+			if (!timerisset(&set->next_timeout) ||
+					timercmp(&waiter->timeout,
+						&set->next_timeout, <))
+				set->next_timeout = waiter->timeout;
+
+			set->time_waiters[i_time] = waiter;
+			i_time++;
+		}
 	}
 }
 
@@ -229,11 +223,13 @@ int waiter_poll(struct waitset *set)
 		timeout_ms = -1;
 	}
 
-
 	rc = poll(set->pollfds, set->n_io_waiters, timeout_ms);
 
-	if (rc < 0)
+	if (rc < 0) {
+		if (errno == EINTR)
+			rc = 0;
 		goto out;
+	}
 
 	for (i = 0; i < set->n_io_waiters; i++) {
 		struct waiter *waiter = set->io_waiters[i];