X-Git-Url: https://gerrit.automotivelinux.org/gerrit/gitweb?a=blobdiff_plain;f=src%2Fjobs.c;h=1ed20fcb3a57722cfa1fcab81d1704f4bfbf5dc5;hb=9fe2dfd3c4df334607083f989346090e1051a565;hp=03fe4254fba8704a504b33eea72949b36e7e4035;hpb=5fdbf43a0af039c7ad64b9de8038d5d7d9cd0c60;p=src%2Fapp-framework-binder.git

diff --git a/src/jobs.c b/src/jobs.c
index 03fe4254..1ed20fcb 100644
--- a/src/jobs.c
+++ b/src/jobs.c
@@ -18,6 +18,7 @@
 #define _GNU_SOURCE
 
 #include <stdlib.h>
+#include <stdint.h>
 #include <unistd.h>
 #include <signal.h>
 #include <time.h>
@@ -26,6 +27,8 @@
 #include <errno.h>
 #include <assert.h>
 
+#include <systemd/sd-event.h>
+
 #include "jobs.h"
 #include "sig-monitor.h"
 #include "verbose.h"
@@ -37,6 +40,9 @@
 #define sig_monitor(to,cb,arg)       (cb(0,arg))
 #endif
 
+#define EVENT_TIMEOUT_TOP  	((uint64_t)-1)
+#define EVENT_TIMEOUT_CHILD	((uint64_t)10000)
+
 /** Internal shortcut for callback */
 typedef void (*job_cb_t)(int, void*, void *, void*);
 
@@ -54,16 +60,26 @@ struct job
 	unsigned dropped: 1; /**< is removed ? */
 };
 
+/** Description of handled event loops */
+struct events
+{
+	struct events *next;
+	struct sd_event *event;
+	uint64_t timeout;
+	unsigned runs: 1;
+};
+
 /** Description of threads */
 struct thread
 {
-	struct thread *next;  /**< next thread of the list */
-	struct thread *upper; /**< upper same thread */
-	struct job *job;      /**< currently processed job */
-	pthread_t tid;        /**< the thread id */
-	unsigned stop: 1;     /**< stop requested */
-	unsigned lowered: 1;  /**< has a lower same thread */
-	unsigned waits: 1;    /**< is waiting? */
+	struct thread *next;   /**< next thread of the list */
+	struct thread *upper;  /**< upper same thread */
+	struct job *job;       /**< currently processed job */
+	struct events *events; /**< currently processed job */
+	pthread_t tid;         /**< the thread id */
+	unsigned stop: 1;      /**< stop requested */
+	unsigned lowered: 1;   /**< has a lower same thread */
+	unsigned waits: 1;     /**< is waiting? */
 };
 
 /* synchronisation of threads */
@@ -75,6 +91,7 @@ static int allowed = 0; /** allowed count of threads */
 static int started = 0; /** started count of threads */
 static int waiting = 0; /** waiting count of threads */
 static int remains = 0; /** allowed count of waiting jobs */
+static int nevents = 0; /** count of events */
 
 /* list of threads */
 static struct thread *threads;
@@ -82,7 +99,7 @@ static _Thread_local struct thread *current;
 
 /* queue of pending jobs */
 static struct job *first_job;
-static struct job *first_events;
+static struct events *first_events;
 static struct job *free_jobs;
 
 /**
@@ -133,59 +150,57 @@ end:
 }
 
 /**
- * Adds 'job1' and 'job2' at the end of the list of jobs, marking it
+ * Adds 'job' at the end of the list of jobs, marking it
  * as blocked if an other job with the same group is pending.
- * @param job1 the first job to add
- * @param job2 the second job to add or NULL
+ * @param job the job to add
  */
-static void job_add2(struct job *job1, struct job *job2)
+static void job_add(struct job *job)
 {
-	void *group1, *group2, *group;
+	void *group;
 	struct job *ijob, **pjob;
 
 	/* prepare to add */
-	group1 = job1->group;
-	job1->next = job2;
-	if (!job2)
-		group2 = NULL;
-	else {
-		job2->next = NULL;
-		group2 = job2->group;
-		if (group2 && group2 == group1)
-			job2->blocked = 1;
-	}
+	group = job->group;
+	job->next = NULL;
 
 	/* search end and blockers */
 	pjob = &first_job;
 	ijob = first_job;
 	while (ijob) {
-		group = ijob->group;
-		if (group) {
-			if (group == group1)
-				job1->blocked = 1;
-			if (group == group2)
-				job2->blocked = 1;
-		}
+		if (group && ijob->group == group)
+			job->blocked = 1;
 		pjob = &ijob->next;
 		ijob = ijob->next;
 	}
 
 	/* queue the jobs */
-	*pjob = job1;
+	*pjob = job;
 }
 
 /**
  * Get the next job to process or NULL if none.
- * @param job the head of the list to search.
  * @return the first job that isn't blocked or NULL
  */
