lkml.org 
[lkml]   [2008]   [Jan]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    SubjectRe: [PATCH] SCSI: fix isa/pcmcia compile problem
    From
    Date

    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 <htejun@gmail.com>
    > ---
    > 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.

    James




    \
     
     \ /
      Last update: 2008-01-18 16:01    [W:4.556 / U:0.016 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site