Commit: a9d5cda0617319b66aeaa96fd2a22c0b12530fc7 - system-coremmmm (git)

system/corennnnn

Commit MetaInfo

• Révision: a9d5cda0617319b66aeaa96fd2a22c0b12530fc7 (tree)
• l'heure: 2016-07-27 03:52:25
• Auteur: Chih-Wei Huang <cwhuang@linu...>
• Commiter: Jaap Jan Meijer

Message de Log

ueventd: load modules in a forked process

Loading some modules (e.g., ath3k) will be blocked to wait for
firmware. The forking is necessary, otherwise ueventd has no
chance to load the firmware.

Change Summary

• modified: init/devices.cpp

Modification

--- a/init/devices.cpp
+++ b/init/devices.cpp
@@ -875,14 +875,12 @@ out:
     return ret;
 }
 
-static int load_module_by_device_modalias(const char *id)
+static int do_load_module_by_device_modalias(const char *id)
 {
     struct listnode *alias_node;
     struct module_alias_node *alias;
     int ret = -1;
 
-    if (!id) goto out;
-
     list_for_each(alias_node, &modules_aliases_map) {
         alias = node_to_item(alias_node, struct module_alias_node, list);
 
@@ -901,27 +899,38 @@ static int load_module_by_device_modalias(const char *id)
                         /* loading was successful */
                         INFO("loaded module %s due to uevents\n", alias->name);
                         ret = 0;
-                        goto out;
+                        break;
                     }
                 }
             }
         }
     }
 
-out:
     return ret;
 }
 
-static void handle_deferred_module_loading()
+static void load_module_by_device_modalias(const char *id)
 {
-    struct listnode *node = NULL;
-    struct listnode *next = NULL;
-    struct module_alias_node *alias = NULL;
+    if (id) {
+        pid_t pid = fork();
+        if (!pid) {
+            exit(do_load_module_by_device_modalias(id));
+        } else if (pid < 0) {
+            ERROR("failed to fork for loading %s\n", id);
+        }
+    }
+}
 
+static void handle_deferred_module_loading()
+{
     /* try to read the module alias mapping if map is empty
      * if succeed, loading all the modules in the queue
      */
     if (!list_empty(&modules_aliases_map)) {
+        struct listnode *node = NULL;
+        struct listnode *next = NULL;
+        struct module_alias_node *alias = NULL;
+
         list_for_each_safe(node, next, &deferred_module_loading_list) {
             alias = node_to_item(node, struct module_alias_node, list);
 
@@ -945,7 +954,7 @@ int module_probe(const char *modalias)
             return -1;
     }
 
-    return load_module_by_device_modalias(modalias);
+    return modalias ? do_load_module_by_device_modalias(modalias) : -1;
 }
 
 static void handle_module_loading(const char *modalias)