From owner-freebsd-current Wed Aug 30 16:24:28 1995 Return-Path: current-owner Received: (from majordom@localhost) by freefall.FreeBSD.org (8.6.11/8.6.6) id QAA08324 for current-outgoing; Wed, 30 Aug 1995 16:24:28 -0700 Received: from godzilla.zeta.org.au (godzilla.zeta.org.au [203.2.228.34]) by freefall.FreeBSD.org (8.6.11/8.6.6) with ESMTP id QAA08312 for ; Wed, 30 Aug 1995 16:24:22 -0700 Received: (from bde@localhost) by godzilla.zeta.org.au (8.6.9/8.6.9) id JAA27135; Thu, 31 Aug 1995 09:21:47 +1000 Date: Thu, 31 Aug 1995 09:21:47 +1000 From: Bruce Evans Message-Id: <199508302321.JAA27135@godzilla.zeta.org.au> To: freebsd-current@FreeBSD.org, jhay@mikom.csir.co.za Subject: Re: pseudo device lkm's broken Sender: current-owner@FreeBSD.org Precedence: bulk >It looks like the pseudo device lkm's are broken. (lkm/if_disc, if_ppp etc) >I think it has something to do with the changes in sys/kernel.h. >Here is the error message for lkm/if_disc: >angel:/usr/src/lkm/if_disc # make >cc -O -I. -DINET -DKERNEL -I/usr/src/lkm/if_disc/../../sys -W -Wcomment >-Wredundant-decls -DPSEUDO_LKM -c /usr/src/lkm/if_disc/../../sys/net/if_disc.c >/usr/src/lkm/if_disc/../../sys/net/if_disc.c: In function `if_disc_load': >/usr/src/lkm/if_disc/../../sys/net/if_disc.c:84: too few arguments to >function `discattach' >*** Error code 1 I added the arg to discattach() and several other functions because sysinit functions are supposed to have one and there were type errors in the kernel, at least when everything is prototyped. lkms are compiled with -DPSEUDO_LKM which now apparently gives a PSEUDO_SET() set macro that is incompatible with the kernel one. This can probably be fixed by adding a dummy ((caddr_t)NULL) arg to init() in the lkm PSEUDO_SET(). The correct fix is probably to get rid of the deprecated kernel PSEUDO_SET() and remove all the dummy args. Even for boot-time sysinits, the args are almost always dummies. Bruce