Fix concurrency issues on event manager
[src/app-framework-binder.git] / src / jobs.c
index f7acebf..8ffd6b6 100644 (file)
 #include <errno.h>
 #include <assert.h>
 
+#include <systemd/sd-event.h>
+
 #include "jobs.h"
 #include "sig-monitor.h"
 #include "verbose.h"
 
-/* describes pending job */
+#if 0
+#define _alert_ "do you really want to remove monitoring?"
+#define sig_monitor_init_timeouts()  ((void)0)
+#define sig_monitor_clean_timeouts() ((void)0)
+#define sig_monitor(to,cb,arg)       (cb(0,arg))
+#endif
+
+/** Internal shortcut for callback */
+typedef void (*job_cb_t)(int, void*, void *, void*);
+
+/** Description of a pending job */
 struct job
 {
-       struct job *next;   /* link to the next job enqueued */
-       void *group;        /* group of the request */
-       void (*callback)(int,void*,void*,void*);     /* processing callback */
-       void *arg1;         /* first arg */
-       void *arg2;         /* second arg */
-       void *arg3;         /* second arg */
-       int timeout;        /* timeout in second for processing the request */
-       int blocked;        /* is an other request blocking this one ? */
+       struct job *next;    /**< link to the next job enqueued */
+       void *group;         /**< group of the request */
+       job_cb_t callback;   /**< processing callback */
+       void *arg1;          /**< first arg */
+       void *arg2;          /**< second arg */
+       void *arg3;          /**< third arg */
+       int timeout;         /**< timeout in second for processing the request */
+       unsigned blocked: 1; /**< is an other request blocking this one ? */
+       unsigned dropped: 1; /**< is removed ? */
+};
+
+/** Description of handled event loops */
+struct events
+{
+       struct events *next;
+       struct sd_event *event;
+       unsigned runs: 1;
 };
 
-/** control of threads */
+/** 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 */
+       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 */
 static pthread_mutex_t mutex = PTHREAD_MUTEX_INITIALIZER;
 static pthread_cond_t  cond = PTHREAD_COND_INITIALIZER;
 
-/* count allowed, started and running threads */
+/* count allowed, started and waiting threads */
 static int allowed = 0; /** allowed count of threads */
 static int started = 0; /** started count of threads */
-static int running = 0; /** running count of threads */
-static int remains = 0; /** remaining count of jobs that can be created */
+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;
@@ -70,23 +94,23 @@ static _Thread_local struct thread *current;
 
 /* queue of pending jobs */
 static struct job *first_job;
-static struct job *first_evloop;
+static struct events *first_events;
 static struct job *free_jobs;
 
 /**
  * Create a new job with the given parameters
- * @param group the group of the job
- * @param timeout the timeout of the job (0 if none)
+ * @param group    the group of the job
+ * @param timeout  the timeout of the job (0 if none)
  * @param callback the function that achieves the job
- * @param arg1 the first argument of the callback
- * @param arg2 the second argument of the callback
- * @param arg3 the third argument of the callback
+ * @param arg1     the first argument of the callback
+ * @param arg2     the second argument of the callback
+ * @param arg3     the third argument of the callback
  * @return the created job unblock or NULL when no more memory
  */
 static struct job *job_create(
                void *group,
                int timeout,
-               void (*callback)(int, void*, void *, void*),
+               job_cb_t callback,
                void *arg1,
                void *arg2,
                void *arg3)
@@ -115,6 +139,7 @@ static struct job *job_create(
        job->arg2 = arg2;
        job->arg3 = arg3;
        job->blocked = 0;
+       job->dropped = 0;
 end:
        return job;
 }
@@ -142,7 +167,7 @@ static void job_add2(struct job *job1, struct job *job2)
                        job2->blocked = 1;
        }
 
