Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 461828 - sys-kernel/genkernel: Default config for amd64/alpha/x86 sets invalid CONFIG_UEVENT_HELPER="/sbin/{hotplug,udevadm}" value (no such file)
Summary: sys-kernel/genkernel: Default config for amd64/alpha/x86 sets invalid CONFIG_...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Hosted Projects
Classification: Unclassified
Component: genkernel (show other bugs)
Hardware: AMD64 Linux
: Normal critical (vote)
Assignee: Gentoo Genkernel Maintainers
URL:
Whiteboard: Default config of genkernel breaks LV...
Keywords: PATCH
Depends on:
Blocks: 505962
  Show dependency tree
 
Reported: 2013-03-15 15:29 UTC by Mike Gilbert
Modified: 2014-05-31 19:59 UTC (History)
7 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
patch for genkernel (0001-Set-deprecated-CONFIG_UEVENT_HELPER_PATH-option-to-a.patch,710 bytes, patch)
2014-05-13 12:26 UTC, Sergey Popov
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Mike Gilbert gentoo-dev 2013-03-15 15:29:47 UTC
The default kernel config for amd64 contains CONFIG_UEVENT_HELPER="/sbin/udevadm".

Based on the description of the option, this really doesn't make any sense; I don't think udevadm does anything with the events. udevd uses the netlink interface, so it doesn't need this anyway.

I think it should be changed to an empty string, or possibly the upstream default (/sbin/hotplug).
Comment 1 Samuli Suominen (RETIRED) gentoo-dev 2013-05-09 09:46:35 UTC
Futhermore /sbin/udevadm is the obsolete location and the new one is /bin/udevadm.
There is a compability symlink over to /sbin/udevadm which will go away at some point.

