Re: [PATCH] Fix socket.c compilation failure when CONFIG_NET=n

Jean Tourrilhes (jt@bougret.hpl.hp.com)
Thu, 9 Jan 2003 10:06:26 -0800


On Tue, Jan 07, 2003 at 03:41:07PM +0900, Miles Bader wrote:
> [I send this to Linus earlier and he ignored it; maybe you're the right
> person...]
>
> In net/socket.c, <linux/wireless.h> is included twice, once conditionally
> (on CONFIG_NET_RADIO || CONFIG_NET_PCMCIA_RADIO) and once unconditionally.
> However, including <linux/wireless.h> defines WIRELESS_EXT, and this causes an
> #ifdef in `sock_ioctl' to reference `dev_ioctl', which isn't defined when
> CONFIG_NET=n, and so results in an unresolved symbol reference in that case.
>
> The following patch fixes this by removing the unconditional include, and only
> keeping the conditional one.
>
> Thanks,
>
> -Miles
>
> diff -ruN -X../cludes linux-2.5.54-moo.orig/net/socket.c linux-2.5.54-moo/net/socket.c
> --- linux-2.5.54-moo.orig/net/socket.c 2002-11-25 10:30:11.000000000 +0900
> +++ linux-2.5.54-moo/net/socket.c 2003-01-06 13:27:17.000000000 +0900
> @@ -75,7 +75,6 @@
> #include <linux/cache.h>
> #include <linux/module.h>
> #include <linux/highmem.h>
> -#include <linux/wireless.h>
> #include <linux/divert.h>
> #include <linux/mount.h>

This was included in 2.5.55, so I guess Linus didn't ignored you.
Regards,

Jean
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/