* [PATCH v3] udev: upgrade to 182
@ 2012-09-06 9:27 Alex DAMIAN
2012-09-11 20:53 ` Saul Wold
0 siblings, 1 reply; 5+ messages in thread
From: Alex DAMIAN @ 2012-09-06 9:27 UTC (permalink / raw)
To: openembedded-core
From: Alexandru DAMIAN <alexandru.damian@intel.com>
This is the final upgrade of udev. Futher upgrades will only
come in conjunction with systemd.
The v4l1 removal patch is deprecated as the bug is fixed inside udev.
There is a new patch fixing the path for default sh interpreter.
New debug binaries are generated, and udev.inc is modified to package
those correctly.
The install locations changed for udevd and udevadm, so the scripts
are updated accordingly.
Signed-off-by: Alexandru DAMIAN <alexandru.damian@intel.com>
Conflicts:
meta/recipes-core/udev/udev_164.bb
---
.../initscripts/initscripts-1.0/volatiles | 2 +-
meta/recipes-core/udev/udev.inc | 12 +++--
...yboard_force_release.sh-shell-script-path.patch | 35 ++++++++++++++
meta/recipes-core/udev/udev/include_resource.patch | 31 ------------
meta/recipes-core/udev/udev/init | 14 +++---
meta/recipes-core/udev/udev/udev-166-v4l1-1.patch | 50 --------------------
meta/recipes-core/udev/udev/udev-cache | 2 +-
meta/recipes-core/udev/udev_164.bb | 9 ----
meta/recipes-core/udev/udev_182.bb | 9 ++++
9 files changed, 61 insertions(+), 103 deletions(-)
create mode 100644 meta/recipes-core/udev/udev/0001-Fixing-keyboard_force_release.sh-shell-script-path.patch
delete mode 100644 meta/recipes-core/udev/udev/include_resource.patch
delete mode 100644 meta/recipes-core/udev/udev/udev-166-v4l1-1.patch
delete mode 100644 meta/recipes-core/udev/udev_164.bb
create mode 100644 meta/recipes-core/udev/udev_182.bb
diff --git a/meta/recipes-core/initscripts/initscripts-1.0/volatiles b/meta/recipes-core/initscripts/initscripts-1.0/volatiles
index b2ae279..e0741aa 100644
--- a/meta/recipes-core/initscripts/initscripts-1.0/volatiles
+++ b/meta/recipes-core/initscripts/initscripts-1.0/volatiles
@@ -36,4 +36,4 @@ f root root 0664 /var/log/wtmp none
f root root 0664 /var/run/utmp none
l root root 0644 /etc/resolv.conf /var/run/resolv.conf
f root root 0644 /var/run/resolv.conf none
-
+l root root 0755 /run /var/run
diff --git a/meta/recipes-core/udev/udev.inc b/meta/recipes-core/udev/udev.inc
index 9cc00e8..fafd9ce 100644
--- a/meta/recipes-core/udev/udev.inc
+++ b/meta/recipes-core/udev/udev.inc
@@ -6,9 +6,10 @@ LICENSE = "GPLv2.0+ & LGPLv2.1+"
LICENSE_${PN} = "GPLv2.0+"
LICENSE_libudev = "LGPLv2.1+"
LICENSE_libgudev = "LGPLv2.1+"
-LIC_FILES_CHKSUM = "file://COPYING;md5=751419260aa954499f7abaabaa882bbe \
- file://libudev/COPYING;md5=a6f89e2100d9b6cdffcea4f398e37343 \
- file://extras/gudev/COPYING;md5=a6f89e2100d9b6cdffcea4f398e37343"
+LIC_FILES_CHKSUM = "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263 \
+ file://src/COPYING;md5=17c4e5fb495e6707ac92a3864926f979 \
+ file://src/gudev/COPYING;md5=fb494485a7d0505308cb68e4997cc266"
+
DEPENDS = "acl glib-2.0 libusb usbutils pciutils gperf-native libxslt-native"
RPROVIDES_${PN} = "hotplug"
@@ -16,6 +17,7 @@ RRECOMMENDS_${PN} += "udev-extraconf usbutils-ids pciutils-ids"
RDEPENDS_libudev = "${PN} (= ${EXTENDPKGV})"
SRC_URI = "${KERNELORG_MIRROR}/linux/utils/kernel/hotplug/udev-${PV}.tar.gz \
+ file://0001-Fixing-keyboard_force_release.sh-shell-script-path.patch \
file://run.rules \
file://udev.rules \
file://devfs-udev.rules \
@@ -32,7 +34,7 @@ inherit autotools pkgconfig update-rc.d
# udevd/udevadm -> /sbin/, libudev.so.* -> /lib/
sbindir = "${base_sbindir}"
-libexecdir = "${base_libdir}/udev"
+libexecdir = "${base_libdir}"
EXTRA_OECONF = "--disable-introspection --with-rootlibdir=${base_libdir} \
--with-pci-ids-path=${datadir}/pci.ids"
@@ -50,6 +52,8 @@ FILES_${PN} += "${libexecdir} ${libdir}/ConsoleKit"
RRECOMMENDS_${PN} += "udev-utils"
FILES_${PN}-dbg += "${libexecdir}/.debug"
+FILES_${PN}-dbg += "${base_libdir}/udev/.debug/"
+FILES_${PN}-dbg += "${base_libdir}/udev/.debug/*"
FILES_${PN}-dev = "${datadir}/pkgconfig/udev.pc"
FILES_libudev = "${base_libdir}/libudev.so.*"
FILES_libudev-dbg = "${base_libdir}/.debug/libudev.so.*"
diff --git a/meta/recipes-core/udev/udev/0001-Fixing-keyboard_force_release.sh-shell-script-path.patch b/meta/recipes-core/udev/udev/0001-Fixing-keyboard_force_release.sh-shell-script-path.patch
new file mode 100644
index 0000000..41deafa
--- /dev/null
+++ b/meta/recipes-core/udev/udev/0001-Fixing-keyboard_force_release.sh-shell-script-path.patch
@@ -0,0 +1,35 @@
+From 0f8290c943da298abd269ca60fd8375dfb219971 Mon Sep 17 00:00:00 2001
+From: Alexandru DAMIAN <alexandru.damian@intel.com>
+Date: Thu, 12 Jul 2012 12:54:48 +0300
+Subject: [PATCH] Fixing keyboard_force_release.sh shell script path
+
+With the introduction of rootprefix, the keyboard-force-release.sh.in
+was modified to be executed with @rootprefix@/bin/sh, which is wrong
+because @rootprefix@ defaults to /usr (which is correct), but the
+shell is always at /bin/sh (IEEE Std 1003.2-1992).
+
+Therefore the interpreter for shell scripts needs to be /bin/sh at all times.
+
+The upstream moved to configurable root prefix, this patch taclkes a
+transition bug, and will not be applied upstream.
+
+Upstream-Status: Inappropriate [legacy version]
+
+Signed-off-by: Alexandru DAMIAN <alexandru.damian@intel.com>
+---
+ src/keymap/keyboard-force-release.sh.in | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/src/keymap/keyboard-force-release.sh.in b/src/keymap/keyboard-force-release.sh.in
+index dd040ce..597a3a6 100755
+--- a/src/keymap/keyboard-force-release.sh.in
++++ b/src/keymap/keyboard-force-release.sh.in
+@@ -1,4 +1,4 @@
+-#!@rootprefix@/bin/sh -e
++#!/bin/sh -e
+ # read list of scancodes, convert hex to decimal and
+ # append to the atkbd force_release sysfs attribute
+ # $1 sysfs devpath for serioX
+--
+1.7.9.5
+
diff --git a/meta/recipes-core/udev/udev/include_resource.patch b/meta/recipes-core/udev/udev/include_resource.patch
deleted file mode 100644
index a46ce59..0000000
--- a/meta/recipes-core/udev/udev/include_resource.patch
+++ /dev/null
@@ -1,31 +0,0 @@
-Fixes errors like below with eglibc 2.16
-
-libudev/libudev-util-private.c:348:16: error: 'PRIO_PROCESS' undeclared (first use in this function)
-
-Signed-off-by: Khem Raj <raj.khem@gmail.com>
-Upstream-Status: Pending
-
-Index: udev-164/libudev/libudev-util-private.c
-===================================================================
---- udev-164.orig/libudev/libudev-util-private.c 2010-08-03 02:42:26.000000000 -0700
-+++ udev-164/libudev/libudev-util-private.c 2012-07-03 10:53:35.082346684 -0700
-@@ -21,6 +21,7 @@
- #include <grp.h>
- #include <sys/wait.h>
- #include <sys/param.h>
-+#include <sys/resource.h>
-
- #include "libudev.h"
- #include "libudev-private.h"
-Index: udev-164/udev/udevd.c
-===================================================================
---- udev-164.orig/udev/udevd.c 2010-10-13 04:08:29.000000000 -0700
-+++ udev-164/udev/udevd.c 2012-07-03 10:53:19.214345914 -0700
-@@ -43,6 +43,7 @@
- #include <sys/ioctl.h>
- #include <sys/inotify.h>
- #include <sys/utsname.h>
-+#include <sys/resource.h>
-
- #include "udev.h"
- #include "sd-daemon.h"
diff --git a/meta/recipes-core/udev/udev/init b/meta/recipes-core/udev/udev/init
index d521685..ca02abb 100644
--- a/meta/recipes-core/udev/udev/init
+++ b/meta/recipes-core/udev/udev/init
@@ -13,7 +13,7 @@ export TZ=/etc/localtime
[ -d /sys/class ] || exit 1
[ -r /proc/mounts ] || exit 1
-[ -x /sbin/udevd ] || exit 1
+[ -x /lib/udev/udevd ] || exit 1
[ -f /etc/default/udev-cache ] && . /etc/default/udev-cache
[ -f /etc/udev/udev.conf ] && . /etc/udev/udev.conf
@@ -70,15 +70,15 @@ kill_udevd > "/dev/null" 2>&1
# trigger the sorted events
echo -e '\000\000\000\000' > /proc/sys/kernel/hotplug
-/sbin/udevd -d
+/lib/udev/udevd -d
-/sbin/udevadm control --env=STARTUP=1
+/usr/bin/udevadm control --env=STARTUP=1
if [ "$not_first_boot" != "" ];then
- /sbin/udevadm trigger --action=add --subsystem-nomatch=tty --subsystem-nomatch=mem --subsystem-nomatch=vc --subsystem-nomatch=vtconsole --subsystem-nomatch=misc --subsystem-nomatch=dcon --subsystem-nomatch=pci_bus --subsystem-nomatch=graphics --subsystem-nomatch=backlight --subsystem-nomatch=video4linux --subsystem-nomatch=platform
- (/sbin/udevadm settle --timeout=3; /sbin/udevadm control --env=STARTUP=)&
+ /usr/bin/udevadm trigger --action=add --subsystem-nomatch=tty --subsystem-nomatch=mem --subsystem-nomatch=vc --subsystem-nomatch=vtconsole --subsystem-nomatch=misc --subsystem-nomatch=dcon --subsystem-nomatch=pci_bus --subsystem-nomatch=graphics --subsystem-nomatch=backlight --subsystem-nomatch=video4linux --subsystem-nomatch=platform
+ (/usr/bin/udevadm settle --timeout=3; /usr/bin/udevadm control --env=STARTUP=)&
else
- /sbin/udevadm trigger --action=add
- /sbin/udevadm settle
+ /usr/bin/udevadm trigger --action=add
+ /usr/bin/udevadm settle
fi
exit 0
diff --git a/meta/recipes-core/udev/udev/udev-166-v4l1-1.patch b/meta/recipes-core/udev/udev/udev-166-v4l1-1.patch
deleted file mode 100644
index 2086fe1..0000000
--- a/meta/recipes-core/udev/udev/udev-166-v4l1-1.patch
+++ /dev/null
@@ -1,50 +0,0 @@
-Upstream-Status: Backport
-
-Submitted By: Matt Burgess <matthew_at_linuxfromscratch_dot_org>
-Date: 2011-03-26
-Initial Package Version: 166
-Upstream Status: From upstream
-Origin: http://git.kernel.org/?p=linux/hotplug/udev.git;a=commitdiff;h=4ace8a43ac2cbbd4d6f5c29fc461c3caa8f8545b
-Description: Fixes a compilation error caused by the removal of the
- Video for Linux 1 API from Linux kernels from 2.6.38
- onwards.
-
-diff -Naur udev-166.orig/extras/v4l_id/v4l_id.c udev-166/extras/v4l_id/v4l_id.c
---- udev-166.orig/extras/v4l_id/v4l_id.c 2009-12-03 12:45:03.000000000 +0000
-+++ udev-166/extras/v4l_id/v4l_id.c 2011-03-25 20:26:33.000000000 +0000
-@@ -28,7 +28,6 @@
- #include <sys/types.h>
- #include <sys/time.h>
- #include <sys/ioctl.h>
--#include <linux/videodev.h>
- #include <linux/videodev2.h>
-
- int main (int argc, char *argv[])
-@@ -39,7 +38,6 @@
- };
- int fd;
- char *device;
-- struct video_capability v1cap;
- struct v4l2_capability v2cap;
-
- while (1) {
-@@ -82,19 +80,6 @@
- if ((v2cap.capabilities & V4L2_CAP_RADIO) > 0)
- printf("radio:");
- printf("\n");
-- } else if (ioctl (fd, VIDIOCGCAP, &v1cap) == 0) {
-- printf("ID_V4L_VERSION=1\n");
-- printf("ID_V4L_PRODUCT=%s\n", v1cap.name);
-- printf("ID_V4L_CAPABILITIES=:");
-- if ((v1cap.type & VID_TYPE_CAPTURE) > 0)
-- printf("capture:");
-- if ((v1cap.type & VID_TYPE_OVERLAY) > 0)
-- printf("video_overlay:");
-- if (v1cap.audios > 0)
-- printf("audio:");
-- if ((v1cap.type & VID_TYPE_TUNER) > 0)
-- printf("tuner:");
-- printf("\n");
- }
-
- close (fd);
diff --git a/meta/recipes-core/udev/udev/udev-cache b/meta/recipes-core/udev/udev/udev-cache
index 9c24e76..0f3d7cd 100644
--- a/meta/recipes-core/udev/udev/udev-cache
+++ b/meta/recipes-core/udev/udev/udev-cache
@@ -12,7 +12,7 @@
export TZ=/etc/localtime
[ -r /proc/mounts ] || exit 1
-[ -x /sbin/udevd ] || exit 1
+[ -x /lib/udev/udevd ] || exit 1
[ -d /sys/class ] || exit 1
[ -f /etc/default/udev-cache ] && . /etc/default/udev-cache
diff --git a/meta/recipes-core/udev/udev_164.bb b/meta/recipes-core/udev/udev_164.bb
deleted file mode 100644
index 4d22bc8..0000000
--- a/meta/recipes-core/udev/udev_164.bb
+++ /dev/null
@@ -1,9 +0,0 @@
-include udev.inc
-
-PR = "r16"
-
-SRC_URI += "file://udev-166-v4l1-1.patch \
- file://include_resource.patch \
- "
-SRC_URI[md5sum] = "fddac2d54761ea34865af9467377ca9f"
-SRC_URI[sha256sum] = "c12e66280b5e1465f6587a8cfa47d7405c4caa7e52ce5dd13478d04f6ec05e5c"
diff --git a/meta/recipes-core/udev/udev_182.bb b/meta/recipes-core/udev/udev_182.bb
new file mode 100644
index 0000000..6678fe1
--- /dev/null
+++ b/meta/recipes-core/udev/udev_182.bb
@@ -0,0 +1,9 @@
+include udev.inc
+
+PR = "r0"
+
+# module-init-tools from kmod_git will provide libkmod runtime
+DEPENDS = "module-init-tools"
+
+SRC_URI[md5sum] = "1b964456177fbf48023dfee7db3a708d"
+SRC_URI[sha256sum] = "7857ed19fafd8f3ca8de410194e8c7336e9eb8a20626ea8a4ba6449b017faba4"
--
1.7.9.5
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH v3] udev: upgrade to 182
2012-09-06 9:27 [PATCH v3] udev: upgrade to 182 Alex DAMIAN
@ 2012-09-11 20:53 ` Saul Wold
2012-09-19 18:02 ` Khem Raj
0 siblings, 1 reply; 5+ messages in thread
From: Saul Wold @ 2012-09-11 20:53 UTC (permalink / raw)
To: Alex DAMIAN; +Cc: openembedded-core
On 09/06/2012 02:27 AM, Alex DAMIAN wrote:
> From: Alexandru DAMIAN <alexandru.damian@intel.com>
>
> This is the final upgrade of udev. Futher upgrades will only
> come in conjunction with systemd.
>
> The v4l1 removal patch is deprecated as the bug is fixed inside udev.
> There is a new patch fixing the path for default sh interpreter.
> New debug binaries are generated, and udev.inc is modified to package
> those correctly.
> The install locations changed for udevd and udevadm, so the scripts
> are updated accordingly.
>
> Signed-off-by: Alexandru DAMIAN <alexandru.damian@intel.com>
>
Alex,
I have had some problems with this patch trying to build it. It was
looking on my host for usbutils version .82 or greater, but we have the
new numbered 006 usbutils and it failed.
Also, why did you not take the meta-oe version, did you compare your
changes with the version is meta-oe? This was mentioned once before?
Thanks
Sau!
> Conflicts:
>
> meta/recipes-core/udev/udev_164.bb
> ---
> .../initscripts/initscripts-1.0/volatiles | 2 +-
> meta/recipes-core/udev/udev.inc | 12 +++--
> ...yboard_force_release.sh-shell-script-path.patch | 35 ++++++++++++++
> meta/recipes-core/udev/udev/include_resource.patch | 31 ------------
> meta/recipes-core/udev/udev/init | 14 +++---
> meta/recipes-core/udev/udev/udev-166-v4l1-1.patch | 50 --------------------
> meta/recipes-core/udev/udev/udev-cache | 2 +-
> meta/recipes-core/udev/udev_164.bb | 9 ----
> meta/recipes-core/udev/udev_182.bb | 9 ++++
> 9 files changed, 61 insertions(+), 103 deletions(-)
> create mode 100644 meta/recipes-core/udev/udev/0001-Fixing-keyboard_force_release.sh-shell-script-path.patch
> delete mode 100644 meta/recipes-core/udev/udev/include_resource.patch
> delete mode 100644 meta/recipes-core/udev/udev/udev-166-v4l1-1.patch
> delete mode 100644 meta/recipes-core/udev/udev_164.bb
> create mode 100644 meta/recipes-core/udev/udev_182.bb
>
> diff --git a/meta/recipes-core/initscripts/initscripts-1.0/volatiles b/meta/recipes-core/initscripts/initscripts-1.0/volatiles
> index b2ae279..e0741aa 100644
> --- a/meta/recipes-core/initscripts/initscripts-1.0/volatiles
> +++ b/meta/recipes-core/initscripts/initscripts-1.0/volatiles
> @@ -36,4 +36,4 @@ f root root 0664 /var/log/wtmp none
> f root root 0664 /var/run/utmp none
> l root root 0644 /etc/resolv.conf /var/run/resolv.conf
> f root root 0644 /var/run/resolv.conf none
> -
> +l root root 0755 /run /var/run
> diff --git a/meta/recipes-core/udev/udev.inc b/meta/recipes-core/udev/udev.inc
> index 9cc00e8..fafd9ce 100644
> --- a/meta/recipes-core/udev/udev.inc
> +++ b/meta/recipes-core/udev/udev.inc
> @@ -6,9 +6,10 @@ LICENSE = "GPLv2.0+ & LGPLv2.1+"
> LICENSE_${PN} = "GPLv2.0+"
> LICENSE_libudev = "LGPLv2.1+"
> LICENSE_libgudev = "LGPLv2.1+"
> -LIC_FILES_CHKSUM = "file://COPYING;md5=751419260aa954499f7abaabaa882bbe \
> - file://libudev/COPYING;md5=a6f89e2100d9b6cdffcea4f398e37343 \
> - file://extras/gudev/COPYING;md5=a6f89e2100d9b6cdffcea4f398e37343"
> +LIC_FILES_CHKSUM = "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263 \
> + file://src/COPYING;md5=17c4e5fb495e6707ac92a3864926f979 \
> + file://src/gudev/COPYING;md5=fb494485a7d0505308cb68e4997cc266"
> +
>
> DEPENDS = "acl glib-2.0 libusb usbutils pciutils gperf-native libxslt-native"
> RPROVIDES_${PN} = "hotplug"
> @@ -16,6 +17,7 @@ RRECOMMENDS_${PN} += "udev-extraconf usbutils-ids pciutils-ids"
> RDEPENDS_libudev = "${PN} (= ${EXTENDPKGV})"
>
> SRC_URI = "${KERNELORG_MIRROR}/linux/utils/kernel/hotplug/udev-${PV}.tar.gz \
> + file://0001-Fixing-keyboard_force_release.sh-shell-script-path.patch \
> file://run.rules \
> file://udev.rules \
> file://devfs-udev.rules \
> @@ -32,7 +34,7 @@ inherit autotools pkgconfig update-rc.d
>
> # udevd/udevadm -> /sbin/, libudev.so.* -> /lib/
> sbindir = "${base_sbindir}"
> -libexecdir = "${base_libdir}/udev"
> +libexecdir = "${base_libdir}"
> EXTRA_OECONF = "--disable-introspection --with-rootlibdir=${base_libdir} \
> --with-pci-ids-path=${datadir}/pci.ids"
>
> @@ -50,6 +52,8 @@ FILES_${PN} += "${libexecdir} ${libdir}/ConsoleKit"
> RRECOMMENDS_${PN} += "udev-utils"
>
> FILES_${PN}-dbg += "${libexecdir}/.debug"
> +FILES_${PN}-dbg += "${base_libdir}/udev/.debug/"
> +FILES_${PN}-dbg += "${base_libdir}/udev/.debug/*"
> FILES_${PN}-dev = "${datadir}/pkgconfig/udev.pc"
> FILES_libudev = "${base_libdir}/libudev.so.*"
> FILES_libudev-dbg = "${base_libdir}/.debug/libudev.so.*"
> diff --git a/meta/recipes-core/udev/udev/0001-Fixing-keyboard_force_release.sh-shell-script-path.patch b/meta/recipes-core/udev/udev/0001-Fixing-keyboard_force_release.sh-shell-script-path.patch
> new file mode 100644
> index 0000000..41deafa
> --- /dev/null
> +++ b/meta/recipes-core/udev/udev/0001-Fixing-keyboard_force_release.sh-shell-script-path.patch
> @@ -0,0 +1,35 @@
> +From 0f8290c943da298abd269ca60fd8375dfb219971 Mon Sep 17 00:00:00 2001
> +From: Alexandru DAMIAN <alexandru.damian@intel.com>
> +Date: Thu, 12 Jul 2012 12:54:48 +0300
> +Subject: [PATCH] Fixing keyboard_force_release.sh shell script path
> +
> +With the introduction of rootprefix, the keyboard-force-release.sh.in
> +was modified to be executed with @rootprefix@/bin/sh, which is wrong
> +because @rootprefix@ defaults to /usr (which is correct), but the
> +shell is always at /bin/sh (IEEE Std 1003.2-1992).
> +
> +Therefore the interpreter for shell scripts needs to be /bin/sh at all times.
> +
> +The upstream moved to configurable root prefix, this patch taclkes a
> +transition bug, and will not be applied upstream.
> +
> +Upstream-Status: Inappropriate [legacy version]
> +
> +Signed-off-by: Alexandru DAMIAN <alexandru.damian@intel.com>
> +---
> + src/keymap/keyboard-force-release.sh.in | 2 +-
> + 1 file changed, 1 insertion(+), 1 deletion(-)
> +
> +diff --git a/src/keymap/keyboard-force-release.sh.in b/src/keymap/keyboard-force-release.sh.in
> +index dd040ce..597a3a6 100755
> +--- a/src/keymap/keyboard-force-release.sh.in
> ++++ b/src/keymap/keyboard-force-release.sh.in
> +@@ -1,4 +1,4 @@
> +-#!@rootprefix@/bin/sh -e
> ++#!/bin/sh -e
> + # read list of scancodes, convert hex to decimal and
> + # append to the atkbd force_release sysfs attribute
> + # $1 sysfs devpath for serioX
> +--
> +1.7.9.5
> +
> diff --git a/meta/recipes-core/udev/udev/include_resource.patch b/meta/recipes-core/udev/udev/include_resource.patch
> deleted file mode 100644
> index a46ce59..0000000
> --- a/meta/recipes-core/udev/udev/include_resource.patch
> +++ /dev/null
> @@ -1,31 +0,0 @@
> -Fixes errors like below with eglibc 2.16
> -
> -libudev/libudev-util-private.c:348:16: error: 'PRIO_PROCESS' undeclared (first use in this function)
> -
> -Signed-off-by: Khem Raj <raj.khem@gmail.com>
> -Upstream-Status: Pending
> -
> -Index: udev-164/libudev/libudev-util-private.c
> -===================================================================
> ---- udev-164.orig/libudev/libudev-util-private.c 2010-08-03 02:42:26.000000000 -0700
> -+++ udev-164/libudev/libudev-util-private.c 2012-07-03 10:53:35.082346684 -0700
> -@@ -21,6 +21,7 @@
> - #include <grp.h>
> - #include <sys/wait.h>
> - #include <sys/param.h>
> -+#include <sys/resource.h>
> -
> - #include "libudev.h"
> - #include "libudev-private.h"
> -Index: udev-164/udev/udevd.c
> -===================================================================
> ---- udev-164.orig/udev/udevd.c 2010-10-13 04:08:29.000000000 -0700
> -+++ udev-164/udev/udevd.c 2012-07-03 10:53:19.214345914 -0700
> -@@ -43,6 +43,7 @@
> - #include <sys/ioctl.h>
> - #include <sys/inotify.h>
> - #include <sys/utsname.h>
> -+#include <sys/resource.h>
> -
> - #include "udev.h"
> - #include "sd-daemon.h"
> diff --git a/meta/recipes-core/udev/udev/init b/meta/recipes-core/udev/udev/init
> index d521685..ca02abb 100644
> --- a/meta/recipes-core/udev/udev/init
> +++ b/meta/recipes-core/udev/udev/init
> @@ -13,7 +13,7 @@ export TZ=/etc/localtime
>
> [ -d /sys/class ] || exit 1
> [ -r /proc/mounts ] || exit 1
> -[ -x /sbin/udevd ] || exit 1
> +[ -x /lib/udev/udevd ] || exit 1
> [ -f /etc/default/udev-cache ] && . /etc/default/udev-cache
> [ -f /etc/udev/udev.conf ] && . /etc/udev/udev.conf
>
> @@ -70,15 +70,15 @@ kill_udevd > "/dev/null" 2>&1
>
> # trigger the sorted events
> echo -e '\000\000\000\000' > /proc/sys/kernel/hotplug
> -/sbin/udevd -d
> +/lib/udev/udevd -d
>
> -/sbin/udevadm control --env=STARTUP=1
> +/usr/bin/udevadm control --env=STARTUP=1
> if [ "$not_first_boot" != "" ];then
> - /sbin/udevadm trigger --action=add --subsystem-nomatch=tty --subsystem-nomatch=mem --subsystem-nomatch=vc --subsystem-nomatch=vtconsole --subsystem-nomatch=misc --subsystem-nomatch=dcon --subsystem-nomatch=pci_bus --subsystem-nomatch=graphics --subsystem-nomatch=backlight --subsystem-nomatch=video4linux --subsystem-nomatch=platform
> - (/sbin/udevadm settle --timeout=3; /sbin/udevadm control --env=STARTUP=)&
> + /usr/bin/udevadm trigger --action=add --subsystem-nomatch=tty --subsystem-nomatch=mem --subsystem-nomatch=vc --subsystem-nomatch=vtconsole --subsystem-nomatch=misc --subsystem-nomatch=dcon --subsystem-nomatch=pci_bus --subsystem-nomatch=graphics --subsystem-nomatch=backlight --subsystem-nomatch=video4linux --subsystem-nomatch=platform
> + (/usr/bin/udevadm settle --timeout=3; /usr/bin/udevadm control --env=STARTUP=)&
> else
> - /sbin/udevadm trigger --action=add
> - /sbin/udevadm settle
> + /usr/bin/udevadm trigger --action=add
> + /usr/bin/udevadm settle
> fi
>
> exit 0
> diff --git a/meta/recipes-core/udev/udev/udev-166-v4l1-1.patch b/meta/recipes-core/udev/udev/udev-166-v4l1-1.patch
> deleted file mode 100644
> index 2086fe1..0000000
> --- a/meta/recipes-core/udev/udev/udev-166-v4l1-1.patch
> +++ /dev/null
> @@ -1,50 +0,0 @@
> -Upstream-Status: Backport
> -
> -Submitted By: Matt Burgess <matthew_at_linuxfromscratch_dot_org>
> -Date: 2011-03-26
> -Initial Package Version: 166
> -Upstream Status: From upstream
> -Origin: http://git.kernel.org/?p=linux/hotplug/udev.git;a=commitdiff;h=4ace8a43ac2cbbd4d6f5c29fc461c3caa8f8545b
> -Description: Fixes a compilation error caused by the removal of the
> - Video for Linux 1 API from Linux kernels from 2.6.38
> - onwards.
> -
> -diff -Naur udev-166.orig/extras/v4l_id/v4l_id.c udev-166/extras/v4l_id/v4l_id.c
> ---- udev-166.orig/extras/v4l_id/v4l_id.c 2009-12-03 12:45:03.000000000 +0000
> -+++ udev-166/extras/v4l_id/v4l_id.c 2011-03-25 20:26:33.000000000 +0000
> -@@ -28,7 +28,6 @@
> - #include <sys/types.h>
> - #include <sys/time.h>
> - #include <sys/ioctl.h>
> --#include <linux/videodev.h>
> - #include <linux/videodev2.h>
> -
> - int main (int argc, char *argv[])
> -@@ -39,7 +38,6 @@
> - };
> - int fd;
> - char *device;
> -- struct video_capability v1cap;
> - struct v4l2_capability v2cap;
> -
> - while (1) {
> -@@ -82,19 +80,6 @@
> - if ((v2cap.capabilities & V4L2_CAP_RADIO) > 0)
> - printf("radio:");
> - printf("\n");
> -- } else if (ioctl (fd, VIDIOCGCAP, &v1cap) == 0) {
> -- printf("ID_V4L_VERSION=1\n");
> -- printf("ID_V4L_PRODUCT=%s\n", v1cap.name);
> -- printf("ID_V4L_CAPABILITIES=:");
> -- if ((v1cap.type & VID_TYPE_CAPTURE) > 0)
> -- printf("capture:");
> -- if ((v1cap.type & VID_TYPE_OVERLAY) > 0)
> -- printf("video_overlay:");
> -- if (v1cap.audios > 0)
> -- printf("audio:");
> -- if ((v1cap.type & VID_TYPE_TUNER) > 0)
> -- printf("tuner:");
> -- printf("\n");
> - }
> -
> - close (fd);
> diff --git a/meta/recipes-core/udev/udev/udev-cache b/meta/recipes-core/udev/udev/udev-cache
> index 9c24e76..0f3d7cd 100644
> --- a/meta/recipes-core/udev/udev/udev-cache
> +++ b/meta/recipes-core/udev/udev/udev-cache
> @@ -12,7 +12,7 @@
> export TZ=/etc/localtime
>
> [ -r /proc/mounts ] || exit 1
> -[ -x /sbin/udevd ] || exit 1
> +[ -x /lib/udev/udevd ] || exit 1
> [ -d /sys/class ] || exit 1
>
> [ -f /etc/default/udev-cache ] && . /etc/default/udev-cache
> diff --git a/meta/recipes-core/udev/udev_164.bb b/meta/recipes-core/udev/udev_164.bb
> deleted file mode 100644
> index 4d22bc8..0000000
> --- a/meta/recipes-core/udev/udev_164.bb
> +++ /dev/null
> @@ -1,9 +0,0 @@
> -include udev.inc
> -
> -PR = "r16"
> -
> -SRC_URI += "file://udev-166-v4l1-1.patch \
> - file://include_resource.patch \
> - "
> -SRC_URI[md5sum] = "fddac2d54761ea34865af9467377ca9f"
> -SRC_URI[sha256sum] = "c12e66280b5e1465f6587a8cfa47d7405c4caa7e52ce5dd13478d04f6ec05e5c"
> diff --git a/meta/recipes-core/udev/udev_182.bb b/meta/recipes-core/udev/udev_182.bb
> new file mode 100644
> index 0000000..6678fe1
> --- /dev/null
> +++ b/meta/recipes-core/udev/udev_182.bb
> @@ -0,0 +1,9 @@
> +include udev.inc
> +
> +PR = "r0"
> +
> +# module-init-tools from kmod_git will provide libkmod runtime
> +DEPENDS = "module-init-tools"
> +
> +SRC_URI[md5sum] = "1b964456177fbf48023dfee7db3a708d"
> +SRC_URI[sha256sum] = "7857ed19fafd8f3ca8de410194e8c7336e9eb8a20626ea8a4ba6449b017faba4"
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v3] udev: upgrade to 182
2012-09-11 20:53 ` Saul Wold
@ 2012-09-19 18:02 ` Khem Raj
2012-09-21 10:38 ` Damian, Alexandru
0 siblings, 1 reply; 5+ messages in thread
From: Khem Raj @ 2012-09-19 18:02 UTC (permalink / raw)
To: Saul Wold; +Cc: openembedded-core
On Tue, Sep 11, 2012 at 1:53 PM, Saul Wold <sgw@linux.intel.com> wrote:
> On 09/06/2012 02:27 AM, Alex DAMIAN wrote:
>>
>> From: Alexandru DAMIAN <alexandru.damian@intel.com>
>>
>> This is the final upgrade of udev. Futher upgrades will only
>> come in conjunction with systemd.
>>
>> The v4l1 removal patch is deprecated as the bug is fixed inside udev.
>> There is a new patch fixing the path for default sh interpreter.
>> New debug binaries are generated, and udev.inc is modified to package
>> those correctly.
>> The install locations changed for udevd and udevadm, so the scripts
>> are updated accordingly.
>>
>> Signed-off-by: Alexandru DAMIAN <alexandru.damian@intel.com>
>>
>
> Alex,
>
> I have had some problems with this patch trying to build it. It was looking
> on my host for usbutils version .82 or greater, but we have the new numbered
> 006 usbutils and it failed.
>
> Also, why did you not take the meta-oe version, did you compare your changes
> with the version is meta-oe? This was mentioned once before?
>
> Thanks
> Sau!
>
>
Alex
whats status of this patch is there is a v4 expected soon
>
>> Conflicts:
>>
>> meta/recipes-core/udev/udev_164.bb
>> ---
>> .../initscripts/initscripts-1.0/volatiles | 2 +-
>> meta/recipes-core/udev/udev.inc | 12 +++--
>> ...yboard_force_release.sh-shell-script-path.patch | 35 ++++++++++++++
>> meta/recipes-core/udev/udev/include_resource.patch | 31 ------------
>> meta/recipes-core/udev/udev/init | 14 +++---
>> meta/recipes-core/udev/udev/udev-166-v4l1-1.patch | 50
>> --------------------
>> meta/recipes-core/udev/udev/udev-cache | 2 +-
>> meta/recipes-core/udev/udev_164.bb | 9 ----
>> meta/recipes-core/udev/udev_182.bb | 9 ++++
>> 9 files changed, 61 insertions(+), 103 deletions(-)
>> create mode 100644
>> meta/recipes-core/udev/udev/0001-Fixing-keyboard_force_release.sh-shell-script-path.patch
>> delete mode 100644 meta/recipes-core/udev/udev/include_resource.patch
>> delete mode 100644 meta/recipes-core/udev/udev/udev-166-v4l1-1.patch
>> delete mode 100644 meta/recipes-core/udev/udev_164.bb
>> create mode 100644 meta/recipes-core/udev/udev_182.bb
>>
>> diff --git a/meta/recipes-core/initscripts/initscripts-1.0/volatiles
>> b/meta/recipes-core/initscripts/initscripts-1.0/volatiles
>> index b2ae279..e0741aa 100644
>> --- a/meta/recipes-core/initscripts/initscripts-1.0/volatiles
>> +++ b/meta/recipes-core/initscripts/initscripts-1.0/volatiles
>> @@ -36,4 +36,4 @@ f root root 0664 /var/log/wtmp none
>> f root root 0664 /var/run/utmp none
>> l root root 0644 /etc/resolv.conf /var/run/resolv.conf
>> f root root 0644 /var/run/resolv.conf none
>> -
>> +l root root 0755 /run /var/run
>> diff --git a/meta/recipes-core/udev/udev.inc
>> b/meta/recipes-core/udev/udev.inc
>> index 9cc00e8..fafd9ce 100644
>> --- a/meta/recipes-core/udev/udev.inc
>> +++ b/meta/recipes-core/udev/udev.inc
>> @@ -6,9 +6,10 @@ LICENSE = "GPLv2.0+ & LGPLv2.1+"
>> LICENSE_${PN} = "GPLv2.0+"
>> LICENSE_libudev = "LGPLv2.1+"
>> LICENSE_libgudev = "LGPLv2.1+"
>> -LIC_FILES_CHKSUM = "file://COPYING;md5=751419260aa954499f7abaabaa882bbe \
>> -
>> file://libudev/COPYING;md5=a6f89e2100d9b6cdffcea4f398e37343 \
>> -
>> file://extras/gudev/COPYING;md5=a6f89e2100d9b6cdffcea4f398e37343"
>> +LIC_FILES_CHKSUM = "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263 \
>> +
>> file://src/COPYING;md5=17c4e5fb495e6707ac92a3864926f979 \
>> +
>> file://src/gudev/COPYING;md5=fb494485a7d0505308cb68e4997cc266"
>> +
>>
>> DEPENDS = "acl glib-2.0 libusb usbutils pciutils gperf-native
>> libxslt-native"
>> RPROVIDES_${PN} = "hotplug"
>> @@ -16,6 +17,7 @@ RRECOMMENDS_${PN} += "udev-extraconf usbutils-ids
>> pciutils-ids"
>> RDEPENDS_libudev = "${PN} (= ${EXTENDPKGV})"
>>
>> SRC_URI =
>> "${KERNELORG_MIRROR}/linux/utils/kernel/hotplug/udev-${PV}.tar.gz \
>> +
>> file://0001-Fixing-keyboard_force_release.sh-shell-script-path.patch \
>> file://run.rules \
>> file://udev.rules \
>> file://devfs-udev.rules \
>> @@ -32,7 +34,7 @@ inherit autotools pkgconfig update-rc.d
>>
>> # udevd/udevadm -> /sbin/, libudev.so.* -> /lib/
>> sbindir = "${base_sbindir}"
>> -libexecdir = "${base_libdir}/udev"
>> +libexecdir = "${base_libdir}"
>> EXTRA_OECONF = "--disable-introspection --with-rootlibdir=${base_libdir}
>> \
>> --with-pci-ids-path=${datadir}/pci.ids"
>>
>> @@ -50,6 +52,8 @@ FILES_${PN} += "${libexecdir} ${libdir}/ConsoleKit"
>> RRECOMMENDS_${PN} += "udev-utils"
>>
>> FILES_${PN}-dbg += "${libexecdir}/.debug"
>> +FILES_${PN}-dbg += "${base_libdir}/udev/.debug/"
>> +FILES_${PN}-dbg += "${base_libdir}/udev/.debug/*"
>> FILES_${PN}-dev = "${datadir}/pkgconfig/udev.pc"
>> FILES_libudev = "${base_libdir}/libudev.so.*"
>> FILES_libudev-dbg = "${base_libdir}/.debug/libudev.so.*"
>> diff --git
>> a/meta/recipes-core/udev/udev/0001-Fixing-keyboard_force_release.sh-shell-script-path.patch
>> b/meta/recipes-core/udev/udev/0001-Fixing-keyboard_force_release.sh-shell-script-path.patch
>> new file mode 100644
>> index 0000000..41deafa
>> --- /dev/null
>> +++
>> b/meta/recipes-core/udev/udev/0001-Fixing-keyboard_force_release.sh-shell-script-path.patch
>> @@ -0,0 +1,35 @@
>> +From 0f8290c943da298abd269ca60fd8375dfb219971 Mon Sep 17 00:00:00 2001
>> +From: Alexandru DAMIAN <alexandru.damian@intel.com>
>> +Date: Thu, 12 Jul 2012 12:54:48 +0300
>> +Subject: [PATCH] Fixing keyboard_force_release.sh shell script path
>> +
>> +With the introduction of rootprefix, the keyboard-force-release.sh.in
>> +was modified to be executed with @rootprefix@/bin/sh, which is wrong
>> +because @rootprefix@ defaults to /usr (which is correct), but the
>> +shell is always at /bin/sh (IEEE Std 1003.2-1992).
>> +
>> +Therefore the interpreter for shell scripts needs to be /bin/sh at all
>> times.
>> +
>> +The upstream moved to configurable root prefix, this patch taclkes a
>> +transition bug, and will not be applied upstream.
>> +
>> +Upstream-Status: Inappropriate [legacy version]
>> +
>> +Signed-off-by: Alexandru DAMIAN <alexandru.damian@intel.com>
>> +---
>> + src/keymap/keyboard-force-release.sh.in | 2 +-
>> + 1 file changed, 1 insertion(+), 1 deletion(-)
>> +
>> +diff --git a/src/keymap/keyboard-force-release.sh.in
>> b/src/keymap/keyboard-force-release.sh.in
>> +index dd040ce..597a3a6 100755
>> +--- a/src/keymap/keyboard-force-release.sh.in
>> ++++ b/src/keymap/keyboard-force-release.sh.in
>> +@@ -1,4 +1,4 @@
>> +-#!@rootprefix@/bin/sh -e
>> ++#!/bin/sh -e
>> + # read list of scancodes, convert hex to decimal and
>> + # append to the atkbd force_release sysfs attribute
>> + # $1 sysfs devpath for serioX
>> +--
>> +1.7.9.5
>> +
>> diff --git a/meta/recipes-core/udev/udev/include_resource.patch
>> b/meta/recipes-core/udev/udev/include_resource.patch
>> deleted file mode 100644
>> index a46ce59..0000000
>> --- a/meta/recipes-core/udev/udev/include_resource.patch
>> +++ /dev/null
>> @@ -1,31 +0,0 @@
>> -Fixes errors like below with eglibc 2.16
>> -
>> -libudev/libudev-util-private.c:348:16: error: 'PRIO_PROCESS' undeclared
>> (first use in this function)
>> -
>> -Signed-off-by: Khem Raj <raj.khem@gmail.com>
>> -Upstream-Status: Pending
>> -
>> -Index: udev-164/libudev/libudev-util-private.c
>> -===================================================================
>> ---- udev-164.orig/libudev/libudev-util-private.c 2010-08-03
>> 02:42:26.000000000 -0700
>> -+++ udev-164/libudev/libudev-util-private.c 2012-07-03
>> 10:53:35.082346684 -0700
>> -@@ -21,6 +21,7 @@
>> - #include <grp.h>
>> - #include <sys/wait.h>
>> - #include <sys/param.h>
>> -+#include <sys/resource.h>
>> -
>> - #include "libudev.h"
>> - #include "libudev-private.h"
>> -Index: udev-164/udev/udevd.c
>> -===================================================================
>> ---- udev-164.orig/udev/udevd.c 2010-10-13 04:08:29.000000000 -0700
>> -+++ udev-164/udev/udevd.c 2012-07-03 10:53:19.214345914 -0700
>> -@@ -43,6 +43,7 @@
>> - #include <sys/ioctl.h>
>> - #include <sys/inotify.h>
>> - #include <sys/utsname.h>
>> -+#include <sys/resource.h>
>> -
>> - #include "udev.h"
>> - #include "sd-daemon.h"
>> diff --git a/meta/recipes-core/udev/udev/init
>> b/meta/recipes-core/udev/udev/init
>> index d521685..ca02abb 100644
>> --- a/meta/recipes-core/udev/udev/init
>> +++ b/meta/recipes-core/udev/udev/init
>> @@ -13,7 +13,7 @@ export TZ=/etc/localtime
>>
>> [ -d /sys/class ] || exit 1
>> [ -r /proc/mounts ] || exit 1
>> -[ -x /sbin/udevd ] || exit 1
>> +[ -x /lib/udev/udevd ] || exit 1
>> [ -f /etc/default/udev-cache ] && . /etc/default/udev-cache
>> [ -f /etc/udev/udev.conf ] && . /etc/udev/udev.conf
>>
>> @@ -70,15 +70,15 @@ kill_udevd > "/dev/null" 2>&1
>>
>> # trigger the sorted events
>> echo -e '\000\000\000\000' > /proc/sys/kernel/hotplug
>> -/sbin/udevd -d
>> +/lib/udev/udevd -d
>>
>> -/sbin/udevadm control --env=STARTUP=1
>> +/usr/bin/udevadm control --env=STARTUP=1
>> if [ "$not_first_boot" != "" ];then
>> - /sbin/udevadm trigger --action=add --subsystem-nomatch=tty
>> --subsystem-nomatch=mem --subsystem-nomatch=vc --subsystem-nomatch=vtconsole
>> --subsystem-nomatch=misc --subsystem-nomatch=dcon
>> --subsystem-nomatch=pci_bus --subsystem-nomatch=graphics
>> --subsystem-nomatch=backlight --subsystem-nomatch=video4linux
>> --subsystem-nomatch=platform
>> - (/sbin/udevadm settle --timeout=3; /sbin/udevadm control
>> --env=STARTUP=)&
>> + /usr/bin/udevadm trigger --action=add --subsystem-nomatch=tty
>> --subsystem-nomatch=mem --subsystem-nomatch=vc --subsystem-nomatch=vtconsole
>> --subsystem-nomatch=misc --subsystem-nomatch=dcon
>> --subsystem-nomatch=pci_bus --subsystem-nomatch=graphics
>> --subsystem-nomatch=backlight --subsystem-nomatch=video4linux
>> --subsystem-nomatch=platform
>> + (/usr/bin/udevadm settle --timeout=3; /usr/bin/udevadm control
>> --env=STARTUP=)&
>> else
>> - /sbin/udevadm trigger --action=add
>> - /sbin/udevadm settle
>> + /usr/bin/udevadm trigger --action=add
>> + /usr/bin/udevadm settle
>> fi
>>
>> exit 0
>> diff --git a/meta/recipes-core/udev/udev/udev-166-v4l1-1.patch
>> b/meta/recipes-core/udev/udev/udev-166-v4l1-1.patch
>> deleted file mode 100644
>> index 2086fe1..0000000
>> --- a/meta/recipes-core/udev/udev/udev-166-v4l1-1.patch
>> +++ /dev/null
>> @@ -1,50 +0,0 @@
>> -Upstream-Status: Backport
>> -
>> -Submitted By: Matt Burgess
>> <matthew_at_linuxfromscratch_dot_org>
>> -Date: 2011-03-26
>> -Initial Package Version: 166
>> -Upstream Status: From upstream
>> -Origin:
>> http://git.kernel.org/?p=linux/hotplug/udev.git;a=commitdiff;h=4ace8a43ac2cbbd4d6f5c29fc461c3caa8f8545b
>> -Description: Fixes a compilation error caused by the removal
>> of the
>> - Video for Linux 1 API from Linux kernels from
>> 2.6.38
>> - onwards.
>> -
>> -diff -Naur udev-166.orig/extras/v4l_id/v4l_id.c
>> udev-166/extras/v4l_id/v4l_id.c
>> ---- udev-166.orig/extras/v4l_id/v4l_id.c 2009-12-03
>> 12:45:03.000000000 +0000
>> -+++ udev-166/extras/v4l_id/v4l_id.c 2011-03-25 20:26:33.000000000
>> +0000
>> -@@ -28,7 +28,6 @@
>> - #include <sys/types.h>
>> - #include <sys/time.h>
>> - #include <sys/ioctl.h>
>> --#include <linux/videodev.h>
>> - #include <linux/videodev2.h>
>> -
>> - int main (int argc, char *argv[])
>> -@@ -39,7 +38,6 @@
>> - };
>> - int fd;
>> - char *device;
>> -- struct video_capability v1cap;
>> - struct v4l2_capability v2cap;
>> -
>> - while (1) {
>> -@@ -82,19 +80,6 @@
>> - if ((v2cap.capabilities & V4L2_CAP_RADIO) > 0)
>> - printf("radio:");
>> - printf("\n");
>> -- } else if (ioctl (fd, VIDIOCGCAP, &v1cap) == 0) {
>> -- printf("ID_V4L_VERSION=1\n");
>> -- printf("ID_V4L_PRODUCT=%s\n", v1cap.name);
>> -- printf("ID_V4L_CAPABILITIES=:");
>> -- if ((v1cap.type & VID_TYPE_CAPTURE) > 0)
>> -- printf("capture:");
>> -- if ((v1cap.type & VID_TYPE_OVERLAY) > 0)
>> -- printf("video_overlay:");
>> -- if (v1cap.audios > 0)
>> -- printf("audio:");
>> -- if ((v1cap.type & VID_TYPE_TUNER) > 0)
>> -- printf("tuner:");
>> -- printf("\n");
>> - }
>> -
>> - close (fd);
>> diff --git a/meta/recipes-core/udev/udev/udev-cache
>> b/meta/recipes-core/udev/udev/udev-cache
>> index 9c24e76..0f3d7cd 100644
>> --- a/meta/recipes-core/udev/udev/udev-cache
>> +++ b/meta/recipes-core/udev/udev/udev-cache
>> @@ -12,7 +12,7 @@
>> export TZ=/etc/localtime
>>
>> [ -r /proc/mounts ] || exit 1
>> -[ -x /sbin/udevd ] || exit 1
>> +[ -x /lib/udev/udevd ] || exit 1
>> [ -d /sys/class ] || exit 1
>>
>> [ -f /etc/default/udev-cache ] && . /etc/default/udev-cache
>> diff --git a/meta/recipes-core/udev/udev_164.bb
>> b/meta/recipes-core/udev/udev_164.bb
>> deleted file mode 100644
>> index 4d22bc8..0000000
>> --- a/meta/recipes-core/udev/udev_164.bb
>> +++ /dev/null
>> @@ -1,9 +0,0 @@
>> -include udev.inc
>> -
>> -PR = "r16"
>> -
>> -SRC_URI += "file://udev-166-v4l1-1.patch \
>> - file://include_resource.patch \
>> - "
>> -SRC_URI[md5sum] = "fddac2d54761ea34865af9467377ca9f"
>> -SRC_URI[sha256sum] =
>> "c12e66280b5e1465f6587a8cfa47d7405c4caa7e52ce5dd13478d04f6ec05e5c"
>> diff --git a/meta/recipes-core/udev/udev_182.bb
>> b/meta/recipes-core/udev/udev_182.bb
>> new file mode 100644
>> index 0000000..6678fe1
>> --- /dev/null
>> +++ b/meta/recipes-core/udev/udev_182.bb
>> @@ -0,0 +1,9 @@
>> +include udev.inc
>> +
>> +PR = "r0"
>> +
>> +# module-init-tools from kmod_git will provide libkmod runtime
>> +DEPENDS = "module-init-tools"
>> +
>> +SRC_URI[md5sum] = "1b964456177fbf48023dfee7db3a708d"
>> +SRC_URI[sha256sum] =
>> "7857ed19fafd8f3ca8de410194e8c7336e9eb8a20626ea8a4ba6449b017faba4"
>>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v3] udev: upgrade to 182
2012-09-19 18:02 ` Khem Raj
@ 2012-09-21 10:38 ` Damian, Alexandru
2012-09-21 20:41 ` Saul Wold
0 siblings, 1 reply; 5+ messages in thread
From: Damian, Alexandru @ 2012-09-21 10:38 UTC (permalink / raw)
To: Khem Raj; +Cc: openembedded-core
[-- Attachment #1: Type: text/plain, Size: 16112 bytes --]
This patch still stands.
I took a look at meta-oe. IMHO, the way the build works here is the right
way.
I had no problems rebasing it to the latest HEAD,
df5f9a3e6dfcf8e8f57132843a024b44c0bd90fc,
and rebuilding.
Tested as follows:
- build core-image-sato
- running qemux86 image
- udevd is started correctly. the image works normally under usage smoke
tests
- udevd is verified to be version 182
If you'll see problems building, please let me have a log.
Alex
On Wed, Sep 19, 2012 at 9:02 PM, Khem Raj <raj.khem@gmail.com> wrote:
> On Tue, Sep 11, 2012 at 1:53 PM, Saul Wold <sgw@linux.intel.com> wrote:
> > On 09/06/2012 02:27 AM, Alex DAMIAN wrote:
> >>
> >> From: Alexandru DAMIAN <alexandru.damian@intel.com>
> >>
> >> This is the final upgrade of udev. Futher upgrades will only
> >> come in conjunction with systemd.
> >>
> >> The v4l1 removal patch is deprecated as the bug is fixed inside udev.
> >> There is a new patch fixing the path for default sh interpreter.
> >> New debug binaries are generated, and udev.inc is modified to package
> >> those correctly.
> >> The install locations changed for udevd and udevadm, so the scripts
> >> are updated accordingly.
> >>
> >> Signed-off-by: Alexandru DAMIAN <alexandru.damian@intel.com>
> >>
> >
> > Alex,
> >
> > I have had some problems with this patch trying to build it. It was
> looking
> > on my host for usbutils version .82 or greater, but we have the new
> numbered
> > 006 usbutils and it failed.
> >
> > Also, why did you not take the meta-oe version, did you compare your
> changes
> > with the version is meta-oe? This was mentioned once before?
> >
> > Thanks
> > Sau!
> >
> >
>
> Alex
>
> whats status of this patch is there is a v4 expected soon
>
> >
> >> Conflicts:
> >>
> >> meta/recipes-core/udev/udev_164.bb
> >> ---
> >> .../initscripts/initscripts-1.0/volatiles | 2 +-
> >> meta/recipes-core/udev/udev.inc | 12 +++--
> >> ...yboard_force_release.sh-shell-script-path.patch | 35
> ++++++++++++++
> >> meta/recipes-core/udev/udev/include_resource.patch | 31 ------------
> >> meta/recipes-core/udev/udev/init | 14 +++---
> >> meta/recipes-core/udev/udev/udev-166-v4l1-1.patch | 50
> >> --------------------
> >> meta/recipes-core/udev/udev/udev-cache | 2 +-
> >> meta/recipes-core/udev/udev_164.bb | 9 ----
> >> meta/recipes-core/udev/udev_182.bb | 9 ++++
> >> 9 files changed, 61 insertions(+), 103 deletions(-)
> >> create mode 100644
> >>
> meta/recipes-core/udev/udev/0001-Fixing-keyboard_force_release.sh-shell-script-path.patch
> >> delete mode 100644 meta/recipes-core/udev/udev/include_resource.patch
> >> delete mode 100644 meta/recipes-core/udev/udev/udev-166-v4l1-1.patch
> >> delete mode 100644 meta/recipes-core/udev/udev_164.bb
> >> create mode 100644 meta/recipes-core/udev/udev_182.bb
> >>
> >> diff --git a/meta/recipes-core/initscripts/initscripts-1.0/volatiles
> >> b/meta/recipes-core/initscripts/initscripts-1.0/volatiles
> >> index b2ae279..e0741aa 100644
> >> --- a/meta/recipes-core/initscripts/initscripts-1.0/volatiles
> >> +++ b/meta/recipes-core/initscripts/initscripts-1.0/volatiles
> >> @@ -36,4 +36,4 @@ f root root 0664 /var/log/wtmp none
> >> f root root 0664 /var/run/utmp none
> >> l root root 0644 /etc/resolv.conf /var/run/resolv.conf
> >> f root root 0644 /var/run/resolv.conf none
> >> -
> >> +l root root 0755 /run /var/run
> >> diff --git a/meta/recipes-core/udev/udev.inc
> >> b/meta/recipes-core/udev/udev.inc
> >> index 9cc00e8..fafd9ce 100644
> >> --- a/meta/recipes-core/udev/udev.inc
> >> +++ b/meta/recipes-core/udev/udev.inc
> >> @@ -6,9 +6,10 @@ LICENSE = "GPLv2.0+ & LGPLv2.1+"
> >> LICENSE_${PN} = "GPLv2.0+"
> >> LICENSE_libudev = "LGPLv2.1+"
> >> LICENSE_libgudev = "LGPLv2.1+"
> >> -LIC_FILES_CHKSUM =
> "file://COPYING;md5=751419260aa954499f7abaabaa882bbe \
> >> -
> >> file://libudev/COPYING;md5=a6f89e2100d9b6cdffcea4f398e37343 \
> >> -
> >> file://extras/gudev/COPYING;md5=a6f89e2100d9b6cdffcea4f398e37343"
> >> +LIC_FILES_CHKSUM =
> "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263 \
> >> +
> >> file://src/COPYING;md5=17c4e5fb495e6707ac92a3864926f979 \
> >> +
> >> file://src/gudev/COPYING;md5=fb494485a7d0505308cb68e4997cc266"
> >> +
> >>
> >> DEPENDS = "acl glib-2.0 libusb usbutils pciutils gperf-native
> >> libxslt-native"
> >> RPROVIDES_${PN} = "hotplug"
> >> @@ -16,6 +17,7 @@ RRECOMMENDS_${PN} += "udev-extraconf usbutils-ids
> >> pciutils-ids"
> >> RDEPENDS_libudev = "${PN} (= ${EXTENDPKGV})"
> >>
> >> SRC_URI =
> >> "${KERNELORG_MIRROR}/linux/utils/kernel/hotplug/udev-${PV}.tar.gz \
> >> +
> >> file://0001-Fixing-keyboard_force_release.sh-shell-script-path.patch \
> >> file://run.rules \
> >> file://udev.rules \
> >> file://devfs-udev.rules \
> >> @@ -32,7 +34,7 @@ inherit autotools pkgconfig update-rc.d
> >>
> >> # udevd/udevadm -> /sbin/, libudev.so.* -> /lib/
> >> sbindir = "${base_sbindir}"
> >> -libexecdir = "${base_libdir}/udev"
> >> +libexecdir = "${base_libdir}"
> >> EXTRA_OECONF = "--disable-introspection
> --with-rootlibdir=${base_libdir}
> >> \
> >> --with-pci-ids-path=${datadir}/pci.ids"
> >>
> >> @@ -50,6 +52,8 @@ FILES_${PN} += "${libexecdir} ${libdir}/ConsoleKit"
> >> RRECOMMENDS_${PN} += "udev-utils"
> >>
> >> FILES_${PN}-dbg += "${libexecdir}/.debug"
> >> +FILES_${PN}-dbg += "${base_libdir}/udev/.debug/"
> >> +FILES_${PN}-dbg += "${base_libdir}/udev/.debug/*"
> >> FILES_${PN}-dev = "${datadir}/pkgconfig/udev.pc"
> >> FILES_libudev = "${base_libdir}/libudev.so.*"
> >> FILES_libudev-dbg = "${base_libdir}/.debug/libudev.so.*"
> >> diff --git
> >>
> a/meta/recipes-core/udev/udev/0001-Fixing-keyboard_force_release.sh-shell-script-path.patch
> >>
> b/meta/recipes-core/udev/udev/0001-Fixing-keyboard_force_release.sh-shell-script-path.patch
> >> new file mode 100644
> >> index 0000000..41deafa
> >> --- /dev/null
> >> +++
> >>
> b/meta/recipes-core/udev/udev/0001-Fixing-keyboard_force_release.sh-shell-script-path.patch
> >> @@ -0,0 +1,35 @@
> >> +From 0f8290c943da298abd269ca60fd8375dfb219971 Mon Sep 17 00:00:00 2001
> >> +From: Alexandru DAMIAN <alexandru.damian@intel.com>
> >> +Date: Thu, 12 Jul 2012 12:54:48 +0300
> >> +Subject: [PATCH] Fixing keyboard_force_release.sh shell script path
> >> +
> >> +With the introduction of rootprefix, the keyboard-force-release.sh.in
> >> +was modified to be executed with @rootprefix@/bin/sh, which is wrong
> >> +because @rootprefix@ defaults to /usr (which is correct), but the
> >> +shell is always at /bin/sh (IEEE Std 1003.2-1992).
> >> +
> >> +Therefore the interpreter for shell scripts needs to be /bin/sh at all
> >> times.
> >> +
> >> +The upstream moved to configurable root prefix, this patch taclkes a
> >> +transition bug, and will not be applied upstream.
> >> +
> >> +Upstream-Status: Inappropriate [legacy version]
> >> +
> >> +Signed-off-by: Alexandru DAMIAN <alexandru.damian@intel.com>
> >> +---
> >> + src/keymap/keyboard-force-release.sh.in | 2 +-
> >> + 1 file changed, 1 insertion(+), 1 deletion(-)
> >> +
> >> +diff --git a/src/keymap/keyboard-force-release.sh.in
> >> b/src/keymap/keyboard-force-release.sh.in
> >> +index dd040ce..597a3a6 100755
> >> +--- a/src/keymap/keyboard-force-release.sh.in
> >> ++++ b/src/keymap/keyboard-force-release.sh.in
> >> +@@ -1,4 +1,4 @@
> >> +-#!@rootprefix@/bin/sh -e
> >> ++#!/bin/sh -e
> >> + # read list of scancodes, convert hex to decimal and
> >> + # append to the atkbd force_release sysfs attribute
> >> + # $1 sysfs devpath for serioX
> >> +--
> >> +1.7.9.5
> >> +
> >> diff --git a/meta/recipes-core/udev/udev/include_resource.patch
> >> b/meta/recipes-core/udev/udev/include_resource.patch
> >> deleted file mode 100644
> >> index a46ce59..0000000
> >> --- a/meta/recipes-core/udev/udev/include_resource.patch
> >> +++ /dev/null
> >> @@ -1,31 +0,0 @@
> >> -Fixes errors like below with eglibc 2.16
> >> -
> >> -libudev/libudev-util-private.c:348:16: error: 'PRIO_PROCESS' undeclared
> >> (first use in this function)
> >> -
> >> -Signed-off-by: Khem Raj <raj.khem@gmail.com>
> >> -Upstream-Status: Pending
> >> -
> >> -Index: udev-164/libudev/libudev-util-private.c
> >> -===================================================================
> >> ---- udev-164.orig/libudev/libudev-util-private.c 2010-08-03
> >> 02:42:26.000000000 -0700
> >> -+++ udev-164/libudev/libudev-util-private.c 2012-07-03
> >> 10:53:35.082346684 -0700
> >> -@@ -21,6 +21,7 @@
> >> - #include <grp.h>
> >> - #include <sys/wait.h>
> >> - #include <sys/param.h>
> >> -+#include <sys/resource.h>
> >> -
> >> - #include "libudev.h"
> >> - #include "libudev-private.h"
> >> -Index: udev-164/udev/udevd.c
> >> -===================================================================
> >> ---- udev-164.orig/udev/udevd.c 2010-10-13 04:08:29.000000000 -0700
> >> -+++ udev-164/udev/udevd.c 2012-07-03 10:53:19.214345914 -0700
> >> -@@ -43,6 +43,7 @@
> >> - #include <sys/ioctl.h>
> >> - #include <sys/inotify.h>
> >> - #include <sys/utsname.h>
> >> -+#include <sys/resource.h>
> >> -
> >> - #include "udev.h"
> >> - #include "sd-daemon.h"
> >> diff --git a/meta/recipes-core/udev/udev/init
> >> b/meta/recipes-core/udev/udev/init
> >> index d521685..ca02abb 100644
> >> --- a/meta/recipes-core/udev/udev/init
> >> +++ b/meta/recipes-core/udev/udev/init
> >> @@ -13,7 +13,7 @@ export TZ=/etc/localtime
> >>
> >> [ -d /sys/class ] || exit 1
> >> [ -r /proc/mounts ] || exit 1
> >> -[ -x /sbin/udevd ] || exit 1
> >> +[ -x /lib/udev/udevd ] || exit 1
> >> [ -f /etc/default/udev-cache ] && . /etc/default/udev-cache
> >> [ -f /etc/udev/udev.conf ] && . /etc/udev/udev.conf
> >>
> >> @@ -70,15 +70,15 @@ kill_udevd > "/dev/null" 2>&1
> >>
> >> # trigger the sorted events
> >> echo -e '\000\000\000\000' > /proc/sys/kernel/hotplug
> >> -/sbin/udevd -d
> >> +/lib/udev/udevd -d
> >>
> >> -/sbin/udevadm control --env=STARTUP=1
> >> +/usr/bin/udevadm control --env=STARTUP=1
> >> if [ "$not_first_boot" != "" ];then
> >> - /sbin/udevadm trigger --action=add --subsystem-nomatch=tty
> >> --subsystem-nomatch=mem --subsystem-nomatch=vc
> --subsystem-nomatch=vtconsole
> >> --subsystem-nomatch=misc --subsystem-nomatch=dcon
> >> --subsystem-nomatch=pci_bus --subsystem-nomatch=graphics
> >> --subsystem-nomatch=backlight --subsystem-nomatch=video4linux
> >> --subsystem-nomatch=platform
> >> - (/sbin/udevadm settle --timeout=3; /sbin/udevadm control
> >> --env=STARTUP=)&
> >> + /usr/bin/udevadm trigger --action=add --subsystem-nomatch=tty
> >> --subsystem-nomatch=mem --subsystem-nomatch=vc
> --subsystem-nomatch=vtconsole
> >> --subsystem-nomatch=misc --subsystem-nomatch=dcon
> >> --subsystem-nomatch=pci_bus --subsystem-nomatch=graphics
> >> --subsystem-nomatch=backlight --subsystem-nomatch=video4linux
> >> --subsystem-nomatch=platform
> >> + (/usr/bin/udevadm settle --timeout=3; /usr/bin/udevadm control
> >> --env=STARTUP=)&
> >> else
> >> - /sbin/udevadm trigger --action=add
> >> - /sbin/udevadm settle
> >> + /usr/bin/udevadm trigger --action=add
> >> + /usr/bin/udevadm settle
> >> fi
> >>
> >> exit 0
> >> diff --git a/meta/recipes-core/udev/udev/udev-166-v4l1-1.patch
> >> b/meta/recipes-core/udev/udev/udev-166-v4l1-1.patch
> >> deleted file mode 100644
> >> index 2086fe1..0000000
> >> --- a/meta/recipes-core/udev/udev/udev-166-v4l1-1.patch
> >> +++ /dev/null
> >> @@ -1,50 +0,0 @@
> >> -Upstream-Status: Backport
> >> -
> >> -Submitted By: Matt Burgess
> >> <matthew_at_linuxfromscratch_dot_org>
> >> -Date: 2011-03-26
> >> -Initial Package Version: 166
> >> -Upstream Status: From upstream
> >> -Origin:
> >>
> http://git.kernel.org/?p=linux/hotplug/udev.git;a=commitdiff;h=4ace8a43ac2cbbd4d6f5c29fc461c3caa8f8545b
> >> -Description: Fixes a compilation error caused by the
> removal
> >> of the
> >> - Video for Linux 1 API from Linux kernels from
> >> 2.6.38
> >> - onwards.
> >> -
> >> -diff -Naur udev-166.orig/extras/v4l_id/v4l_id.c
> >> udev-166/extras/v4l_id/v4l_id.c
> >> ---- udev-166.orig/extras/v4l_id/v4l_id.c 2009-12-03
> >> 12:45:03.000000000 +0000
> >> -+++ udev-166/extras/v4l_id/v4l_id.c 2011-03-25 20:26:33.000000000
> >> +0000
> >> -@@ -28,7 +28,6 @@
> >> - #include <sys/types.h>
> >> - #include <sys/time.h>
> >> - #include <sys/ioctl.h>
> >> --#include <linux/videodev.h>
> >> - #include <linux/videodev2.h>
> >> -
> >> - int main (int argc, char *argv[])
> >> -@@ -39,7 +38,6 @@
> >> - };
> >> - int fd;
> >> - char *device;
> >> -- struct video_capability v1cap;
> >> - struct v4l2_capability v2cap;
> >> -
> >> - while (1) {
> >> -@@ -82,19 +80,6 @@
> >> - if ((v2cap.capabilities & V4L2_CAP_RADIO) > 0)
> >> - printf("radio:");
> >> - printf("\n");
> >> -- } else if (ioctl (fd, VIDIOCGCAP, &v1cap) == 0) {
> >> -- printf("ID_V4L_VERSION=1\n");
> >> -- printf("ID_V4L_PRODUCT=%s\n", v1cap.name);
> >> -- printf("ID_V4L_CAPABILITIES=:");
> >> -- if ((v1cap.type & VID_TYPE_CAPTURE) > 0)
> >> -- printf("capture:");
> >> -- if ((v1cap.type & VID_TYPE_OVERLAY) > 0)
> >> -- printf("video_overlay:");
> >> -- if (v1cap.audios > 0)
> >> -- printf("audio:");
> >> -- if ((v1cap.type & VID_TYPE_TUNER) > 0)
> >> -- printf("tuner:");
> >> -- printf("\n");
> >> - }
> >> -
> >> - close (fd);
> >> diff --git a/meta/recipes-core/udev/udev/udev-cache
> >> b/meta/recipes-core/udev/udev/udev-cache
> >> index 9c24e76..0f3d7cd 100644
> >> --- a/meta/recipes-core/udev/udev/udev-cache
> >> +++ b/meta/recipes-core/udev/udev/udev-cache
> >> @@ -12,7 +12,7 @@
> >> export TZ=/etc/localtime
> >>
> >> [ -r /proc/mounts ] || exit 1
> >> -[ -x /sbin/udevd ] || exit 1
> >> +[ -x /lib/udev/udevd ] || exit 1
> >> [ -d /sys/class ] || exit 1
> >>
> >> [ -f /etc/default/udev-cache ] && . /etc/default/udev-cache
> >> diff --git a/meta/recipes-core/udev/udev_164.bb
> >> b/meta/recipes-core/udev/udev_164.bb
> >> deleted file mode 100644
> >> index 4d22bc8..0000000
> >> --- a/meta/recipes-core/udev/udev_164.bb
> >> +++ /dev/null
> >> @@ -1,9 +0,0 @@
> >> -include udev.inc
> >> -
> >> -PR = "r16"
> >> -
> >> -SRC_URI += "file://udev-166-v4l1-1.patch \
> >> - file://include_resource.patch \
> >> - "
> >> -SRC_URI[md5sum] = "fddac2d54761ea34865af9467377ca9f"
> >> -SRC_URI[sha256sum] =
> >> "c12e66280b5e1465f6587a8cfa47d7405c4caa7e52ce5dd13478d04f6ec05e5c"
> >> diff --git a/meta/recipes-core/udev/udev_182.bb
> >> b/meta/recipes-core/udev/udev_182.bb
> >> new file mode 100644
> >> index 0000000..6678fe1
> >> --- /dev/null
> >> +++ b/meta/recipes-core/udev/udev_182.bb
> >> @@ -0,0 +1,9 @@
> >> +include udev.inc
> >> +
> >> +PR = "r0"
> >> +
> >> +# module-init-tools from kmod_git will provide libkmod runtime
> >> +DEPENDS = "module-init-tools"
> >> +
> >> +SRC_URI[md5sum] = "1b964456177fbf48023dfee7db3a708d"
> >> +SRC_URI[sha256sum] =
> >> "7857ed19fafd8f3ca8de410194e8c7336e9eb8a20626ea8a4ba6449b017faba4"
> >>
> >
> > _______________________________________________
> > Openembedded-core mailing list
> > Openembedded-core@lists.openembedded.org
> > http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>
[-- Attachment #2: Type: text/html, Size: 21572 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v3] udev: upgrade to 182
2012-09-21 10:38 ` Damian, Alexandru
@ 2012-09-21 20:41 ` Saul Wold
0 siblings, 0 replies; 5+ messages in thread
From: Saul Wold @ 2012-09-21 20:41 UTC (permalink / raw)
To: Damian, Alexandru; +Cc: Koen Kooi, openembedded-core
On 09/21/2012 03:38 AM, Damian, Alexandru wrote:
> This patch still stands.
>
> I took a look at meta-oe. IMHO, the way the build works here is the
> right way.
>
> I had no problems rebasing it to the latest HEAD,
> df5f9a3e6dfcf8e8f57132843a024b44c0bd90fc,
> and rebuilding.
>
Ok, I am have figured out what's going on!
There are 2 issues with this patch, which I have fixed and in the
stage/master_under_test.
First issue, is that the patch would not apply directly to master same
HEAD ref you mentioned above, this was due to a PR bump between when you
sent this patch and now.
Next issue was the usbutils issue I mentioned before, not only to I just
rebuild on my system, but I do from scratch builds, so I found this.
The DEPENDS line you added in the .bb file overwrites the data you have
in the .inc file since you use "=", this caused usbutils to not be
installed before udev started it's configuration. It could have caused
other failures also. This should be DEPENDS += "module_init_tools"
Also, can you (or Koen) explain what the differences are between your
version and the meta-oe version, I know there is some packaging
differences and I almost wonder if those should be a separate recipe?
The other difference I saw where in configuration, specifically:
ac_cv_file__usr_share_pci_ids=no \
ac_cv_file__usr_share_hwdata_pci_ids=no \
ac_cv_file__usr_share_misc_pci_ids=yes \
Looks like those changes will be good to have since we might be having
some host contamination issues.
I also think the other changes for setting the correct dirs would be
good for the flexibility of the filesystem layout
I have made those change to the patch also.
Sau!
> Tested as follows:
> - build core-image-sato
> - running qemux86 image
> - udevd is started correctly. the image works normally under usage smoke
> tests
> - udevd is verified to be version 182
>
> If you'll see problems building, please let me have a log.
>
> Alex
>
> On Wed, Sep 19, 2012 at 9:02 PM, Khem Raj <raj.khem@gmail.com
> <mailto:raj.khem@gmail.com>> wrote:
>
> On Tue, Sep 11, 2012 at 1:53 PM, Saul Wold <sgw@linux.intel.com
> <mailto:sgw@linux.intel.com>> wrote:
> > On 09/06/2012 02:27 AM, Alex DAMIAN wrote:
> >>
> >> From: Alexandru DAMIAN <alexandru.damian@intel.com
> <mailto:alexandru.damian@intel.com>>
> >>
> >> This is the final upgrade of udev. Futher upgrades will only
> >> come in conjunction with systemd.
> >>
> >> The v4l1 removal patch is deprecated as the bug is fixed inside
> udev.
> >> There is a new patch fixing the path for default sh interpreter.
> >> New debug binaries are generated, and udev.inc is modified to
> package
> >> those correctly.
> >> The install locations changed for udevd and udevadm, so the scripts
> >> are updated accordingly.
> >>
> >> Signed-off-by: Alexandru DAMIAN <alexandru.damian@intel.com
> <mailto:alexandru.damian@intel.com>>
> >>
> >
> > Alex,
> >
> > I have had some problems with this patch trying to build it. It
> was looking
> > on my host for usbutils version .82 or greater, but we have the
> new numbered
> > 006 usbutils and it failed.
> >
> > Also, why did you not take the meta-oe version, did you compare
> your changes
> > with the version is meta-oe? This was mentioned once before?
> >
> > Thanks
> > Sau!
> >
> >
>
> Alex
>
> whats status of this patch is there is a v4 expected soon
>
> >
> >> Conflicts:
> >>
> >> meta/recipes-core/udev/udev_164.bb <http://udev_164.bb>
> >> ---
> >> .../initscripts/initscripts-1.0/volatiles | 2 +-
> >> meta/recipes-core/udev/udev.inc | 12 +++--
> >> ...yboard_force_release.sh-shell-script-path.patch | 35
> ++++++++++++++
> >> meta/recipes-core/udev/udev/include_resource.patch | 31
> ------------
> >> meta/recipes-core/udev/udev/init | 14 +++---
> >> meta/recipes-core/udev/udev/udev-166-v4l1-1.patch | 50
> >> --------------------
> >> meta/recipes-core/udev/udev/udev-cache | 2 +-
> >> meta/recipes-core/udev/udev_164.bb <http://udev_164.bb>
> | 9 ----
> >> meta/recipes-core/udev/udev_182.bb <http://udev_182.bb>
> | 9 ++++
> >> 9 files changed, 61 insertions(+), 103 deletions(-)
> >> create mode 100644
> >>
> meta/recipes-core/udev/udev/0001-Fixing-keyboard_force_release.sh-shell-script-path.patch
> >> delete mode 100644
> meta/recipes-core/udev/udev/include_resource.patch
> >> delete mode 100644
> meta/recipes-core/udev/udev/udev-166-v4l1-1.patch
> >> delete mode 100644 meta/recipes-core/udev/udev_164.bb
> <http://udev_164.bb>
> >> create mode 100644 meta/recipes-core/udev/udev_182.bb
> <http://udev_182.bb>
> >>
> >> diff --git a/meta/recipes-core/initscripts/initscripts-1.0/volatiles
> >> b/meta/recipes-core/initscripts/initscripts-1.0/volatiles
> >> index b2ae279..e0741aa 100644
> >> --- a/meta/recipes-core/initscripts/initscripts-1.0/volatiles
> >> +++ b/meta/recipes-core/initscripts/initscripts-1.0/volatiles
> >> @@ -36,4 +36,4 @@ f root root 0664 /var/log/wtmp none
> >> f root root 0664 /var/run/utmp none
> >> l root root 0644 /etc/resolv.conf /var/run/resolv.conf
> >> f root root 0644 /var/run/resolv.conf none
> >> -
> >> +l root root 0755 /run /var/run
> >> diff --git a/meta/recipes-core/udev/udev.inc
> >> b/meta/recipes-core/udev/udev.inc
> >> index 9cc00e8..fafd9ce 100644
> >> --- a/meta/recipes-core/udev/udev.inc
> >> +++ b/meta/recipes-core/udev/udev.inc
> >> @@ -6,9 +6,10 @@ LICENSE = "GPLv2.0+ & LGPLv2.1+"
> >> LICENSE_${PN} = "GPLv2.0+"
> >> LICENSE_libudev = "LGPLv2.1+"
> >> LICENSE_libgudev = "LGPLv2.1+"
> >> -LIC_FILES_CHKSUM =
> "file://COPYING;md5=751419260aa954499f7abaabaa882bbe \
> >> -
> >> file://libudev/COPYING;md5=a6f89e2100d9b6cdffcea4f398e37343 \
> >> -
> >> file://extras/gudev/COPYING;md5=a6f89e2100d9b6cdffcea4f398e37343"
> >> +LIC_FILES_CHKSUM =
> "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263 \
> >> +
> >> file://src/COPYING;md5=17c4e5fb495e6707ac92a3864926f979 \
> >> +
> >> file://src/gudev/COPYING;md5=fb494485a7d0505308cb68e4997cc266"
> >> +
> >>
> >> DEPENDS = "acl glib-2.0 libusb usbutils pciutils gperf-native
> >> libxslt-native"
> >> RPROVIDES_${PN} = "hotplug"
> >> @@ -16,6 +17,7 @@ RRECOMMENDS_${PN} += "udev-extraconf usbutils-ids
> >> pciutils-ids"
> >> RDEPENDS_libudev = "${PN} (= ${EXTENDPKGV})"
> >>
> >> SRC_URI =
> >> "${KERNELORG_MIRROR}/linux/utils/kernel/hotplug/udev-${PV}.tar.gz \
> >> +
> >>
> file://0001-Fixing-keyboard_force_release.sh-shell-script-path.patch \
> >> file://run.rules \
> >> file://udev.rules \
> >> file://devfs-udev.rules \
> >> @@ -32,7 +34,7 @@ inherit autotools pkgconfig update-rc.d
> >>
> >> # udevd/udevadm -> /sbin/, libudev.so.* -> /lib/
> >> sbindir = "${base_sbindir}"
> >> -libexecdir = "${base_libdir}/udev"
> >> +libexecdir = "${base_libdir}"
> >> EXTRA_OECONF = "--disable-introspection
> --with-rootlibdir=${base_libdir}
> >> \
> >> --with-pci-ids-path=${datadir}/pci.ids"
> >>
> >> @@ -50,6 +52,8 @@ FILES_${PN} += "${libexecdir}
> ${libdir}/ConsoleKit"
> >> RRECOMMENDS_${PN} += "udev-utils"
> >>
> >> FILES_${PN}-dbg += "${libexecdir}/.debug"
> >> +FILES_${PN}-dbg += "${base_libdir}/udev/.debug/"
> >> +FILES_${PN}-dbg += "${base_libdir}/udev/.debug/*"
> >> FILES_${PN}-dev = "${datadir}/pkgconfig/udev.pc"
> >> FILES_libudev = "${base_libdir}/libudev.so.*"
> >> FILES_libudev-dbg = "${base_libdir}/.debug/libudev.so.*"
> >> diff --git
> >>
> a/meta/recipes-core/udev/udev/0001-Fixing-keyboard_force_release.sh-shell-script-path.patch
> >>
> b/meta/recipes-core/udev/udev/0001-Fixing-keyboard_force_release.sh-shell-script-path.patch
> >> new file mode 100644
> >> index 0000000..41deafa
> >> --- /dev/null
> >> +++
> >>
> b/meta/recipes-core/udev/udev/0001-Fixing-keyboard_force_release.sh-shell-script-path.patch
> >> @@ -0,0 +1,35 @@
> >> +From 0f8290c943da298abd269ca60fd8375dfb219971 Mon Sep 17
> 00:00:00 2001
> >> +From: Alexandru DAMIAN <alexandru.damian@intel.com
> <mailto:alexandru.damian@intel.com>>
> >> +Date: Thu, 12 Jul 2012 12:54:48 +0300
> >> +Subject: [PATCH] Fixing keyboard_force_release.sh shell script path
> >> +
> >> +With the introduction of rootprefix, the
> keyboard-force-release.sh.in <http://keyboard-force-release.sh.in>
> >> +was modified to be executed with @rootprefix@/bin/sh, which is
> wrong
> >> +because @rootprefix@ defaults to /usr (which is correct), but the
> >> +shell is always at /bin/sh (IEEE Std 1003.2-1992).
> >> +
> >> +Therefore the interpreter for shell scripts needs to be /bin/sh
> at all
> >> times.
> >> +
> >> +The upstream moved to configurable root prefix, this patch
> taclkes a
> >> +transition bug, and will not be applied upstream.
> >> +
> >> +Upstream-Status: Inappropriate [legacy version]
> >> +
> >> +Signed-off-by: Alexandru DAMIAN <alexandru.damian@intel.com
> <mailto:alexandru.damian@intel.com>>
> >> +---
> >> + src/keymap/keyboard-force-release.sh.in
> <http://keyboard-force-release.sh.in> | 2 +-
> >> + 1 file changed, 1 insertion(+), 1 deletion(-)
> >> +
> >> +diff --git a/src/keymap/keyboard-force-release.sh.in
> <http://keyboard-force-release.sh.in>
> >> b/src/keymap/keyboard-force-release.sh.in
> <http://keyboard-force-release.sh.in>
> >> +index dd040ce..597a3a6 100755
> >> +--- a/src/keymap/keyboard-force-release.sh.in
> <http://keyboard-force-release.sh.in>
> >> ++++ b/src/keymap/keyboard-force-release.sh.in
> <http://keyboard-force-release.sh.in>
> >> +@@ -1,4 +1,4 @@
> >> +-#!@rootprefix@/bin/sh -e
> >> ++#!/bin/sh -e
> >> + # read list of scancodes, convert hex to decimal and
> >> + # append to the atkbd force_release sysfs attribute
> >> + # $1 sysfs devpath for serioX
> >> +--
> >> +1.7.9.5
> >> +
> >> diff --git a/meta/recipes-core/udev/udev/include_resource.patch
> >> b/meta/recipes-core/udev/udev/include_resource.patch
> >> deleted file mode 100644
> >> index a46ce59..0000000
> >> --- a/meta/recipes-core/udev/udev/include_resource.patch
> >> +++ /dev/null
> >> @@ -1,31 +0,0 @@
> >> -Fixes errors like below with eglibc 2.16
> >> -
> >> -libudev/libudev-util-private.c:348:16: error: 'PRIO_PROCESS'
> undeclared
> >> (first use in this function)
> >> -
> >> -Signed-off-by: Khem Raj <raj.khem@gmail.com
> <mailto:raj.khem@gmail.com>>
> >> -Upstream-Status: Pending
> >> -
> >> -Index: udev-164/libudev/libudev-util-private.c
> >> -===================================================================
> >> ---- udev-164.orig/libudev/libudev-util-private.c 2010-08-03
> >> 02:42:26.000000000 -0700
> >> -+++ udev-164/libudev/libudev-util-private.c 2012-07-03
> >> 10:53:35.082346684 -0700
> >> -@@ -21,6 +21,7 @@
> >> - #include <grp.h>
> >> - #include <sys/wait.h>
> >> - #include <sys/param.h>
> >> -+#include <sys/resource.h>
> >> -
> >> - #include "libudev.h"
> >> - #include "libudev-private.h"
> >> -Index: udev-164/udev/udevd.c
> >> -===================================================================
> >> ---- udev-164.orig/udev/udevd.c 2010-10-13 04:08:29.000000000 -0700
> >> -+++ udev-164/udev/udevd.c 2012-07-03 10:53:19.214345914 -0700
> >> -@@ -43,6 +43,7 @@
> >> - #include <sys/ioctl.h>
> >> - #include <sys/inotify.h>
> >> - #include <sys/utsname.h>
> >> -+#include <sys/resource.h>
> >> -
> >> - #include "udev.h"
> >> - #include "sd-daemon.h"
> >> diff --git a/meta/recipes-core/udev/udev/init
> >> b/meta/recipes-core/udev/udev/init
> >> index d521685..ca02abb 100644
> >> --- a/meta/recipes-core/udev/udev/init
> >> +++ b/meta/recipes-core/udev/udev/init
> >> @@ -13,7 +13,7 @@ export TZ=/etc/localtime
> >>
> >> [ -d /sys/class ] || exit 1
> >> [ -r /proc/mounts ] || exit 1
> >> -[ -x /sbin/udevd ] || exit 1
> >> +[ -x /lib/udev/udevd ] || exit 1
> >> [ -f /etc/default/udev-cache ] && . /etc/default/udev-cache
> >> [ -f /etc/udev/udev.conf ] && . /etc/udev/udev.conf
> >>
> >> @@ -70,15 +70,15 @@ kill_udevd > "/dev/null" 2>&1
> >>
> >> # trigger the sorted events
> >> echo -e '\000\000\000\000' > /proc/sys/kernel/hotplug
> >> -/sbin/udevd -d
> >> +/lib/udev/udevd -d
> >>
> >> -/sbin/udevadm control --env=STARTUP=1
> >> +/usr/bin/udevadm control --env=STARTUP=1
> >> if [ "$not_first_boot" != "" ];then
> >> - /sbin/udevadm trigger --action=add --subsystem-nomatch=tty
> >> --subsystem-nomatch=mem --subsystem-nomatch=vc
> --subsystem-nomatch=vtconsole
> >> --subsystem-nomatch=misc --subsystem-nomatch=dcon
> >> --subsystem-nomatch=pci_bus --subsystem-nomatch=graphics
> >> --subsystem-nomatch=backlight --subsystem-nomatch=video4linux
> >> --subsystem-nomatch=platform
> >> - (/sbin/udevadm settle --timeout=3; /sbin/udevadm control
> >> --env=STARTUP=)&
> >> + /usr/bin/udevadm trigger --action=add
> --subsystem-nomatch=tty
> >> --subsystem-nomatch=mem --subsystem-nomatch=vc
> --subsystem-nomatch=vtconsole
> >> --subsystem-nomatch=misc --subsystem-nomatch=dcon
> >> --subsystem-nomatch=pci_bus --subsystem-nomatch=graphics
> >> --subsystem-nomatch=backlight --subsystem-nomatch=video4linux
> >> --subsystem-nomatch=platform
> >> + (/usr/bin/udevadm settle --timeout=3; /usr/bin/udevadm
> control
> >> --env=STARTUP=)&
> >> else
> >> - /sbin/udevadm trigger --action=add
> >> - /sbin/udevadm settle
> >> + /usr/bin/udevadm trigger --action=add
> >> + /usr/bin/udevadm settle
> >> fi
> >>
> >> exit 0
> >> diff --git a/meta/recipes-core/udev/udev/udev-166-v4l1-1.patch
> >> b/meta/recipes-core/udev/udev/udev-166-v4l1-1.patch
> >> deleted file mode 100644
> >> index 2086fe1..0000000
> >> --- a/meta/recipes-core/udev/udev/udev-166-v4l1-1.patch
> >> +++ /dev/null
> >> @@ -1,50 +0,0 @@
> >> -Upstream-Status: Backport
> >> -
> >> -Submitted By: Matt Burgess
> >> <matthew_at_linuxfromscratch_dot_org>
> >> -Date: 2011-03-26
> >> -Initial Package Version: 166
> >> -Upstream Status: From upstream
> >> -Origin:
> >>
> http://git.kernel.org/?p=linux/hotplug/udev.git;a=commitdiff;h=4ace8a43ac2cbbd4d6f5c29fc461c3caa8f8545b
> >> -Description: Fixes a compilation error caused by
> the removal
> >> of the
> >> - Video for Linux 1 API from Linux
> kernels from
> >> 2.6.38
> >> - onwards.
> >> -
> >> -diff -Naur udev-166.orig/extras/v4l_id/v4l_id.c
> >> udev-166/extras/v4l_id/v4l_id.c
> >> ---- udev-166.orig/extras/v4l_id/v4l_id.c 2009-12-03
> >> 12:45:03.000000000 +0000
> >> -+++ udev-166/extras/v4l_id/v4l_id.c 2011-03-25
> 20:26:33.000000000
> >> +0000
> >> -@@ -28,7 +28,6 @@
> >> - #include <sys/types.h>
> >> - #include <sys/time.h>
> >> - #include <sys/ioctl.h>
> >> --#include <linux/videodev.h>
> >> - #include <linux/videodev2.h>
> >> -
> >> - int main (int argc, char *argv[])
> >> -@@ -39,7 +38,6 @@
> >> - };
> >> - int fd;
> >> - char *device;
> >> -- struct video_capability v1cap;
> >> - struct v4l2_capability v2cap;
> >> -
> >> - while (1) {
> >> -@@ -82,19 +80,6 @@
> >> - if ((v2cap.capabilities & V4L2_CAP_RADIO) > 0)
> >> - printf("radio:");
> >> - printf("\n");
> >> -- } else if (ioctl (fd, VIDIOCGCAP, &v1cap) == 0) {
> >> -- printf("ID_V4L_VERSION=1\n");
> >> -- printf("ID_V4L_PRODUCT=%s\n", v1cap.name
> <http://v1cap.name>);
> >> -- printf("ID_V4L_CAPABILITIES=:");
> >> -- if ((v1cap.type & VID_TYPE_CAPTURE) > 0)
> >> -- printf("capture:");
> >> -- if ((v1cap.type & VID_TYPE_OVERLAY) > 0)
> >> -- printf("video_overlay:");
> >> -- if (v1cap.audios > 0)
> >> -- printf("audio:");
> >> -- if ((v1cap.type & VID_TYPE_TUNER) > 0)
> >> -- printf("tuner:");
> >> -- printf("\n");
> >> - }
> >> -
> >> - close (fd);
> >> diff --git a/meta/recipes-core/udev/udev/udev-cache
> >> b/meta/recipes-core/udev/udev/udev-cache
> >> index 9c24e76..0f3d7cd 100644
> >> --- a/meta/recipes-core/udev/udev/udev-cache
> >> +++ b/meta/recipes-core/udev/udev/udev-cache
> >> @@ -12,7 +12,7 @@
> >> export TZ=/etc/localtime
> >>
> >> [ -r /proc/mounts ] || exit 1
> >> -[ -x /sbin/udevd ] || exit 1
> >> +[ -x /lib/udev/udevd ] || exit 1
> >> [ -d /sys/class ] || exit 1
> >>
> >> [ -f /etc/default/udev-cache ] && . /etc/default/udev-cache
> >> diff --git a/meta/recipes-core/udev/udev_164.bb <http://udev_164.bb>
> >> b/meta/recipes-core/udev/udev_164.bb <http://udev_164.bb>
> >> deleted file mode 100644
> >> index 4d22bc8..0000000
> >> --- a/meta/recipes-core/udev/udev_164.bb <http://udev_164.bb>
> >> +++ /dev/null
> >> @@ -1,9 +0,0 @@
> >> -include udev.inc
> >> -
> >> -PR = "r16"
> >> -
> >> -SRC_URI += "file://udev-166-v4l1-1.patch \
> >> - file://include_resource.patch \
> >> - "
> >> -SRC_URI[md5sum] = "fddac2d54761ea34865af9467377ca9f"
> >> -SRC_URI[sha256sum] =
> >> "c12e66280b5e1465f6587a8cfa47d7405c4caa7e52ce5dd13478d04f6ec05e5c"
> >> diff --git a/meta/recipes-core/udev/udev_182.bb <http://udev_182.bb>
> >> b/meta/recipes-core/udev/udev_182.bb <http://udev_182.bb>
> >> new file mode 100644
> >> index 0000000..6678fe1
> >> --- /dev/null
> >> +++ b/meta/recipes-core/udev/udev_182.bb <http://udev_182.bb>
> >> @@ -0,0 +1,9 @@
> >> +include udev.inc
> >> +
> >> +PR = "r0"
> >> +
> >> +# module-init-tools from kmod_git will provide libkmod runtime
> >> +DEPENDS = "module-init-tools"
> >> +
> >> +SRC_URI[md5sum] = "1b964456177fbf48023dfee7db3a708d"
> >> +SRC_URI[sha256sum] =
> >> "7857ed19fafd8f3ca8de410194e8c7336e9eb8a20626ea8a4ba6449b017faba4"
> >>
> >
> > _______________________________________________
> > Openembedded-core mailing list
> > Openembedded-core@lists.openembedded.org
> <mailto:Openembedded-core@lists.openembedded.org>
> > http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>
>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2012-09-21 20:53 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-09-06 9:27 [PATCH v3] udev: upgrade to 182 Alex DAMIAN
2012-09-11 20:53 ` Saul Wold
2012-09-19 18:02 ` Khem Raj
2012-09-21 10:38 ` Damian, Alexandru
2012-09-21 20:41 ` Saul Wold
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox