Skip to content
Browse files

FL-34: add busybox-1.21 patch

  • Loading branch information...
1 parent 27702f3 commit 2fce9ac53e54f05444154e2e455f43f01cdc6d2c @angryvincent angryvincent committed May 18, 2013
Showing with 705 additions and 0 deletions.
  1. +701 −0 sys-fs/mdev-bb/files/busybox-1.21.0-mdev.patch
  2. +4 −0 sys-fs/mdev-bb/mdev-bb-1.21.0.ebuild
View
701 sys-fs/mdev-bb/files/busybox-1.21.0-mdev.patch
@@ -0,0 +1,701 @@
+--- busybox-1.21.0/util-linux/mdev.c
++++ busybox-1.21.0-mdev/util-linux/mdev.c
+@@ -80,7 +80,7 @@
+ //usage: IF_FEATURE_MDEV_CONF(
+ //usage: "\n"
+ //usage: "It uses /etc/mdev.conf with lines\n"
+-//usage: " [-]DEVNAME UID:GID PERM"
++//usage: " [-][ENV=regex;]...DEVNAME UID:GID PERM"
+ //usage: IF_FEATURE_MDEV_RENAME(" [>|=PATH]|[!]")
+ //usage: IF_FEATURE_MDEV_EXEC(" [@|$|*PROG]")
+ //usage: "\n"
+@@ -230,9 +230,34 @@
+ * SUBSYSTEM=block
+ */
+
+-static const char keywords[] ALIGN1 = "add\0remove\0change\0";
++#define DEBUG_LVL 2
++
++#if DEBUG_LVL >= 1
++# define dbg1(...) do { if (G.verbose) bb_error_msg(__VA_ARGS__); } while(0)
++#else
++# define dbg1(...) ((void)0)
++#endif
++#if DEBUG_LVL >= 2
++# define dbg2(...) do { if (G.verbose >= 2) bb_error_msg(__VA_ARGS__); } while(0)
++#else
++# define dbg2(...) ((void)0)
++#endif
++#if DEBUG_LVL >= 3
++# define dbg3(...) do { if (G.verbose >= 3) bb_error_msg(__VA_ARGS__); } while(0)
++#else
++# define dbg3(...) ((void)0)
++#endif
++
++
++static const char keywords[] ALIGN1 = "add\0remove\0"; // "change\0"
+ enum { OP_add, OP_remove };
+
++struct envmatch {
++ struct envmatch *next;
++ char *envname;
++ regex_t match;
++};
++
+ struct rule {
+ bool keep_matching;
+ bool regex_compiled;
+@@ -243,12 +268,14 @@ struct rule {
+ char *ren_mov;
+ IF_FEATURE_MDEV_EXEC(char *r_cmd;)
+ regex_t match;
++ struct envmatch *envmatch;
+ };
+
+ struct globals {
+ int root_major, root_minor;
+ smallint verbose;
+ char *subsystem;
++ char *subsys_env; /* for putenv("SUBSYSTEM=subsystem") */
+ #if ENABLE_FEATURE_MDEV_CONF
+ const char *filename;
+ parser_t *parser;
+@@ -256,6 +283,7 @@ struct globals {
+ unsigned rule_idx;
+ #endif
+ struct rule cur_rule;
++ char timestr[sizeof("60.123456")];
+ } FIX_ALIASING;
+ #define G (*(struct globals*)&bb_common_bufsiz1)
+ #define INIT_G() do { \
+@@ -267,15 +295,8 @@ struct globals {
+ /* Prevent infinite loops in /sys symlinks */
+ #define MAX_SYSFS_DEPTH 3
+
+-/* We use additional 64+ bytes in make_device() */
+-#define SCRATCH_SIZE 80
+-
+-#if 0
+-# define dbg(...) bb_error_msg(__VA_ARGS__)
+-#else
+-# define dbg(...) ((void)0)
+-#endif
+-
++/* We use additional bytes in make_device() */
++#define SCRATCH_SIZE 128
+
+ #if ENABLE_FEATURE_MDEV_CONF
+
+@@ -288,14 +309,65 @@ static void make_default_cur_rule(void)
+
+ static void clean_up_cur_rule(void)
+ {
++ struct envmatch *e;
++
+ free(G.cur_rule.envvar);
++ free(G.cur_rule.ren_mov);
+ if (G.cur_rule.regex_compiled)
+ regfree(&G.cur_rule.match);
+- free(G.cur_rule.ren_mov);
+ IF_FEATURE_MDEV_EXEC(free(G.cur_rule.r_cmd);)
++ e = G.cur_rule.envmatch;
++ while (e) {
++ free(e->envname);
++ regfree(&e->match);
++ e = e->next;
++ }
+ make_default_cur_rule();
+ }
+
++/* In later versions, endofname is in libbb */
++#define endofname mdev_endofname
++static
++const char* FAST_FUNC
++endofname(const char *name)
++{
++#define is_name(c) ((c) == '_' || isalpha((unsigned char)(c)))
++#define is_in_name(c) ((c) == '_' || isalnum((unsigned char)(c)))
++ if (!is_name(*name))
++ return name;
++ while (*++name) {
++ if (!is_in_name(*name))
++ break;
++ }
++ return name;
++}
++
++static char *parse_envmatch_pfx(char *val)
++{
++ struct envmatch **nextp = &G.cur_rule.envmatch;
++
++ for (;;) {
++ struct envmatch *e;
++ char *semicolon;
++ char *eq = strchr(val, '=');
++ if (!eq /* || eq == val? */)
++ return val;
++ if (endofname(val) != eq)
++ return val;
++ semicolon = strchr(eq, ';');
++ if (!semicolon)
++ return val;
++ /* ENVVAR=regex;... */
++ *nextp = e = xzalloc(sizeof(*e));
++ nextp = &e->next;
++ e->envname = xstrndup(val, eq - val);
++ *semicolon = '\0';
++ xregcomp(&e->match, eq + 1, REG_EXTENDED);
++ *semicolon = ';';
++ val = semicolon + 1;
++ }
++}
++
+ static void parse_next_rule(void)
+ {
+ /* Note: on entry, G.cur_rule is set to default */
+@@ -308,12 +380,13 @@ static void parse_next_rule(void)
+ break;
+
+ /* Fields: [-]regex uid:gid mode [alias] [cmd] */
+- dbg("token1:'%s'", tokens[1]);
++ dbg3("token1:'%s'", tokens[1]);
+
+ /* 1st field */
+ val = tokens[0];
+ G.cur_rule.keep_matching = ('-' == val[0]);
+ val += G.cur_rule.keep_matching; /* swallow leading dash */
++ val = parse_envmatch_pfx(val);
+ if (val[0] == '@') {
+ /* @major,minor[-minor2] */
+ /* (useful when name is ambiguous:
+@@ -328,8 +401,10 @@ static void parse_next_rule(void)
+ if (sc == 2)
+ G.cur_rule.min1 = G.cur_rule.min0;
+ } else {
++ char *eq = strchr(val, '=');
+ if (val[0] == '$') {
+- char *eq = strchr(++val, '=');
++ /* $ENVVAR=regex ... */
++ val++;
+ if (!eq) {
+ bb_error_msg("bad $envvar=regex on line %d", G.parser->lineno);
+ goto next_rule;
+@@ -373,7 +448,7 @@ static void parse_next_rule(void)
+ clean_up_cur_rule();
+ } /* while (config_read) */
+
+- dbg("config_close(G.parser)");
++ dbg3("config_close(G.parser)");
+ config_close(G.parser);
+ G.parser = NULL;
+
+@@ -390,7 +465,7 @@ static const struct rule *next_rule(void
+
+ /* Open conf file if we didn't do it yet */
+ if (!G.parser && G.filename) {
+- dbg("config_open('%s')", G.filename);
++ dbg3("config_open('%s')", G.filename);
+ G.parser = config_open2(G.filename, fopen_for_read);
+ G.filename = NULL;
+ }
+@@ -399,7 +474,7 @@ static const struct rule *next_rule(void
+ /* mdev -s */
+ /* Do we have rule parsed already? */
+ if (G.rule_vec[G.rule_idx]) {
+- dbg("< G.rule_vec[G.rule_idx:%d]=%p", G.rule_idx, G.rule_vec[G.rule_idx]);
++ dbg3("< G.rule_vec[G.rule_idx:%d]=%p", G.rule_idx, G.rule_vec[G.rule_idx]);
+ return G.rule_vec[G.rule_idx++];
+ }
+ make_default_cur_rule();
+@@ -416,13 +491,28 @@ static const struct rule *next_rule(void
+ rule = memcpy(xmalloc(sizeof(G.cur_rule)), &G.cur_rule, sizeof(G.cur_rule));
+ G.rule_vec = xrealloc_vector(G.rule_vec, 4, G.rule_idx);
+ G.rule_vec[G.rule_idx++] = rule;
+- dbg("> G.rule_vec[G.rule_idx:%d]=%p", G.rule_idx, G.rule_vec[G.rule_idx]);
++ dbg3("> G.rule_vec[G.rule_idx:%d]=%p", G.rule_idx, G.rule_vec[G.rule_idx]);
+ }
+ }
+
+ return rule;
+ }
+
++static int env_matches(struct envmatch *e)
++{
++ while (e) {
++ int r;
++ char *val = getenv(e->envname);
++ if (!val)
++ return 0;
++ r = regexec(&e->match, val, /*size*/ 0, /*range[]*/ NULL, /*eflags*/ 0);
++ if (r != 0) /* no match */
++ return 0;
++ e = e->next;
++ }
++ return 1;
++}
++
+ #else
+
+ # define next_rule() (&G.cur_rule)
+@@ -468,7 +558,7 @@ static char *build_alias(char *alias, co
+
+ /* mknod in /dev based on a path like "/sys/block/hda/hda1"
+ * NB1: path parameter needs to have SCRATCH_SIZE scratch bytes
+- * after NUL, but we promise to not mangle (IOW: to restore if needed)
++ * after NUL, but we promise to not mangle (IOW: to restore NUL if needed)
+ * path string.
+ * NB2: "mdev -s" may call us many times, do not leak memory/fds!
+ *
+@@ -478,9 +568,7 @@ static char *build_alias(char *alias, co
+ static void make_device(char *device_name, char *path, int operation)
+ {
+ int major, minor, type, len;
+-
+- if (G.verbose)
+- bb_error_msg("device: %s, %s", device_name, path);
++ char *path_end = path + strlen(path);
+
+ /* Try to read major/minor string. Note that the kernel puts \n after
+ * the data, so we don't need to worry about null terminating the string
+@@ -489,29 +577,51 @@ static void make_device(char *device_nam
+ */
+ major = -1;
+ if (operation == OP_add) {
+- char *dev_maj_min = path + strlen(path);
+-
+- strcpy(dev_maj_min, "/dev");
+- len = open_read_close(path, dev_maj_min + 1, 64);
+- *dev_maj_min = '\0';
++ strcpy(path_end, "/dev");
++ len = open_read_close(path, path_end + 1, SCRATCH_SIZE - 1);
++ *path_end = '\0';
+ if (len < 1) {
+ if (!ENABLE_FEATURE_MDEV_EXEC)
+ return;
+ /* no "dev" file, but we can still run scripts
+ * based on device name */
+- } else if (sscanf(++dev_maj_min, "%u:%u", &major, &minor) == 2) {
+- if (G.verbose)
+- bb_error_msg("maj,min: %u,%u", major, minor);
++ } else if (sscanf(path_end + 1, "%u:%u", &major, &minor) == 2) {
++ dbg1("dev %u,%u", major, minor);
+ } else {
+ major = -1;
+ }
+ }
+ /* else: for delete, -1 still deletes the node, but < -1 suppresses that */
+
+- /* Determine device name, type, major and minor */
+- if (!device_name)
+- device_name = (char*) bb_basename(path);
+- /* http://kernel.org/doc/pending/hotplug.txt says that only
++ /* Determine device name */
++ if (!device_name) {
++ /*
++ * There was no $DEVNAME envvar (for example, mdev -s never has).
++ * But it is very useful: it contains the *path*, not only basename,
++ * Thankfully, uevent file has it.
++ * Example of .../sound/card0/controlC0/uevent file on Linux-3.7.7:
++ * MAJOR=116
++ * MINOR=7
++ * DEVNAME=snd/controlC0
++ */
++ strcpy(path_end, "/uevent");
++ len = open_read_close(path, path_end + 1, SCRATCH_SIZE - 1);
++ if (len < 0)
++ len = 0;
++ *path_end = '\0';
++ path_end[1 + len] = '\0';
++ device_name = strstr(path_end + 1, "\nDEVNAME=");
++ if (device_name) {
++ device_name += sizeof("\nDEVNAME=")-1;
++ strchrnul(device_name, '\n')[0] = '\0';
++ } else {
++ /* Fall back to just basename */
++ device_name = (char*) bb_basename(path);
++ }
++ }
++ /* Determine device type */
++ /*
++ * http://kernel.org/doc/pending/hotplug.txt says that only
+ * "/sys/block/..." is for block devices. "/sys/bus" etc is not.
+ * But since 2.6.25 block devices are also in /sys/class/block.
+ * We use strstr("/block/") to forestall future surprises.
+@@ -537,6 +647,8 @@ static void make_device(char *device_nam
+ rule = next_rule();
+
+ #if ENABLE_FEATURE_MDEV_CONF
++ if (!env_matches(rule->envmatch))
++ continue;
+ if (rule->maj >= 0) { /* @maj,min rule */
+ if (major != rule->maj)
+ continue;
+@@ -547,7 +659,7 @@ static void make_device(char *device_nam
+ }
+ if (rule->envvar) { /* $envvar=regex rule */
+ str_to_match = getenv(rule->envvar);
+- dbg("getenv('%s'):'%s'", rule->envvar, str_to_match);
++ dbg3("getenv('%s'):'%s'", rule->envvar, str_to_match);
+ if (!str_to_match)
+ continue;
+ }
+@@ -555,7 +667,7 @@ static void make_device(char *device_nam
+
+ if (rule->regex_compiled) {
+ int regex_match = regexec(&rule->match, str_to_match, ARRAY_SIZE(off), off, 0);
+- dbg("regex_match for '%s':%d", str_to_match, regex_match);
++ dbg3("regex_match for '%s':%d", str_to_match, regex_match);
+ //bb_error_msg("matches:");
+ //for (int i = 0; i < ARRAY_SIZE(off); i++) {
+ // if (off[i].rm_so < 0) continue;
+@@ -574,9 +686,8 @@ static void make_device(char *device_nam
+ }
+ /* else: it's final implicit "match-all" rule */
+ rule_matches:
++ dbg2("rule matched, line %d", G.parser ? G.parser->lineno : -1);
+ #endif
+- dbg("rule matched");
+-
+ /* Build alias name */
+ alias = NULL;
+ if (ENABLE_FEATURE_MDEV_RENAME && rule->ren_mov) {
+@@ -619,34 +730,30 @@ static void make_device(char *device_nam
+ }
+ }
+ }
+- dbg("alias:'%s'", alias);
++ dbg3("alias:'%s'", alias);
+
+ command = NULL;
+ IF_FEATURE_MDEV_EXEC(command = rule->r_cmd;)
+ if (command) {
+- const char *s = "$@*";
+- const char *s2 = strchr(s, command[0]);
+-
+ /* Are we running this command now?
+- * Run $cmd on delete, @cmd on create, *cmd on both
++ * Run @cmd on create, $cmd on delete, *cmd on any
+ */
+- if (s2 - s != (operation == OP_remove) || *s2 == '*') {
+- /* We are here if: '*',
+- * or: '@' and delete = 0,
+- * or: '$' and delete = 1
+- */
++ if ((command[0] == '@' && operation == OP_add)
++ || (command[0] == '$' && operation == OP_remove)
++ || (command[0] == '*')
++ ) {
+ command++;
+ } else {
+ command = NULL;
+ }
+ }
+- dbg("command:'%s'", command);
++ dbg3("command:'%s'", command);
+
+ /* "Execute" the line we found */
+ node_name = device_name;
+ if (ENABLE_FEATURE_MDEV_RENAME && alias) {
+ node_name = alias = build_alias(alias, device_name);
+- dbg("alias2:'%s'", alias);
++ dbg3("alias2:'%s'", alias);
+ }
+
+ if (operation == OP_add && major >= 0) {
+@@ -656,8 +763,17 @@ static void make_device(char *device_nam
+ mkdir_recursive(node_name);
+ *slash = '/';
+ }
+- if (G.verbose)
+- bb_error_msg("mknod: %s (%d,%d) %o", node_name, major, minor, rule->mode | type);
++ if (ENABLE_FEATURE_MDEV_CONF) {
++ dbg1("mknod %s (%d,%d) %o"
++ " %u:%u",
++ node_name, major, minor, rule->mode | type,
++ rule->ugid.uid, rule->ugid.gid
++ );
++ } else {
++ dbg1("mknod %s (%d,%d) %o",
++ node_name, major, minor, rule->mode | type
++ );
++ }
+ if (mknod(node_name, rule->mode | type, makedev(major, minor)) && errno != EEXIST)
+ bb_perror_msg("can't create '%s'", node_name);
+ if (ENABLE_FEATURE_MDEV_CONF) {
+@@ -671,8 +787,7 @@ static void make_device(char *device_nam
+ //TODO: on devtmpfs, device_name already exists and symlink() fails.
+ //End result is that instead of symlink, we have two nodes.
+ //What should be done?
+- if (G.verbose)
+- bb_error_msg("symlink: %s", device_name);
++ dbg1("symlink: %s", device_name);
+ symlink(node_name, device_name);
+ }
+ }
+@@ -681,27 +796,21 @@ static void make_device(char *device_nam
+ if (ENABLE_FEATURE_MDEV_EXEC && command) {
+ /* setenv will leak memory, use putenv/unsetenv/free */
+ char *s = xasprintf("%s=%s", "MDEV", node_name);
+- char *s1 = xasprintf("%s=%s", "SUBSYSTEM", G.subsystem);
+ putenv(s);
+- putenv(s1);
+- if (G.verbose)
+- bb_error_msg("running: %s", command);
++ dbg1("running: %s", command);
+ if (system(command) == -1)
+ bb_perror_msg("can't run '%s'", command);
+- bb_unsetenv_and_free(s1);
+ bb_unsetenv_and_free(s);
+ }
+
+ if (operation == OP_remove && major >= -1) {
+ if (ENABLE_FEATURE_MDEV_RENAME && alias) {
+ if (aliaslink == '>') {
+- if (G.verbose)
+- bb_error_msg("unlink: %s", device_name);
++ dbg1("unlink: %s", device_name);
+ unlink(device_name);
+ }
+ }
+- if (G.verbose)
+- bb_error_msg("unlink: %s", node_name);
++ dbg1("unlink: %s", node_name);
+ unlink(node_name);
+ }
+
+@@ -746,9 +855,16 @@ static int FAST_FUNC dirAction(const cha
+ * under /sys/class/ */
+ if (1 == depth) {
+ free(G.subsystem);
++ if (G.subsys_env) {
++ bb_unsetenv_and_free(G.subsys_env);
++ G.subsys_env = NULL;
++ }
+ G.subsystem = strrchr(fileName, '/');
+- if (G.subsystem)
++ if (G.subsystem) {
+ G.subsystem = xstrdup(G.subsystem + 1);
++ G.subsys_env = xasprintf("%s=%s", "SUBSYSTEM", G.subsystem);
++ putenv(G.subsys_env);
++ }
+ }
+
+ return (depth >= MAX_SYSFS_DEPTH ? SKIP : TRUE);
+@@ -813,12 +929,107 @@ static void load_firmware(const char *fi
+ full_write(loading_fd, "-1", 2);
+
+ out:
++ xchdir("/dev");
+ if (ENABLE_FEATURE_CLEAN_UP) {
+ close(firmware_fd);
+ close(loading_fd);
+ }
+ }
+
++static char *curtime(void)
++{
++ struct timeval tv;
++ gettimeofday(&tv, NULL);
++ sprintf(G.timestr, "%u.%06u", (unsigned)tv.tv_sec % 60, (unsigned)tv.tv_usec);
++ return G.timestr;
++}
++
++static void open_mdev_log(const char *seq, unsigned my_pid)
++{
++ int logfd = open("mdev.log", O_WRONLY | O_APPEND);
++ if (logfd >= 0) {
++ xmove_fd(logfd, STDERR_FILENO);
++ G.verbose = 2;
++ applet_name = xasprintf("%s[%s]", applet_name, seq ? seq : utoa(my_pid));
++ }
++}
++
++/* If it exists, does /dev/mdev.seq match $SEQNUM?
++ * If it does not match, earlier mdev is running
++ * in parallel, and we need to wait.
++ * Active mdev pokes us with SIGCHLD to check the new file.
++ */
++static int
++wait_for_seqfile(const char *seq)
++{
++ /* We time out after 2 sec */
++ static const struct timespec ts = { 0, 32*1000*1000 };
++ int timeout = 2000 / 32;
++ int seq_fd = -1;
++ int do_once = 1;
++ sigset_t set_CHLD;
++
++ sigemptyset(&set_CHLD);
++ sigaddset(&set_CHLD, SIGCHLD);
++ sigprocmask(SIG_BLOCK, &set_CHLD, NULL);
++
++ for (;;) {
++ int seqlen;
++ char seqbuf[sizeof(int)*3 + 2];
++
++ if (seq_fd < 0) {
++ seq_fd = open("mdev.seq", O_RDWR);
++ if (seq_fd < 0)
++ break;
++ }
++ seqlen = pread(seq_fd, seqbuf, sizeof(seqbuf) - 1, 0);
++ if (seqlen < 0) {
++ close(seq_fd);
++ seq_fd = -1;
++ break;
++ }
++ seqbuf[seqlen] = '\0';
++ if (seqbuf[0] == '\n') {
++ /* seed file: write out seq ASAP */
++ xwrite_str(seq_fd, seq);
++ xlseek(seq_fd, 0, SEEK_SET);
++ dbg2("first seq written");
++ break;
++ }
++ if (strcmp(seq, seqbuf) == 0) {
++ /* correct idx */
++ break;
++ }
++ if (do_once) {
++ dbg2("%s waiting for '%s'", curtime(), seqbuf);
++ do_once = 0;
++ }
++ if (sigtimedwait(&set_CHLD, NULL, &ts) >= 0) {
++ dbg3("woken up");
++ continue; /* don't decrement timeout! */
++ }
++ if (--timeout == 0) {
++ dbg1("%s waiting for '%s'", "timed out", seqbuf);
++ break;
++ }
++ }
++ sigprocmask(SIG_UNBLOCK, &set_CHLD, NULL);
++ return seq_fd;
++}
++
++static void signal_mdevs(unsigned my_pid)
++{
++ procps_status_t* p = NULL;
++ while ((p = procps_scan(p, PSSCAN_ARGV0)) != NULL) {
++ if (p->pid != my_pid
++ && p->argv0
++ && strcmp(bb_basename(p->argv0), "mdev") == 0
++ ) {
++ kill(p->pid, SIGCHLD);
++ }
++ }
++}
++
+ int mdev_main(int argc, char **argv) MAIN_EXTERNALLY_VISIBLE;
+ int mdev_main(int argc UNUSED_PARAM, char **argv)
+ {
+@@ -840,8 +1051,8 @@ int mdev_main(int argc UNUSED_PARAM, cha
+ xchdir("/dev");
+
+ if (argv[1] && strcmp(argv[1], "-s") == 0) {
+- /* Scan:
+- * mdev -s
++ /*
++ * Scan: mdev -s
+ */
+ struct stat st;
+
+@@ -853,6 +1064,8 @@ int mdev_main(int argc UNUSED_PARAM, cha
+ G.root_major = major(st.st_dev);
+ G.root_minor = minor(st.st_dev);
+
++ putenv((char*)"ACTION=add");
++
+ /* ACTION_FOLLOWLINKS is needed since in newer kernels
+ * /sys/block/loop* (for example) are symlinks to dirs,
+ * not real directories.
+@@ -878,11 +1091,13 @@ int mdev_main(int argc UNUSED_PARAM, cha
+ char *action;
+ char *env_devname;
+ char *env_devpath;
++ unsigned my_pid;
++ int seq_fd;
+ smalluint op;
+
+ /* Hotplug:
+ * env ACTION=... DEVPATH=... SUBSYSTEM=... [SEQNUM=...] mdev
+- * ACTION can be "add" or "remove"
++ * ACTION can be "add", "remove", "change"
+ * DEVPATH is like "/block/sda" or "/class/input/mice"
+ */
+ action = getenv("ACTION");
+@@ -893,39 +1108,20 @@ int mdev_main(int argc UNUSED_PARAM, cha
+ if (!action || !env_devpath /*|| !G.subsystem*/)
+ bb_show_usage();
+ fw = getenv("FIRMWARE");
+- /* If it exists, does /dev/mdev.seq match $SEQNUM?
+- * If it does not match, earlier mdev is running
+- * in parallel, and we need to wait */
+ seq = getenv("SEQNUM");
+- if (seq) {
+- int timeout = 2000 / 32; /* 2000 msec */
+- do {
+- int seqlen;
+- char seqbuf[sizeof(int)*3 + 2];
+-
+- seqlen = open_read_close("mdev.seq", seqbuf, sizeof(seqbuf) - 1);
+- if (seqlen < 0) {
+- seq = NULL;
+- break;
+- }
+- seqbuf[seqlen] = '\0';
+- if (seqbuf[0] == '\n' /* seed file? */
+- || strcmp(seq, seqbuf) == 0 /* correct idx? */
+- ) {
+- break;
+- }
+- usleep(32*1000);
+- } while (--timeout);
+- }
+
+- {
+- int logfd = open("/dev/mdev.log", O_WRONLY | O_APPEND);
+- if (logfd >= 0) {
+- xmove_fd(logfd, STDERR_FILENO);
+- G.verbose = 1;
+- bb_error_msg("seq: %s action: %s", seq, action);
+- }
+- }
++ my_pid = getpid();
++ open_mdev_log(seq, my_pid);
++
++ seq_fd = seq ? wait_for_seqfile(seq) : -1;
++
++ dbg1("%s "
++ "ACTION:%s SUBSYSTEM:%s DEVNAME:%s DEVPATH:%s"
++ "%s%s",
++ curtime(),
++ action, G.subsystem, env_devname, env_devpath,
++ fw ? " FW:" : "", fw ? fw : ""
++ );
+
+ snprintf(temp, PATH_MAX, "/sys%s", env_devpath);
+ if (op == OP_remove) {
+@@ -935,16 +1131,18 @@ int mdev_main(int argc UNUSED_PARAM, cha
+ if (!fw)
+ make_device(env_devname, temp, op);
+ }
+- else if (op == OP_add) {
++ else {
+ make_device(env_devname, temp, op);
+ if (ENABLE_FEATURE_MDEV_LOAD_FIRMWARE) {
+- if (fw)
++ if (op == OP_add && fw)
+ load_firmware(fw, temp);
+ }
+ }
+
+- if (seq) {
+- xopen_xwrite_close("mdev.seq", utoa(xatou(seq) + 1));
++ dbg1("%s exiting", curtime());
++ if (seq_fd >= 0) {
++ xwrite_str(seq_fd, utoa(xatou(seq) + 1));
++ signal_mdevs(my_pid);
+ }
+ }
+
View
4 sys-fs/mdev-bb/mdev-bb-1.21.0.ebuild
@@ -22,6 +22,10 @@ RDEPEND="!sys-apps/busybox[mdev]"
S=${WORKDIR}/${MY_P}
QA_PRESTRIPPED="/sbin/mdev"
+src_prepare(){
+ epatch "${FILESDIR}/busybox-1.21.0-mdev.patch"
+}
+
src_configure() {
cat >"${S}/tmp.config" <<-END_OF_CONFIG
CONFIG_HAVE_DOT_CONFIG=y

0 comments on commit 2fce9ac

Please sign in to comment.
Something went wrong with that request. Please try again.