-       /* search end and blackers */
+       /* search end and blockers */
        pjob = &first_job;
        ijob = first_job;
        while (ijob) {
@@ -163,22 +188,32 @@ static void job_add2(struct job *job1, struct job *job2)
 
 /**
  * Get the next job to process or NULL if none.
- * The returned job if any isn't removed from
- * the list of jobs.
- * @return the job to process
+ * @return the first job that isn't blocked or NULL
  */
 static inline struct job *job_get()
 {
-       struct job *job;
-
-       job = first_job;
+       struct job *job = first_job;
        while (job && job->blocked)
                job = job->next;
        return job;
 }
 
 /**
- * Releases the processed '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
+ * pending job of the same group if any.
  * @param job the job to release
  */
 static inline void job_release(struct job *job)
@@ -210,80 +245,132 @@ static inline void job_release(struct job *job)
        free_jobs = job;
 }
 
-/** monitored call to the job */
+/**
+ * Monitored normal callback for a job.
+ * This function is called by the monitor
+ * to run the job 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 job to run
+ */
 static void job_call(int signum, void *arg)
 {
        struct job *job = arg;
        job->callback(signum, job->arg1, job->arg2, job->arg3);
 }
 
-/** monitored cancel of the job */
+/**
+ * Monitored cancel callback for a job.
+ * This function is called by the monitor
+ * to cancel the job when the safe environment
+ * is set.
+ * @param signum 0 on normal flow or the number
+ *               of the signal that interrupted the normal
+ *               flow, isn't used
+ * @param arg    the job to run
+ */
 static void job_cancel(int signum, void *arg)
 {
        job_call(SIGABRT, arg);
 }
 
