package/busybox: support spaces in module aliases in mdev
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Fri, 1 Jul 2016 18:56:19 +0000 (21:56 +0300)
committerThomas Petazzoni <thomas.petazzoni@free-electrons.com>
Sat, 2 Jul 2016 11:15:50 +0000 (13:15 +0200)
The new change which enabled automatic module loading on boot does not handle
the cases when module alias includes spaces. It prevents modules to be loaded
since script fails:

  % find /sys/ -name modalias | xargs sort -u
  sort: /sys/devices/platform/Fixed: No such file or directory

First alias in question is "platform:Fixed MDIO bus".

Amend the script to support above like cases.

Fixes: 07f46c2b6dae ("package/busybox: support automatic module loading with mdev")
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
package/busybox/S10mdev

index 4cb31de42d03ae0e110dc2bdd3eb23b596bc8ec1..63ca955b1c551671dc18adac4f7e961430c1a675 100644 (file)
@@ -9,7 +9,7 @@ case "$1" in
        echo /sbin/mdev >/proc/sys/kernel/hotplug
        /sbin/mdev -s
        # coldplug modules
-       find /sys/ -name modalias | xargs sort -u | xargs modprobe -abq
+       find /sys/ -name modalias -print0 | xargs -0 sort -u -z | xargs -0 modprobe -abq
        ;;
   stop)
        ;;