ARI: Add ability to raise arbitrary User Events
[asterisk/asterisk.git] / main / taskprocessor.c
index bd94103..0582f76 100644 (file)
@@ -1,7 +1,7 @@
 /*
  * Asterisk -- An open source telephony toolkit.
  *
- * Copyright (C) 2007-2008, Digium, Inc.
+ * Copyright (C) 2007-2013, Digium, Inc.
  *
  * Dwayne M. Hubbard <dhubbard@digium.com>
  *
@@ -37,7 +37,7 @@ ASTERISK_FILE_VERSION(__FILE__, "$Revision$")
 #include "asterisk/astobj2.h"
 #include "asterisk/cli.h"
 #include "asterisk/taskprocessor.h"
-
+#include "asterisk/sem.h"
 
 /*!
  * \brief tps_task structure is queued to a taskprocessor
@@ -48,11 +48,15 @@ ASTERISK_FILE_VERSION(__FILE__, "$Revision$")
  */
 struct tps_task {
        /*! \brief The execute() task callback function pointer */
-       int (*execute)(void *datap);
+       union {
+               int (*execute)(void *datap);
+               int (*execute_local)(struct ast_taskprocessor_local *local);
+       } callback;
        /*! \brief The data pointer for the task execute() function */
        void *datap;
        /*! \brief AST_LIST_ENTRY overhead */
        AST_LIST_ENTRY(tps_task) list;
+       unsigned int wants_local:1;
 };
 
 /*! \brief tps_taskprocessor_stats maintain statistics for a taskprocessor. */
