lkml.org 
[lkml]   [2008]   [Apr]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: Build failure (with insane config)

* Roland Dreier <rdreier@cisco.com> wrote:

> 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.

well, the TCP/IP checksumming functions are in lib portions on every
architecture i checked, and the exports are handled by the networking
code. And the symbol used is named ip_compute_csum(). So i think the
correct solution is to make CONFIG_VIDEO_BT848 depend on CONFIG_NET ...

especially on embedded there's an advantage in linking out unused core
kernel functions. But even if we did this change on x86, other
architectures might (correctly) complain about x86 going it alone. x86
gets the most use by far, so if we start introducing build assymetries
like this it will just mess up the build on other architectures.

Ingo


\
 
 \ /
  Last update: 2008-04-28 10:11    [W:0.048 / U:0.036 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site