Empty string sounds good to me.
Comment 2 Samuli Suominen (RETIRED) gentoo-dev 2014-05-11 10:41:00 UTC
(In reply to Samuli Suominen from comment #1)
> Futhermore /sbin/udevadm is the obsolete location and the new one is
> /bin/udevadm.
> There is a compability symlink over to /sbin/udevadm which will go away at
> some point.

FYI, this compability symlink no longer exists in stable udev, or systemd for that matter
Comment 3 Samuli Suominen (RETIRED) gentoo-dev 2014-05-12 14:04:06 UTC
17:00 <@ssuominen> genkernel is really abandoned entirely?
17:00 <@ssuominen> i wonder if we should proceed with lastriting it, been broken for more than year now regarding it's default kernel config, like bug 461828
17:00 <+willikins> ssuominen: https://bugs.gentoo.org/461828 "sys-kernel/genkernel: Default config for amd64 sets CONFIG_UEVENT_HELPER="/sbin/udevadm""; Gentoo Hosted Projects, genkernel; CONF; 
                   floppym:genkernel
17:01 <@ssuominen> ~400 days should be enough to give people time to fix it
17:02 <@Zero_Chaos> ssuominen: genkernel isn't abandoned at all, but I believe they expect the arch teams to maintain the kernel configs.
17:02 <@ssuominen> ok, CCing amd64@ then to take responsibility...

-> CC amd64@g.o
Comment 4 Sergey Popov gentoo-dev 2014-05-13 12:15:38 UTC
This is NOT critical issue, this option is completely deprecated on new kernels(i set up this to empty string around 3.0 and did not have problems from that time). Non-existant binaries, that you may try to put there will be simple ignored.

However, for the sake of consistency, this should be empty string in default config too.
Comment 5 Sergey Popov gentoo-dev 2014-05-13 12:26:08 UTC
Created attachment 376820 [details, diff]
patch for genkernel

<amd64 hat>
Dear genkernel maintaners, please apply attached patch to genkernel repo, thanks in advance.
</amd64 hat>
Comment 6 Samuli Suominen (RETIRED) gentoo-dev 2014-05-13 12:54:40 UTC
(In reply to Sergey Popov from comment #4)
> This is NOT critical issue, this option is completely deprecated on new
> kernels(i set up this to empty string around 3.0 and did not have problems
> from that time). Non-existant binaries, that you may try to put there will
> be simple ignored.
> 
> However, for the sake of consistency, this should be empty string in default
> config too.

I have this option in 3.13.4 kernel, and current stable is 3.12, so the issue is still critical
Comment 7 Mike Gilbert gentoo-dev 2014-05-13 14:11:20 UTC
lvm2 has some issues that are triggered when the uevent helper is not found or exits with non-zero status. I don't have the bug(s) handy.

I'm not sure I would call this "critical", but it is more than minor.
Comment 8 Pacho Ramos gentoo-dev 2014-05-15 07:17:46 UTC
(In reply to Sergey Popov from comment #5)
> Created attachment 376820 [details, diff] [details, diff]
> patch for genkernel
> 
> <amd64 hat>
> Dear genkernel maintaners, please apply attached patch to genkernel repo,
> thanks in advance.
> </amd64 hat>

What about applying it in our ebuilds until it lands genkernel repo?
Comment 9 Sergey Popov gentoo-dev 2014-05-15 08:39:05 UTC
(In reply to Samuli Suominen from comment #6)
> (In reply to Sergey Popov from comment #4)
> > This is NOT critical issue, this option is completely deprecated on new
> > kernels(i set up this to empty string around 3.0 and did not have problems
> > from that time). Non-existant binaries, that you may try to put there will
> > be simple ignored.
> > 
> > However, for the sake of consistency, this should be empty string in default
> > config too.
> 
> I have this option in 3.13.4 kernel, and current stable is 3.12, so the
> issue is still critical

You do not get my point. Incorrect set of this option leading to no damage at all. No unbootable systems. No missing udev events. So, i do not understand why this is marked critical again.

But, as i said earlier, no doubt that it should be fixed
Comment 10 Sergey Popov gentoo-dev 2014-05-15 08:40:31 UTC
(In reply to Mike Gilbert from comment #7)
> lvm2 has some issues that are triggered when the uevent helper is not found
> or exits with non-zero status. I don't have the bug(s) handy.
> 
> I'm not sure I would call this "critical", but it is more than minor.

Ahhh... LVM stuff. Yeah, that maybe a culprit if non-udev device detection is not enabled. So yeah, it can be called critical, i was wrong.
Comment 11 Samuli Suominen (RETIRED) gentoo-dev 2014-05-30 15:11:18 UTC
the incorrect setting breaks lvm2 as pointed out, here are the links,

https://www.redhat.com/archives/linux-lvm/2012-August/msg00025.html
https://bugs.gentoo.org/show_bug.cgi?id=479756
Comment 12 Alexander Tsoy 2014-05-30 15:41:56 UTC
See also: bug 493874
Comment 13 Samuli Suominen (RETIRED) gentoo-dev 2014-05-30 16:18:07 UTC
found more broken arches,

genkernel-3.4.49.1/arch/x86/kernel-config:CONFIG_UEVENT_HELPER_PATH="/sbin/hotplug"
genkernel-3.4.49.1/arch/alpha/kernel-config:CONFIG_UEVENT_HELPER_PATH="/sbin/hotplug"
genkernel-3.4.49.1/defaults/kernel-config:CONFIG_UEVENT_HELPER_PATH="/sbin/hotplug"
Comment 14 Rick Farina (Zero_Chaos) gentoo-dev 2014-05-30 16:50:27 UTC
If no one is maintaining the configs, perhaps we should simply make defconfig or the releng configs default?  Not that the releng configs are any more up to date mind you, but at least that's one set of gentoo specific  kernel configs to neglect instead of two...
Comment 15 Jeff (JD) Horelick (RETIRED) gentoo-dev 2014-05-30 19:12:39 UTC
(In reply to Rick Farina (Zero_Chaos) from comment #14)
> If no one is maintaining the configs, perhaps we should simply make
> defconfig or the releng configs default?  Not that the releng configs are
> any more up to date mind you, but at least that's one set of gentoo specific
> kernel configs to neglect instead of two...

Speaking for the x86 team, we'd support either making the releng configs default or making `make allmodconfig` the default.

Personally, I feel a lot of people need genkernel to just work and would either spend the extra time compiling to have it "just work" or, if they couldn't afford the compile time, they'd be well-versed enough to know about genkernel --menuconfig to disable the stuff they don't need.
Comment 16 Jorge Manuel B. S. Vicetto (RETIRED) Gentoo Infrastructure gentoo-dev 2014-05-31 12:45:30 UTC
(In reply to Samuli Suominen from comment #11)
> the incorrect setting breaks lvm2 as pointed out, here are the links,
> 
> https://www.redhat.com/archives/linux-lvm/2012-August/msg00025.html
> https://bugs.gentoo.org/show_bug.cgi?id=479756

Yes, this broke infra boxes. We had to update the kernel config to clear that setting.

(In reply to Rick Farina (Zero_Chaos) from comment #14)
> If no one is maintaining the configs, perhaps we should simply make
> defconfig or the releng configs default?  Not that the releng configs are
> any more up to date mind you, but at least that's one set of gentoo specific
> kernel configs to neglect instead of two...

The releng kernel configs for amd64 / x86 have been kept by me. I've been doing updates every 6 months / 1 year - so they're not that "old".
In any case, someone needs to step up and maintain the configs for every affected arch.
Comment 17 Samuli Suominen (RETIRED) gentoo-dev 2014-05-31 15:24:08 UTC
Sorry, I don't want to be a drama queen (as someone said), but I really would rather avoid getting pulled in by force to the genkernel development :-(
Comment 18 Pacho Ramos gentoo-dev 2014-05-31 15:39:18 UTC
(In reply to Samuli Suominen from comment #17)
> Sorry, I don't want to be a drama queen (as someone said), but I really
> would rather avoid getting pulled in by force to the genkernel development
> :-(

Why don't we simply apply the patch from comment #5 to the ebuild? Also, why don't we move from genkernel to genkernel-next? :/
Comment 19 Robin Johnson archtester Gentoo Infrastructure gentoo-dev Security 2014-05-31 19:59:23 UTC
InCVS