eudev: restore userspace firmware loading support
This sycncs eudev with systemd. This is required for old kernels to work and does same backport as did in: ,---- | commit 31163abc7bedd477c1aad10e6d1f47dbee194513 | Author: Jonathan Liu <net147@gmail.com> | Date: Thu Mar 19 15:38:32 2015 +1100 | | systemd: restore userspace firmware loading support | | This changes the minimum required Linux version from 3.7 back to 3.0. | | [YOCTO #7409] | | Signed-off-by: Jonathan Liu <net147@gmail.com> | Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org> `---- (From OE-Core rev: d6b139ef1b52ee4842f8706c1b8b950cc50a3d54) Signed-off-by: Lauren Post <lauren.post@nxp.com> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
This commit is contained in:
parent
c711caae83
commit
020e35cfd9
|
@ -0,0 +1,28 @@
|
|||
From 4f0a722489154da99e7f6b3051afde984eed2f74 Mon Sep 17 00:00:00 2001
|
||||
From: Jonathan Liu <net147@gmail.com>
|
||||
Date: Thu, 19 Mar 2015 15:01:29 +1100
|
||||
Subject: [PATCH] Revert "rules: remove firmware loading rules"
|
||||
|
||||
This reverts commit 70e7d754ddb356fb1a2942b262f8cee9650e2a19.
|
||||
Userspace firmware loading support is needed for Linux < 3.7.
|
||||
|
||||
Upstream-Status: Inappropriate [OE specific]
|
||||
|
||||
Signed-off-by: Jonathan Liu <net147@gmail.com>
|
||||
---
|
||||
rules/50-firmware.rules | 3 +++
|
||||
1 file changed, 3 insertions(+)
|
||||
create mode 100644 rules/50-firmware.rules
|
||||
|
||||
diff --git a/rules/50-firmware.rules b/rules/50-firmware.rules
|
||||
new file mode 100644
|
||||
index 0000000..f0ae684
|
||||
--- /dev/null
|
||||
+++ b/rules/50-firmware.rules
|
||||
@@ -0,0 +1,3 @@
|
||||
+# do not edit this file, it will be overwritten on update
|
||||
+
|
||||
+SUBSYSTEM=="firmware", ACTION=="add", RUN{builtin}="firmware"
|
||||
--
|
||||
2.3.3
|
||||
|
|
@ -0,0 +1,364 @@
|
|||
From e415372cc7a2f52e70e1cfa8c6c1f633b411355d Mon Sep 17 00:00:00 2001
|
||||
From: Lauren Post <lauren.post@nxp.com>
|
||||
Date: Wed, 8 Jun 2016 06:51:56 -0500
|
||||
Subject: [PATCH] Revert "udev: remove userspace firmware loading support"
|
||||
|
||||
This reverts commit 3b717594600fa717cdf9bcfd0c7c1b703b245482.
|
||||
|
||||
Conflicts:
|
||||
configure.ac
|
||||
src/udev/udevd.c
|
||||
|
||||
Upstream-Status: Inappropriate [OE specific]
|
||||
|
||||
Signed-off-by: Lauren Post <lauren.post@nxp.com>
|
||||
---
|
||||
configure.ac | 39 +++++++++-
|
||||
rules/Makefile.am | 5 ++
|
||||
src/udev/Makefile.am | 10 +++
|
||||
src/udev/udev-builtin-firmware.c | 154 +++++++++++++++++++++++++++++++++++++++
|
||||
src/udev/udev-builtin.c | 3 +
|
||||
src/udev/udev.h | 6 ++
|
||||
src/udev/udevd.c | 13 ++++
|
||||
7 files changed, 227 insertions(+), 3 deletions(-)
|
||||
create mode 100644 src/udev/udev-builtin-firmware.c
|
||||
|
||||
diff --git a/configure.ac b/configure.ac
|
||||
index 8691891..65028c2 100644
|
||||
--- a/configure.ac
|
||||
+++ b/configure.ac
|
||||
@@ -238,9 +238,42 @@ AC_CHECK_DECL([unshare],
|
||||
[#include <sched.h>])
|
||||
|
||||
# ------------------------------------------------------------------------------
|
||||
-AC_PATH_TOOL(GPERF, gperf)
|
||||
-if test -z "$GPERF" ; then
|
||||
- AC_MSG_ERROR([*** gperf not found])
|
||||
+AC_ARG_WITH(firmware-path,
|
||||
+ AS_HELP_STRING([--with-firmware-path=DIR[[[:DIR[...]]]]],
|
||||
+ [Firmware search path (default=ROOTPREFIX/lib/firmware/updates:ROOTPREFIX/lib/firmware)]),
|
||||
+ [], [with_firmware_path="$rootprefix/lib/firmware/updates:$rootprefix/lib/firmware"])
|
||||
+OLD_IFS=$IFS
|
||||
+IFS=:
|
||||
+for i in $with_firmware_path; do
|
||||
+ if test "x${FIRMWARE_PATH}" = "x"; then
|
||||
+ FIRMWARE_PATH="\\\"${i}/\\\""
|
||||
+ else
|
||||
+ FIRMWARE_PATH="${FIRMWARE_PATH}, \\\"${i}/\\\""
|
||||
+ fi
|
||||
+done
|
||||
+IFS=$OLD_IFS
|
||||
+AC_SUBST(FIRMWARE_PATH)
|
||||
+AS_IF([test "x${FIRMWARE_PATH}" != "x"], [ AC_DEFINE(HAVE_FIRMWARE, 1, [Define if FIRMWARE is available]) ])
|
||||
+AM_CONDITIONAL(ENABLE_FIRMWARE, [test "x${FIRMWARE_PATH}" != "x"])
|
||||
+
|
||||
+# ------------------------------------------------------------------------------
|
||||
+AC_ARG_ENABLE([gudev],
|
||||
+ AS_HELP_STRING([--disable-gudev], [disable Gobject libudev support @<:@default=enabled@:>@]),
|
||||
+ [], [enable_gudev=yes])
|
||||
+AS_IF([test "x$enable_gudev" = "xyes"], [ PKG_CHECK_MODULES([GLIB], [glib-2.0 >= 2.22.0 gobject-2.0 >= 2.22.0]) ])
|
||||
+AM_CONDITIONAL([ENABLE_GUDEV], [test "x$enable_gudev" = "xyes"])
|
||||
+
|
||||
+# ------------------------------------------------------------------------------
|
||||
+AC_ARG_ENABLE([keymap],
|
||||
+ AS_HELP_STRING([--disable-keymap], [disable keymap fixup support @<:@default=enabled@:>@]),
|
||||
+ [], [enable_keymap=yes])
|
||||
+
|
||||
+if test "x$enable_keymap" = "xyes"; then
|
||||
+ AC_PATH_TOOL(GPERF, gperf)
|
||||
+ if test -z "$GPERF" ; then
|
||||
+ AC_MSG_ERROR([*** gperf not found])
|
||||
+ fi
|
||||
+ AC_DEFINE([ENABLE_KEYMAP], [1], [Define if we are enabling rule generator])
|
||||
fi
|
||||
|
||||
# ------------------------------------------------------------------------------
|
||||
diff --git a/rules/Makefile.am b/rules/Makefile.am
|
||||
index 24c099c..d714ae3 100644
|
||||
--- a/rules/Makefile.am
|
||||
+++ b/rules/Makefile.am
|
||||
@@ -22,6 +22,11 @@ dist_udevrules_DATA += \
|
||||
80-net-name-slot.rules
|
||||
endif
|
||||
|
||||
+if ENABLE_FIRMWARE
|
||||
+dist_udevrules_DATA += \
|
||||
+ 50-firmware.rules
|
||||
+endif
|
||||
+
|
||||
if HAVE_BLKID
|
||||
dist_udevrules_DATA += \
|
||||
60-persistent-storage.rules
|
||||
diff --git a/src/udev/Makefile.am b/src/udev/Makefile.am
|
||||
index 401af01..742fbc8 100644
|
||||
--- a/src/udev/Makefile.am
|
||||
+++ b/src/udev/Makefile.am
|
||||
@@ -53,12 +53,18 @@ libudev_core_la_SOURCES = \
|
||||
udev-ctrl.c \
|
||||
udev-builtin.c \
|
||||
udev-builtin-btrfs.c \
|
||||
+ udev-builtin-firmware.c \
|
||||
udev-builtin-hwdb.c \
|
||||
udev-builtin-input_id.c \
|
||||
udev-builtin-net_id.c \
|
||||
udev-builtin-path_id.c \
|
||||
udev-builtin-usb_id.c
|
||||
|
||||
+if ENABLE_FIRMWARE
|
||||
+libudev_core_la_SOURCES += \
|
||||
+ udev-builtin-firmware.c
|
||||
+endif
|
||||
+
|
||||
include_HEADERS = \
|
||||
udev.h
|
||||
|
||||
@@ -86,6 +92,10 @@ libudev_core_la_LIBADD += \
|
||||
$(KMOD_LIBS)
|
||||
endif
|
||||
|
||||
+libudev_core_la_CPPFLAGS = \
|
||||
+ $(AM_CPPFLAGS) \
|
||||
+ -DFIRMWARE_PATH="$(FIRMWARE_PATH)"
|
||||
+
|
||||
#
|
||||
# Extras
|
||||
#
|
||||
diff --git a/src/udev/udev-builtin-firmware.c b/src/udev/udev-builtin-firmware.c
|
||||
new file mode 100644
|
||||
index 0000000..bd8c2fb
|
||||
--- /dev/null
|
||||
+++ b/src/udev/udev-builtin-firmware.c
|
||||
@@ -0,0 +1,154 @@
|
||||
+/*
|
||||
+ * firmware - Kernel firmware loader
|
||||
+ *
|
||||
+ * Copyright (C) 2009 Piter Punk <piterpunk@slackware.com>
|
||||
+ * Copyright (C) 2009-2011 Kay Sievers <kay@vrfy.org>
|
||||
+ *
|
||||
+ * This program is free software; you can redistribute it and/or
|
||||
+ * modify it under the terms of the GNU General Public License as
|
||||
+ * published by the Free Software Foundation; either version 2 of the
|
||||
+ * License, or (at your option) any later version.
|
||||
+ *
|
||||
+ * This program is distributed in the hope that it will be useful, but
|
||||
+ * WITHOUT ANY WARRANTY; without even the implied warranty of
|
||||
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
|
||||
+ * General Public License for more details:*
|
||||
+ */
|
||||
+
|
||||
+#include <unistd.h>
|
||||
+#include <stdlib.h>
|
||||
+#include <string.h>
|
||||
+#include <stdio.h>
|
||||
+#include <getopt.h>
|
||||
+#include <errno.h>
|
||||
+#include <stdbool.h>
|
||||
+#include <sys/utsname.h>
|
||||
+#include <sys/stat.h>
|
||||
+
|
||||
+#include "udev.h"
|
||||
+
|
||||
+static bool set_loading(struct udev *udev, char *loadpath, const char *state) {
|
||||
+ FILE *ldfile;
|
||||
+
|
||||
+ ldfile = fopen(loadpath, "we");
|
||||
+ if (ldfile == NULL) {
|
||||
+ log_error("error: can not open '%s'", loadpath);
|
||||
+ return false;
|
||||
+ };
|
||||
+ fprintf(ldfile, "%s\n", state);
|
||||
+ fclose(ldfile);
|
||||
+ return true;
|
||||
+}
|
||||
+
|
||||
+static bool copy_firmware(struct udev *udev, const char *source, const char *target, size_t size) {
|
||||
+ char *buf;
|
||||
+ FILE *fsource = NULL, *ftarget = NULL;
|
||||
+ bool ret = false;
|
||||
+
|
||||
+ buf = malloc(size);
|
||||
+ if (buf == NULL) {
|
||||
+ log_error("No memory available to load firmware file");
|
||||
+ return false;
|
||||
+ }
|
||||
+
|
||||
+ log_debug("writing '%s' (%zi) to '%s'", source, size, target);
|
||||
+
|
||||
+ fsource = fopen(source, "re");
|
||||
+ if (fsource == NULL)
|
||||
+ goto exit;
|
||||
+ ftarget = fopen(target, "we");
|
||||
+ if (ftarget == NULL)
|
||||
+ goto exit;
|
||||
+ if (fread(buf, size, 1, fsource) != 1)
|
||||
+ goto exit;
|
||||
+ if (fwrite(buf, size, 1, ftarget) == 1)
|
||||
+ ret = true;
|
||||
+exit:
|
||||
+ if (ftarget != NULL)
|
||||
+ fclose(ftarget);
|
||||
+ if (fsource != NULL)
|
||||
+ fclose(fsource);
|
||||
+ free(buf);
|
||||
+ return ret;
|
||||
+}
|
||||
+
|
||||
+static int builtin_firmware(struct udev_device *dev, int argc, char *argv[], bool test) {
|
||||
+ struct udev *udev = udev_device_get_udev(dev);
|
||||
+ static const char *searchpath[] = { FIRMWARE_PATH };
|
||||
+ char loadpath[UTIL_PATH_SIZE];
|
||||
+ char datapath[UTIL_PATH_SIZE];
|
||||
+ char fwpath[UTIL_PATH_SIZE];
|
||||
+ const char *firmware;
|
||||
+ FILE *fwfile = NULL;
|
||||
+ struct utsname kernel;
|
||||
+ struct stat statbuf;
|
||||
+ unsigned int i;
|
||||
+ int rc = EXIT_SUCCESS;
|
||||
+
|
||||
+ firmware = udev_device_get_property_value(dev, "FIRMWARE");
|
||||
+ if (firmware == NULL) {
|
||||
+ log_error("firmware parameter missing");
|
||||
+ rc = EXIT_FAILURE;
|
||||
+ goto exit;
|
||||
+ }
|
||||
+
|
||||
+ /* lookup firmware file */
|
||||
+ uname(&kernel);
|
||||
+ for (i = 0; i < ELEMENTSOF(searchpath); i++) {
|
||||
+ strscpyl(fwpath, sizeof(fwpath), searchpath[i], kernel.release, "/", firmware, NULL);
|
||||
+ fwfile = fopen(fwpath, "re");
|
||||
+ if (fwfile != NULL)
|
||||
+ break;
|
||||
+
|
||||
+ strscpyl(fwpath, sizeof(fwpath), searchpath[i], firmware, NULL);
|
||||
+ fwfile = fopen(fwpath, "re");
|
||||
+ if (fwfile != NULL)
|
||||
+ break;
|
||||
+ }
|
||||
+
|
||||
+ strscpyl(loadpath, sizeof(loadpath), udev_device_get_syspath(dev), "/loading", NULL);
|
||||
+
|
||||
+ if (fwfile == NULL) {
|
||||
+ log_debug("did not find firmware file '%s'", firmware);
|
||||
+ rc = EXIT_FAILURE;
|
||||
+ /*
|
||||
+ * Do not cancel the request in the initrd, the real root might have
|
||||
+ * the firmware file and the 'coldplug' run in the real root will find
|
||||
+ * this pending request and fulfill or cancel it.
|
||||
+ * */
|
||||
+ if (!in_initrd())
|
||||
+ set_loading(udev, loadpath, "-1");
|
||||
+ goto exit;
|
||||
+ }
|
||||
+
|
||||
+ if (stat(fwpath, &statbuf) < 0 || statbuf.st_size == 0) {
|
||||
+ if (!in_initrd())
|
||||
+ set_loading(udev, loadpath, "-1");
|
||||
+ rc = EXIT_FAILURE;
|
||||
+ goto exit;
|
||||
+ }
|
||||
+
|
||||
+ if (!set_loading(udev, loadpath, "1"))
|
||||
+ goto exit;
|
||||
+
|
||||
+ strscpyl(datapath, sizeof(datapath), udev_device_get_syspath(dev), "/data", NULL);
|
||||
+ if (!copy_firmware(udev, fwpath, datapath, statbuf.st_size)) {
|
||||
+ log_error("error sending firmware '%s' to device", firmware);
|
||||
+ set_loading(udev, loadpath, "-1");
|
||||
+ rc = EXIT_FAILURE;
|
||||
+ goto exit;
|
||||
+ };
|
||||
+
|
||||
+ set_loading(udev, loadpath, "0");
|
||||
+exit:
|
||||
+ if (fwfile)
|
||||
+ fclose(fwfile);
|
||||
+ return rc;
|
||||
+}
|
||||
+
|
||||
+const struct udev_builtin udev_builtin_firmware = {
|
||||
+ .name = "firmware",
|
||||
+ .cmd = builtin_firmware,
|
||||
+ .help = "kernel firmware loader",
|
||||
+ .run_once = true,
|
||||
+};
|
||||
diff --git a/src/udev/udev-builtin.c b/src/udev/udev-builtin.c
|
||||
index 74b3bdc..3657106 100644
|
||||
--- a/src/udev/udev-builtin.c
|
||||
+++ b/src/udev/udev-builtin.c
|
||||
@@ -34,6 +34,9 @@ static const struct udev_builtin *builtins[] = {
|
||||
[UDEV_BUILTIN_BLKID] = &udev_builtin_blkid,
|
||||
#endif
|
||||
[UDEV_BUILTIN_BTRFS] = &udev_builtin_btrfs,
|
||||
+#ifdef HAVE_FIRMWARE
|
||||
+ [UDEV_BUILTIN_FIRMWARE] = &udev_builtin_firmware,
|
||||
+#endif
|
||||
[UDEV_BUILTIN_HWDB] = &udev_builtin_hwdb,
|
||||
[UDEV_BUILTIN_INPUT_ID] = &udev_builtin_input_id,
|
||||
[UDEV_BUILTIN_KEYBOARD] = &udev_builtin_keyboard,
|
||||
diff --git a/src/udev/udev.h b/src/udev/udev.h
|
||||
index 198cb2c..01a1e9c 100644
|
||||
--- a/src/udev/udev.h
|
||||
+++ b/src/udev/udev.h
|
||||
@@ -149,6 +149,9 @@ enum udev_builtin_cmd {
|
||||
UDEV_BUILTIN_BLKID,
|
||||
#endif
|
||||
UDEV_BUILTIN_BTRFS,
|
||||
+#ifdef HAVE_FIRMWARE
|
||||
+ UDEV_BUILTIN_FIRMWARE,
|
||||
+#endif
|
||||
UDEV_BUILTIN_HWDB,
|
||||
UDEV_BUILTIN_INPUT_ID,
|
||||
UDEV_BUILTIN_KEYBOARD,
|
||||
@@ -173,6 +176,9 @@ struct udev_builtin {
|
||||
extern const struct udev_builtin udev_builtin_blkid;
|
||||
#endif
|
||||
extern const struct udev_builtin udev_builtin_btrfs;
|
||||
+#ifdef HAVE_FIRMWARE
|
||||
+extern const struct udev_builtin udev_builtin_firmware;
|
||||
+#endif
|
||||
extern const struct udev_builtin udev_builtin_hwdb;
|
||||
extern const struct udev_builtin udev_builtin_input_id;
|
||||
extern const struct udev_builtin udev_builtin_keyboard;
|
||||
diff --git a/src/udev/udevd.c b/src/udev/udevd.c
|
||||
index b1de97a..35655d8 100644
|
||||
--- a/src/udev/udevd.c
|
||||
+++ b/src/udev/udevd.c
|
||||
@@ -101,6 +101,9 @@ struct event {
|
||||
bool is_block;
|
||||
usec_t start_usec;
|
||||
bool warned;
|
||||
+#ifdef HAVE_FIRMWARE
|
||||
+ bool nodelay;
|
||||
+#endif
|
||||
};
|
||||
|
||||
static inline struct event *node_to_event(struct udev_list_node *node) {
|
||||
@@ -491,6 +494,10 @@ static int event_queue_insert(struct udev_device *dev) {
|
||||
event->devnum = udev_device_get_devnum(dev);
|
||||
event->is_block = streq("block", udev_device_get_subsystem(dev));
|
||||
event->ifindex = udev_device_get_ifindex(dev);
|
||||
+#ifdef HAVE_FIRMWARE
|
||||
+ if (streq(udev_device_get_subsystem(dev), "firmware"))
|
||||
+ event->nodelay = true;
|
||||
+#endif
|
||||
|
||||
log_debug("seq %llu queued, '%s' '%s'", udev_device_get_seqnum(dev),
|
||||
udev_device_get_action(dev), udev_device_get_subsystem(dev));
|
||||
@@ -566,6 +573,12 @@ static bool is_devpath_busy(struct event *event) {
|
||||
return true;
|
||||
}
|
||||
|
||||
+#ifdef HAVE_FIRMWARE
|
||||
+ /* allow to bypass the dependency tracking */
|
||||
+ if (event->nodelay)
|
||||
+ continue;
|
||||
+#endif
|
||||
+
|
||||
/* parent device event found */
|
||||
if (event->devpath[common] == '/') {
|
||||
event->delaying_seqnum = loop_event->seqnum;
|
||||
--
|
||||
1.9.1
|
||||
|
|
@ -8,6 +8,8 @@ DEPENDS = "glib-2.0 glib-2.0-native gperf-native kmod libxslt-native util-linux"
|
|||
PROVIDES = "udev"
|
||||
|
||||
SRC_URI = "https://github.com/gentoo/${BPN}/archive/v${PV}.tar.gz;downloadfilename=${BP}.tar.gz \
|
||||
file://0014-Revert-rules-remove-firmware-loading-rules.patch \
|
||||
file://Revert-udev-remove-userspace-firmware-loading-suppor.patch \
|
||||
file://devfs-udev.rules \
|
||||
file://init \
|
||||
file://links.conf \
|
||||
|
|
Loading…
Reference in New Issue