Bug 234669 – update to sys-fs/udev-124-r1 should not have started udevd

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Jan 2008-08-13 19:25:58 UTC]

An update from udev-119 to 124-r1 just gave the message "restarting udevd now" at the end of the emerge and started "/sbin/udevd --daemon" (now found in ps) despite the fact that the system in question did not have a running udevd previously and has 

RC_DEVICES="static"
RC_DEVICE_TARBALL="no"

listed in /etc/conf.d/rc. The lines responsible are easily found in the ebuild, no check is done there to see if udevd is wanted on the system or if the user is only installing udevd and using static /dev at the moment.

Reproducible: Didn't try



Expected Results:  
Check /etc/conf.d/rc and start udevd only if RC_DEVICES is set accordingly.

emerge --info:

Portage 2.1.4.4 (hardened/x86/2.6, gcc-3.4.6, glibc-2.5-r4, 2.6.23-hardened-r12 i686)
=================================================================
System uname: 2.6.23-hardened-r12 i686 VIA C7 Processor 1000MHz
Timestamp of tree: Tue, 12 Aug 2008 06:15:01 +0000
app-shells/bash:     3.2_p33
dev-lang/python:     2.4.4-r14
dev-python/pycrypto: 2.0.1-r6
sys-apps/baselayout: 1.12.11.1
sys-apps/sandbox:    1.2.18.1-r2
sys-devel/autoconf:  2.13, 2.61-r1
sys-devel/automake:  1.10.1
sys-devel/binutils:  2.18-r1
sys-devel/gcc-config: 1.4.0-r4
sys-devel/libtool:   1.5.26
virtual/os-headers:  2.6.23-r3
ACCEPT_KEYWORDS="x86"
CBUILD="i686-pc-linux-gnu"
CFLAGS="-march=i686 -O2 -pipe -mmmx -msse -msse2 -fforce-addr"
CHOST="i686-pc-linux-gnu"
CONFIG_PROTECT="/etc /var/bind /var/qmail/alias /var/qmail/control"
CONFIG_PROTECT_MASK="/etc/env.d /etc/gconf /etc/revdep-rebuild /etc/terminfo /etc/udev/rules.d /usr/share/X11/xkb"
CXXFLAGS="-march=i686 -O2 -pipe -mmmx -msse -msse2 -fforce-addr"
DISTDIR="/usr/portage/distfiles"
FEATURES="collision-protect distlocks metadata-transfer parallel-fetch sandbox sfperms strict test unmerge-orphans userfetch"
GENTOO_MIRRORS="http://mirrors.sec.informatik.tu-darmstadt.de/gentoo/ http://gentoo.oregonstate.edu/ http://www.ibiblio.org/pub/Linux/distributions/gentoo"
LINGUAS=""
MAKEOPTS="-j2"
PKGDIR="/usr/portage/packages"
PORTAGE_RSYNC_OPTS="--recursive --links --safe-links --perms --times --compress --force --whole-file --delete --stats --timeout=180 --exclude=/distfiles --exclude=/local --exclude=/packages"
PORTAGE_TMPDIR="/var/tmp"
PORTDIR="/usr/portage"
SYNC="rsync://rsync.europe.gentoo.org/gentoo-portage"
USE="acpi bash-completion bzip2 caps cracklib crypt cups hardened imap maildir midi mmx ncurses nls nptl nptlonly pam pic readline samba sse sse2 ssl tcpd unicode urandom usb x86 xinetd xorg zlib" ALSA_CARDS="ali5451 als4000 atiixp atiixp-modem bt87x ca0106 cmipci emu10k1 emu10k1x ens1370 ens1371 es1938 es1968 fm801 hda-intel intel8x0 intel8x0m maestro3 trident usb-audio via82xx via82xx-modem ymfpci" ALSA_PCM_PLUGINS="adpcm alaw asym copy dmix dshare dsnoop empty extplug file hooks iec958 ioplug ladspa lfloat linear meter mmap_emul mulaw multi null plug rate route share shm softvol" APACHE2_MODULES="actions alias auth_basic authn_alias authn_anon authn_dbm authn_default authn_file authz_dbm authz_default authz_groupfile authz_host authz_owner authz_user autoindex cache dav dav_fs dav_lock deflate dir disk_cache env expires ext_filter file_cache filter headers include info log_config logio mem_cache mime mime_magic negotiation rewrite setenvif speling status unique_id userdir usertrack vhost_alias" ELIBC="glibc" INPUT_DEVICES="mouse keyboard" KERNEL="linux" LCD_DEVICES="bayrad cfontz cfontz633 glk hd44780 lb216 lcdm001 mtxorb ncurses text" USERLAND="GNU" VIDEO_CARDS="apm ark chips cirrus cyrix dummy fbdev glint i128 i740 i810 imstt mach64 mga neomagic nsc nv r128 radeon rendition s3 s3virge savage siliconmotion sis sisusb tdfx tga trident tseng v4l vesa vga vmware voodoo"
Unset:  CPPFLAGS, CTARGET, EMERGE_DEFAULT_OPTS, INSTALL_MASK, LANG, LC_ALL, LDFLAGS, PORTAGE_COMPRESS, PORTAGE_COMPRESS_FLAGS, PORTAGE_RSYNC_EXTRA_OPTS, PORTDIR_OVERLAY

[Comment 1 Russell Harmon 2008-08-14 02:20:28 UTC]

Created attachment 162855 [details, diff]
udev-check-usage.patch

This patch is rather hackish, but should work fine for those using openrc.

[Comment 2 Matthias Schwarzott 2008-08-14 11:49:30 UTC]

We could also check if udevd is running before and only restarted it in that case.

Adding condition like this
if pidof -s /sbin/udevd &>/dev/null; then
   restart it
fi

For now only the killing of udevd is wrapped into that condition but not the starting.

[Comment 3 SpanKY 2008-08-20 07:00:27 UTC]

if the solution in Bug 129204 was merged, this would "go away" ...

[Comment 4 Matthias Schwarzott 2009-07-16 08:46:49 UTC]

Fixed for udev-124-r{1,2}. All newer ebuilds were already fixed.