mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH] CONFIG_NET=n - lots of link time errors
@ 2007-07-21 16:27 Jan Engelhardt
  2007-07-22  2:12 ` David Miller
  0 siblings, 1 reply; 3+ messages in thread
From: Jan Engelhardt @ 2007-07-21 16:27 UTC (permalink / raw)
  To: netdev; +Cc: Linux Kernel Mailing List, Andrew Morton

[-- Attachment #1: Type: TEXT/PLAIN, Size: 1430 bytes --]

Hi,


enable everything from Drivers > Networking, but deselect CONFIG_NET. 
This throws a ton of linking errors (when using CONFIG_MODULES=n), and 
probably unresolved symbols (MODULES=m). Happens in current -git, but I 
believe it dates much farther back, because this is imo just a simple 
Kconfig issue. I think we need to hand out a big 'if NET'-endif block in 
drivers/net/Kconfig. The full error log is like 120 K in size, and I 
doubt it would make sense to post it, given instructions above. I'll 
attach my (i386) .config for anyone to see it for themselves, and a 
patch proposal right below. Is it ok to add 'depends on NET' on 
NETDEVICES, or are there some network devices that can live without 
CONFIG_NET?

Thanks,
	Jan
===

Enabling drivers from "Devices > Networking" (in menuconfig), for 
example SLIP and/or PLIP, throws link time errors when CONFIG_NET itself 
is =n. Have CONFIG_NETDEVICES depend on CONFIG_NET.

Signed-off-by: Jan Engelhardt <jengelh@gmx.de>

---
 drivers/net/Kconfig |    1 +
 1 file changed, 1 insertion(+)

Index: linux-2.6.23/drivers/net/Kconfig
===================================================================
--- linux-2.6.23.orig/drivers/net/Kconfig
+++ linux-2.6.23/drivers/net/Kconfig
@@ -5,6 +5,7 @@
 
 menuconfig NETDEVICES
 	default y if UML
+	depends on NET
 	bool "Network device support"
 	---help---
 	  You can say N here if you don't intend to connect your Linux box to

[-- Attachment #2: Type: APPLICATION/X-CONFIG, Size: 19173 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] CONFIG_NET=n - lots of link time errors
  2007-07-21 16:27 [PATCH] CONFIG_NET=n - lots of link time errors Jan Engelhardt
@ 2007-07-22  2:12 ` David Miller
  2007-07-22 10:10   ` Jan Engelhardt
  0 siblings, 1 reply; 3+ messages in thread
From: David Miller @ 2007-07-22  2:12 UTC (permalink / raw)
  To: jengelh; +Cc: netdev, linux-kernel, akpm

From: Jan Engelhardt <jengelh@computergmbh.de>
Date: Sat, 21 Jul 2007 18:27:38 +0200 (CEST)

> Enabling drivers from "Devices > Networking" (in menuconfig), for 
> example SLIP and/or PLIP, throws link time errors when CONFIG_NET itself 
> is =n. Have CONFIG_NETDEVICES depend on CONFIG_NET.
> 
> Signed-off-by: Jan Engelhardt <jengelh@gmx.de>

This is the second time I've seen this change in the past few
days, and people seem to hit it quite readily with randconfig.

It seems reasonable and I'll apply it, thanks Jan.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] CONFIG_NET=n - lots of link time errors
  2007-07-22  2:12 ` David Miller
@ 2007-07-22 10:10   ` Jan Engelhardt
  0 siblings, 0 replies; 3+ messages in thread
From: Jan Engelhardt @ 2007-07-22 10:10 UTC (permalink / raw)
  To: David Miller; +Cc: netdev, linux-kernel, akpm


On Jul 21 2007 19:12, David Miller wrote:
>> Enabling drivers from "Devices > Networking" (in menuconfig), for 
>> example SLIP and/or PLIP, throws link time errors when CONFIG_NET itself 
>> is =n. Have CONFIG_NETDEVICES depend on CONFIG_NET.
>> 
>> Signed-off-by: Jan Engelhardt <jengelh@gmx.de>
>
>This is the second time I've seen this change in the past few
>days, and people seem to hit it quite readily with randconfig.
>
>It seems reasonable and I'll apply it, thanks Jan.

While randconfig is nice and good, this error actually came by manually
tuning the .config for a specific goal (namely: make a small kernel for an
ancient i386 without modifying any code). Turning off CONFIG_NET
was then just an idea away, because I knew that TCP was quite big.


	Jan
-- 

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2007-07-22 10:10 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-07-21 16:27 [PATCH] CONFIG_NET=n - lots of link time errors Jan Engelhardt
2007-07-22  2:12 ` David Miller
2007-07-22 10:10   ` Jan Engelhardt

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®