chiark / gitweb /
Merge commit '5f03ed8a56d308af72db8a48ab66ed68667af2c6'
authorKay Sievers <kay.sievers@vrfy.org>
Wed, 21 Jan 2009 13:42:30 +0000 (14:42 +0100)
committerKay Sievers <kay.sievers@vrfy.org>
Wed, 21 Jan 2009 13:42:30 +0000 (14:42 +0100)
Conflicts:
udev/udevd.c

1  2 
udev/udevd.c

diff --combined udev/udevd.c
index 9a278ce1f39023487edbbda54a5a2d9afe8f1d66,9a8d7918000b5df7bdb3e83e00c14caedf63085f..13a10240d775f6f38a9d61682f2fcf35ec3704f7
@@@ -29,7 -29,6 +29,7 @@@
  #include <getopt.h>
  #include <dirent.h>
  #include <sys/select.h>
 +#include <sys/poll.h>
  #include <sys/wait.h>
  #include <sys/stat.h>
  #include <sys/ioctl.h>
@@@ -59,17 -58,15 +59,17 @@@ static void log_fn(struct udev *udev, i
        }
  }
  
 +static void reap_sigchilds(void);
 +
  static int debug_trace;
  static struct udev_rules *rules;
  static struct udev_ctrl *udev_ctrl;
  static struct udev_monitor *kernel_monitor;
  static int inotify_fd = -1;
 -static int signal_pipe[2] = {-1, -1};
  static volatile int sigchilds_waiting;
  static volatile int udev_exit;
  static volatile int reload_config;
 +static volatile int signal_received;
  static int run_exec_q;
  static int stop_exec_q;
  static int max_childs;
