From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759980AbXGaMI3 (ORCPT ); Tue, 31 Jul 2007 08:08:29 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1759459AbXGaMIN (ORCPT ); Tue, 31 Jul 2007 08:08:13 -0400 Received: from mx12.go2.pl ([193.17.41.142]:34402 "EHLO poczta.o2.pl" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1759502AbXGaMIL (ORCPT ); Tue, 31 Jul 2007 08:08:11 -0400 Date: Tue, 31 Jul 2007 14:17:35 +0200 From: Jarek Poplawski To: Gabriel C Cc: Andrew Morton , linux-kernel@vger.kernel.org, netdev@vger.kernel.org, jason.wessel@windriver.com, amitkale@linsyssoft.com Subject: Re: NETPOLL=y , NETDEVICES=n compile error ( Re: 2.6.23-rc1-mm1 ) Message-ID: <20070731121735.GA1046@ff.dom.local> References: <20070731083210.GA1797@ff.dom.local> <46AF0B8C.7070006@googlemail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <46AF0B8C.7070006@googlemail.com> User-Agent: Mutt/1.4.2.2i Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Jul 31, 2007 at 12:14:36PM +0200, Gabriel C wrote: > Jarek Poplawski wrote: > > On 28-07-2007 20:42, Gabriel C wrote: > >> Andrew Morton wrote: > >>> On Sat, 28 Jul 2007 17:44:45 +0200 Gabriel C wrote: > >>> > >>>> Hi, > >>>> > >>>> I got this compile error with a randconfig ( http://194.231.229.228/MM/randconfig-auto-82.broken.netpoll.c ). > >>>> > >>>> ... > >>>> > >>>> net/core/netpoll.c: In function 'netpoll_poll': > >>>> net/core/netpoll.c:155: error: 'struct net_device' has no member named 'poll_controller' > >>>> net/core/netpoll.c:159: error: 'struct net_device' has no member named 'poll_controller' > >>>> net/core/netpoll.c: In function 'netpoll_setup': > >>>> net/core/netpoll.c:670: error: 'struct net_device' has no member named 'poll_controller' > >>>> make[2]: *** [net/core/netpoll.o] Error 1 > >>>> make[1]: *** [net/core] Error 2 > >>>> make: *** [net] Error 2 > >>>> make: *** Waiting for unfinished jobs.... > >>>> > >>>> ... > >>>> > >>>> > >>>> I think is because KGDBOE selects just NETPOLL. > >>>> > >>> Looks like it. > >>> > >>> Select went and selected NETPOLL and NETPOLL_TRAP but things like > >>> CONFIG_NETDEVICES and CONFIG_NET_POLL_CONTROLLER remain unset. `select' > >>> remains evil. > > ... > >> I think there may be a logical issue ( again if I got it right ). > >> We need some ethernet card to work with kgdboe right ? but we don't have any if !NETDEVICES && !NET_ETHERNET. > >> > >> So maybe some ' depends on ... && NETDEVICES!=n && NET_ETHERNET!=n ' is needed too ? > > > > IMHO, the only logical issue here is netpoll.c mustn't use > > CONFIG_NET_POLL_CONTROLLER code without #ifdef if it doesn't > > add this dependency itself. > > > > Well it does if NETDEVICES && if NET_ETHERNET which booth are N when !NETDEVICES is why KGDBOE uses select and not depends on. "does if XXX" means may "use if XXX". > Now KGDBOE just selects NETPOLL and NETPOLL_TRAP. > Adding 'select CONFIG_NET_POLL_CONTROLLER' let kgdboe compiles but the question is does it work without any ethernet card ? Why kgdboe should care what netpoll needs? So, I hope, you are adding this select under config NETPOLL. On the other hand, if NETPOLL should depend on NET_POLL_CONTROLLER there is probably no reason to have them both. The "does it work" question isn't logical issue, so it's irrelevant here... Jarek P.