From bfffa28ba0483b49770326c4f567950ea80d0fb6 Mon Sep 17 00:00:00 2001
From: =?utf8?q?Jos=C3=A9=20Bollo?= <jose.bollo@iot.bzh>
Date: Thu, 31 Aug 2017 08:32:11 +0200
Subject: [PATCH] jobs: optimized version of fix starve
MIME-Version: 1.0
Content-Type: text/plain; charset=utf8
Content-Transfer-Encoding: 8bit

Signed-off-by: José Bollo <jose.bollo@iot.bzh>
---
 src/jobs.c | 47 +++++++++++++++++++----------------------------
 1 file changed, 19 insertions(+), 28 deletions(-)

diff --git a/src/jobs.c b/src/jobs.c
index e3aa5c99..ed812f81 100644
--- a/src/jobs.c
+++ b/src/jobs.c
@@ -300,17 +300,18 @@ static void thread_run(volatile struct thread *me)
 	if (current) {
 		current->lowered = 1;
 		evto = EVENT_TIMEOUT_CHILD;
+		me->events = current->events;
 	} else {
 		started++;
 		sig_monitor_init_timeouts();
 		evto = EVENT_TIMEOUT_TOP;
+		me->events = NULL;
 	}
 	me->next = threads;
 	threads = (struct thread*)me;
 	current = (struct thread*)me;
 
 	/* loop until stopped */
-	me->events = NULL;
 	while (!me->stop) {
 		/* get a job */
 		job = job_get(first_job);
@@ -336,43 +337,33 @@ static void thread_run(volatile struct thread *me)
 			}
 		} else {
 			/* no job, check events */
-			thr = (struct thread*)me;
-			events = NULL;
-			while (thr && !(events = thr->events))
-				thr = thr->upper;
-			if (events && !events->runs) {
+			events = me->events;
+			if (!events || events->runs)
+				events = events_get();
+			if (events) {
 				/* run the events */
+				events->used = 1;
 				events->runs = 1;
 				events->timeout = evto;
 				me->events = events;
 				pthread_mutex_unlock(&mutex);
 				sig_monitor(0, events_call, events);
 				pthread_mutex_lock(&mutex);
+				events->used = 0;
 				events->runs = 0;
 				me->events = NULL;
-			} else {
-				/* no owned event, check events */
-				events = events_get();
-				if (events) {
-					/* run the events */
-					events->used = 1;
-					events->runs = 1;
-					events->timeout = evto;
-					me->events = events;
-					pthread_mutex_unlock(&mutex);
-					sig_monitor(0, events_call, events);
-					pthread_mutex_lock(&mutex);
-					events->used = 0;
-					events->runs = 0;
-					me->events = NULL;
-				} else {
-					/* no job and not events */
-					waiting++;
-					me->waits = 1;
-					pthread_cond_wait(&cond, &mutex);
-					me->waits = 0;
-					waiting--;
+				thr = me->upper;
+				while (thr && thr->events == events) {
+					thr->events = NULL;
+					thr = thr->upper;
 				}
+			} else {
+				/* no job and not events */
+				waiting++;
+				me->waits = 1;
+				pthread_cond_wait(&cond, &mutex);
+				me->waits = 0;
+				waiting--;
 			}
 		}
 	}
-- 
2.16.6