aboutsummaryrefslogtreecommitdiffstats
path: root/src/main_notify.c
diff options
context:
space:
mode:
authorMax Kellermann <max@duempel.org>2008-12-30 19:20:36 +0100
committerMax Kellermann <max@duempel.org>2008-12-30 19:20:36 +0100
commit03e650aa9e9a95575fccd51ec9f669abae52fe7e (patch)
tree6d1447445c41dc1322d017bb78f1e603d06d0237 /src/main_notify.c
parent10b5966bf656bc25261171f6fbf4ea35eff246af (diff)
downloadmpd-03e650aa9e9a95575fccd51ec9f669abae52fe7e.tar.gz
mpd-03e650aa9e9a95575fccd51ec9f669abae52fe7e.tar.xz
mpd-03e650aa9e9a95575fccd51ec9f669abae52fe7e.zip
main_notify: make the read side of the pipe blocking
Currently, both sides of the pipe are blocking, although we do not need blocking read(). Convert it back to blocking. Eliminate the select() from wait_main_task().
Diffstat (limited to '')
-rw-r--r--src/main_notify.c21
1 files changed, 6 insertions, 15 deletions
diff --git a/src/main_notify.c b/src/main_notify.c
index 68df0b44b..f821f76aa 100644
--- a/src/main_notify.c
+++ b/src/main_notify.c
@@ -63,7 +63,12 @@ static int ioops_consume(int fd_count, fd_set * rfds,
void init_main_notify(void)
{
main_task = g_thread_self();
- init_async_pipe(main_pipe);
+
+ if (pipe(main_pipe) < 0)
+ g_error("Couldn't open pipe: %s", strerror(errno));
+ if (set_nonblocking(main_pipe[1]) < 0)
+ g_error("Couldn't set non-blocking I/O: %s", strerror(errno));
+
main_notify_IO.fdset = ioops_fdset;
main_notify_IO.consume = ioops_consume;
registerIO(&main_notify_IO);
@@ -96,19 +101,5 @@ void main_notify_unlock(void)
void wait_main_task(void)
{
- fd_set rfds;
- int ret;
-
- assert(main_task == g_thread_self());
-
- do {
- FD_ZERO(&rfds);
- FD_SET(main_pipe[0], &rfds);
- ret = select(main_pipe[0] + 1, &rfds, NULL, NULL, NULL);
- } while (ret == 0 || (ret < 0 && (errno == EAGAIN || errno == EINTR)));
-
- if (ret < 0)
- g_error("select() failed: %s", strerror(errno));
-
consume_pipe();
}