-static inline struct job *job_get(struct job *job)
+static inline struct job *job_get()
 {
+	struct job *job = first_job;
 	while (job && job->blocked)
 		job = job->next;
 	return job;
 }
 
+/**
+ * Get the next events to process or NULL if none.
+ * @return the first events that isn't running or NULL
+ */
+static inline struct events *events_get()
+{
+	struct events *events = first_events;
+	while (events && events->runs)
+		events = events->next;
+	return events;
+}
+
 /**
  * Releases the processed 'job': removes it
  * from the list of jobs and unblock the first
@@ -221,48 +236,6 @@ static inline void job_release(struct job *job)
 	free_jobs = job;
 }
 
-/**
- * Releases the events 'job': removes it
- * from the list of events.
- * @param job the event to release
- */
-static inline void events_release(struct job *job)
-{
-	struct job *ijob, **pjob;
-
-	/* first unqueue the job */
-	pjob = &first_events;
-	ijob = first_events;
-	while (ijob != job) {
-		pjob = &ijob->next;
-		ijob = ijob->next;
-	}
-	*pjob = job->next;
-
-	/* recycle the job */
-	job->next = free_jobs;
-	free_jobs = job;
-}
-
-/**
- * Get the events of 'key' if existing.
- * @param key the key to search
- * @return the found events or NULL if none existing has key
- */
-static inline struct job *events_of_key(void *key)
-{
-	struct job *job;
-
-	if (!key)
-		job = NULL;
-	else {
-		job = first_events;
-		while (job && (job->dropped || job->group != key))
-			job = job->next;
-	}
-	return job;
-}
-
 /**
  * Monitored normal callback for a job.
  * This function is called by the monitor
@@ -294,6 +267,23 @@ static void job_cancel(int signum, void *arg)
 	job_call(SIGABRT, arg);
 }
 
+/**
+ * Monitored normal callback for events.
+ * This function is called by the monitor
+ * to run the event loop when the safe environment
+ * is set.
+ * @param signum 0 on normal flow or the number
+ *               of the signal that interrupted the normal
+ *               flow
+ * @param arg     the events to run
+ */
+static void events_call(int signum, void *arg)
+{
+	struct events *events = arg;
+	if (!signum)
+		sd_event_run(events->event, events->timeout);
+}
+
 /**
  * Main processing loop of threads processing jobs.
  * The loop must be called with the mutex locked
@@ -301,10 +291,12 @@ static void job_cancel(int signum, void *arg)
  * @param me the description of the thread to use
  * TODO: how are timeout handled when reentering?
  */
-static void thread_run(struct thread *me)
+static void thread_run(volatile struct thread *me)
 {
 	struct thread **prv;
 	struct job *job;
+	struct events *events;
+	uint64_t evto;
 
 	/* initialize description of itself and link it in the list */
 	me->tid = pthread_self();
@@ -312,16 +304,22 @@ static void thread_run(struct thread *me)
 	me->lowered = 0;
 	me->waits = 0;
 	me->upper = current;
-	if (current)
+	if (current) {
 		current->lowered = 1;
-	else
+		evto = EVENT_TIMEOUT_CHILD;
+	} else {
+		started++;
 		sig_monitor_init_timeouts();
-	current = me;
+		evto = EVENT_TIMEOUT_TOP;
+	}
 	me->next = threads;
-	threads = me;
-	started++;
+	threads = (struct thread*)me;
+	current = (struct thread*)me;
+
+	NOTICE("job thread starting %d(/%d) %s", started, allowed, me->upper ? "child" : "parent");
 
 	/* loop until stopped */
+	me->events = NULL;
 	while (!me->stop) {
 		/* get a job */
 		job = job_get(first_job);
@@ -338,18 +336,26 @@ static void thread_run(struct thread *me)
 
 			/* release the run job */
 			job_release(job);
+
+			/* release event if any */
+			events = me->events;
+			if (events) {
+				events->runs = 0;
+				me->events = NULL;
+			}
 		} else {
 			/* no job, check events */
-			job = job_get(first_events);
-			if (job) {
+			events = events_get();
+			if (events) {
 				/* run the events */
-				job->blocked = 1;
+				events->runs = 1;
+				events->timeout = evto;
+				me->events = events;
 				pthread_mutex_unlock(&mutex);
-				sig_monitor(job->timeout, job_call, job);
+				sig_monitor(0, events_call, events);
 				pthread_mutex_lock(&mutex);
-				job->blocked = 0;
-				if (job->dropped)
-					events_release(job);
+				events->runs = 0;
+				me->events = NULL;
 			} else {
 				/* no job and not events */
 				waiting++;
@@ -360,18 +366,20 @@ static void thread_run(struct thread *me)
 			}
 		}
 	}
+	NOTICE("job thread stoping %d(/%d) %s", started, allowed, me->upper ? "child" : "parent");
 
 	/* unlink the current thread and cleanup */
-	started--;
 	prv = &threads;
 	while (*prv != me)
 		prv = &(*prv)->next;
 	*prv = me->next;
 	current = me->upper;
-	if (current)
+	if (current) {
 		current->lowered = 0;
-	else
+	} else {
 		sig_monitor_clean_timeouts();
+		started--;
+	}
 }
 
 /**
@@ -543,7 +551,7 @@ int jobs_queue3(
 
 	/* queues the job */
 	remains--;
