Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1762249AbYARX2H (ORCPT ); Fri, 18 Jan 2008 18:28:07 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1758939AbYARX1w (ORCPT ); Fri, 18 Jan 2008 18:27:52 -0500 Received: from rv-out-0910.google.com ([209.85.198.186]:46062 "EHLO rv-out-0910.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1758094AbYARX1v (ORCPT ); Fri, 18 Jan 2008 18:27:51 -0500 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=message-id:date:from:user-agent:mime-version:to:cc:subject:references:in-reply-to:x-enigmail-version:content-type:content-transfer-encoding; b=Fm+J2kuX/48HVQPQs2BoliEOFeuvg0lDXDkpDEmu2WOuNeZHDUBaGlC8iCN16PFXQ2dkXr9hWDdZXtQXv/+HaswPon0eGjpnMwifBoKq4Qk2pLQMDtmZOfvYWXEPcW+7qsIdgq7gMflfthlCkeeGGnXRhPERw0bH66bPq2/zjLI= Message-ID: <479135EE.2090009@gmail.com> Date: Sat, 19 Jan 2008 08:27:42 +0900 From: Tejun Heo User-Agent: Thunderbird 2.0.0.9 (X11/20070801) MIME-Version: 1.0 To: James Bottomley CC: Kamalesh Babulal , Andrew Morton , linux-kernel@vger.kernel.org, linux-scsi@vger.kernel.org, fischer@norbit.de, Andy Whitcroft , Balbir Singh , Samuel Ortiz Subject: Re: [PATCH] SCSI: fix isa/pcmcia compile problem References: <20080117023514.9df393cf.akpm@linux-foundation.org> <478F7F2B.9000801@linux.vnet.ibm.com> <20080117111104.3baa878e.akpm@linux-foundation.org> <47904927.1040000@linux.vnet.ibm.com> <47905348.1000709@gmail.com> <1200668315.3111.3.camel@localhost.localdomain> In-Reply-To: <1200668315.3111.3.camel@localhost.localdomain> X-Enigmail-Version: 0.95.5 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2037 Lines: 46 James Bottomley wrote: > On Fri, 2008-01-18 at 16:20 +0900, Tejun Heo wrote: >> aha152x.c and fdomain are built twice - once for the isa driver and >> once for the PCMCIA one. Through #ifdefs, the compiled codes are >> slightly different; thus, global symbols need to be given different >> names depending on which flavor is being built. This patch adds >> GLOBAL() macro to aha152x.h and fdomain.h which change the symbol >> depending on PCMCIA. >> >> This bug has always existed but has been masked by the fact the >> drivers/scsi/pcmcia used subdir-(y|m) instead of obj-(y|m) which made >> drivers/scsi/pcmcia/built_in.o not linked into the kernel and thus >> avoided the duplicate symbols during compilation. >> >> Signed-off-by: Tejun Heo >> --- >> Ah... missed that one. Here's the updated version. > > Actually, isn't the better fix just to return to the original behaviour? > > As you pointed out, using the subdir instead of obj meant that although > the modules were built, the drivers were never linked into the main > kernel. According to the records, this has been the default forever, so > there can be no-one anywhere relying on these drivers being built in. > Actually, as old style pcmcia drivers, I'm not sure there's much value > building them into the kernel anyway. > > So just modify scsi/pcmcia/Kconfig to make them all depend on m. Yeap, there is no problem if you don't allow them to be linked into the kernel. If that's how you want it, please go ahead. I personally think it's a bit odd to disallow building into kernel because of the peculiarity of the implementation (including c files and compiling them slightly differently) and also no one reporting doesn't necessarily mean no one has attempted it and failed. Thanks. -- tejun -- 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/