2006-03-23 16:38:59

by Russell King

[permalink] [raw]
Subject: 2.6.16-git6: build failure: ksysfs.c (h7201_defconfig)

Building h7201_defconfig on ARM provokes these build errors:

LD .tmp_vmlinux1
kernel/built-in.o: In function `uevent_seqnum_show':
ksysfs.c:(.text+0x1f258): undefined reference to `uevent_seqnum'
kernel/built-in.o: In function `uevent_helper_show':
ksysfs.c:(.text+0x1f280): undefined reference to `uevent_helper'
kernel/built-in.o: In function `uevent_helper_store':
ksysfs.c:(.text+0x1f2e0): undefined reference to `uevent_helper'
kernel/built-in.o:(.data+0xd1c): undefined reference to `uevent_helper'
make: *** [.tmp_vmlinux1] Error 1
make: Leaving directory `/var/tmp/kernel-orig'

--
Russell King
Linux kernel 2.6 ARM Linux - http://www.arm.linux.org.uk/
maintainer of: 2.6 Serial core


2006-03-23 16:51:25

by Greg KH

[permalink] [raw]
Subject: Re: 2.6.16-git6: build failure: ksysfs.c (h7201_defconfig)

On Thu, Mar 23, 2006 at 04:38:52PM +0000, Russell King wrote:
> Building h7201_defconfig on ARM provokes these build errors:
>
> LD .tmp_vmlinux1
> kernel/built-in.o: In function `uevent_seqnum_show':
> ksysfs.c:(.text+0x1f258): undefined reference to `uevent_seqnum'
> kernel/built-in.o: In function `uevent_helper_show':
> ksysfs.c:(.text+0x1f280): undefined reference to `uevent_helper'
> kernel/built-in.o: In function `uevent_helper_store':
> ksysfs.c:(.text+0x1f2e0): undefined reference to `uevent_helper'
> kernel/built-in.o:(.data+0xd1c): undefined reference to `uevent_helper'
> make: *** [.tmp_vmlinux1] Error 1
> make: Leaving directory `/var/tmp/kernel-orig'

Ugh, CONFIG_NET is not set, yet CONFIG_HOTPLUG is. I was wrong with my
assumption that no one would ever need that :)

I have a patch in my queue from Andrew that fixes it up, I'll send it on
to Linus later today to fix this. Thanks for letting me know.

greg k-h

2006-03-26 00:02:14

by Russell King

[permalink] [raw]
Subject: Re: 2.6.16-git6: build failure: ksysfs.c (h7201_defconfig)

On Thu, Mar 23, 2006 at 08:51:08AM -0800, Greg KH wrote:
> On Thu, Mar 23, 2006 at 04:38:52PM +0000, Russell King wrote:
> > Building h7201_defconfig on ARM provokes these build errors:
> >
> > LD .tmp_vmlinux1
> > kernel/built-in.o: In function `uevent_seqnum_show':
> > ksysfs.c:(.text+0x1f258): undefined reference to `uevent_seqnum'
> > kernel/built-in.o: In function `uevent_helper_show':
> > ksysfs.c:(.text+0x1f280): undefined reference to `uevent_helper'
> > kernel/built-in.o: In function `uevent_helper_store':
> > ksysfs.c:(.text+0x1f2e0): undefined reference to `uevent_helper'
> > kernel/built-in.o:(.data+0xd1c): undefined reference to `uevent_helper'
> > make: *** [.tmp_vmlinux1] Error 1
> > make: Leaving directory `/var/tmp/kernel-orig'
>
> Ugh, CONFIG_NET is not set, yet CONFIG_HOTPLUG is. I was wrong with my
> assumption that no one would ever need that :)
>
> I have a patch in my queue from Andrew that fixes it up, I'll send it on
> to Linus later today to fix this. Thanks for letting me know.

2.6.16-git9 is better, but still fails on this defconfig. We now
have:

LD .tmp_vmlinux1
kernel/built-in.o:(.data+0xd1c): undefined reference to `uevent_helper'
make: *** [.tmp_vmlinux1] Error 1
make: Leaving directory `/var/tmp/kernel-orig'

which seems to be a reference from sysfs.c for ...proc.../sys/kernel/hotplug.

This brings up an interesting point - if CONFIG_NET is not set and
CONFIG_HOTPLUG is set, don't we want to call /sbin/hotplug? If
...proc.../sys/kernel/hotplug isn't present then what?

Or are we back in the business of breaking the userspace expectations
in random ways?

--
Russell King
Linux kernel 2.6 ARM Linux - http://www.arm.linux.org.uk/
maintainer of: 2.6 Serial core