lkml.org 
[lkml]   [2011]   [Mar]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] netconsole: Fix strange kconfig dependency on configfs
From
Date
On Thu, 2011-03-10 at 21:44 -0500, Steven Rostedt wrote:
> Doing ktest rand config tests, I hit the following build bug:
>
> LD .tmp_vmlinux1
> drivers/built-in.o: In function `netconsole_target_get':
> /home/rostedt/work/autotest/nobackup/linux-test.git/drivers/net/netconsole.c:127: undefined reference to `config_item_get'
> drivers/built-in.o: In function `netconsole_target_put':
> /home/rostedt/work/autotest/nobackup/linux-test.git/drivers/net/netconsole.c:133: undefined reference to `config_item_put'
> drivers/built-in.o: In function `netconsole_target_get':
> /home/rostedt/work/autotest/nobackup/linux-test.git/drivers/net/netconsole.c:127: undefined reference to `config_item_get'
> drivers/built-in.o: In function `netconsole_target_put':
> /home/rostedt/work/autotest/nobackup/linux-test.git/drivers/net/netconsole.c:133: undefined reference to `config_item_put'
> drivers/built-in.o: In function `drop_netconsole_target':
> /home/rostedt/work/autotest/nobackup/linux-test.git/drivers/net/netconsole.c:634: undefined reference to `config_item_put'
> drivers/built-in.o: In function `make_netconsole_target':
> /home/rostedt/work/autotest/nobackup/linux-test.git/drivers/net/netconsole.c:607: undefined reference to `config_item_init_type_name'
> drivers/built-in.o: In function `dynamic_netconsole_init':
> /home/rostedt/work/autotest/nobackup/linux-test.git/drivers/net/netconsole.c:109: undefined reference to `config_group_init'
> /home/rostedt/work/autotest/nobackup/linux-test.git/drivers/net/netconsole.c:111: undefined reference to `configfs_register_subsystem'
> drivers/built-in.o: In function `dynamic_netconsole_exit':
> /home/rostedt/work/autotest/nobackup/linux-test.git/drivers/net/netconsole.c:116: undefined reference to `configfs_unregister_subsystem'
> make[1]: *** [.tmp_vmlinux1] Error 1
>
> The .config had the following settings:
>
> CONFIG_CONFIGFS_FS=m
> CONFIG_NETCONSOLE=y
> CONFIG_NETCONSOLE_DYNAMIC=y
>
> NETCONSOLE_DYNAMIC depends on NETCONSOLE and CONFIGFS_FS, but
> NETCONSOLE_DYNAMIC is just a boolean and not a tristate, where both
> NETCONSOLE and CONFIGFS_FS are tristates and can be built as modules.
>
> As we can see from the resulting config, NETCONSOLE_DYNAMIC can be set
> to be builtin when CONFIGFS_FS which it depends on is built as a module.
>
> The simplest solution is not to allow NETCONSOLE_DYNAMIC be enabled if
> CONFIGFS_FS is a module but NETCONSOLE is built in, as
> NETCONSOLE_DYNAMIC depends on code that is only compiled in when
> CONFIGFS_FS is set.
>
> Cc: Michal Marek <mmarek@suse.cz>
> Cc: Nicholas A. Bellinger <nab@linux-iscsi.org>
> Cc: Joel Becker <jlbec@evilplan.org>
> Cc: Randy Dunlap <randy.dunlap@oracle.com>
> CC: Stephen Rothwell <sfr@canb.auug.org.au>
> Cc: James Bottomley <James.Bottomley@suse.de>
> Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
>

Hi Steven,

This particular case was reported by Ingo and fixed back in January by
converting this to properly use 'select CONFIGFS', but it appears that
this series did not get picked up.

[PATCH RESEND 0/4] Fix kconfig breakage wrt to CONFIGFS_FS
http://marc.info/?l=linux-netdev&m=129615549321425&w=2

Would someone be so kind as to carry these..?

Thanks,

--nab

> ---
> drivers/net/Kconfig | 1 +
> 1 file changed, 1 insertion(+)
>
> Index: linux-test.git/drivers/net/Kconfig
> ===================================================================
> --- linux-test.git.orig/drivers/net/Kconfig 2011-03-10 18:19:54.644008709 -0500
> +++ linux-test.git/drivers/net/Kconfig 2011-03-10 18:21:21.320194133 -0500
> @@ -3390,6 +3390,7 @@ config NETCONSOLE
> config NETCONSOLE_DYNAMIC
> bool "Dynamic reconfiguration of logging targets"
> depends on NETCONSOLE && SYSFS && CONFIGFS_FS
> + depends on !(NETCONSOLE=y && CONFIGFS_FS=m)
> help
> This option enables the ability to dynamically reconfigure target
> parameters (interface, IP addresses, port numbers, MAC addresses)
>
>



\
 
 \ /
  Last update: 2011-03-12 03:55    [W:0.062 / U:0.220 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site