jshn: shorten JSON_VAR to J_V

[project/libubox.git] / uloop.c

diff --git a/uloop.c b/uloop.c
index db8cacd..9a77ce4 100644 (file)
--- a/uloop.c
+++ b/uloop.c
@@ -43,6 +43,14 @@ struct uloop_fd_event {
        unsigned int events;
 };
 
        unsigned int events;
 };
 

+struct uloop_fd_stack {
+       struct uloop_fd_stack *next;
+       struct uloop_fd *fd;
+       unsigned int events;
+};
+
+static struct uloop_fd_stack *fd_stack = NULL;
+

 #define ULOOP_MAX_EVENTS 10
 
 static struct list_head timeouts = LIST_HEAD_INIT(timeouts);
 #define ULOOP_MAX_EVENTS 10
 
 static struct list_head timeouts = LIST_HEAD_INIT(timeouts);


@@ -50,7 +58,6 @@ static struct list_head processes = LIST_HEAD_INIT(processes);
 
 static int poll_fd = -1;
 bool uloop_cancelled = false;
 
 static int poll_fd = -1;
 bool uloop_cancelled = false;

-bool uloop_handle_sigchld = true;

 static bool do_sigchld = false;
 
 static struct uloop_fd_event cur_fds[ULOOP_MAX_EVENTS];
 static bool do_sigchld = false;
 
 static struct uloop_fd_event cur_fds[ULOOP_MAX_EVENTS];


@@ -166,7 +173,8 @@ static int uloop_fetch_events(int timeout)
 
                if (events[n].flags & EV_ERROR) {
                        u->error = true;
 
                if (events[n].flags & EV_ERROR) {
                        u->error = true;

-                       uloop_fd_delete(u);
+                       if (!(u->flags & ULOOP_ERROR_CB))
+                               uloop_fd_delete(u);

                }
 
                if(events[n].filter == EVFILT_READ)
                }
 
                if(events[n].filter == EVFILT_READ)


@@ -179,8 +187,10 @@ static int uloop_fetch_events(int timeout)
                else if (!ev)
                        cur->fd = NULL;
 
                else if (!ev)
                        cur->fd = NULL;
 

+               cur->events = ev;

                if (u->flags & ULOOP_EDGE_DEFER) {
                        u->flags &= ~ULOOP_EDGE_DEFER;
                if (u->flags & ULOOP_EDGE_DEFER) {
                        u->flags &= ~ULOOP_EDGE_DEFER;

+                       u->flags |= ULOOP_EDGE_TRIGGER;

                        register_kevent(u, u->flags);
                }
        }
                        register_kevent(u, u->flags);
                }
        }


@@ -229,6 +239,7 @@ static int register_poll(struct uloop_fd *fd, unsigned int flags)
 
        ev.data.fd = fd->fd;
        ev.data.ptr = fd;
 
        ev.data.fd = fd->fd;
        ev.data.ptr = fd;

+       fd->flags = flags;

 
        return epoll_ctl(poll_fd, op, fd->fd, &ev);
 }
 
        return epoll_ctl(poll_fd, op, fd->fd, &ev);
 }


@@ -237,6 +248,7 @@ static struct epoll_event events[ULOOP_MAX_EVENTS];
 
 static int __uloop_fd_delete(struct uloop_fd *sock)
 {
 
 static int __uloop_fd_delete(struct uloop_fd *sock)
 {

+       sock->flags = 0;

        return epoll_ctl(poll_fd, EPOLL_CTL_DEL, sock->fd, 0);
 }
 
        return epoll_ctl(poll_fd, EPOLL_CTL_DEL, sock->fd, 0);
 }
 


@@ -256,7 +268,8 @@ static int uloop_fetch_events(int timeout)
 
                if (events[n].events & (EPOLLERR|EPOLLHUP)) {
                        u->error = true;
 
                if (events[n].events & (EPOLLERR|EPOLLHUP)) {
                        u->error = true;

-                       uloop_fd_delete(u);
+                       if (!(u->flags & ULOOP_ERROR_CB))
+                               uloop_fd_delete(u);

                }
 
                if(!(events[n].events & (EPOLLRDHUP|EPOLLIN|EPOLLOUT|EPOLLERR|EPOLLHUP))) {
                }
 
                if(!(events[n].events & (EPOLLRDHUP|EPOLLIN|EPOLLOUT|EPOLLERR|EPOLLHUP))) {


@@ -281,6 +294,32 @@ static int uloop_fetch_events(int timeout)
 
 #endif
 
 
 #endif
 

+static bool uloop_fd_stack_event(struct uloop_fd *fd, int events)
+{
+       struct uloop_fd_stack *cur;
+
+       /*
+        * Do not buffer events for level-triggered fds, they will keep firing.
+        * Caller needs to take care of recursion issues.
+        */
+       if (!(fd->flags & ULOOP_EDGE_TRIGGER))
+               return false;
+
+       for (cur = fd_stack; cur; cur = cur->next) {
+               if (cur->fd != fd)
+                       continue;
+
+               if (events < 0)
+                       cur->fd = NULL;
+               else
+                       cur->events |= events | ULOOP_EVENT_BUFFERED;
+
+               return true;
+       }
+
+       return false;
+}
+

 static void uloop_run_events(int timeout)
 {
        struct uloop_fd_event *cur;
 static void uloop_run_events(int timeout)
 {
        struct uloop_fd_event *cur;


@@ -294,17 +333,33 @@ static void uloop_run_events(int timeout)
        }
 
        while (cur_nfds > 0) {
        }
 
        while (cur_nfds > 0) {

+               struct uloop_fd_stack stack_cur;
+               unsigned int events;
+

                cur = &cur_fds[cur_fd++];
                cur_nfds--;
 
                fd = cur->fd;
                cur = &cur_fds[cur_fd++];
                cur_nfds--;
 
                fd = cur->fd;

+               events = cur->events;

                if (!fd)
                        continue;
 
                if (!fd->cb)
                        continue;
 
                if (!fd)
                        continue;
 
                if (!fd->cb)
                        continue;
 

-               fd->cb(fd, cur->events);
+               if (uloop_fd_stack_event(fd, cur->events))
+                       continue;
+
+               stack_cur.next = fd_stack;
+               stack_cur.fd = fd;
+               fd_stack = &stack_cur;
+               do {
+                       stack_cur.events = 0;
+                       fd->cb(fd, events);
+                       events = stack_cur.events & ULOOP_EVENT_MASK;
+               } while (stack_cur.fd && events);
+               fd_stack = stack_cur.next;
+

                return;
        }
 }
                return;
        }
 }


