[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: Bug#752742: udev: fails to boot if CONFIG_UEVENT_HELPER is disabled



Am 26.06.2014 08:43, schrieb Arto Jantunen:
> Package: udev
> Version: 204-12
> Severity: normal
> Tags: patch
> 
> The udev initramfs script is executed with set -e, and attempts to clear
> out the contents of /sys/kernel/uevent_helper at boot. If
> CONFIG_UEVENT_HELPER is not set this file doesn't exist, setting it
> fails and the boot stops. The script should check that the file exists
> and is writable before trying to write to it.
> 
> A trivial patch to do that is attached.

Makes sense.

> Additionally udev.init and udev.postinst have checks for the existence
> of that file which would stop udev from being started under sysvinit if
> that compatibility option is not set. I'm not entirely sure what should
> be done about them.

Seems reasonable to drop the

  if [ ! -e /sys/kernel/uevent_helper ]; then
    echo "udev requires hotplug support, not started."
    return 1
  fi

checks from postinst and the udev SysV init script.
udev has been switched over to use the netlink based interface for
communicating with the kernel a long time ago.

Marco, at least I don't see a good reason to keep them. Do you agree?

> The kernel team should possibly be notified about keeping this option
> enabled for the jessie release to avoid breaking partial upgrades.

I don't think they have any plans to change that. But CCed the kernel
team just in case.


Just curious, was this triggered by [1]. Usually CONFIG_UEVENT_HELPER is
set (at least by distros for compat reasons) and contains "".

Cheers,
Michael

[1] http://www.spinics.net/linux/lists/kernel/msg1770793.html

-- 
Why is it that all of the instruments seeking intelligent life in the
universe are pointed away from Earth?

Attachment: signature.asc
Description: OpenPGP digital signature


Reply to: