Re: linux-next: scsi tree build failure

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



cc: linux-scsi added

On Thu, 2009-04-02 at 16:49 +1100, Stephen Rothwell wrote:
> Hi James,
> 
> Today's linux-next build (powerpc allyesconfig) failed like this:
> 
> drivers/scsi/fcoe/fcoe.c:982: error: __ksymtab_fcoe_fc_crc causes a section type conflict
> drivers/scsi/fcoe/fcoe.c:982: error: __ksymtab_fcoe_fc_crc causes a section type conflict
> 
> Reverting commit 0c3cb56ec3321b23e9826cccf822470717be6e08 ("[SCSI] fcoe:
> cleans up libfcoe.h and adds fcoe.h for fcoe module") makes the error go
> away.  I have no idea why - the only clue is that the only global
> declaration of fcoe_fc_crc() is removed by that commit.
> 
> Using a different compiler version (4.1.2) also fixed the problem.

Yes, that would be why I didn't see the issue.

> I also had to revert commits 60390544fcc212e0657cf1eea639691d09651943
> ("[SCSI] fcoe, libfc: fix double fcoe_softc memory alloc") and
> ba070644bff4390c965e99b1997f2cbec4157b46 ("[SCSI] fcoe, libfcoe: Add
> support for FIP. FCoE discovery and keep-alive") since they followed the
> one above.
> 
> I wonder if that function even needs to be exported as it is only used
> withing the one file.

I think it's because the above commit removes the external declaration
of fcoe_fc_crc ... so now we have an export symbol with no header
defining it.  I think the correct patch is just not to export the
symbol.

James

---

diff --git a/drivers/scsi/fcoe/fcoe.c b/drivers/scsi/fcoe/fcoe.c
index 5cdda6e..94e1e31 100644
--- a/drivers/scsi/fcoe/fcoe.c
+++ b/drivers/scsi/fcoe/fcoe.c
@@ -979,7 +979,6 @@ u32 fcoe_fc_crc(struct fc_frame *fp)
 	}
 	return crc;
 }
-EXPORT_SYMBOL_GPL(fcoe_fc_crc);
 
 /**
  * fcoe_xmit() - FCoE frame transmit function


--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]
  Powered by Linux