From mboxrd@z Thu Jan 1 00:00:00 1970 From: Randy Dunlap Subject: Re: [PATCH] scsi: fix bnx2i build errors Date: Wed, 05 May 2010 08:48:18 -0700 Message-ID: <4BE19342.7010507@oracle.com> References: <20100504103001.33fcbe88.randy.dunlap@oracle.com> <1273067699.5227.56.camel@mulgrave.site> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Return-path: Received: from rcsinet10.oracle.com ([148.87.113.121]:61846 "EHLO rcsinet10.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1758810Ab0EEPtL (ORCPT ); Wed, 5 May 2010 11:49:11 -0400 In-Reply-To: <1273067699.5227.56.camel@mulgrave.site> Sender: linux-scsi-owner@vger.kernel.org List-Id: linux-scsi@vger.kernel.org To: James Bottomley Cc: scsi , Anil Veerabhadrappa , akpm On 05/05/10 06:54, James Bottomley wrote: > On Tue, 2010-05-04 at 10:30 -0700, Randy Dunlap wrote: >> From: Randy Dunlap >> >> bnx2i should depend on NET since it selects SCSI_ISCSI_ATTRS, >> which depends on NET. >> >> Also move the dependencies together. >> >> The "depends" change fixes multiple build errors when CONFIG_NET is >> not enabled: >> ERROR: "skb_trim" [drivers/scsi/scsi_transport_iscsi.ko] undefined! >> ERROR: "netlink_kernel_create" [drivers/scsi/scsi_transport_iscsi.ko] undefined! >> ERROR: "netlink_kernel_release" [drivers/scsi/scsi_transport_iscsi.ko] undefined! >> ERROR: "skb_pull" [drivers/scsi/scsi_transport_iscsi.ko] undefined! >> ERROR: "init_net" [drivers/scsi/scsi_transport_iscsi.ko] undefined! >> ERROR: "__alloc_skb" [drivers/scsi/scsi_transport_iscsi.ko] undefined! >> ERROR: "netlink_broadcast" [drivers/scsi/scsi_transport_iscsi.ko] undefined! >> ERROR: "kfree_skb" [drivers/scsi/scsi_transport_iscsi.ko] undefined! >> ERROR: "skb_put" [drivers/scsi/scsi_transport_iscsi.ko] undefined! >> >> Signed-off-by: Randy Dunlap >> Cc: Anil Veerabhadrappa >> --- >> drivers/scsi/bnx2i/Kconfig | 3 ++- >> 1 file changed, 2 insertions(+), 1 deletion(-) >> >> --- lnx-2634-rc6.orig/drivers/scsi/bnx2i/Kconfig >> +++ lnx-2634-rc6/drivers/scsi/bnx2i/Kconfig >> @@ -1,10 +1,11 @@ >> config SCSI_BNX2_ISCSI >> tristate "Broadcom NetXtreme II iSCSI support" >> + depends on NET >> + depends on PCI > > Actually, let's not fix it this way. While we still have this annoying > bug in the Kconfig system where it doesn't respect the dependencies of a > select, let's not build traps like this ... the rule has to be selected > attributes must select their dependencies, that way the selection just > works(tm). > > Of course, fixing Kconfig would be the better option ... > > James > > --- > > diff --git a/drivers/scsi/Kconfig b/drivers/scsi/Kconfig > index 75f2336..804ba3d 100644 > --- a/drivers/scsi/Kconfig > +++ b/drivers/scsi/Kconfig > @@ -308,7 +308,8 @@ config SCSI_FC_TGT_ATTRS > > config SCSI_ISCSI_ATTRS > tristate "iSCSI Transport Attributes" > - depends on SCSI && NET > + depends on SCSI > + select NET > help > If you wish to export transport-specific information about > each attached iSCSI device to sysfs, say Y. > > I would Nak that but it would probably do no good. Selecting NET just adds way too much code to be hidden like this. Yes, of course, kconfig should be fixed... -- ~Randy *** Remember to use Documentation/SubmitChecklist when testing your code ***