X-Git-Url: https://gerrit.automotivelinux.org/gerrit/gitweb?a=blobdiff_plain;f=src%2Fjobs.c;h=26fc01571dba753a0165dc28299ef62c31c41512;hb=44b24775deb71c45c474ff97553a1514674bf2ad;hp=9c81700a54f84535a3b6900ca54488603c353ae3;hpb=cab05ba72abeb335b456b0820e053c94b91986e7;p=src%2Fapp-framework-binder.git diff --git a/src/jobs.c b/src/jobs.c index 9c81700a..26fc0157 100644 --- a/src/jobs.c +++ b/src/jobs.c @@ -42,10 +42,9 @@ #endif #include "jobs.h" -#include "fdev-epoll.h" #include "sig-monitor.h" #include "verbose.h" - +#include "fdev-epoll.h" #if 0 #define _alert_ "do you really want to remove signal monitoring?" #define sig_monitor_init_timeouts() ((void)0) @@ -92,8 +91,8 @@ struct thread struct thread *upper; /**< upper same thread */ struct job *job; /**< currently processed job */ pthread_t tid; /**< the thread id */ - unsigned stop: 1; /**< stop requested */ - unsigned waits: 1; /**< is waiting? */ + volatile unsigned stop: 1; /**< stop requested */ + volatile unsigned waits: 1; /**< is waiting? */ }; /** @@ -133,6 +132,9 @@ static struct job *free_jobs; /* event loop */ static struct evloop evloop[1]; static struct fdev_epoll *fdevepoll; +#if !defined(REMOVE_SYSTEMD_EVENT) +__attribute__((unused)) +#endif static int waitevt; /** @@ -336,6 +338,9 @@ static void evloop_run(int signum, void *arg) * flow * @param arg the events to run */ +#if !defined(REMOVE_SYSTEMD_EVENT) +__attribute__((unused)) +#endif static void monitored_wait_and_dispatch(int signum, void *arg) { struct fdev_epoll *fdev_epoll = arg; @@ -355,6 +360,9 @@ static void thread_run(volatile struct thread *me) { struct thread **prv; struct job *job; +#if !defined(REMOVE_SYSTEMD_EVENT) + struct evloop *el; +#endif /* initialize description of itself and link it in the list */ me->tid = pthread_self(); @@ -378,7 +386,7 @@ static void thread_run(volatile struct thread *me) } /* get a job */ - job = job_get(first_job); + job = job_get(); if (job) { /* prepare running the job */ remains++; /* increases count of job that can wait */ @@ -392,6 +400,28 @@ static void thread_run(volatile struct thread *me) /* release the run job */ job_release(job); +#if !defined(REMOVE_SYSTEMD_EVENT) + } else { + /* no job, check events */ + el = &evloop[0]; + if (el->sdev && !__atomic_load_n(&el->state, __ATOMIC_RELAXED)) { + /* run the events */ + __atomic_store_n(&el->state, EVLOOP_STATE_LOCK|EVLOOP_STATE_RUN|EVLOOP_STATE_WAIT, __ATOMIC_RELAXED); + current_evloop = el; + pthread_mutex_unlock(&mutex); + sig_monitor(0, evloop_run, el); + pthread_mutex_lock(&mutex); + } else { + /* no job and not events */ + running--; + if (!running) + ERROR("Entering job deep sleep! Check your bindings."); + me->waits = 1; + pthread_cond_wait(&cond, &mutex); + me->waits = 0; + running++; + } +#else } else if (waitevt) { /* no job and not events */ running--; @@ -408,6 +438,7 @@ static void thread_run(volatile struct thread *me) sig_monitor(0, monitored_wait_and_dispatch, get_fdevepoll()); pthread_mutex_lock(&mutex); waitevt = 0; +#endif } } @@ -604,7 +635,7 @@ static int do_sync( * of interrupted flow, the context 'closure' as given and * a 'jobloop' reference that must be used when the job is * terminated to unlock the current execution flow. - * @param arg the argument to the callback + * @param closure the argument to the callback * @return 0 on success or -1 in case of error */ int jobs_enter( @@ -685,12 +716,15 @@ static int on_evloop_efd(sd_event_source *s, int fd, uint32_t revents, void *use struct evloop *evloop = userdata; read(evloop->efd, &x, sizeof x); pthread_mutex_lock(&mutex); - pthread_cond_broadcast(&evloop->cond); + pthread_cond_broadcast(&evloop->cond); pthread_mutex_unlock(&mutex); return 1; } /* temporary hack */ +#if !defined(REMOVE_SYSTEMD_EVENT) +__attribute__((unused)) +#endif static void evloop_callback(void *arg, uint32_t event, struct fdev *fdev) { sig_monitor(0, evloop_run, arg); @@ -727,6 +761,15 @@ static struct sd_event *get_sd_event_locked() rc = sd_event_add_io(el->sdev, NULL, el->efd, EPOLLIN, on_evloop_efd, el); if (rc < 0) { ERROR("can't register eventfd"); +#if !defined(REMOVE_SYSTEMD_EVENT) + sd_event_unref(el->sdev); + el->sdev = NULL; +error2: + close(el->efd); +error1: + return NULL; + } +#else goto error3; } /* handle the event loop */ @@ -744,6 +787,7 @@ error1: fdev_set_autoclose(el->fdev, 0); fdev_set_events(el->fdev, EPOLLIN); fdev_set_callback(el->fdev, evloop_callback, el); +#endif } /* attach the event loop to the current thread */