-/* main loop of processing threads */
-static void thread_run(struct thread *me)
+/**
+ * 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, (uint64_t) -1);
+}
+
+/**
+ * Main processing loop of threads processing jobs.
+ * The loop must be called with the mutex locked
+ * and it returns with the mutex locked.
+ * @param me the description of the thread to use
+ * TODO: how are timeout handled when reentering?
+ */
+static void thread_run(volatile struct thread *me)
 {
        struct thread **prv;
        struct job *job;
+       struct events *events;
 
-       /* init */
+       /* initialize description of itself and link it in the list */
        me->tid = pthread_self();
        me->stop = 0;
        me->lowered = 0;
+       me->waits = 0;
        me->upper = current;
        if (current)
                current->lowered = 1;
        else
                sig_monitor_init_timeouts();
-       current = me;
+       current = (struct thread*)me;
        me->next = threads;
-       threads = me;
+       threads = (struct thread*)me;
+       started++;
 
        /* loop until stopped */
-       running++;
+       me->events = NULL;
        while (!me->stop) {
                /* get a job */
-               job = job_get();
-               if (!job && first_job && running == 0) {
-                       /* sad situation!! should not happen */
-                       ERROR("threads are blocked!");
-                       job = first_job;
-                       first_job = job->next;
-               }
+               job = job_get(first_job);
                if (job) {
+                       /* prepare running the job */
+                       remains++; /* increases count of job that can wait */
+                       job->blocked = 1; /* mark job as blocked */
+                       me->job = job; /* record the job (only for terminate) */
+
                        /* run the job */
-                       remains++;
-                       job->blocked = 1;
-                       me->job = job;
                        pthread_mutex_unlock(&mutex);
                        sig_monitor(job->timeout, job_call, job);
                        pthread_mutex_lock(&mutex);
+
+                       /* 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 evloop */
-                       job = first_evloop;
-                       if (job) {
-                               /* evloop */
-                               first_evloop = job->next;
+                       /* no job, check events */
+                       events = events_get();
+                       if (events) {
+                               /* run the events */
+                               events->runs = 1;
+                               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->next = first_evloop;
-                               first_evloop = job;
+                               events->runs = 0;
+                               me->events = NULL;
                        } else {
-                               /* no job and not evloop */
-                               running--;
+                               /* no job and not events */
+                               waiting++;
+                               me->waits = 1;
                                pthread_cond_wait(&cond, &mutex);
-                               running++;
+                               me->waits = 0;
+                               waiting--;
                        }
                }
        }
-       running--;
 
-       /* uninit */
+       /* unlink the current thread and cleanup */
+       started--;
        prv = &threads;
        while (*prv != me)
                prv = &(*prv)->next;
@@ -293,11 +380,14 @@ static void thread_run(struct thread *me)
                current->lowered = 0;
        else
                sig_monitor_clean_timeouts();
-       pthread_mutex_unlock(&mutex);
 }
 
-/* main loop of processing threads */
-static void *thread_create(void *data)
+/**
+ * Entry point for created threads.
+ * @param data not used
+ * @return NULL
+ */
+static void *thread_main(void *data)
 {
        struct thread me;
 
@@ -307,55 +397,90 @@ static void *thread_create(void *data)
        return NULL;
 }
 
-/* start a new thread */
+/**
+ * Starts a new thread
+ * @return 0 in case of success or -1 in case of error
+ */
 static int start_one_thread()
 {
        pthread_t tid;
        int rc;
 
-       assert(started < allowed);
-
-       started++;
-       rc = pthread_create(&tid, NULL, thread_create, NULL);
+       rc = pthread_create(&tid, NULL, thread_main, NULL);
        if (rc != 0) {
-               started--;
-               errno = rc;
+               /* errno = rc; */
                WARNING("not able to start thread: %m");
                rc = -1;
        }
        return rc;
 }
 
-static int start_one_thread_if_needed()
-{
-       int rc;
-
-       if (started == running && started < allowed) {
-               /* all threads are busy and a new can be started */
-               rc = start_one_thread();
-               if (rc < 0 && started == 0)
-                       return rc; /* no thread available */
-       }
-       return 0;
-}
-
+/**
+ * Queues a new asynchronous job represented by 'callback'
+ * for the 'group' and the 'timeout'.
+ * Jobs are queued FIFO and are possibly executed in parallel
+ * concurrently except for job of the same group that are
+ * executed sequentially in FIFO order.
+ * @param group    The group of the job or NULL when no group.
+ * @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
+ */
 int jobs_queue0(
                void *group,
                int timeout,
                void (*callback)(int signum))
 {
-       return jobs_queue3(group, timeout, (void(*)(int,void*,void*,void*))callback, NULL, NULL, NULL);
+       return jobs_queue3(group, timeout, (job_cb_t)callback, NULL, NULL, NULL);
 }
 
+/**
+ * Queues a new asynchronous job represented by 'callback' and 'arg1'
+ * for the 'group' and the 'timeout'.
+ * Jobs are queued FIFO and are possibly executed in parallel
+ * concurrently except for job of the same group that are
+ * executed sequentially in FIFO order.
+ * @param group    The group of the job or NULL when no group.
+ * @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_queue(
                void *group,
                int timeout,
                void (*callback)(int, void*),
                void *arg)
 {
-       return jobs_queue3(group, timeout, (void(*)(int,void*,void*,void*))callback, arg, NULL, NULL);
+       return jobs_queue3(group, timeout, (job_cb_t)callback, arg, NULL, NULL);
 }
 
+/**
+ * Queues a new asynchronous job represented by 'callback' and 'arg[12]'
+ * for the 'group' and the 'timeout'.
+ * Jobs are queued FIFO and are possibly executed in parallel
+ * concurrently except for job of the same group that are
+ * executed sequentially in FIFO order.
+ * @param group    The group of the job or NULL when no group.
+ * @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_queue2(
                void *group,
                int timeout,
@@ -363,10 +488,28 @@ int jobs_queue2(
                void *arg1,
                void *arg2)
 {
-       return jobs_queue3(group, timeout, (void(*)(int,void*,void*,void*))callback, arg1, arg2, NULL);
+       return jobs_queue3(group, timeout, (job_cb_t)callback, arg1, arg2, NULL);
 }
 
-/* queue the job to the 'callback' using a separate thread if available */
+/**
+ * Queues a new asynchronous job represented by 'callback' and 'arg[123]'
+ * for the 'group' and the 'timeout'.
+ * Jobs are queued FIFO and are possibly executed in parallel
+ * concurrently except for job of the same group that are
+ * executed sequentially in FIFO order.
+ * @param group    The group of the job or NULL when no group.
+ * @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
+ */
 int jobs_queue3(
                void *group,
                int timeout,
@@ -397,20 +540,22 @@ int jobs_queue3(
        }
 
        /* start a thread if needed */
-       rc = start_one_thread_if_needed();
-       if (rc < 0) {
-               /* failed to start threading */
-               info = "can't start first thread";
-               goto error2;
+       if (waiting == 0 && started < allowed) {
+               /* all threads are busy and a new can be started */
+               rc = start_one_thread();
+               if (rc < 0 && started == 0) {
+                       info = "can't start first thread";
+                       goto error2;
+               }
        }
 
        /* queues the job */
        remains--;
        job_add2(job, NULL);
-       pthread_mutex_unlock(&mutex);
 
        /* signal an existing job */
        pthread_cond_signal(&cond);
+       pthread_mutex_unlock(&mutex);
        return 0;
 
 error2:
@@ -422,57 +567,100 @@ error:
        return -1;
 }
 
-/* initialise the threads */
-int jobs_init(int allowed_count, int start_count, int waiter_count)
-{
-       /* records the allowed count */
-       allowed = allowed_count;
-       started = 0;
-       running = 0;
-       remains = waiter_count;
-
-       /* start at least one thread */
-       pthread_mutex_lock(&mutex);
-       while (started < start_count && start_one_thread() == 0);
-       pthread_mutex_unlock(&mutex);
-
-       /* end */
-       return -(started != start_count);
-}
-
+/**
+ * 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
+ */
 int jobs_invoke0(
                int timeout,
                void (*callback)(int signum))
 {
-       return jobs_invoke3(timeout, (void(*)(int,void*,void*,void*))callback, NULL, NULL, NULL);
+       return jobs_invoke3(timeout, (job_cb_t)callback, NULL, NULL, NULL);
 }
 
+/**
+ * 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, (void(*)(int,void*,void*,void*))callback, arg, NULL, NULL);
+       return jobs_invoke3(timeout, (job_cb_t)callback, arg, NULL, NULL);
 }
 
+/**
+ * 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, (void(*)(int,void*,void*,void*))callback, arg1, arg2, NULL);
+       return jobs_invoke3(timeout, (job_cb_t)callback, arg1, arg2, NULL);
 }
 
+/**
+ * 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)
 {
        struct thread *t = arg1;
        pthread_mutex_lock(&mutex);
        t->stop = 1;
+       if (t->waits)
+               pthread_cond_broadcast(&cond);
        pthread_mutex_unlock(&mutex);
 }
 
-/* invoke the job to the 'callback' using a separate thread if available */
+/**
+ * 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
+ */
 int jobs_invoke3(
                int timeout,
                void (*callback)(int, void*, void *, void*),
@@ -480,9 +668,7 @@ int jobs_invoke3(
                void *arg2,
                void *arg3)
 {
-       const char *info;
        struct job *job1, *job2;
-       int rc;
        struct thread me;
        
        pthread_mutex_lock(&mutex);
@@ -491,47 +677,75 @@ int jobs_invoke3(
        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;
-               info = "out of memory";
-               goto error;
-       }
-
-       /* start a thread if needed */
-       rc = start_one_thread_if_needed();
-       if (rc < 0) {
-               /* failed to start threading */
-               info = "can't start first thread";
-               goto error;
+               if (job1) {
+                       job1->next = free_jobs;
+                       free_jobs = job1;
+               }
+               if (job2) {
+                       job2->next = free_jobs;
+                       free_jobs = job2;
+               }
+               pthread_mutex_unlock(&mutex);
+               return -1;
        }
 
        /* queues the job */
        job_add2(job1, job2);
 
-       /* run untill stopped */
+       /* run until stopped */
        thread_run(&me);
        pthread_mutex_unlock(&mutex);
        return 0;
+}
 
-error:
-       if (job1) {
-               job1->next = free_jobs;
-               free_jobs = job1;
-       }
-       if (job2) {
-               job2->next = free_jobs;
-               free_jobs = job2;
-       }
-       ERROR("can't process job with threads: %s, %m", info);
+/**
+ * Initialise the job stuff.
+ * @param allowed_count Maximum count of thread for jobs (can be 0,
+ *                      see 'jobs_add_me' for merging new threads)
+ * @param start_count   Count of thread to start now, must be lower.
+ * @param waiter_count  Maximum count of jobs that can be waiting.
+ * @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 rc, launched;
+
+       assert(allowed_count >= 0);
+       assert(start_count >= 0);
+       assert(waiter_count > 0);
+       assert(start_count <= allowed_count);
+
+       /* records the allowed count */
+       allowed = allowed_count;
+       started = 0;
+       waiting = 0;
+       remains = waiter_count;
+
+       /* start at least one thread */
+       pthread_mutex_lock(&mutex);
+       launched = 0;
+       while (launched < start_count && start_one_thread() == 0)
+               launched++;
+       rc = -(launched != start_count);
        pthread_mutex_unlock(&mutex);
-       return -1;
+
+       /* end */
+       if (rc)
+               ERROR("Not all threads can be started");
+       return rc;
 }
 
-/* terminate all the threads and all pending requests */
+/**
+ * Terminate all the threads and cancel all pending jobs.
+ */
 void jobs_terminate()
 {
        struct job *job, *head, *tail;
-       pthread_t me, other;
+       pthread_t me, *others;
        struct thread *t;
+       int count;
 
        /* how am i? */
        me = pthread_self();
@@ -539,23 +753,42 @@ void jobs_terminate()
        /* request all threads to stop */
        pthread_mutex_lock(&mutex);
        allowed = 0;
-       for(;;) {
-               /* search the next thread to stop */
-               t = threads;
-               while (t && pthread_equal(t->tid, me))
-                       t = t->next;
-               if (!t)
-                       break;
-               /* stop it */
-               other = t->tid;
+
+       /* count the number of threads */
+       count = 0;
+       t = threads;
+       while (t) {
+               if (!t->upper && !pthread_equal(t->tid, me))
+                       count++;
+               t = t->next;
+       }
+
+       /* fill the array of threads */
+       others = alloca(count * sizeof *others);
+       count = 0;
+       t = threads;
+       while (t) {
+               if (!t->upper && !pthread_equal(t->tid, me))
+                       others[count++] = t->tid;
+               t = t->next;
+       }
+
+       /* stops the threads */
+       t = threads;
+       while (t) {
                t->stop = 1;
-               pthread_mutex_unlock(&mutex);
-               pthread_cond_broadcast(&cond);
-               pthread_join(other, NULL);
-               pthread_mutex_lock(&mutex);
+               t = t->next;
        }
 
+       /* wait the threads */
+       pthread_cond_broadcast(&cond);
+       pthread_mutex_unlock(&mutex);
+       while (count)
+               pthread_join(others[--count], NULL);
+       pthread_mutex_lock(&mutex);
+
        /* cancel pending jobs of other threads */
+       remains = 0;
        head = first_job;
        first_job = NULL;
        tail = NULL;
@@ -587,24 +820,13 @@ void jobs_terminate()
        pthread_mutex_unlock(&mutex);
 }
 
-int jobs_add_event_loop(void *key, int timeout, void (*evloop)(int signum, void*), void *closure)
-{
-       struct job *job;
-
-       pthread_mutex_lock(&mutex);
-       job = job_create(key, timeout, (void (*)(int,  void *, void *, void *))evloop, closure, NULL, NULL);
-       if (job) {
-               /* adds the loop */
-               job->next = first_evloop;
-               first_evloop = job;
-
-               /* signal the loop */
-               pthread_cond_signal(&cond);
-       }
-       pthread_mutex_unlock(&mutex);
-       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;
@@ -626,3 +848,66 @@ int jobs_add_me()
 }
 
 
+struct sd_event *jobs_get_sd_event()
+{
+       struct events *events;
+       struct thread *me;
+       int rc;
+
+       pthread_mutex_lock(&mutex);
+
+       /* 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");
+                                       else {
+                                               free(events);
+                                               ERROR("creation of sd_event failed: %m");
+                                               events = NULL;
+                                               errno = -rc;
+                                       } 
+                               }
+                       }
+               }
+               if (events) {
+                       /* */
+                       me = current;
+                       if (me) {
+                               events->runs = 1;
+                               me->events = events;
+                       } else {
+                               WARNING("event returned for unknown thread!");
+                       }
+               }
+       }
+       pthread_mutex_unlock(&mutex);
+       return events ? events->event : NULL;
+}
+