-	job_add2(job, NULL);
+	job_add(job);
 
 	/* signal an existing job */
 	pthread_cond_signal(&cond);
@@ -560,155 +568,161 @@ error:
 }
 
 /**
- * Run a asynchronous job represented by 'callback'
- * with the 'timeout' but only returns after job completion.
- * @param timeout  The maximum execution time in seconds of the job
- *                 or 0 for unlimited time.
- * @param callback The function to execute for achieving the job.
- *                 Its first parameter is either 0 on normal flow
- *                 or the signal number that broke the normal flow.
- * @return 0 in case of success or -1 in case of error
+ * Enter a synchronisation point: activates the job given by 'callback'
+ * @param group the gro
  */
-int jobs_invoke0(
+int jobs_enter(
+		void *group,
 		int timeout,
-		void (*callback)(int signum))
+		void (*callback)(int signum, void *closure, struct jobloop *jobloop),
+		void *closure)
 {
-	return jobs_invoke3(timeout, (job_cb_t)callback, NULL, NULL, NULL);
-}
+	
+	struct job *job;
+	struct thread me;
 
-/**
- * Run a asynchronous job represented by 'callback' and 'arg1'
- * with the 'timeout' but only returns after job completion.
- * @param timeout  The maximum execution time in seconds of the job
- *                 or 0 for unlimited time.
- * @param callback The function to execute for achieving the job.
- *                 Its first parameter is either 0 on normal flow
- *                 or the signal number that broke the normal flow.
- *                 The remaining parameter is the parameter 'arg1'
- *                 given here.
- * @param arg1     The second argument for 'callback'
- * @return 0 in case of success or -1 in case of error
- */
-int jobs_invoke(
-		int timeout,
-		void (*callback)(int, void*),
-		void *arg)
-{
-	return jobs_invoke3(timeout, (job_cb_t)callback, arg, NULL, NULL);
-}
+	pthread_mutex_lock(&mutex);
 
-/**
- * Run a asynchronous job represented by 'callback' and 'arg[12]'
- * with the 'timeout' but only returns after job completion.
- * @param timeout  The maximum execution time in seconds of the job
- *                 or 0 for unlimited time.
- * @param callback The function to execute for achieving the job.
- *                 Its first parameter is either 0 on normal flow
- *                 or the signal number that broke the normal flow.
- *                 The remaining parameters are the parameters 'arg[12]'
- *                 given here.
- * @param arg1     The second argument for 'callback'
- * @param arg2     The third argument for 'callback'
- * @return 0 in case of success or -1 in case of error
- */
-int jobs_invoke2(
-		int timeout,
-		void (*callback)(int, void*, void*),
-		void *arg1,
-		void *arg2)
-{
-	return jobs_invoke3(timeout, (job_cb_t)callback, arg1, arg2, NULL);
+	/* allocates the job */
+	job = job_create(group, timeout, (job_cb_t)callback, closure, &me, NULL);
+	if (!job) {
+		ERROR("out of memory");
+		errno = ENOMEM;
+		pthread_mutex_unlock(&mutex);
+		return -1;
+	}
+
+	/* queues the job */
+	job_add(job);
+
+	/* run until stopped */
+	thread_run(&me);
+	pthread_mutex_unlock(&mutex);
+	return 0;
 }
 
