[Xenomai-git] Matthias Schneider : copperplate/notifier: drop pointless select() loop from sighandler

[git repository hosting]

Module: xenomai-forge
Branch: rtdm-api-waitqueues
Commit: 2ac9031e9ac788d3349315693a70d16d08e9be03
URL:    
http://git.xenomai.org/?p=xenomai-forge.git;a=commit;h=2ac9031e9ac788d3349315693a70d16d08e9be03

Author: Matthias Schneider <ma30002...@yahoo.de>
Date:   Wed Apr 23 15:44:53 2014 +0200

copperplate/notifier: drop pointless select() loop from sighandler

---

 lib/copperplate/notifier.c |   24 +-----------------------
 1 file changed, 1 insertion(+), 23 deletions(-)

diff --git a/lib/copperplate/notifier.c b/lib/copperplate/notifier.c
index d069a66..7fa48f3 100644
--- a/lib/copperplate/notifier.c
+++ b/lib/copperplate/notifier.c
@@ -34,41 +34,21 @@ static pthread_mutex_t notifier_lock;
 
 static struct sigaction notifier_old_sa;
 
-static fd_set notifier_rset;
-
 static void notifier_sighandler(int sig, siginfo_t *siginfo, void *uc)
 {
-       static struct timeval tv = { .tv_sec = 0, .tv_usec = 0 };
        int ret, matched = 0;
        struct notifier *nf;
-       fd_set rfds;
        pid_t tid;
        char c;
 
        tid = copperplate_get_tid();
 
-       if (siginfo->si_code == SI_SIGIO) {
-               FD_ZERO(&rfds);
-               FD_SET(siginfo->si_fd, &rfds);
-       } else {
-               /*
-                * We will have to find out by ourselves which fd was
-                * notified.
-                */
-               rfds = notifier_rset;
-               do
-                       ret = __STD(select(FD_SETSIZE, &rfds, NULL, NULL, &tv));
-               while (ret == -1 && errno == EINTR);
-               if (ret <= 0)
-                       goto hand_over;
-       }
-
        if (pvlist_empty(&notifier_list))
                goto hand_over;
 
        /* We may NOT alter the notifier list, but only scan it. */
        pvlist_for_each_entry(nf, &notifier_list, link) {
-               if (!FD_ISSET(nf->psfd[0], &rfds))
+               if (nf->psfd[0] != siginfo->si_fd)
                        continue;
                /*
                 * Ignore misdirected notifications. We want those to
@@ -173,7 +153,6 @@ int notifier_init(struct notifier *nf,
        push_cleanup_lock(&notifier_lock);
        lock_notifier_list(&oset);
        pvlist_append(&nf->link, &notifier_list);
-       FD_SET(nf->psfd[0], &notifier_rset);
        unlock_notifier_list(&oset);
        pop_cleanup_lock(&notifier_lock);
 
@@ -208,7 +187,6 @@ void notifier_destroy(struct notifier *nf)
        push_cleanup_lock(&notifier_lock);
        lock_notifier_list(&oset);
        pvlist_remove(&nf->link);
-       FD_CLR(nf->psfd[0], &notifier_rset);
        unlock_notifier_list(&oset);
        pop_cleanup_lock(&notifier_lock);
        __STD(close(nf->psfd[0]));


_______________________________________________
Xenomai-git mailing list
Xenomai-git@xenomai.org
http://www.xenomai.org/mailman/listinfo/xenomai-git