@@@ -197,6 -194,8 +197,6 @@@ static void event_fork(struct udev_even
                udev_ctrl_unref(udev_ctrl);
                if (inotify_fd >= 0)
                        close(inotify_fd);
 -              close(signal_pipe[READ_END]);
 -              close(signal_pipe[WRITE_END]);
                logging_close();
                logging_init("udevd-event");
                setpriority(PRIO_PROCESS, 0, UDEV_PRIORITY);
@@@ -405,7 -404,6 +405,7 @@@ static void event_queue_manager(struct 
        struct udev_list_node *loop;
        struct udev_list_node *tmp;
  
 +start_over:
        if (udev_list_is_empty(&event_list)) {
                if (childs > 0) {
                        err(udev, "event list empty, but childs count is %i", childs);
  
                event_fork(loop_event);
                dbg(udev, "moved seq %llu to running list\n", udev_device_get_seqnum(loop_event->dev));
 +
 +              /* retry if events finished in the meantime */
 +              if (sigchilds_waiting) {
 +                      sigchilds_waiting = 0;
 +                      reap_sigchilds();
 +                      goto start_over;
 +              }
        }
  }
  
@@@ -529,7 -520,8 +529,7 @@@ static void asmlinkage sig_handler(int 
                        break;
        }
  
 -      /* write to pipe, which will wakeup select() in our mainloop */
 -      write(signal_pipe[WRITE_END], "", 1);
 +      signal_received = 1;
  }
  
  static void udev_done(int pid, int exitstatus)
@@@ -641,19 -633,24 +641,21 @@@ static void export_initial_seqnum(struc
  int main(int argc, char *argv[])
  {
        struct udev *udev;
 -      int err;
        int fd;
        struct sigaction act;
 -      fd_set readfds;
        const char *value;
        int daemonize = 0;
+       int resolve_names = 1;
        static const struct option options[] = {
                { "daemon", no_argument, NULL, 'd' },
                { "debug-trace", no_argument, NULL, 't' },
                { "debug", no_argument, NULL, 'D' },
                { "help", no_argument, NULL, 'h' },
                { "version", no_argument, NULL, 'V' },
+               { "resolve-names", required_argument, NULL, 'N' },
                {}
        };
        int rc = 1;
 -      int maxfd;
  
        udev = udev_new();
        if (udev == NULL)
                        if (udev_get_log_priority(udev) < LOG_INFO)
                                udev_set_log_priority(udev, LOG_INFO);
                        break;
+               case 'N':
+                       if (strcmp (optarg, "early") == 0) {
+                               resolve_names = 1;
+                       } else if (strcmp (optarg, "never") == 0) {
+                               resolve_names = -1;
+                       } else {
+                               fprintf(stderr, "resolve-names must be early or never\n");
+                               err(udev, "resolve-names must be early or never\n");
+                               goto exit;
+                       }
+                       break;
                case 'h':
-                       printf("Usage: udevd [--help] [--daemon] [--debug-trace] [--debug] [--version]\n");
+                       printf("Usage: udevd [--help] [--daemon] [--debug-trace] [--debug] [--resolve-names=early|never] [--version]\n");
                        goto exit;
                case 'V':
                        printf("%s\n", VERSION);
        }
        udev_monitor_set_receive_buffer_size(kernel_monitor, 128*1024*1024);
  
-       rules = udev_rules_new(udev, 1);
 -      err = pipe(signal_pipe);
 -      if (err < 0) {
 -              err(udev, "error getting pipes: %m\n");
 -              goto exit;
 -      }
 -
 -      err = fcntl(signal_pipe[READ_END], F_GETFL, 0);
 -      if (err < 0) {
 -              err(udev, "error fcntl on read pipe: %m\n");
 -              goto exit;
 -      }
 -      err = fcntl(signal_pipe[READ_END], F_SETFL, err | O_NONBLOCK);
 -      if (err < 0) {
 -              err(udev, "error fcntl on read pipe: %m\n");
 -              goto exit;
 -      }
 -
 -      err = fcntl(signal_pipe[WRITE_END], F_GETFL, 0);
 -      if (err < 0) {
 -              err(udev, "error fcntl on write pipe: %m\n");
 -              goto exit;
 -      }
 -      err = fcntl(signal_pipe[WRITE_END], F_SETFL, err | O_NONBLOCK);
 -      if (err < 0) {
 -              err(udev, "error fcntl on write pipe: %m\n");
 -              goto exit;
 -      }
 -
+       rules = udev_rules_new(udev, resolve_names);
        if (rules == NULL) {
                err(udev, "error reading rules\n");
                goto exit;
        memset(&act, 0x00, sizeof(struct sigaction));
        act.sa_handler = (void (*)(int)) sig_handler;
        sigemptyset(&act.sa_mask);
 -      act.sa_flags = SA_RESTART;
        sigaction(SIGINT, &act, NULL);
        sigaction(SIGTERM, &act, NULL);
        sigaction(SIGCHLD, &act, NULL);
                max_childs = strtoul(value, NULL, 10);
        info(udev, "initialize max_childs to %u\n", max_childs);
  
 -      maxfd = udev_ctrl_get_fd(udev_ctrl);
 -      maxfd = UDEV_MAX(maxfd, udev_monitor_get_fd(kernel_monitor));
 -      maxfd = UDEV_MAX(maxfd, signal_pipe[READ_END]);
 -      maxfd = UDEV_MAX(maxfd, inotify_fd);
        while (!udev_exit) {
 +              sigset_t blocked_mask, orig_mask;
 +              struct pollfd pfd[4];
 +              struct pollfd *ctrl_poll, *monitor_poll, *inotify_poll = NULL;
 +              int nfds = 0;
                int fdcount;
  
 -              FD_ZERO(&readfds);
 -              FD_SET(signal_pipe[READ_END], &readfds);
 -              FD_SET(udev_ctrl_get_fd(udev_ctrl), &readfds);
 -              FD_SET(udev_monitor_get_fd(kernel_monitor), &readfds);
 -              if (inotify_fd >= 0)
 -                      FD_SET(inotify_fd, &readfds);
 -              fdcount = select(maxfd+1, &readfds, NULL, NULL, NULL);
 +              sigfillset(&blocked_mask);
 +              sigprocmask(SIG_SETMASK, &blocked_mask, &orig_mask);
 +              if (signal_received) {
 +                      sigprocmask(SIG_SETMASK, &orig_mask, NULL);
 +                      goto handle_signals;
 +              }
 +
 +              ctrl_poll = &pfd[nfds++];
 +              ctrl_poll->fd = udev_ctrl_get_fd(udev_ctrl);
 +              ctrl_poll->events = POLLIN;
 +
 +              monitor_poll = &pfd[nfds++];
 +              monitor_poll->fd = udev_monitor_get_fd(kernel_monitor);
 +              monitor_poll->events = POLLIN;
 +
 +              if (inotify_fd >= 0) {
 +                      inotify_poll = &pfd[nfds++];
 +                      inotify_poll->fd = inotify_fd;
 +                      inotify_poll->events = POLLIN;
 +              }
 +
 +              fdcount = ppoll(pfd, nfds, NULL, &orig_mask);
 +              sigprocmask(SIG_SETMASK, &orig_mask, NULL);
                if (fdcount < 0) {
 -                      if (errno != EINTR)
 -                              err(udev, "error in select: %m\n");
 +                      if (errno == EINTR)
 +                              goto handle_signals;
 +                      err(udev, "error in select: %m\n");
                        continue;
                }
  
                /* get control message */
 -              if (FD_ISSET(udev_ctrl_get_fd(udev_ctrl), &readfds))
 +              if (ctrl_poll->revents & POLLIN)
                        handle_ctrl_msg(udev_ctrl);
  
                /* get kernel uevent */
 -              if (FD_ISSET(udev_monitor_get_fd(kernel_monitor), &readfds)) {
 +              if (monitor_poll->revents & POLLIN) {
                        struct udev_device *dev;
  
                        dev = udev_monitor_receive_device(kernel_monitor);
                        }
                }
  
 -              /* received a signal, clear our notification pipe */
 -              if (FD_ISSET(signal_pipe[READ_END], &readfds)) {
 -                      char buf[256];
 -
 -                      read(signal_pipe[READ_END], &buf, sizeof(buf));
 -              }
 -
                /* rules directory inotify watch */
 -              if ((inotify_fd >= 0) && FD_ISSET(inotify_fd, &readfds)) {
 +              if (inotify_poll && (inotify_poll->revents & POLLIN)) {
                        int nbytes;
  
                        /* discard all possible events, we can just reload the config */
                        }
                }
  
 +handle_signals:
 +              signal_received = 0;
 +
                /* rules changed, set by inotify or a HUP signal */
                if (reload_config) {
                        struct udev_rules *rules_new;
  
                        reload_config = 0;
-                       rules_new = udev_rules_new(udev, 1);
+                       rules_new = udev_rules_new(udev, resolve_names);
                        if (rules_new != NULL) {
                                udev_rules_unref(rules);
                                rules = rules_new;
        rc = 0;
  exit:
        udev_rules_unref(rules);
 -      if (signal_pipe[READ_END] >= 0)
 -              close(signal_pipe[READ_END]);
 -      if (signal_pipe[WRITE_END] >= 0)
 -              close(signal_pipe[WRITE_END]);
        udev_ctrl_unref(udev_ctrl);
        if (inotify_fd >= 0)
                close(inotify_fd);