Update copyright dates

[src/app-framework-binder.git] / src / fdev-epoll.c

diff --git a/src/fdev-epoll.c b/src/fdev-epoll.c
index 80c037d..262ec84 100644 (file)
--- a/src/fdev-epoll.c
+++ b/src/fdev-epoll.c
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2018 "IoT.bzh"
+ * Copyright (C) 2015-2020 "IoT.bzh"
  * Author José Bollo <jose.bollo@iot.bzh>
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
@@ -23,15 +23,30 @@
 #include "fdev.h"
 #include "fdev-epoll.h"
 
+/*
+ * For sake of simplicity there is no struct fdev_epoll.
+ * Instead, the file descriptor of the internal epoll is used
+ * and wrapped in a pseudo pointer to a pseudo struct.
+ */
 #define epollfd(fdev_epoll)  ((int)(intptr_t)fdev_epoll)
 
+/*
+ * disable callback for fdev
+ *
+ * refs to fdev must not be counted here
+ */
 static void disable(void *closure, const struct fdev *fdev)
 {
        struct fdev_epoll *fdev_epoll = closure;
        epoll_ctl(epollfd(fdev_epoll), EPOLL_CTL_DEL, fdev_fd(fdev), 0);
 }
 
-static void enable(void *closure, const struct fdev *fdev)
+/*
+ * enable callback for fdev
+ *
+ * refs to fdev must not be counted here
+ */
+static void enable_or_update(void *closure, const struct fdev *fdev, int op, int err)
 {
        struct fdev_epoll *fdev_epoll = closure;
        struct epoll_event event;
@@ -40,18 +55,45 @@ static void enable(void *closure, const struct fdev *fdev)
        fd = fdev_fd(fdev);
        event.events = fdev_events(fdev);
        event.data.ptr = (void*)fdev;
-       rc = epoll_ctl(epollfd(fdev_epoll), EPOLL_CTL_MOD, fd, &event);
-       if (rc < 0 && errno == ENOENT)
-               epoll_ctl(epollfd(fdev_epoll), EPOLL_CTL_ADD, fd, &event);
+       rc = epoll_ctl(epollfd(fdev_epoll), op, fd, &event);
+       if (rc < 0 && errno == err)
+               epoll_ctl(epollfd(fdev_epoll), (EPOLL_CTL_MOD + EPOLL_CTL_ADD) - op, fd, &event);
+}
+
+/*
+ * enable callback for fdev
+ *
+ * refs to fdev must not be counted here
+ */
+static void enable(void *closure, const struct fdev *fdev)
+{
+       enable_or_update(closure, fdev, EPOLL_CTL_ADD, EEXIST);
+}
+
+/*
+ * update callback for fdev
+ *
+ * refs to fdev must not be counted here
+ */
+static void update(void *closure, const struct fdev *fdev)
+{
+       enable_or_update(closure, fdev, EPOLL_CTL_MOD, ENOENT);
 }
 
+/*
+ * unref is not handled here
+ */
 static struct fdev_itf itf =
 {
        .unref = 0,
        .disable = disable,
-       .enable = enable
+       .enable = enable,
+       .update = update
 };
 
+/*
+ * create an fdev_epoll
+ */
 struct fdev_epoll *fdev_epoll_create()
 {
        int fd = epoll_create1(EPOLL_CLOEXEC);
@@ -62,16 +104,25 @@ struct fdev_epoll *fdev_epoll_create()
        return fd < 0 ? 0 : (struct fdev_epoll*)(intptr_t)fd;
 }
 
+/*
+ * destroy the fdev_epoll
+ */
 void fdev_epoll_destroy(struct fdev_epoll *fdev_epoll)
 {
        close(epollfd(fdev_epoll));
 }
 
+/*
+ * get pollable fd for the fdev_epoll
+ */
 int fdev_epoll_fd(struct fdev_epoll *fdev_epoll)
 {
        return epollfd(fdev_epoll);
 }
 
+/*
+ * create an fdev linked to the 'fdev_epoll' for 'fd'
+ */
 struct fdev *fdev_epoll_add(struct fdev_epoll *fdev_epoll, int fd)
 {
        struct fdev *fdev;
@@ -82,16 +133,20 @@ struct fdev *fdev_epoll_add(struct fdev_epoll *fdev_epoll, int fd)
        return fdev;
 }
 
-void fdev_epoll_wait_and_dispatch(struct fdev_epoll *fdev_epoll, int timeout_ms)
+/*
+ * get pollable fd for the fdev_epoll
+ */
+int fdev_epoll_wait_and_dispatch(struct fdev_epoll *fdev_epoll, int timeout_ms)
 {
        struct fdev *fdev;
-       struct epoll_event events[8];
-       int rc, i;
+       struct epoll_event events;
+       int rc;
 
-       rc = epoll_wait(epollfd(fdev_epoll), events, sizeof events / sizeof *events, timeout_ms < 0 ? -1 : timeout_ms);
-       for (i = 0 ; i < rc ; i++) {
-               fdev = events[i].data.ptr;
-               fdev_dispatch(fdev, events[i].events);
+       rc = epoll_wait(epollfd(fdev_epoll), &events, 1, timeout_ms < 0 ? -1 : timeout_ms);
+       if (rc == 1) {
+               fdev = events.data.ptr;
+               fdev_dispatch(fdev, events.events);
        }
+       return rc;
 }