-/**
- * Stops the thread pointed by 'arg1'. Used with
- * invoke familly to return to the caller after completion.
- * @param signum Unused
- * @param arg1   The thread to stop
- * @param arg2   Unused
- * @param arg3   Unused
- */
-static void unlock_invoker(int signum, void *arg1, void *arg2, void *arg3)
+int jobs_leave(struct jobloop *jobloop)
 {
-	struct thread *t = arg1;
+	struct thread *t;
 	pthread_mutex_lock(&mutex);
-	t->stop = 1;
-	if (t->waits)
-		pthread_cond_broadcast(&cond);
+
+	t = threads;
+	while (t && t != (struct thread*)jobloop)
+		t = t->next;
+	if (!t) {
+		errno = EINVAL;
+	} else {
+		t->stop = 1;
+		if (t->waits)
+			pthread_cond_broadcast(&cond);
+	}
 	pthread_mutex_unlock(&mutex);
+	return -!t;
 }
 
 /**
- * Run a asynchronous job represented by 'callback' and 'arg[123]'
- * with the 'timeout' but only returns after job completion.
- * @param timeout  The maximum execution time in seconds of the job
- *                 or 0 for unlimited time.
- * @param callback The function to execute for achieving the job.
- *                 Its first parameter is either 0 on normal flow
- *                 or the signal number that broke the normal flow.
- *                 The remaining parameters are the parameters 'arg[123]'
- *                 given here.
- * @param arg1     The second argument for 'callback'
- * @param arg2     The third argument for 'callback'
- * @param arg3     The forth argument for 'callback'
- * @return 0 in case of success or -1 in case of error
+ * Gets a sd_event item for the current thread.
+ * @return a sd_event or NULL in case of error
  */