@@ -69,6 +73,7 @@ struct ast_taskprocessor {
        const char *name;
        /*! \brief Taskprocessor statistics */
        struct tps_taskprocessor_stats *stats;
+       void *local_data;
        /*! \brief Taskprocessor current queue size */
        long tps_queue_size;
        /*! \brief Taskprocessor queue */
@@ -76,7 +81,29 @@ struct ast_taskprocessor {
        /*! \brief Taskprocessor singleton list entry */
        AST_LIST_ENTRY(ast_taskprocessor) list;
        struct ast_taskprocessor_listener *listener;
+       /*! Indicates if the taskprocessor is currently executing a task */
+       unsigned int executing:1;
+};
+
+/*!
+ * \brief A listener for taskprocessors
+ *
+ * \since 12.0.0
+ *
+ * When a taskprocessor's state changes, the listener
+ * is notified of the change. This allows for tasks
+ * to be addressed in whatever way is appropriate for
+ * the module using the taskprocessor.
+ */
+struct ast_taskprocessor_listener {
+       /*! The callbacks the taskprocessor calls into to notify of state changes */
+       const struct ast_taskprocessor_listener_callbacks *callbacks;
+       /*! The taskprocessor that the listener is listening to */
+       struct ast_taskprocessor *tps;
+       /*! Data private to the listener */
+       void *user_data;
 };
+
 #define TPS_MAX_BUCKETS 7
 /*! \brief tps_singletons is the astobj2 container for taskprocessor singletons */
 static struct ao2_container *tps_singletons;
@@ -92,9 +119,6 @@ static int tps_hash_cb(const void *obj, const int flags);
 /*! \brief The astobj2 compare callback for taskprocessors */
 static int tps_cmp_cb(void *obj, void *arg, int flags);
 
-/*! \brief The task processing function executed by a taskprocessor */
-static void *tps_processing_function(void *data);
-
 /*! \brief Destroy the taskprocessor when its refcount reaches zero */
 static void tps_taskprocessor_destroy(void *tps);
 
@@ -117,85 +141,129 @@ static struct ast_cli_entry taskprocessor_clis[] = {
 
 struct default_taskprocessor_listener_pvt {
        pthread_t poll_thread;
-       ast_mutex_t lock;
-       ast_cond_t cond;
-       int wake_up;
        int dead;
+       struct ast_sem sem;
 };
 
-static void default_tps_wake_up(struct default_taskprocessor_listener_pvt *pvt, int should_die)
+static void default_listener_pvt_destroy(struct default_taskprocessor_listener_pvt *pvt)
 {
-       SCOPED_MUTEX(lock, &pvt->lock); 
-       pvt->wake_up = 1;
-       pvt->dead = should_die;
-       ast_cond_signal(&pvt->cond);
+       ast_assert(pvt->dead);
+       ast_sem_destroy(&pvt->sem);
+       ast_free(pvt);
 }
 
-static void default_listener_destroy(void *obj)
+static void default_listener_pvt_dtor(struct ast_taskprocessor_listener *listener)
 {
-       struct ast_taskprocessor_listener *listener = obj;
-       struct default_taskprocessor_listener_pvt *pvt = listener->private_data;
+       struct default_taskprocessor_listener_pvt *pvt = listener->user_data;
 
-       default_tps_wake_up(pvt, 1);
-       pthread_join(pvt->poll_thread, NULL);
-       pvt->poll_thread = AST_PTHREADT_NULL;
-       ast_mutex_destroy(&pvt->lock);
-       ast_cond_destroy(&pvt->cond);
-       ast_free(pvt);
+       default_listener_pvt_destroy(pvt);
 
-       ao2_ref(listener->tps, -1);
-       listener->tps = NULL;
+       listener->user_data = NULL;
 }
 
-static int default_tps_idle(struct default_taskprocessor_listener_pvt *pvt)
+/*!
+ * \brief Function that processes tasks in the taskprocessor
+ * \internal
+ */
+static void *default_tps_processing_function(void *data)
 {
-       SCOPED_MUTEX(lock, &pvt->lock);
-       while (!pvt->wake_up) {
-               ast_cond_wait(&pvt->cond, lock);
+       struct ast_taskprocessor_listener *listener = data;
+       struct ast_taskprocessor *tps = listener->tps;
+       struct default_taskprocessor_listener_pvt *pvt = listener->user_data;
+       int sem_value;
+       int res;
+
+       while (!pvt->dead) {
+               res = ast_sem_wait(&pvt->sem);
+               if (res != 0 && errno != EINTR) {
+                       ast_log(LOG_ERROR, "ast_sem_wait(): %s\n",
+                               strerror(errno));
+                       /* Just give up */
+                       break;
+               }
+               ast_taskprocessor_execute(tps);
        }
-       pvt->wake_up = 0;
-       return pvt->dead;
+
+       /* No posting to a dead taskprocessor! */
+       res = ast_sem_getvalue(&pvt->sem, &sem_value);
+       ast_assert(res == 0 && sem_value == 0);
+
+       /* Free the shutdown reference (see default_listener_shutdown) */
+       ao2_t_ref(listener->tps, -1, "tps-shutdown");
+
+       return NULL;
 }
 
-/* this is the task processing worker function */
-static void *tps_processing_function(void *data)
+static int default_listener_start(struct ast_taskprocessor_listener *listener)
 {
-       struct ast_taskprocessor_listener *listener = data;
-       struct ast_taskprocessor *tps = listener->tps;
-       struct default_taskprocessor_listener_pvt *pvt = listener->private_data;
-       int dead = 0;
+       struct default_taskprocessor_listener_pvt *pvt = listener->user_data;
 
-       while (!dead) {
-               if (!ast_taskprocessor_execute(tps)) {
-                       dead = default_tps_idle(pvt);
-               }
+       if (ast_pthread_create(&pvt->poll_thread, NULL, default_tps_processing_function, listener)) {
+               return -1;
        }
-       return NULL;
+
+       return 0;
 }
 
 static void default_task_pushed(struct ast_taskprocessor_listener *listener, int was_empty)
 {
-       struct default_taskprocessor_listener_pvt *pvt = listener->private_data;
+       struct default_taskprocessor_listener_pvt *pvt = listener->user_data;
 
-       if (was_empty) {
-               default_tps_wake_up(pvt, 0);
+       if (ast_sem_post(&pvt->sem) != 0) {
+               ast_log(LOG_ERROR, "Failed to notify of enqueued task: %s\n",
+                       strerror(errno));
        }
 }
 
-static void default_emptied(struct ast_taskprocessor_listener *listener)
+static int default_listener_die(void *data)
 {
-       /* No-op */
+       struct default_taskprocessor_listener_pvt *pvt = data;
+       pvt->dead = 1;
+       return 0;
 }
 
-static struct ast_taskprocessor_listener_callbacks default_listener_callbacks = {
+static void default_listener_shutdown(struct ast_taskprocessor_listener *listener)
+{
+       struct default_taskprocessor_listener_pvt *pvt = listener->user_data;
+       int res;
+
+       /* Hold a reference during shutdown */
+       ao2_t_ref(listener->tps, +1, "tps-shutdown");
+
+       ast_taskprocessor_push(listener->tps, default_listener_die, pvt);
+
+       ast_assert(pvt->poll_thread != AST_PTHREADT_NULL);
+
+       if (pthread_equal(pthread_self(), pvt->poll_thread)) {
+               res = pthread_detach(pvt->poll_thread);
+               if (res != 0) {
+                       ast_log(LOG_ERROR, "pthread_detach(): %s\n", strerror(errno));
+               }
+       } else {
+               res = pthread_join(pvt->poll_thread, NULL);
+               if (res != 0) {
+                       ast_log(LOG_ERROR, "pthread_join(): %s\n", strerror(errno));
+               }
+       }
+       pvt->poll_thread = AST_PTHREADT_NULL;
+}
+
+static const struct ast_taskprocessor_listener_callbacks default_listener_callbacks = {
+       .start = default_listener_start,
        .task_pushed = default_task_pushed,
-       .emptied = default_emptied,
+       .shutdown = default_listener_shutdown,
+       .dtor = default_listener_pvt_dtor,
 };
 
-/*! \internal \brief Clean up resources on Asterisk shutdown */
+/*!
+ * \internal
+ * \brief Clean up resources on Asterisk shutdown
+ */
 static void tps_shutdown(void)
 {
+       ast_cli_unregister_multiple(taskprocessor_clis, ARRAY_LEN(taskprocessor_clis));
        ao2_t_ref(tps_singletons, -1, "Unref tps_singletons in shutdown");
+       tps_singletons = NULL;
 }
 
 /* initialize the taskprocessor container and register CLI operations */
@@ -219,19 +287,48 @@ int ast_tps_init(void)
 static struct tps_task *tps_task_alloc(int (*task_exe)(void *datap), void *datap)
 {
        struct tps_task *t;
-       if ((t = ast_calloc(1, sizeof(*t)))) {
-               t->execute = task_exe;
-               t->datap = datap;
+       if (!task_exe) {
+               ast_log(LOG_ERROR, "task_exe is NULL!\n");
+               return NULL;
+       }
+
+       t = ast_calloc(1, sizeof(*t));
+       if (!t) {
+               ast_log(LOG_ERROR, "failed to allocate task!\n");
+               return NULL;
        }
+
+       t->callback.execute = task_exe;
+       t->datap = datap;
+
+       return t;
+}
+
+static struct tps_task *tps_task_alloc_local(int (*task_exe)(struct ast_taskprocessor_local *local), void *datap)
+{
+       struct tps_task *t;
+       if (!task_exe) {
+               ast_log(LOG_ERROR, "task_exe is NULL!\n");
+               return NULL;
+       }
+
+       t = ast_calloc(1, sizeof(*t));
+       if (!t) {
+               ast_log(LOG_ERROR, "failed to allocate task!\n");
+               return NULL;
+       }
+
+       t->callback.execute_local = task_exe;
+       t->datap = datap;
+       t->wants_local = 1;
+
        return t;
 }
 
 /* release task resources */
 static void *tps_task_free(struct tps_task *task)
 {
-       if (task) {
-               ast_free(task);
-       }
+       ast_free(task);
        return NULL;
 }
 
@@ -347,7 +444,7 @@ static char *cli_tps_report(struct ast_cli_entry *e, int cmd, struct ast_cli_arg
                qsize = p->tps_queue_size;
                maxqsize = p->stats->max_qsize;
                processed = p->stats->_tasks_processed_count;
-               ast_cli(a->fd, "\n%24s   %17ld %12ld %12ld", name, processed, qsize, maxqsize);
+               ast_cli(a->fd, "\n%24s   %17lu %12lu %12lu", name, processed, qsize, maxqsize);
                ao2_ref(p, -1);
        }
        ao2_iterator_destroy(&i);
@@ -378,6 +475,7 @@ static int tps_cmp_cb(void *obj, void *arg, int flags)
 static void tps_taskprocessor_destroy(void *tps)
 {
        struct ast_taskprocessor *t = tps;
+       struct tps_task *task;
 
        if (!tps) {
                ast_log(LOG_ERROR, "missing taskprocessor\n");
@@ -385,19 +483,16 @@ static void tps_taskprocessor_destroy(void *tps)
        }
        ast_debug(1, "destroying taskprocessor '%s'\n", t->name);
        /* free it */
-       if (t->stats) {
-               ast_free(t->stats);
-               t->stats = NULL;
-       }
+       ast_free(t->stats);
+       t->stats = NULL;
        ast_free((char *) t->name);
        if (t->listener) {
-               /* This code should not be reached since the listener
-                * should have been destroyed before the taskprocessor could
-                * be destroyed
-                */
                ao2_ref(t->listener, -1);
                t->listener = NULL;
        }
+       while ((task = AST_LIST_REMOVE_HEAD(&t->tps_queue, list))) {
+               tps_task_free(task);
+       }
 }
 
 /* pop the front task and return it */
@@ -405,15 +500,9 @@ static struct tps_task *tps_taskprocessor_pop(struct ast_taskprocessor *tps)
 {
        struct tps_task *task;
 
-       if (!tps) {
-               ast_log(LOG_ERROR, "missing taskprocessor\n");
-               return NULL;
-       }
-       ao2_lock(tps);
        if ((task = AST_LIST_REMOVE_HEAD(&tps->tps_queue, list))) {
                tps->tps_queue_size--;
        }
-       ao2_unlock(tps);
        return task;
 }
 
@@ -432,67 +521,67 @@ const char *ast_taskprocessor_name(struct ast_taskprocessor *tps)
        return tps->name;
 }
 
-static struct ast_taskprocessor_listener *default_listener_alloc(void)
+static void listener_shutdown(struct ast_taskprocessor_listener *listener)
+{
+       listener->callbacks->shutdown(listener);
+       ao2_ref(listener->tps, -1);
+}
+
+static void taskprocessor_listener_dtor(void *obj)
+{
+       struct ast_taskprocessor_listener *listener = obj;
+
+       if (listener->callbacks->dtor) {
+               listener->callbacks->dtor(listener);
+       }
+}
+
+struct ast_taskprocessor_listener *ast_taskprocessor_listener_alloc(const struct ast_taskprocessor_listener_callbacks *callbacks, void *user_data)
 {
        struct ast_taskprocessor_listener *listener;
-       struct default_taskprocessor_listener_pvt *pvt;
 
-       listener = ao2_alloc(sizeof(*listener), default_listener_destroy);
+       listener = ao2_alloc(sizeof(*listener), taskprocessor_listener_dtor);
        if (!listener) {
                return NULL;
        }
-       pvt = ast_calloc(1, sizeof(*pvt));
-       if (!pvt) {
-               ao2_ref(listener, -1);
-               return NULL;
-       }
-       listener->callbacks = &default_listener_callbacks;
-       listener->private_data = pvt;
-       ast_cond_init(&pvt->cond, NULL);
-       ast_mutex_init(&pvt->lock);
-       pvt->poll_thread = AST_PTHREADT_NULL;
-       if (ast_pthread_create(&pvt->poll_thread, NULL, tps_processing_function, listener) < 0) {
-               ao2_ref(listener, -1);
-               return NULL;
-       }
+       listener->callbacks = callbacks;
+       listener->user_data = user_data;
+
        return listener;
 }
 
-/* Provide a reference to a taskprocessor.  Create the taskprocessor if necessary, but don't
- * create the taskprocessor if we were told via ast_tps_options to return a reference only
- * if it already exists */
-struct ast_taskprocessor *ast_taskprocessor_get(const char *name, enum ast_tps_options create)
+struct ast_taskprocessor *ast_taskprocessor_listener_get_tps(const struct ast_taskprocessor_listener *listener)
 {
-       struct ast_taskprocessor *p;
-       struct ast_taskprocessor_listener *listener;
+       ao2_ref(listener->tps, +1);
+       return listener->tps;
+}
 
-       if (ast_strlen_zero(name)) {
-               ast_log(LOG_ERROR, "requesting a nameless taskprocessor!!!\n");
+void *ast_taskprocessor_listener_get_user_data(const struct ast_taskprocessor_listener *listener)
+{
+       return listener->user_data;
+}
+
+static void *default_listener_pvt_alloc(void)
+{
+       struct default_taskprocessor_listener_pvt *pvt;
+
+       pvt = ast_calloc(1, sizeof(*pvt));
+       if (!pvt) {
                return NULL;
        }
-       p = ao2_find(tps_singletons, name, OBJ_KEY);
-       if (p) {
-               ao2_unlock(tps_singletons);
-               return p;
-       }
-       if (create & TPS_REF_IF_EXISTS) {
-               /* calling function does not want a new taskprocessor to be created if it doesn't already exist */
+       pvt->poll_thread = AST_PTHREADT_NULL;
+       if (ast_sem_init(&pvt->sem, 0, 0) != 0) {
+               ast_log(LOG_ERROR, "ast_sem_init(): %s\n", strerror(errno));
+               ast_free(pvt);
                return NULL;
        }
-       /* Create a new taskprocessor. Start by creating a default listener */
-       listener = default_listener_alloc();
-
-       p = ast_taskprocessor_create_with_listener(name, listener);
-       ao2_ref(listener, -1);
-       return p;
-
+       return pvt;
 }
 
-struct ast_taskprocessor *ast_taskprocessor_create_with_listener(const char *name, struct ast_taskprocessor_listener *listener)
+static struct ast_taskprocessor *__allocate_taskprocessor(const char *name, struct ast_taskprocessor_listener *listener)
 {
        RAII_VAR(struct ast_taskprocessor *, p,
-                       ao2_alloc(sizeof(*p), tps_taskprocessor_destroy),
-                       ao2_cleanup);
+                       ao2_alloc(sizeof(*p), tps_taskprocessor_destroy), ao2_cleanup);
 
        if (!p) {
                ast_log(LOG_WARNING, "failed to create taskprocessor '%s'\n", name);
@@ -519,17 +608,84 @@ struct ast_taskprocessor *ast_taskprocessor_create_with_listener(const char *nam
                return NULL;
        }
 
+       if (p->listener->callbacks->start(p->listener)) {
+               ast_log(LOG_ERROR, "Unable to start taskprocessor listener for taskprocessor %s\n", p->name);
+               ast_taskprocessor_unreference(p);
+               return NULL;
+       }
+
        /* RAII_VAR will decrement the refcount at the end of the function.
         * Since we want to pass back a reference to p, we bump the refcount
         */
        ao2_ref(p, +1);
        return p;
+
+}
+
+/* Provide a reference to a taskprocessor.  Create the taskprocessor if necessary, but don't
+ * create the taskprocessor if we were told via ast_tps_options to return a reference only
+ * if it already exists */
+struct ast_taskprocessor *ast_taskprocessor_get(const char *name, enum ast_tps_options create)
+{
+       struct ast_taskprocessor *p;
+       struct ast_taskprocessor_listener *listener;
+       struct default_taskprocessor_listener_pvt *pvt;
+
+       if (ast_strlen_zero(name)) {
+               ast_log(LOG_ERROR, "requesting a nameless taskprocessor!!!\n");
+               return NULL;
+       }
+       p = ao2_find(tps_singletons, name, OBJ_KEY);
+       if (p) {
+               return p;
+       }
+       if (create & TPS_REF_IF_EXISTS) {
+               /* calling function does not want a new taskprocessor to be created if it doesn't already exist */
+               return NULL;
+       }
+       /* Create a new taskprocessor. Start by creating a default listener */
+       pvt = default_listener_pvt_alloc();
+       if (!pvt) {
+               return NULL;
+       }
+       listener = ast_taskprocessor_listener_alloc(&default_listener_callbacks, pvt);
+       if (!listener) {
+               default_listener_pvt_destroy(pvt);
+               return NULL;
+       }
+
+       p = __allocate_taskprocessor(name, listener);
+       if (!p) {
+               ao2_ref(listener, -1);
+               return NULL;
+       }
+
+       /* Unref listener here since the taskprocessor has gained a reference to the listener */
+       ao2_ref(listener, -1);
+       return p;
+}
+
+struct ast_taskprocessor *ast_taskprocessor_create_with_listener(const char *name, struct ast_taskprocessor_listener *listener)
+{
+       struct ast_taskprocessor *p = ao2_find(tps_singletons, name, OBJ_KEY);
+
+       if (p) {
+               ast_taskprocessor_unreference(p);
+               return NULL;
+       }
+       return __allocate_taskprocessor(name, listener);
+}
+
+void ast_taskprocessor_set_local(struct ast_taskprocessor *tps,
+       void *local_data)
+{
+       SCOPED_AO2LOCK(lock, tps);
+       tps->local_data = local_data;
 }
 
 /* decrement the taskprocessor reference count and unlink from the container if necessary */
 void *ast_taskprocessor_unreference(struct ast_taskprocessor *tps)
 {
-       struct ast_taskprocessor_listener *listener;
        if (!tps) {
                return NULL;
        }
@@ -543,49 +699,82 @@ void *ast_taskprocessor_unreference(struct ast_taskprocessor *tps)
         * 3. The listener
         */
        ao2_unlink(tps_singletons, tps);
-       listener = tps->listener;
-       tps->listener = NULL;
-       ao2_ref(listener, -1);
+       listener_shutdown(tps->listener);
        return NULL;
 }
 
 /* push the task into the taskprocessor queue */
-int ast_taskprocessor_push(struct ast_taskprocessor *tps, int (*task_exe)(void *datap), void *datap)
+static int taskprocessor_push(struct ast_taskprocessor *tps, struct tps_task *t)
 {
-       struct tps_task *t;
        int previous_size;
+       int was_empty;
 
-       if (!tps || !task_exe) {
-               ast_log(LOG_ERROR, "%s is missing!!\n", (tps) ? "task callback" : "taskprocessor");
+       if (!tps) {
+               ast_log(LOG_ERROR, "tps is NULL!\n");
                return -1;
        }
-       if (!(t = tps_task_alloc(task_exe, datap))) {
-               ast_log(LOG_ERROR, "failed to allocate task!  Can't push to '%s'\n", tps->name);
+
+       if (!t) {
+               ast_log(LOG_ERROR, "t is NULL!\n");
                return -1;
        }
+
        ao2_lock(tps);
        AST_LIST_INSERT_TAIL(&tps->tps_queue, t, list);
        previous_size = tps->tps_queue_size++;
+       /* The currently executing task counts as still in queue */
+       was_empty = tps->executing ? 0 : previous_size == 0;
        ao2_unlock(tps);
-       tps->listener->callbacks->task_pushed(tps->listener, previous_size ? 0 : 1);
+       tps->listener->callbacks->task_pushed(tps->listener, was_empty);
        return 0;
 }
 
+int ast_taskprocessor_push(struct ast_taskprocessor *tps, int (*task_exe)(void *datap), void *datap)
+{
+       return taskprocessor_push(tps, tps_task_alloc(task_exe, datap));
+}
+
+int ast_taskprocessor_push_local(struct ast_taskprocessor *tps, int (*task_exe)(struct ast_taskprocessor_local *datap), void *datap)
+{
+       return taskprocessor_push(tps, tps_task_alloc_local(task_exe, datap));
+}
+
 int ast_taskprocessor_execute(struct ast_taskprocessor *tps)
 {
+       struct ast_taskprocessor_local local;
        struct tps_task *t;
        int size;
 
-       if (!(t = tps_taskprocessor_pop(tps))) {
+       ao2_lock(tps);
+       t = tps_taskprocessor_pop(tps);
+       if (!t) {
+               ao2_unlock(tps);
                return 0;
        }
 
-       t->execute(t->datap);
+       tps->executing = 1;
 
+       if (t->wants_local) {
+               local.local_data = tps->local_data;
+               local.data = t->datap;
+       }
+       ao2_unlock(tps);
+
+       if (t->wants_local) {
+               t->callback.execute_local(&local);
+       } else {
+               t->callback.execute(t->datap);
+       }
        tps_task_free(t);
 
        ao2_lock(tps);
+       /* We need to check size in the same critical section where we reset the
+        * executing bit. Avoids a race condition where a task is pushed right
+        * after we pop an empty stack.
+        */
+       tps->executing = 0;
        size = tps_taskprocessor_depth(tps);
+       /* If we executed a task, bump the stats */
        if (tps->stats) {
                tps->stats->_tasks_processed_count++;
                if (size > tps->stats->max_qsize) {
@@ -594,9 +783,9 @@ int ast_taskprocessor_execute(struct ast_taskprocessor *tps)
        }
        ao2_unlock(tps);
 
-       if (size == 0) {
+       /* If we executed a task, check for the transition to empty */
+       if (size == 0 && tps->listener->callbacks->emptied) {
                tps->listener->callbacks->emptied(tps->listener);
-               return 0;
        }
-       return 1;
+       return size > 0;
 }