Re: netconsole build breakage (Re: [GIT] Networking)

* Nicholas A. Bellinger [off-list ref] wrote:

On Wed, 2011-01-19 at 18:08 +0800, Américo Wang wrote:

quoted

On Wed, Jan 19, 2011 at 10:59:20AM +0100, Ingo Molnar wrote:

quoted

FYI, there's a .38-rc1 build failure that triggers rather often:

drivers/built-in.o: In function `drop_netconsole_target':
netconsole.c:(.text+0x130146): undefined reference to `config_item_put'
drivers/built-in.o: In function `write_msg':
netconsole.c:(.text+0x1301aa): undefined reference to `config_item_get'
netconsole.c:(.text+0x130217): undefined reference to `config_item_put'
drivers/built-in.o: In function `netconsole_netdev_event':
netconsole.c:(.text+0x1302ab): undefined reference to `config_item_get'
...

Triggered by this configuration:

CONFIG_CONFIGFS_FS=m
CONFIG_NETCONSOLE=y

Should be "depends on CONFIGFS_FS=y".

Sorry for breaking this one folks..

Where this was left yesterday was to change NETCONSOLE_DYNAMIC, DLM and
OCFS2_FS symbols to use 'select configfs' instead of 'depends on SYSFS
&& CONFIGFS':

http://marc.info/?l=linux-kernel&m=129539400709508&w=2

but unfortuately this did not make it into .38-rc1 in time..

Using 'select CONFIGFS_FS' here for NETCONSOLE_DYNAMIC with the
following patches should do the trick.

   git://git.kernel.org/pub/scm/linux/kernel/git/nab/scsi-post-merge-2.6.git for-linus

Thanks,

Nicholas Bellinger (3):
  net: Make NETCONSOLE_DYNAMIC use select CONFIGFS_FS
  dlm: Make DLM use select CONFIGFS_FS
  ocfs2: Make OCFS2_FS use select CONFIGFS_FS

 drivers/net/Kconfig |    3 ++-
 fs/dlm/Kconfig      |    4 ++--
 fs/ocfs2/Kconfig    |    3 ++-
 3 files changed, 6 insertions(+), 4 deletions(-)

Ping? This is still broken in Linus's tree as of today ... simple builds like 
allmodconfig still fail.

Thanks,

	Ingo