-int jobs_invoke3(
-		int timeout,
-		void (*callback)(int, void*, void *, void*),
-		void *arg1,
-		void *arg2,
-		void *arg3)
+struct sd_event *jobs_get_sd_event()
 {
-	struct job *job1, *job2;
-	struct thread me;
-	
+	struct events *events;
+	struct thread *me;
+	int rc;
+
 	pthread_mutex_lock(&mutex);
 
-	/* allocates the job */
-	job1 = job_create(&me, timeout, callback, arg1, arg2, arg3);
-	job2 = job_create(&me, 0, unlock_invoker, &me, NULL, NULL);
-	if (!job1 || !job2) {
-		ERROR("out of memory");
-		errno = ENOMEM;
-		if (job1) {
-			job1->next = free_jobs;
-			free_jobs = job1;
+	/* search events on stack */
+	me = current;
+	while (me && !me->events)
+		me = me->upper;
+	if (me)
+		/* return the stacked events */
+		events = me->events;
+	else {
+		/* search an available events */
+		events = events_get();
+		if (!events) {
+			/* not found, check if creation possible */
+			if (nevents >= allowed) {
+				ERROR("not possible to add a new event");
+				events = NULL;
+			} else {
+				events = malloc(sizeof *events);
+				if (events && (rc = sd_event_new(&events->event)) >= 0) {
+					if (nevents < started || start_one_thread() >= 0) {
+						events->runs = 0;
+						events->next = first_events;
+						first_events = events;
+					} else {
+						ERROR("can't start thread for events");
+						sd_event_unref(events->event);
+						free(events);
+						events = NULL;
+					}
+				} else {
+					if (!events) {
+						ERROR("out of memory");
+						errno = ENOMEM;
+					} else {
+						free(events);
+						ERROR("creation of sd_event failed: %m");
+						events = NULL;
+						errno = -rc;
+					} 
+				}
+			}
 		}
-		if (job2) {
-			job2->next = free_jobs;
-			free_jobs = job2;
+		if (events) {
+			/* */
+			me = current;
+			if (me) {
+				events->runs = 1;
+				me->events = events;
+			} else {
+				WARNING("event returned for unknown thread!");
+			}
 		}
-		pthread_mutex_unlock(&mutex);
-		return -1;
 	}
-
-	/* queues the job */
-	job_add2(job1, job2);
-
-	/* run until stopped */
-	thread_run(&me);
 	pthread_mutex_unlock(&mutex);
-	return 0;
+	return events ? events->event : NULL;
 }
 
 /**
- * Initialise the job stuff.
- * @param allowed_count Maximum count of thread for jobs (can be 0,
- *                      see 'jobs_add_me' for merging new threads)
+ * Enter the jobs processing loop.
+ * @param allowed_count Maximum count of thread for jobs including this one
  * @param start_count   Count of thread to start now, must be lower.
  * @param waiter_count  Maximum count of jobs that can be waiting.
+ * @param start         The start routine to activate (can't be NULL)
  * @return 0 in case of success or -1 in case of error.
  */
-int jobs_init(int allowed_count, int start_count, int waiter_count)
+int jobs_start(int allowed_count, int start_count, int waiter_count, void (*start)())
 {
 	int rc, launched;
+	struct thread me;
+	struct job *job;
 
-	assert(allowed_count >= 0);
+	assert(allowed_count >= 1);
 	assert(start_count >= 0);
 	assert(waiter_count > 0);
 	assert(start_count <= allowed_count);
 
+	rc = -1;
+	pthread_mutex_lock(&mutex);
+
+	/* check whether already running */
+	if (current || allowed) {
+		ERROR("thread already started");
+		errno = EINVAL;
+		goto error;
+	}
+
+	/* start */
+	if (sig_monitor_init() < 0) {
+		ERROR("failed to initialise signal handlers");
+		goto error;
+	}
+
 	/* records the allowed count */
 	allowed = allowed_count;
 	started = 0;
@@ -716,16 +730,30 @@ int jobs_init(int allowed_count, int start_count, int waiter_count)
 	remains = waiter_count;
 
 	/* start at least one thread */
-	pthread_mutex_lock(&mutex);
 	launched = 0;
-	while (launched < start_count && start_one_thread() == 0)
+	while ((launched + 1) < start_count) {
+		if (start_one_thread() != 0) {
+			ERROR("Not all threads can be started");
+			goto error;
+		}
 		launched++;
-	rc = -(launched != start_count);
-	pthread_mutex_unlock(&mutex);
+	}
+
+	/* queue the start job */
+	job = job_create(NULL, 0, (job_cb_t)start, NULL, NULL, NULL);
+	if (!job) {
+		ERROR("out of memory");
+		errno = ENOMEM;
+		goto error;
+	}
+	job_add(job);
+	remains--;
 
-	/* end */
-	if (rc)
-		ERROR("Not all threads can be started");
+	/* run until end */
+	thread_run(&me);
+	rc = 0;
+error:
+	pthread_mutex_unlock(&mutex);
 	return rc;
 }
 
@@ -812,101 +840,3 @@ void jobs_terminate()
 	pthread_mutex_unlock(&mutex);
 }
 
-/**
- * Adds the events waiter/dispatcher to the list of events waiters/dispatchers
- * to monitor.
- * @param key     A key to register the events waiter/dispatcher (see
- *                'jobs_del_events')
- * @param timeout Timeout in second of the function or 0 if none
- * @param events  The callback, the first argument is 0 for normal
- *                flow or the signal number when normal flow failed
- * @param closure The closure to give to the callback as secondd argument
- * @return 0 in case of success or -1 in case of error
- */
-int jobs_add_events(void *key, int timeout, void (*events)(int signum, void*), void *closure)
-{
-	struct job *job;
-
-	pthread_mutex_lock(&mutex);
-
-	/* look at an already existsing events for same key */
-	job = events_of_key(key);
-	if (job) {
-		pthread_mutex_unlock(&mutex);
-		ERROR("events of key %p already exist", key);
-		errno = EEXIST;
-		return -1;
-	}
-
-	/* creates the job */
-	job = job_create(key, timeout, (job_cb_t)events, closure, NULL, NULL);
-	if (!job) {
-		pthread_mutex_unlock(&mutex);
-		ERROR("Can't create events, out of memory");
-		errno = ENOMEM;
-		return -1;
-	}
-
-	/* adds the loop */
-	job->next = first_events;
-	first_events = job;
-
-	/* signal the loop */
-	if (waiting)
-		pthread_cond_signal(&cond);
-	pthread_mutex_unlock(&mutex);
-	return 0;
-}
-
-/**
- * Removes the events of 'key'
- * @param key The key of the events to remove
- * @return 0 in case of success or -1 in case of error
- */
-int jobs_del_events(void *key)
-{
-	struct job *job;
-
-	pthread_mutex_lock(&mutex);
-	job = events_of_key(key);
-	if (job)
-		if (job->blocked)
-			job->dropped = 1;
-		else
-			events_release(job);
-	pthread_mutex_unlock(&mutex);
-	if (!job) {
-		ERROR("events of key %p not found", key);
-		errno = ENOENT;
-	}
-	return -!job;
-}
-
-/**
- * Adds the current thread to the pool of threads
- * processing the jobs. Returns normally when the threads are
- * terminated or immediately with an error if the thread is
- * already in the pool.
- * @return 0 in case of success or -1 in case of error
- */
-int jobs_add_me()
-{
-	struct thread me;
-
-	/* check whether already running */
-	if (current) {
-		ERROR("thread already running");
-		errno = EINVAL;
-		return -1;
-	}
-
-	/* allowed... */
-	pthread_mutex_lock(&mutex);
-	allowed++;
-	thread_run(&me);
-	allowed--;
-	pthread_mutex_unlock(&mutex);
-	return 0;
-}
-
-