]> code.delx.au - pulseaudio/blobdiff - src/pulse/mainloop.c
merge 'lennart' branch back into trunk.
[pulseaudio] / src / pulse / mainloop.c
index 32f1a845d4655261e840d6fe14828447e8df82b1..ad4e4e97df5473fa19281294a4416882a25c8170 100644 (file)
@@ -2,17 +2,20 @@
 
 /***
   This file is part of PulseAudio.
+
+  Copyright 2004-2006 Lennart Poettering
+  Copyright 2006 Pierre Ossman <ossman@cendio.se> for Cendio AB
+
   PulseAudio is free software; you can redistribute it and/or modify
   it under the terms of the GNU Lesser General Public License as published
   by the Free Software Foundation; either version 2 of the License,
   or (at your option) any later version.
+
   PulseAudio is distributed in the hope that it will be useful, but
   WITHOUT ANY WARRANTY; without even the implied warranty of
   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
   General Public License for more details.
+
   You should have received a copy of the GNU Lesser General Public License
   along with PulseAudio; if not, write to the Free Software
   Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307
 #include <unistd.h>
 #include <stdlib.h>
 #include <string.h>
-#include <assert.h>
 #include <fcntl.h>
 #include <errno.h>
 
-#ifdef HAVE_SYS_POLL_H
-#include <sys/poll.h>
+#ifdef HAVE_POLL_H
+#include <poll.h>
 #else
-#include "../pulsecore/poll.h"
+#include <pulsecore/poll.h>
 #endif
 
-#include "../pulsecore/winsock.h"
-
 #ifndef HAVE_PIPE
-#include "../pulsecore/pipe.h"
+#include <pulsecore/pipe.h>
 #endif
 
-#include <pulsecore/core-error.h>
 #include <pulse/timeval.h>
 #include <pulse/xmalloc.h>
 
 #include <pulsecore/core-util.h>
-#include <pulsecore/idxset.h>
+#include <pulsecore/llist.h>
 #include <pulsecore/log.h>
+#include <pulsecore/core-error.h>
+#include <pulsecore/winsock.h>
+#include <pulsecore/macro.h>
 
 #include "mainloop.h"
 
 struct pa_io_event {
     pa_mainloop *mainloop;
     int dead;
+
     int fd;
     pa_io_event_flags_t events;
-    void (*callback) (pa_mainloop_api*a, pa_io_event *e, int fd, pa_io_event_flags_t f, void *userdata);
     struct pollfd *pollfd;
+
+    pa_io_event_cb_t callback;
     void *userdata;
-    void (*destroy_callback) (pa_mainloop_api*a, pa_io_event *e, void *userdata);
+    pa_io_event_destroy_cb_t destroy_callback;
+
+    PA_LLIST_FIELDS(pa_io_event);
 };
 
 struct pa_time_event {
     pa_mainloop *mainloop;
     int dead;
+
     int enabled;
     struct timeval timeval;
-    void (*callback)(pa_mainloop_api*a, pa_time_event *e, const struct timeval*tv, void *userdata);
+
+    pa_time_event_cb_t callback;
     void *userdata;
-    void (*destroy_callback) (pa_mainloop_api*a, pa_time_event *e, void *userdata);
+    pa_time_event_destroy_cb_t destroy_callback;
+
+    PA_LLIST_FIELDS(pa_time_event);
 };
 
 struct pa_defer_event {
     pa_mainloop *mainloop;
     int dead;
+
     int enabled;
-    void (*callback)(pa_mainloop_api*a, pa_defer_event*e, void *userdata);
+
+    pa_defer_event_cb_t callback;
     void *userdata;
-    void (*destroy_callback) (pa_mainloop_api*a, pa_defer_event *e, void *userdata);
+    pa_defer_event_destroy_cb_t destroy_callback;
+
+    PA_LLIST_FIELDS(pa_defer_event);
 };
 
 struct pa_mainloop {
-    pa_idxset *io_events, *time_events, *defer_events;
-    int io_events_scan_dead, defer_events_scan_dead, time_events_scan_dead;
+    PA_LLIST_HEAD(pa_io_event, io_events);
+    PA_LLIST_HEAD(pa_time_event, time_events);
+    PA_LLIST_HEAD(pa_defer_event, defer_events);
+
+    int n_enabled_defer_events, n_enabled_time_events, n_io_events;
+    int io_events_please_scan, time_events_please_scan, defer_events_please_scan;
 
     struct pollfd *pollfds;
     unsigned max_pollfds, n_pollfds;
     int rebuild_pollfds;
 
     int prepared_timeout;
+    pa_time_event *cached_next_time_event;
 
     int quit, retval;
     pa_mainloop_api api;
 
-    int deferred_pending;
-
     int wakeup_pipe[2];
+    int wakeup_pipe_type;
+    int wakeup_requested;
 
     enum {
         STATE_PASSIVE,
@@ -111,33 +130,55 @@ struct pa_mainloop {
 
     pa_poll_func poll_func;
     void *poll_func_userdata;
+    int poll_func_ret;
 };
 
+static short map_flags_to_libc(pa_io_event_flags_t flags) {
+    return
+        (flags & PA_IO_EVENT_INPUT ? POLLIN : 0) |
+        (flags & PA_IO_EVENT_OUTPUT ? POLLOUT : 0) |
+        (flags & PA_IO_EVENT_ERROR ? POLLERR : 0) |
+        (flags & PA_IO_EVENT_HANGUP ? POLLHUP : 0);
+}
+
+static pa_io_event_flags_t map_flags_from_libc(short flags) {
+    return
+        (flags & POLLIN ? PA_IO_EVENT_INPUT : 0) |
+        (flags & POLLOUT ? PA_IO_EVENT_OUTPUT : 0) |
+        (flags & POLLERR ? PA_IO_EVENT_ERROR : 0) |
+        (flags & POLLHUP ? PA_IO_EVENT_HANGUP : 0);
+}
+
 /* IO events */
 static pa_io_event* mainloop_io_new(
-    pa_mainloop_api*a,
-    int fd,
-    pa_io_event_flags_t events,
-    void (*callback) (pa_mainloop_api*a, pa_io_event *e, int fd, pa_io_event_flags_t events, void *userdata),
-    void *userdata) {
-    
+        pa_mainloop_api*a,
+        int fd,
+        pa_io_event_flags_t events,
+        pa_io_event_cb_t callback,
+        void *userdata) {
+
     pa_mainloop *m;
     pa_io_event *e;
 
-    assert(a && a->userdata && fd >= 0 && callback);
+    pa_assert(a);
+    pa_assert(a->userdata);
+    pa_assert(fd >= 0);
+    pa_assert(callback);
+
     m = a->userdata;
-    assert(a == &m->api);
+    pa_assert(a == &m->api);
 
-    e = pa_xmalloc(sizeof(pa_io_event));
+    e = pa_xnew(pa_io_event, 1);
     e->mainloop = m;
     e->dead = 0;
 
     e->fd = fd;
     e->events = events;
+    e->pollfd = NULL;
+
     e->callback = callback;
     e->userdata = userdata;
     e->destroy_callback = NULL;
-    e->pollfd = NULL;
 
 #ifdef OS_IS_WIN32
     {
@@ -153,14 +194,15 @@ static pa_io_event* mainloop_io_new(
         if ((select((SELECT_TYPE_ARG1) fd, NULL, NULL, SELECT_TYPE_ARG234 &xset,
                     SELECT_TYPE_ARG5 &tv) == -1) &&
              (WSAGetLastError() == WSAENOTSOCK)) {
-            pa_log_warn(__FILE__": WARNING: cannot monitor non-socket file descriptors.");
+            pa_log_warn("Cannot monitor non-socket file descriptors.");
             e->dead = 1;
         }
     }
 #endif
 
-    pa_idxset_put(m->io_events, e, NULL);
+    PA_LLIST_PREPEND(pa_io_event, m->io_events, e);
     m->rebuild_pollfds = 1;
+    m->n_io_events ++;
 
     pa_mainloop_wakeup(m);
 
@@ -168,48 +210,69 @@ static pa_io_event* mainloop_io_new(
 }
 
 static void mainloop_io_enable(pa_io_event *e, pa_io_event_flags_t events) {
-    assert(e && e->mainloop);
+    pa_assert(e);
+    pa_assert(!e->dead);
+
+    if (e->events == events)
+        return;
 
     e->events = events;
-    e->mainloop->rebuild_pollfds = 1;
+
+    if (e->pollfd)
+        e->pollfd->events = map_flags_to_libc(events);
+    else
+        e->mainloop->rebuild_pollfds = 1;
 
     pa_mainloop_wakeup(e->mainloop);
 }
 
 static void mainloop_io_free(pa_io_event *e) {
-    assert(e && e->mainloop);
+    pa_assert(e);
+    pa_assert(!e->dead);
 
-    e->dead = e->mainloop->io_events_scan_dead = e->mainloop->rebuild_pollfds = 1;
+    e->dead = 1;
+    e->mainloop->io_events_please_scan ++;
+
+    e->mainloop->n_io_events --;
+    e->mainloop->rebuild_pollfds = 1;
 
     pa_mainloop_wakeup(e->mainloop);
 }
 
-static void mainloop_io_set_destroy(pa_io_event *e, void (*callback)(pa_mainloop_api*a, pa_io_event *e, void *userdata)) {
-    assert(e);
+static void mainloop_io_set_destroy(pa_io_event *e, pa_io_event_destroy_cb_t callback) {
+    pa_assert(e);
+
     e->destroy_callback = callback;
 }
 
 /* Defer events */
-static pa_defer_event* mainloop_defer_new(pa_mainloop_api*a, void (*callback) (pa_mainloop_api*a, pa_defer_event *e, void *userdata), void *userdata) {
+static pa_defer_event* mainloop_defer_new(
+        pa_mainloop_api*a,
+        pa_defer_event_cb_t callback,
+        void *userdata) {
+
     pa_mainloop *m;
     pa_defer_event *e;
 
-    assert(a && a->userdata && callback);
+    pa_assert(a);
+    pa_assert(a->userdata);
+    pa_assert(callback);
+
     m = a->userdata;
-    assert(a == &m->api);
+    pa_assert(a == &m->api);
 
-    e = pa_xmalloc(sizeof(pa_defer_event));
+    e = pa_xnew(pa_defer_event, 1);
     e->mainloop = m;
     e->dead = 0;
 
     e->enabled = 1;
+    m->n_enabled_defer_events++;
+
     e->callback = callback;
     e->userdata = userdata;
     e->destroy_callback = NULL;
 
-    pa_idxset_put(m->defer_events, e, NULL);
-
-    m->deferred_pending++;
+    PA_LLIST_PREPEND(pa_defer_event, m->defer_events, e);
 
     pa_mainloop_wakeup(e->mainloop);
 
@@ -217,86 +280,134 @@ static pa_defer_event* mainloop_defer_new(pa_mainloop_api*a, void (*callback) (p
 }
 
 static void mainloop_defer_enable(pa_defer_event *e, int b) {
-    assert(e);
+    pa_assert(e);
+    pa_assert(!e->dead);
 
     if (e->enabled && !b) {
-        assert(e->mainloop->deferred_pending > 0);
-        e->mainloop->deferred_pending--;
+        pa_assert(e->mainloop->n_enabled_defer_events > 0);
+        e->mainloop->n_enabled_defer_events--;
     } else if (!e->enabled && b) {
-        e->mainloop->deferred_pending++;
+        e->mainloop->n_enabled_defer_events++;
         pa_mainloop_wakeup(e->mainloop);
     }
-    
+
     e->enabled = b;
 }
 
 static void mainloop_defer_free(pa_defer_event *e) {
-    assert(e);
-    e->dead = e->mainloop->defer_events_scan_dead = 1;
+    pa_assert(e);
+    pa_assert(!e->dead);
+
+    e->dead = 1;
+    e->mainloop->defer_events_please_scan ++;
 
     if (e->enabled) {
+        pa_assert(e->mainloop->n_enabled_defer_events > 0);
+        e->mainloop->n_enabled_defer_events--;
         e->enabled = 0;
-        assert(e->mainloop->deferred_pending > 0);
-        e->mainloop->deferred_pending--;
     }
 }
 
-static void mainloop_defer_set_destroy(pa_defer_event *e, void (*callback)(pa_mainloop_api*a, pa_defer_event *e, void *userdata)) {
-    assert(e);
+static void mainloop_defer_set_destroy(pa_defer_event *e, pa_defer_event_destroy_cb_t callback) {
+    pa_assert(e);
+    pa_assert(!e->dead);
+
     e->destroy_callback = callback;
 }
 
 /* Time events */
-static pa_time_event* mainloop_time_new(pa_mainloop_api*a, const struct timeval *tv, void (*callback) (pa_mainloop_api*a, pa_time_event*e, const struct timeval *tv, void *userdata), void *userdata) {
+static pa_time_event* mainloop_time_new(
+        pa_mainloop_api*a,
+        const struct timeval *tv,
+        pa_time_event_cb_t callback,
+        void *userdata) {
+
     pa_mainloop *m;
     pa_time_event *e;
 
-    assert(a && a->userdata && callback);
+    pa_assert(a);
+    pa_assert(a->userdata);
+    pa_assert(callback);
+
     m = a->userdata;
-    assert(a == &m->api);
+    pa_assert(a == &m->api);
 
-    e = pa_xmalloc(sizeof(pa_time_event));
+    e = pa_xnew(pa_time_event, 1);
     e->mainloop = m;
     e->dead = 0;
 
-    e->enabled = !!tv;
-    if (tv)
+    if ((e->enabled = !!tv)) {
         e->timeval = *tv;
 
+        m->n_enabled_time_events++;
+
+        if (m->cached_next_time_event) {
+            pa_assert(m->cached_next_time_event->enabled);
+
+            if (pa_timeval_cmp(tv, &m->cached_next_time_event->timeval) < 0)
+                m->cached_next_time_event = e;
+        }
+    }
+
     e->callback = callback;
     e->userdata = userdata;
     e->destroy_callback = NULL;
 
-    pa_idxset_put(m->time_events, e, NULL);
+    PA_LLIST_PREPEND(pa_time_event, m->time_events, e);
 
     if (e->enabled)
         pa_mainloop_wakeup(m);
-    
+
     return e;
 }
 
 static void mainloop_time_restart(pa_time_event *e, const struct timeval *tv) {
-    assert(e);
+    pa_assert(e);
+    pa_assert(!e->dead);
 
-    if (tv) {
-        e->enabled = 1;
-        e->timeval = *tv;
+    if (e->enabled && !tv) {
+        pa_assert(e->mainloop->n_enabled_time_events > 0);
+        e->mainloop->n_enabled_time_events--;
+    } else if (!e->enabled && tv)
+        e->mainloop->n_enabled_time_events++;
 
+    if ((e->enabled = !!tv)) {
+        e->timeval = *tv;
         pa_mainloop_wakeup(e->mainloop);
-    } else
-        e->enabled = 0;
+    }
+
+    if (e->mainloop->cached_next_time_event && e->enabled) {
+        pa_assert(e->mainloop->cached_next_time_event->enabled);
+
+        if (pa_timeval_cmp(tv, &e->mainloop->cached_next_time_event->timeval) < 0)
+            e->mainloop->cached_next_time_event = e;
+    } else if (e->mainloop->cached_next_time_event == e)
+        e->mainloop->cached_next_time_event = NULL;
 }
 
 static void mainloop_time_free(pa_time_event *e) {
-    assert(e);
+    pa_assert(e);
+    pa_assert(!e->dead);
+
+    e->dead = 1;
+    e->mainloop->time_events_please_scan ++;
+
+    if (e->enabled) {
+        pa_assert(e->mainloop->n_enabled_time_events > 0);
+        e->mainloop->n_enabled_time_events--;
+        e->enabled = 0;
+    }
 
-    e->dead = e->mainloop->time_events_scan_dead = 1;
+    if (e->mainloop->cached_next_time_event == e)
+        e->mainloop->cached_next_time_event = NULL;
 
     /* no wakeup needed here. Think about it! */
 }
 
-static void mainloop_time_set_destroy(pa_time_event *e, void (*callback)(pa_mainloop_api*a, pa_time_event *e, void *userdata)) {
-    assert(e);
+static void mainloop_time_set_destroy(pa_time_event *e, pa_time_event_destroy_cb_t callback) {
+    pa_assert(e);
+    pa_assert(!e->dead);
+
     e->destroy_callback = callback;
 }
 
@@ -304,13 +415,15 @@ static void mainloop_time_set_destroy(pa_time_event *e, void (*callback)(pa_main
 
 static void mainloop_quit(pa_mainloop_api*a, int retval) {
     pa_mainloop *m;
-    assert(a && a->userdata);
+
+    pa_assert(a);
+    pa_assert(a->userdata);
     m = a->userdata;
-    assert(a == &m->api);
+    pa_assert(a == &m->api);
 
     pa_mainloop_quit(m, retval);
 }
-    
+
 static const pa_mainloop_api vtable = {
     .userdata = NULL,
 
@@ -323,37 +436,43 @@ static const pa_mainloop_api vtable = {
     .time_restart = mainloop_time_restart,
     .time_free = mainloop_time_free,
     .time_set_destroy = mainloop_time_set_destroy,
-    
+
     .defer_new = mainloop_defer_new,
     .defer_enable = mainloop_defer_enable,
     .defer_free = mainloop_defer_free,
     .defer_set_destroy = mainloop_defer_set_destroy,
-    
+
     .quit = mainloop_quit,
 };
 
 pa_mainloop *pa_mainloop_new(void) {
     pa_mainloop *m;
 
-    m = pa_xmalloc(sizeof(pa_mainloop));
+    m = pa_xnew(pa_mainloop, 1);
 
+    m->wakeup_pipe_type = 0;
     if (pipe(m->wakeup_pipe) < 0) {
-        pa_log_error(__FILE__": ERROR: cannot create wakeup pipe");
+        pa_log_error("ERROR: cannot create wakeup pipe");
         pa_xfree(m);
         return NULL;
     }
 
-    pa_make_nonblock_fd(m->wakeup_pipe[0]);
-    pa_make_nonblock_fd(m->wakeup_pipe[1]);
+    pa_make_fd_nonblock(m->wakeup_pipe[0]);
+    pa_make_fd_nonblock(m->wakeup_pipe[1]);
+    pa_make_fd_cloexec(m->wakeup_pipe[0]);
+    pa_make_fd_cloexec(m->wakeup_pipe[1]);
+    m->wakeup_requested = 0;
 
-    m->io_events = pa_idxset_new(NULL, NULL);
-    m->defer_events = pa_idxset_new(NULL, NULL);
-    m->time_events = pa_idxset_new(NULL, NULL);
+    PA_LLIST_HEAD_INIT(pa_io_event, m->io_events);
+    PA_LLIST_HEAD_INIT(pa_time_event, m->time_events);
+    PA_LLIST_HEAD_INIT(pa_defer_event, m->defer_events);
 
-    assert(m->io_events && m->defer_events && m->time_events);
+    m->n_enabled_defer_events = m->n_enabled_time_events = m->n_io_events = 0;
+    m->io_events_please_scan = m->time_events_please_scan = m->defer_events_please_scan = 0;
+
+    m->cached_next_time_event = NULL;
+    m->prepared_timeout = 0;
 
-    m->io_events_scan_dead = m->defer_events_scan_dead = m->time_events_scan_dead = 0;
-    
     m->pollfds = NULL;
     m->max_pollfds = m->n_pollfds = 0;
     m->rebuild_pollfds = 1;
@@ -363,107 +482,155 @@ pa_mainloop *pa_mainloop_new(void) {
     m->api = vtable;
     m->api.userdata = m;
 
-    m->deferred_pending = 0;
-
     m->state = STATE_PASSIVE;
 
     m->poll_func = NULL;
     m->poll_func_userdata = NULL;
+    m->poll_func_ret = -1;
 
-    m->retval = -1;
-    
     return m;
 }
 
-static int io_foreach(void *p, uint32_t PA_GCC_UNUSED idx, int *del, void*userdata) {
-    pa_io_event *e = p;
-    int *all = userdata;
-    assert(e && del && all);
+static void cleanup_io_events(pa_mainloop *m, int force) {
+    pa_io_event *e;
 
-    if (!*all && !e->dead)
-        return 0;
-    
-    if (e->destroy_callback)
-        e->destroy_callback(&e->mainloop->api, e, e->userdata);
-    pa_xfree(e);
-    *del = 1;
-    return 0;
+    e = m->io_events;
+    while (e) {
+        pa_io_event *n = e->next;
+
+        if (!force && m->io_events_please_scan <= 0)
+            break;
+
+        if (force || e->dead) {
+            PA_LLIST_REMOVE(pa_io_event, m->io_events, e);
+
+            if (e->dead) {
+                pa_assert(m->io_events_please_scan > 0);
+                m->io_events_please_scan--;
+            }
+
+            if (e->destroy_callback)
+                e->destroy_callback(&m->api, e, e->userdata);
+
+            pa_xfree(e);
+
+            m->rebuild_pollfds = 1;
+        }
+
+        e = n;
+    }
+
+    pa_assert(m->io_events_please_scan == 0);
 }
 
-static int time_foreach(void *p, uint32_t PA_GCC_UNUSED idx, int *del, void*userdata) {
-    pa_time_event *e = p;
-    int *all = userdata;
-    assert(e && del && all);
+static void cleanup_time_events(pa_mainloop *m, int force) {
+    pa_time_event *e;
 
-    if (!*all && !e->dead)
-        return 0;
-    
-    if (e->destroy_callback)
-        e->destroy_callback(&e->mainloop->api, e, e->userdata);
-    pa_xfree(e);
-    *del = 1;
-    return 0;
+    e = m->time_events;
+    while (e) {
+        pa_time_event *n = e->next;
+
+        if (!force && m->time_events_please_scan <= 0)
+            break;
+
+        if (force || e->dead) {
+            PA_LLIST_REMOVE(pa_time_event, m->time_events, e);
+
+            if (e->dead) {
+                pa_assert(m->time_events_please_scan > 0);
+                m->time_events_please_scan--;
+            }
+
+            if (!e->dead && e->enabled) {
+                pa_assert(m->n_enabled_time_events > 0);
+                m->n_enabled_time_events--;
+                e->enabled = 0;
+            }
+
+            if (e->destroy_callback)
+                e->destroy_callback(&m->api, e, e->userdata);
+
+            pa_xfree(e);
+        }
+
+        e = n;
+    }
+
+    pa_assert(m->time_events_please_scan == 0);
 }
 
-static int defer_foreach(void *p, PA_GCC_UNUSED uint32_t idx, int *del, void*userdata) {
-    pa_defer_event *e = p;
-    int *all = userdata;
-    assert(e && del && all);
+static void cleanup_defer_events(pa_mainloop *m, int force) {
+    pa_defer_event *e;
 
-    if (!*all && !e->dead)
-        return 0;
-    
-    if (e->destroy_callback)
-        e->destroy_callback(&e->mainloop->api, e, e->userdata);
-    pa_xfree(e);
-    *del = 1;
-    return 0;
+    e = m->defer_events;
+    while (e) {
+        pa_defer_event *n = e->next;
+
+        if (!force && m->defer_events_please_scan <= 0)
+            break;
+
+        if (force || e->dead) {
+            PA_LLIST_REMOVE(pa_defer_event, m->defer_events, e);
+
+            if (e->dead) {
+                pa_assert(m->defer_events_please_scan > 0);
+                m->defer_events_please_scan--;
+            }
+
+            if (!e->dead && e->enabled) {
+                pa_assert(m->n_enabled_defer_events > 0);
+                m->n_enabled_defer_events--;
+                e->enabled = 0;
+            }
+
+            if (e->destroy_callback)
+                e->destroy_callback(&m->api, e, e->userdata);
+
+            pa_xfree(e);
+        }
+
+        e = n;
+    }
+
+    pa_assert(m->defer_events_please_scan == 0);
 }
 
-void pa_mainloop_free(pa_mainloop* m) {
-    int all = 1;
-    assert(m);
 
-    pa_idxset_foreach(m->io_events, io_foreach, &all);
-    pa_idxset_foreach(m->time_events, time_foreach, &all);
-    pa_idxset_foreach(m->defer_events, defer_foreach, &all);
+void pa_mainloop_free(pa_mainloop* m) {
+    pa_assert(m);
 
-    pa_idxset_free(m->io_events, NULL, NULL);
-    pa_idxset_free(m->time_events, NULL, NULL);
-    pa_idxset_free(m->defer_events, NULL, NULL);
+    cleanup_io_events(m, 1);
+    cleanup_defer_events(m, 1);
+    cleanup_time_events(m, 1);
 
     pa_xfree(m->pollfds);
 
-    if (m->wakeup_pipe[0] >= 0)
-        close(m->wakeup_pipe[0]);
-    if (m->wakeup_pipe[1] >= 0)
-        close(m->wakeup_pipe[1]);
+    pa_close_pipe(m->wakeup_pipe);
 
     pa_xfree(m);
 }
 
 static void scan_dead(pa_mainloop *m) {
-    int all = 0;
-    assert(m);
+    pa_assert(m);
+
+    if (m->io_events_please_scan)
+        cleanup_io_events(m, 0);
 
-    if (m->io_events_scan_dead)
-        pa_idxset_foreach(m->io_events, io_foreach, &all);
-    if (m->time_events_scan_dead)
-        pa_idxset_foreach(m->time_events, time_foreach, &all);
-    if (m->defer_events_scan_dead)
-        pa_idxset_foreach(m->defer_events, defer_foreach, &all);
+    if (m->time_events_please_scan)
+        cleanup_time_events(m, 0);
 
-    m->io_events_scan_dead = m->time_events_scan_dead = m->defer_events_scan_dead = 0;
+    if (m->defer_events_please_scan)
+        cleanup_defer_events(m, 0);
 }
 
 static void rebuild_pollfds(pa_mainloop *m) {
     pa_io_event*e;
     struct pollfd *p;
-    uint32_t idx = PA_IDXSET_INVALID;
     unsigned l;
 
-    l = pa_idxset_size(m->io_events) + 1;
+    l = m->n_io_events + 1;
     if (m->max_pollfds < l) {
+        l *= 2;
         m->pollfds = pa_xrealloc(m->pollfds, sizeof(struct pollfd)*l);
         m->max_pollfds = l;
     }
@@ -479,7 +646,7 @@ static void rebuild_pollfds(pa_mainloop *m) {
         m->n_pollfds++;
     }
 
-    for (e = pa_idxset_first(m->io_events, &idx); e; e = pa_idxset_next(m->io_events, &idx)) {
+    for (e = m->io_events; e; e = e->next) {
         if (e->dead) {
             e->pollfd = NULL;
             continue;
@@ -487,11 +654,7 @@ static void rebuild_pollfds(pa_mainloop *m) {
 
         e->pollfd = p;
         p->fd = e->fd;
-        p->events =
-            ((e->events & PA_IO_EVENT_INPUT) ? POLLIN : 0) |
-            ((e->events & PA_IO_EVENT_OUTPUT) ? POLLOUT : 0) |
-            POLLHUP |
-            POLLERR;
+        p->events = map_flags_to_libc(e->events);
         p->revents = 0;
 
         p++;
@@ -502,41 +665,39 @@ static void rebuild_pollfds(pa_mainloop *m) {
 }
 
 static int dispatch_pollfds(pa_mainloop *m) {
-    uint32_t idx = PA_IDXSET_INVALID;
     pa_io_event *e;
-    int r = 0;
+    int r = 0, k;
 
-    for (e = pa_idxset_first(m->io_events, &idx); e && !m->quit; e = pa_idxset_next(m->io_events, &idx)) {
+    pa_assert(m->poll_func_ret > 0);
+
+    for (e = m->io_events, k = m->poll_func_ret; e && !m->quit && k > 0; e = e->next) {
         if (e->dead || !e->pollfd || !e->pollfd->revents)
             continue;
-        
-        assert(e->pollfd->fd == e->fd && e->callback);
-        e->callback(&m->api, e, e->fd,
-                    (e->pollfd->revents & POLLHUP ? PA_IO_EVENT_HANGUP : 0) |
-                    (e->pollfd->revents & POLLIN ? PA_IO_EVENT_INPUT : 0) |
-                    (e->pollfd->revents & POLLOUT ? PA_IO_EVENT_OUTPUT : 0) |
-                    (e->pollfd->revents & POLLERR ? PA_IO_EVENT_ERROR : 0),
-                    e->userdata);
+
+        pa_assert(e->pollfd->fd == e->fd);
+        pa_assert(e->callback);
+        e->callback(&m->api, e, e->fd, map_flags_from_libc(e->pollfd->revents), e->userdata);
         e->pollfd->revents = 0;
         r++;
+
+        k--;
     }
 
     return r;
 }
 
 static int dispatch_defer(pa_mainloop *m) {
-    uint32_t idx;
     pa_defer_event *e;
     int r = 0;
 
-    if (!m->deferred_pending)
+    if (m->n_enabled_defer_events <= 0)
         return 0;
 
-    for (e = pa_idxset_first(m->defer_events, &idx); e && !m->quit; e = pa_idxset_next(m->defer_events, &idx)) {
+    for (e = m->defer_events; e && !m->quit; e = e->next) {
         if (e->dead || !e->enabled)
             continue;
-        assert(e->callback);
+
+        pa_assert(e->callback);
         e->callback(&m->api, e, e->userdata);
         r++;
     }
@@ -544,73 +705,76 @@ static int dispatch_defer(pa_mainloop *m) {
     return r;
 }
 
-static int calc_next_timeout(pa_mainloop *m) {
-    uint32_t idx;
-    pa_time_event *e;
-    struct timeval now;
-    int t = -1;
-    int got_time = 0;
+static pa_time_event* find_next_time_event(pa_mainloop *m) {
+    pa_time_event *t, *n = NULL;
+    pa_assert(m);
 
-    if (pa_idxset_isempty(m->time_events))
-        return -1;
+    if (m->cached_next_time_event)
+        return m->cached_next_time_event;
 
-    for (e = pa_idxset_first(m->time_events, &idx); e; e = pa_idxset_next(m->time_events, &idx)) {
-        int tmp;
-        
-        if (e->dead || !e->enabled)
+    for (t = m->time_events; t; t = t->next) {
+
+        if (t->dead || !t->enabled)
             continue;
 
-        /* Let's save a system call */
-        if (!got_time) {
-            pa_gettimeofday(&now);
-            got_time = 1;
+        if (!n || pa_timeval_cmp(&t->timeval, &n->timeval) < 0) {
+            n = t;
+
+            /* Shortcut for tv = { 0, 0 } */
+            if (n->timeval.tv_sec <= 0)
+                break;
         }
+    }
 
-        if (e->timeval.tv_sec < now.tv_sec || (e->timeval.tv_sec == now.tv_sec && e->timeval.tv_usec <= now.tv_usec)) 
-            return 0;
+    m->cached_next_time_event = n;
+    return n;
+}
 
-        tmp = (e->timeval.tv_sec - now.tv_sec)*1000;
-            
-        if (e->timeval.tv_usec > now.tv_usec)
-            tmp += (e->timeval.tv_usec - now.tv_usec)/1000;
-        else
-            tmp -= (now.tv_usec - e->timeval.tv_usec)/1000;
+static int calc_next_timeout(pa_mainloop *m) {
+    pa_time_event *t;
+    struct timeval now;
+    pa_usec_t usec;
 
-        if (tmp == 0)
-            return 0;
-        else if (t == -1 || tmp < t)
-            t = tmp;
-    }
+    if (!m->n_enabled_time_events)
+        return -1;
+
+    t = find_next_time_event(m);
+    pa_assert(t);
+
+    if (t->timeval.tv_sec <= 0)
+        return 0;
 
-    return t;
+    pa_gettimeofday(&now);
+
+    if (pa_timeval_cmp(&t->timeval, &now) <= 0)
+        return 0;
+
+    usec = pa_timeval_diff(&t->timeval, &now);
+    return (int) (usec / 1000);
 }
 
 static int dispatch_timeout(pa_mainloop *m) {
-    uint32_t idx;
     pa_time_event *e;
     struct timeval now;
-    int got_time = 0;
     int r = 0;
-    assert(m);
+    pa_assert(m);
 
-    if (pa_idxset_isempty(m->time_events))
+    if (m->n_enabled_time_events <= 0)
         return 0;
 
-    for (e = pa_idxset_first(m->time_events, &idx); e && !m->quit; e = pa_idxset_next(m->time_events, &idx)) {
-        
+    pa_gettimeofday(&now);
+
+    for (e = m->time_events; e && !m->quit; e = e->next) {
+
         if (e->dead || !e->enabled)
             continue;
 
-        /* Let's save a system call */
-        if (!got_time) {
-            pa_gettimeofday(&now);
-            got_time = 1;
-        }
-        
-        if (e->timeval.tv_sec < now.tv_sec || (e->timeval.tv_sec == now.tv_sec && e->timeval.tv_usec <= now.tv_usec)) {
-            assert(e->callback);
+        if (pa_timeval_cmp(&e->timeval, &now) <= 0) {
+            pa_assert(e->callback);
+
+            /* Disable time event */
+            mainloop_time_restart(e, NULL);
 
-            e->enabled = 0;
             e->callback(&m->api, e, &e->timeval, e->userdata);
 
             r++;
@@ -622,26 +786,31 @@ static int dispatch_timeout(pa_mainloop *m) {
 
 void pa_mainloop_wakeup(pa_mainloop *m) {
     char c = 'W';
-    assert(m);
+    pa_assert(m);
 
-    if (m->wakeup_pipe[1] >= 0)
-        pa_write(m->wakeup_pipe[1], &c, sizeof(c));
+    if (m->wakeup_pipe[1] >= 0 && m->state == STATE_POLLING) {
+        pa_write(m->wakeup_pipe[1], &c, sizeof(c), &m->wakeup_pipe_type);
+        m->wakeup_requested++;
+    }
 }
 
 static void clear_wakeup(pa_mainloop *m) {
     char c[10];
 
-    assert(m);
+    pa_assert(m);
 
     if (m->wakeup_pipe[0] < 0)
         return;
 
-    while (pa_read(m->wakeup_pipe[0], &c, sizeof(c)) == sizeof(c));
+    if (m->wakeup_requested) {
+        while (pa_read(m->wakeup_pipe[0], &c, sizeof(c), &m->wakeup_pipe_type) == sizeof(c));
+        m->wakeup_requested = 0;
+    }
 }
 
 int pa_mainloop_prepare(pa_mainloop *m, int timeout) {
-    assert(m);
-    assert(m->state == STATE_PASSIVE);
+    pa_assert(m);
+    pa_assert(m->state == STATE_PASSIVE);
 
     clear_wakeup(m);
     scan_dead(m);
@@ -649,11 +818,10 @@ int pa_mainloop_prepare(pa_mainloop *m, int timeout) {
     if (m->quit)
         goto quit;
 
-    if (!m->deferred_pending) {
-    
+    if (m->n_enabled_defer_events <= 0) {
         if (m->rebuild_pollfds)
             rebuild_pollfds(m);
-        
+
         m->prepared_timeout = calc_next_timeout(m);
         if (timeout >= 0 && (timeout < m->prepared_timeout || m->prepared_timeout < 0))
             m->prepared_timeout = timeout;
@@ -668,34 +836,34 @@ quit:
 }
 
 int pa_mainloop_poll(pa_mainloop *m) {
-    int r;
-
-    assert(m);
-    assert(m->state == STATE_PREPARED);
+    pa_assert(m);
+    pa_assert(m->state == STATE_PREPARED);
 
     if (m->quit)
         goto quit;
 
     m->state = STATE_POLLING;
 
-    if (m->deferred_pending)
-        r = 0;
+    if (m->n_enabled_defer_events )
+        m->poll_func_ret = 0;
     else {
+        pa_assert(!m->rebuild_pollfds);
+
         if (m->poll_func)
-            r = m->poll_func(m->pollfds, m->n_pollfds, m->prepared_timeout, m->poll_func_userdata);
+            m->poll_func_ret = m->poll_func(m->pollfds, m->n_pollfds, m->prepared_timeout, m->poll_func_userdata);
         else
-            r = poll(m->pollfds, m->n_pollfds, m->prepared_timeout);
+            m->poll_func_ret = poll(m->pollfds, m->n_pollfds, m->prepared_timeout);
 
-        if (r < 0) {
+        if (m->poll_func_ret < 0) {
             if (errno == EINTR)
-                r = 0;
+                m->poll_func_ret = 0;
             else
-                pa_log(__FILE__": poll(): %s", pa_cstrerror(errno));
+                pa_log("poll(): %s", pa_cstrerror(errno));
         }
     }
 
-    m->state = r < 0 ? STATE_PASSIVE : STATE_POLLED;
-    return r;
+    m->state = m->poll_func_ret < 0 ? STATE_PASSIVE : STATE_POLLED;
+    return m->poll_func_ret;
 
 quit:
     m->state = STATE_QUIT;
@@ -705,27 +873,28 @@ quit:
 int pa_mainloop_dispatch(pa_mainloop *m) {
     int dispatched = 0;
 
-    assert(m);
-    assert(m->state == STATE_POLLED);
+    pa_assert(m);
+    pa_assert(m->state == STATE_POLLED);
 
     if (m->quit)
         goto quit;
-    
-    if (m->deferred_pending)
+
+    if (m->n_enabled_defer_events)
         dispatched += dispatch_defer(m);
     else {
-        dispatched += dispatch_timeout(m);
-        
+        if (m->n_enabled_time_events)
+            dispatched += dispatch_timeout(m);
+
         if (m->quit)
             goto quit;
-        
-        dispatched += dispatch_pollfds(m);
 
+        if (m->poll_func_ret > 0)
+            dispatched += dispatch_pollfds(m);
     }
-    
+
     if (m->quit)
         goto quit;
-    
+
     m->state = STATE_PASSIVE;
 
     return dispatched;
@@ -736,13 +905,13 @@ quit:
 }
 
 int pa_mainloop_get_retval(pa_mainloop *m) {
-    assert(m);
+    pa_assert(m);
     return m->retval;
 }
 
 int pa_mainloop_iterate(pa_mainloop *m, int block, int *retval) {
     int r;
-    assert(m);
+    pa_assert(m);
 
     if ((r = pa_mainloop_prepare(m, block ? -1 : 0)) < 0)
         goto quit;
@@ -756,7 +925,7 @@ int pa_mainloop_iterate(pa_mainloop *m, int block, int *retval) {
     return r;
 
 quit:
-    
+
     if ((r == -2) && retval)
         *retval = pa_mainloop_get_retval(m);
     return r;
@@ -764,7 +933,7 @@ quit:
 
 int pa_mainloop_run(pa_mainloop *m, int *retval) {
     int r;
-    
+
     while ((r = pa_mainloop_iterate(m, 1, retval)) >= 0);
 
     if (r == -2)
@@ -776,7 +945,7 @@ int pa_mainloop_run(pa_mainloop *m, int *retval) {
 }
 
 void pa_mainloop_quit(pa_mainloop *m, int retval) {
-    assert(m);
+    pa_assert(m);
 
     m->quit = 1;
     m->retval = retval;
@@ -784,56 +953,13 @@ void pa_mainloop_quit(pa_mainloop *m, int retval) {
 }
 
 pa_mainloop_api* pa_mainloop_get_api(pa_mainloop*m) {
-    assert(m);
+    pa_assert(m);
     return &m->api;
 }
 
 void pa_mainloop_set_poll_func(pa_mainloop *m, pa_poll_func poll_func, void *userdata) {
-    assert(m);
+    pa_assert(m);
 
     m->poll_func = poll_func;
     m->poll_func_userdata = userdata;
 }
-
-
-#if 0
-void pa_mainloop_dump(pa_mainloop *m) {
-    assert(m);
-
-    pa_log(__FILE__": Dumping mainloop sources START");
-    
-    {
-        uint32_t idx = PA_IDXSET_INVALID;
-        pa_io_event *e;
-        for (e = pa_idxset_first(m->io_events, &idx); e; e = pa_idxset_next(m->io_events, &idx)) {
-            if (e->dead)
-                continue;
-            
-            pa_log(__FILE__": kind=io fd=%i events=%i callback=%p userdata=%p", e->fd, (int) e->events, (void*) e->callback, (void*) e->userdata);
-        }
-    }
-    {
-        uint32_t idx = PA_IDXSET_INVALID;
-        pa_defer_event *e;
-        for (e = pa_idxset_first(m->defer_events, &idx); e; e = pa_idxset_next(m->defer_events, &idx)) {
-            if (e->dead)
-                continue;
-            
-            pa_log(__FILE__": kind=defer enabled=%i callback=%p userdata=%p", e->enabled, (void*) e->callback, (void*) e->userdata);
-        }
-    }
-    {
-        uint32_t idx = PA_IDXSET_INVALID;
-        pa_time_event *e;
-        for (e = pa_idxset_first(m->time_events, &idx); e; e = pa_idxset_next(m->time_events, &idx)) {
-            if (e->dead)
-                continue;
-            
-            pa_log(__FILE__": kind=time enabled=%i time=%lu.%lu callback=%p userdata=%p", e->enabled, (unsigned long) e->timeval.tv_sec, (unsigned long) e->timeval.tv_usec, (void*) e->callback, (void*) e->userdata);
-        }
-    }
-
-    pa_log(__FILE__": Dumping mainloop sources STOP");
-
-}
-#endif