@@ -329,6 +384,7 @@ int uloop_fd_add(struct uloop_fd *sock, unsigned int flags)
 
        sock->registered = true;
        sock->eof = false;
 
        sock->registered = true;
        sock->eof = false;

+       sock->error = false;

 
 out:
        return ret;
 
 out:
        return ret;


@@ -338,16 +394,18 @@ int uloop_fd_delete(struct uloop_fd *fd)
 {
        int i;
 
 {
        int i;
 

-       if (!fd->registered)
-               return 0;
-

        for (i = 0; i < cur_nfds; i++) {
                if (cur_fds[cur_fd + i].fd != fd)
                        continue;
 
                cur_fds[cur_fd + i].fd = NULL;
        }
        for (i = 0; i < cur_nfds; i++) {
                if (cur_fds[cur_fd + i].fd != fd)
                        continue;
 
                cur_fds[cur_fd + i].fd = NULL;
        }

+
+       if (!fd->registered)
+               return 0;
+

        fd->registered = false;
        fd->registered = false;

+       uloop_fd_stack_event(fd, -1);

        return __uloop_fd_delete(fd);
 }
 
        return __uloop_fd_delete(fd);
 }
 


@@ -500,19 +558,37 @@ static void uloop_sigchld(int signo)
        do_sigchld = true;
 }
 
        do_sigchld = true;
 }
 

-static void uloop_setup_signals(void)
+static void uloop_install_handler(int signum, void (*handler)(int), struct sigaction* old, bool add)

 {
        struct sigaction s;
 {
        struct sigaction s;

+       struct sigaction *act;

 
 

-       memset(&s, 0, sizeof(struct sigaction));
-       s.sa_handler = uloop_handle_sigint;
-       s.sa_flags = 0;
-       sigaction(SIGINT, &s, NULL);
+       act = NULL;
+       sigaction(signum, NULL, &s);

 
 

-       if (uloop_handle_sigchld) {
-               s.sa_handler = uloop_sigchld;
-               sigaction(SIGCHLD, &s, NULL);
+       if (add) {
+               if (s.sa_handler == SIG_DFL) { /* Do not override existing custom signal handlers */
+                       memcpy(old, &s, sizeof(struct sigaction));
+                       s.sa_handler = handler;
+                       s.sa_flags = 0;
+                       act = &s;
+               }

        }
        }

+       else if (s.sa_handler == handler) { /* Do not restore if someone modified our handler */
+                       act = old;
+       }
+
+       if (act != NULL)
+               sigaction(signum, act, NULL);
+}
+
+static void uloop_setup_signals(bool add)
+{
+       static struct sigaction old_sigint, old_sigchld, old_sigterm;
+
+       uloop_install_handler(SIGINT, uloop_handle_sigint, &old_sigint, add);
+       uloop_install_handler(SIGTERM, uloop_handle_sigint, &old_sigterm, add);
+       uloop_install_handler(SIGCHLD, uloop_sigchld, &old_sigchld, add);

 }
 
 static int uloop_get_next_timeout(struct timeval *tv)
 }
 
 static int uloop_get_next_timeout(struct timeval *tv)


@@ -565,9 +641,17 @@ static void uloop_clear_processes(void)
 
 void uloop_run(void)
 {
 
 void uloop_run(void)
 {

+       static int recursive_calls = 0;

        struct timeval tv;
 
        struct timeval tv;
 

-       uloop_setup_signals();
+       /*
+        * Handlers are only updated for the first call to uloop_run() (and restored
+        * when this call is done).
+        */
+       if (!recursive_calls++)
+               uloop_setup_signals(true);
+
+       uloop_cancelled = false;

        while(!uloop_cancelled)
        {
                uloop_gettime(&tv);
        while(!uloop_cancelled)
        {
                uloop_gettime(&tv);


@@ -577,8 +661,12 @@ void uloop_run(void)
 
                if (do_sigchld)
                        uloop_handle_processes();
 
                if (do_sigchld)
                        uloop_handle_processes();

+               uloop_gettime(&tv);

                uloop_run_events(uloop_get_next_timeout(&tv));
        }
                uloop_run_events(uloop_get_next_timeout(&tv));
        }

+
+       if (!--recursive_calls)
+               uloop_setup_signals(false);

 }
 
 void uloop_done(void)
 }
 
 void uloop_done(void)