[lkml]   [2001]   [Sep]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
Patch in this message
Subject2.2.20pre10: cannot link ISDN into kernel, netif_wake_queue

Well, I already reported something like this for 2.2.20pre9 and was told
there would be a patch fixing it going to Alan, but this doesn't seem to
be the case.

First, HiSax_setup in drivers/isdn/hisax/config.c is still declared
static, so linking the kernel fails if the HiSax driver is compiled into
the kernel and not as a module.

Second, if I remove the "static", I get the following error during link:

drivers/isdn/isdn.a: In function `isdn_ppp_ioctl':
drivers/isdn/isdn.a(.text+0x1f659): undefined reference to
make: *** [vmlinux] Error 1

Grepping through the source for uses of netif_wake_queue, I see that some
users of this function define it themselves and some don't. My guess is
that all users that don't declare it themselves are broken like the ISDN
stuff and cause the kernel link stage to fail. There is some stuff in
drivers/net and drivers/usb that has this problem.

Fix for the HiSax_setup being declared static problem is easy, patch
below, but I don't know about the netif_wake_queue problem.

I think this is a show-stopper for making 2.2.20pre10 the final 2.2.20, I
can't even link a kernel this way.

--- linux-2.2.20pre10/drivers/isdn/hisax/config.c Wed Sep 12 01:31:33 2001
+++ linux-2.2.20pre10-new/drivers/isdn/hisax/config.c Wed Sep 12 01:46:54 2001
@@ -434,7 +434,7 @@

#ifndef MODULE
-static void __init HiSax_setup(char *str, int *ints)
+void __init HiSax_setup(char *str, int *ints)
int i, j, argc;
argc = ints[0];
Ciao, Pascal

-<[, netmail 2:241/215.72, home ]>-

To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
More majordomo info at
Please read the FAQ at

 \ /
  Last update: 2005-03-22 13:03    [W:0.027 / U:9.948 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site