Messages in this thread | | | From | Roland Dreier <> | Subject | Build failure (with insane config) | Date | Sun, 27 Apr 2008 20:34:12 -0700 |
| |
For x86-64, arch/x86/lib/Makefile has:
lib-y += csum-partial_64.o csum-copy_64.o csum-wrappers_64.o
so in particular ip_compute_csum() (from csum-partial_64.c) gets put in libs, so even though it is exported, it gets thrown away at link time if there are no non-modular users.
However, if someone has an insane config with CONFIG_NET=n and CONFIG_VIDEO_BT848=m, then ip_compute_csum() gets discarded but the modular code in drivers/media/video/bt8xx/bttv-cards.c calls it and...
ERROR: "ip_compute_csum" [drivers/media/video/bt8xx/bttv.ko] undefined!
I guess one fix is just to put at least csum-partial_64.c in obj-y instead; it does seem broken to have files with EXPORT_SYMBOL() be lib-y, since it is really an invitation for this problem. Other ideas would be for bttv to depend on or select CONFIG_NET.
[It's a long story why I noticed this problem, but maybe we want to fix it to save people doing randconfig test builds or something like that. Or maybe we don't really care, since the class of real systems with video capture cards but no networking is probably quite small]
- R.
| |