* linux-next: scsi tree build failure
@ 2008-12-15 14:37 Stephen Rothwell
2008-12-15 14:52 ` James Bottomley
0 siblings, 1 reply; 37+ messages in thread
From: Stephen Rothwell @ 2008-12-15 14:37 UTC (permalink / raw)
To: James Bottomley; +Cc: linux-next, Robert Love
[-- Attachment #1: Type: text/plain, Size: 663 bytes --]
Hi James,
Today's linux-next build (powerpc allyesconfig) failed like this:
drivers/scsi/libfc/fc_fcp.c: In function 'fc_eh_host_reset':
drivers/scsi/libfc/fc_fcp.c:1987: error: implicit declaration of function 'msleep'
That file needs to include <linux/delay.h>.
Introduced by commit 389571f347629d879f9044582bbbafb43e4fe00b ("[SCSI]
libfc: A modular Fibre Channel library") from the scsi tree. I reverted
that commit and commit 7aba33c5cfc7a9c104cc7469558c7849f373c3da ("[SCSI]
fcoe: Fibre Channel over Ethernet") which depended on it.
--
Cheers,
Stephen Rothwell sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/
[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]
^ permalink raw reply [flat|nested] 37+ messages in thread
* Re: linux-next: scsi tree build failure
2008-12-15 14:37 linux-next: scsi tree build failure Stephen Rothwell
@ 2008-12-15 14:52 ` James Bottomley
2008-12-17 12:20 ` Stephen Rothwell
0 siblings, 1 reply; 37+ messages in thread
From: James Bottomley @ 2008-12-15 14:52 UTC (permalink / raw)
To: Stephen Rothwell; +Cc: linux-next, Robert Love
On Tue, 2008-12-16 at 01:37 +1100, Stephen Rothwell wrote:
> Hi James,
>
> Today's linux-next build (powerpc allyesconfig) failed like this:
Heh, I really need to get this terrasoft system integrated into my
builds
> drivers/scsi/libfc/fc_fcp.c: In function 'fc_eh_host_reset':
> drivers/scsi/libfc/fc_fcp.c:1987: error: implicit declaration of function 'msleep'
>
> That file needs to include <linux/delay.h>.
Thanks, that's exactly the problem.
Robert, please reroll the patches with this fix in ... I'll just
exchange the new patches for the ones in my tree.
> Introduced by commit 389571f347629d879f9044582bbbafb43e4fe00b ("[SCSI]
> libfc: A modular Fibre Channel library") from the scsi tree. I reverted
> that commit and commit 7aba33c5cfc7a9c104cc7469558c7849f373c3da ("[SCSI]
> fcoe: Fibre Channel over Ethernet") which depended on it.
James
^ permalink raw reply [flat|nested] 37+ messages in thread
* Re: linux-next: scsi tree build failure
2008-12-15 14:52 ` James Bottomley
@ 2008-12-17 12:20 ` Stephen Rothwell
2008-12-21 3:30 ` Stephen Rothwell
0 siblings, 1 reply; 37+ messages in thread
From: Stephen Rothwell @ 2008-12-17 12:20 UTC (permalink / raw)
To: James Bottomley; +Cc: linux-next, Robert Love
[-- Attachment #1: Type: text/plain, Size: 1183 bytes --]
Hi James,
On Mon, 15 Dec 2008 08:52:42 -0600 James Bottomley <James.Bottomley@HansenPartnership.com> wrote:
>
> On Tue, 2008-12-16 at 01:37 +1100, Stephen Rothwell wrote:
> >
> > Today's linux-next build (powerpc allyesconfig) failed like this:
>
> Heh, I really need to get this terrasoft system integrated into my
> builds
>
> > drivers/scsi/libfc/fc_fcp.c: In function 'fc_eh_host_reset':
> > drivers/scsi/libfc/fc_fcp.c:1987: error: implicit declaration of function 'msleep'
> >
> > That file needs to include <linux/delay.h>.
>
> Thanks, that's exactly the problem.
>
> Robert, please reroll the patches with this fix in ... I'll just
> exchange the new patches for the ones in my tree.
>
> > Introduced by commit 389571f347629d879f9044582bbbafb43e4fe00b ("[SCSI]
> > libfc: A modular Fibre Channel library") from the scsi tree. I reverted
> > that commit and commit 7aba33c5cfc7a9c104cc7469558c7849f373c3da ("[SCSI]
> > fcoe: Fibre Channel over Ethernet") which depended on it.
I have dropped the scsi tree until this is fixed up.
--
Cheers,
Stephen Rothwell sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/
[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]
^ permalink raw reply [flat|nested] 37+ messages in thread
* Re: linux-next: scsi tree build failure
2008-12-17 12:20 ` Stephen Rothwell
@ 2008-12-21 3:30 ` Stephen Rothwell
2008-12-22 1:59 ` James Bottomley
0 siblings, 1 reply; 37+ messages in thread
From: Stephen Rothwell @ 2008-12-21 3:30 UTC (permalink / raw)
To: James Bottomley; +Cc: linux-next, Robert Love
[-- Attachment #1: Type: text/plain, Size: 1369 bytes --]
Hi James, Robert,
On Wed, 17 Dec 2008 23:20:49 +1100 Stephen Rothwell <sfr@canb.auug.org.au> wrote:
>
> On Mon, 15 Dec 2008 08:52:42 -0600 James Bottomley <James.Bottomley@HansenPartnership.com> wrote:
> >
> > On Tue, 2008-12-16 at 01:37 +1100, Stephen Rothwell wrote:
> > >
> > > Today's linux-next build (powerpc allyesconfig) failed like this:
> >
> > Heh, I really need to get this terrasoft system integrated into my
> > builds
> >
> > > drivers/scsi/libfc/fc_fcp.c: In function 'fc_eh_host_reset':
> > > drivers/scsi/libfc/fc_fcp.c:1987: error: implicit declaration of function 'msleep'
> > >
> > > That file needs to include <linux/delay.h>.
> >
> > Thanks, that's exactly the problem.
> >
> > Robert, please reroll the patches with this fix in ... I'll just
> > exchange the new patches for the ones in my tree.
> >
> > > Introduced by commit 389571f347629d879f9044582bbbafb43e4fe00b ("[SCSI]
> > > libfc: A modular Fibre Channel library") from the scsi tree. I reverted
> > > that commit and commit 7aba33c5cfc7a9c104cc7469558c7849f373c3da ("[SCSI]
> > > fcoe: Fibre Channel over Ethernet") which depended on it.
>
> I have dropped the scsi tree until this is fixed up.
I still haven't see a fix for this ...
--
Cheers,
Stephen Rothwell sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/
[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]
^ permalink raw reply [flat|nested] 37+ messages in thread
* Re: linux-next: scsi tree build failure
2008-12-21 3:30 ` Stephen Rothwell
@ 2008-12-22 1:59 ` James Bottomley
2008-12-22 2:38 ` Stephen Rothwell
0 siblings, 1 reply; 37+ messages in thread
From: James Bottomley @ 2008-12-22 1:59 UTC (permalink / raw)
To: Stephen Rothwell; +Cc: linux-next, Robert Love
On Sun, 2008-12-21 at 14:30 +1100, Stephen Rothwell wrote:
> Hi James, Robert,
>
> On Wed, 17 Dec 2008 23:20:49 +1100 Stephen Rothwell <sfr@canb.auug.org.au> wrote:
> >
> > On Mon, 15 Dec 2008 08:52:42 -0600 James Bottomley <James.Bottomley@HansenPartnership.com> wrote:
> > >
> > > On Tue, 2008-12-16 at 01:37 +1100, Stephen Rothwell wrote:
> > > >
> > > > Today's linux-next build (powerpc allyesconfig) failed like this:
> > >
> > > Heh, I really need to get this terrasoft system integrated into my
> > > builds
> > >
> > > > drivers/scsi/libfc/fc_fcp.c: In function 'fc_eh_host_reset':
> > > > drivers/scsi/libfc/fc_fcp.c:1987: error: implicit declaration of function 'msleep'
> > > >
> > > > That file needs to include <linux/delay.h>.
> > >
> > > Thanks, that's exactly the problem.
> > >
> > > Robert, please reroll the patches with this fix in ... I'll just
> > > exchange the new patches for the ones in my tree.
> > >
> > > > Introduced by commit 389571f347629d879f9044582bbbafb43e4fe00b ("[SCSI]
> > > > libfc: A modular Fibre Channel library") from the scsi tree. I reverted
> > > > that commit and commit 7aba33c5cfc7a9c104cc7469558c7849f373c3da ("[SCSI]
> > > > fcoe: Fibre Channel over Ethernet") which depended on it.
> >
> > I have dropped the scsi tree until this is fixed up.
>
> I still haven't see a fix for this ...
OK, gave up waiting for an updated patch, so I fixed this myself in my
tree. scsi-misc should be safe to go again.
Thanks,
James
^ permalink raw reply [flat|nested] 37+ messages in thread
* Re: linux-next: scsi tree build failure
2008-12-22 1:59 ` James Bottomley
@ 2008-12-22 2:38 ` Stephen Rothwell
0 siblings, 0 replies; 37+ messages in thread
From: Stephen Rothwell @ 2008-12-22 2:38 UTC (permalink / raw)
To: James Bottomley; +Cc: linux-next, Robert Love
[-- Attachment #1: Type: text/plain, Size: 426 bytes --]
Hi James,
On Sun, 21 Dec 2008 19:59:25 -0600 James Bottomley <James.Bottomley@HansenPartnership.com> wrote:
>
> OK, gave up waiting for an updated patch, so I fixed this myself in my
> tree. scsi-misc should be safe to go again.
Thanks, I will fetch it and add it at the end of the linux-next tree
today.
--
Cheers,
Stephen Rothwell sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/
[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]
^ permalink raw reply [flat|nested] 37+ messages in thread
* Re: linux-next: scsi tree build failure
2009-06-15 21:40 ` Benjamin Herrenschmidt
@ 2009-06-16 2:24 ` James Bottomley
0 siblings, 0 replies; 37+ messages in thread
From: James Bottomley @ 2009-06-16 2:24 UTC (permalink / raw)
To: Benjamin Herrenschmidt
Cc: Stephen Rothwell, linux-next, linux-kernel, Geert Uytterhoeven
On Tue, 2009-06-16 at 07:40 +1000, Benjamin Herrenschmidt wrote:
> On Mon, 2009-06-15 at 10:11 -0500, James Bottomley wrote:
> > OK, dumped the offending patch, so it should be safe to take scsi-misc
> > again.
> >
> > I'm so used to merging these type of conversions, I didn't check that
> > it
> > was using a ps3 specific wrapper instead of a generic device one.
>
> Do I have your Ack to merge the patch via the powerpc tree ?
Ack
James
^ permalink raw reply [flat|nested] 37+ messages in thread
* Re: linux-next: scsi tree build failure
2009-06-15 15:11 ` James Bottomley
@ 2009-06-15 21:40 ` Benjamin Herrenschmidt
2009-06-16 2:24 ` James Bottomley
0 siblings, 1 reply; 37+ messages in thread
From: Benjamin Herrenschmidt @ 2009-06-15 21:40 UTC (permalink / raw)
To: James Bottomley
Cc: Stephen Rothwell, linux-next, linux-kernel, Geert Uytterhoeven
On Mon, 2009-06-15 at 10:11 -0500, James Bottomley wrote:
> OK, dumped the offending patch, so it should be safe to take scsi-misc
> again.
>
> I'm so used to merging these type of conversions, I didn't check that
> it
> was using a ps3 specific wrapper instead of a generic device one.
Do I have your Ack to merge the patch via the powerpc tree ?
Cheers,
Ben.
^ permalink raw reply [flat|nested] 37+ messages in thread
* Re: linux-next: scsi tree build failure
2009-06-15 2:17 Stephen Rothwell
@ 2009-06-15 15:11 ` James Bottomley
2009-06-15 21:40 ` Benjamin Herrenschmidt
0 siblings, 1 reply; 37+ messages in thread
From: James Bottomley @ 2009-06-15 15:11 UTC (permalink / raw)
To: Stephen Rothwell
Cc: linux-next, linux-kernel, Geert Uytterhoeven, Benjamin Herrenschmidt
On Mon, 2009-06-15 at 12:17 +1000, Stephen Rothwell wrote:
> Hi James,
>
> Today's linux-next build (powerpc ppc64_defconfig) failed like this:
>
> drivers/scsi/ps3rom.c: In function 'ps3rom_interrupt':
> drivers/scsi/ps3rom.c:302: error: implicit declaration of function 'ps3_system_bus_get_drvdata'
> drivers/scsi/ps3rom.c:302: warning: assignment makes pointer from integer without a cast
> drivers/scsi/ps3rom.c: In function 'ps3rom_probe':
> drivers/scsi/ps3rom.c:390: error: implicit declaration of function 'ps3_system_bus_set_drvdata'
> drivers/scsi/ps3rom.c: In function 'ps3rom_remove':
> drivers/scsi/ps3rom.c:421: warning: initialization makes pointer from integer without a cast
>
> Caused by commit 4381bd58427116cd7689831998771ddd58078af9 ("[SCSI]
> ps3rom: Use ps3_system_bus_[gs]et_drvdata() instead of direct access").
> This is number 21 in a series and depends on number 19 which is not yet
> merged. If you instead Ack this patch, Ben will merge it through the
> powerpc tree.
>
> For today, I have used the scsi tree from next-20090612.
OK, dumped the offending patch, so it should be safe to take scsi-misc
again.
I'm so used to merging these type of conversions, I didn't check that it
was using a ps3 specific wrapper instead of a generic device one.
James
^ permalink raw reply [flat|nested] 37+ messages in thread
* linux-next: scsi tree build failure
@ 2009-06-15 2:17 Stephen Rothwell
2009-06-15 15:11 ` James Bottomley
0 siblings, 1 reply; 37+ messages in thread
From: Stephen Rothwell @ 2009-06-15 2:17 UTC (permalink / raw)
To: James Bottomley
Cc: linux-next, linux-kernel, Geert Uytterhoeven, Benjamin Herrenschmidt
[-- Attachment #1: Type: text/plain, Size: 1093 bytes --]
Hi James,
Today's linux-next build (powerpc ppc64_defconfig) failed like this:
drivers/scsi/ps3rom.c: In function 'ps3rom_interrupt':
drivers/scsi/ps3rom.c:302: error: implicit declaration of function 'ps3_system_bus_get_drvdata'
drivers/scsi/ps3rom.c:302: warning: assignment makes pointer from integer without a cast
drivers/scsi/ps3rom.c: In function 'ps3rom_probe':
drivers/scsi/ps3rom.c:390: error: implicit declaration of function 'ps3_system_bus_set_drvdata'
drivers/scsi/ps3rom.c: In function 'ps3rom_remove':
drivers/scsi/ps3rom.c:421: warning: initialization makes pointer from integer without a cast
Caused by commit 4381bd58427116cd7689831998771ddd58078af9 ("[SCSI]
ps3rom: Use ps3_system_bus_[gs]et_drvdata() instead of direct access").
This is number 21 in a series and depends on number 19 which is not yet
merged. If you instead Ack this patch, Ben will merge it through the
powerpc tree.
For today, I have used the scsi tree from next-20090612.
--
Cheers,
Stephen Rothwell sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/
[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]
^ permalink raw reply [flat|nested] 37+ messages in thread
* Re: linux-next: scsi tree build failure
2009-04-03 1:16 ` James Bottomley
@ 2009-04-03 2:37 ` Stephen Rothwell
0 siblings, 0 replies; 37+ messages in thread
From: Stephen Rothwell @ 2009-04-03 2:37 UTC (permalink / raw)
To: James Bottomley; +Cc: Robert Love, linux-next, Dev, Vasu, linux-scsi
[-- Attachment #1: Type: text/plain, Size: 327 bytes --]
Hi James,
On Fri, 03 Apr 2009 01:16:40 +0000 James Bottomley <James.Bottomley@HansenPartnership.com> wrote:
>
> I fixed up the patch in my tree, so you should not need this next time
> around.
Great, thanks.
--
Cheers,
Stephen Rothwell sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/
[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]
^ permalink raw reply [flat|nested] 37+ messages in thread
* Re: linux-next: scsi tree build failure
2009-04-03 0:26 ` Stephen Rothwell
@ 2009-04-03 1:16 ` James Bottomley
2009-04-03 2:37 ` Stephen Rothwell
0 siblings, 1 reply; 37+ messages in thread
From: James Bottomley @ 2009-04-03 1:16 UTC (permalink / raw)
To: Stephen Rothwell; +Cc: Robert Love, linux-next, Dev, Vasu, linux-scsi
On Fri, 2009-04-03 at 11:26 +1100, Stephen Rothwell wrote:
> Hi James,
>
> On Thu, 02 Apr 2009 18:23:42 +0000 James Bottomley <James.Bottomley@HansenPartnership.com> wrote:
> >
> > On Thu, 2009-04-02 at 10:19 -0700, Robert Love wrote:
> > >
> > > I built using gcc version 4.3.2 (on x86_64) and didn't see this either.
>
> It may be powerpc specific, I guess.
>
> > > > 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
> > > >
> > > >
> > Right at the moment, I need someone to verify that this is the fix,
> > since I don't have a complaining compiler myself. If it's this simple,
> > I can adjust the patch accordingly.
>
> This fixes it for me. I will apply that patch to linux-next for today
> (instead of the reverts I did yesterday).
Perfect, thanks!
I fixed up the patch in my tree, so you should not need this next time
around.
James
^ permalink raw reply [flat|nested] 37+ messages in thread
* Re: linux-next: scsi tree build failure
2009-04-02 18:23 ` James Bottomley
@ 2009-04-03 0:26 ` Stephen Rothwell
2009-04-03 1:16 ` James Bottomley
0 siblings, 1 reply; 37+ messages in thread
From: Stephen Rothwell @ 2009-04-03 0:26 UTC (permalink / raw)
To: James Bottomley; +Cc: Robert Love, linux-next, Dev, Vasu, linux-scsi
[-- Attachment #1: Type: text/plain, Size: 1129 bytes --]
Hi James,
On Thu, 02 Apr 2009 18:23:42 +0000 James Bottomley <James.Bottomley@HansenPartnership.com> wrote:
>
> On Thu, 2009-04-02 at 10:19 -0700, Robert Love wrote:
> >
> > I built using gcc version 4.3.2 (on x86_64) and didn't see this either.
It may be powerpc specific, I guess.
> > > 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
> > >
> > >
> Right at the moment, I need someone to verify that this is the fix,
> since I don't have a complaining compiler myself. If it's this simple,
> I can adjust the patch accordingly.
This fixes it for me. I will apply that patch to linux-next for today
(instead of the reverts I did yesterday).
--
Cheers,
Stephen Rothwell sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/
[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]
^ permalink raw reply [flat|nested] 37+ messages in thread
* Re: linux-next: scsi tree build failure
2009-04-02 17:19 ` Robert Love
@ 2009-04-02 18:23 ` James Bottomley
2009-04-03 0:26 ` Stephen Rothwell
0 siblings, 1 reply; 37+ messages in thread
From: James Bottomley @ 2009-04-02 18:23 UTC (permalink / raw)
To: Robert Love; +Cc: Stephen Rothwell, linux-next, Dev, Vasu, linux-scsi
On Thu, 2009-04-02 at 10:19 -0700, Robert Love wrote:
> On Thu, 2009-04-02 at 07:22 -0700, James Bottomley wrote:
> > 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 built using gcc version 4.3.2 (on x86_64) and didn't see this either.
>
> > > 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
> >
> >
> How can we resolve this? Should we provide an updated patch without this
> line or a new patch to remove the line?
Right at the moment, I need someone to verify that this is the fix,
since I don't have a complaining compiler myself. If it's this simple,
I can adjust the patch accordingly.
James
^ permalink raw reply [flat|nested] 37+ messages in thread
* Re: linux-next: scsi tree build failure
2009-04-02 14:22 ` James Bottomley
@ 2009-04-02 17:19 ` Robert Love
2009-04-02 18:23 ` James Bottomley
0 siblings, 1 reply; 37+ messages in thread
From: Robert Love @ 2009-04-02 17:19 UTC (permalink / raw)
To: James Bottomley; +Cc: Stephen Rothwell, linux-next, Dev, Vasu, linux-scsi
On Thu, 2009-04-02 at 07:22 -0700, James Bottomley wrote:
> 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 built using gcc version 4.3.2 (on x86_64) and didn't see this either.
> > 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
>
>
How can we resolve this? Should we provide an updated patch without this
line or a new patch to remove the line?
^ permalink raw reply [flat|nested] 37+ messages in thread
* Re: linux-next: scsi tree build failure
2009-04-02 5:49 Stephen Rothwell
@ 2009-04-02 14:22 ` James Bottomley
2009-04-02 17:19 ` Robert Love
0 siblings, 1 reply; 37+ messages in thread
From: James Bottomley @ 2009-04-02 14:22 UTC (permalink / raw)
To: Stephen Rothwell; +Cc: linux-next, Vasu Dev, Robert Love, linux-scsi
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
^ permalink raw reply related [flat|nested] 37+ messages in thread
* linux-next: scsi tree build failure
@ 2009-04-02 5:49 Stephen Rothwell
2009-04-02 14:22 ` James Bottomley
0 siblings, 1 reply; 37+ messages in thread
From: Stephen Rothwell @ 2009-04-02 5:49 UTC (permalink / raw)
To: James Bottomley; +Cc: linux-next, Vasu Dev, Robert Love
[-- Attachment #1: Type: text/plain, Size: 1107 bytes --]
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.
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.
--
Cheers,
Stephen Rothwell sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/
[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]
^ permalink raw reply [flat|nested] 37+ messages in thread
* Re: linux-next: scsi tree build failure
2009-03-16 10:39 Stephen Rothwell
@ 2009-03-16 13:22 ` James Bottomley
0 siblings, 0 replies; 37+ messages in thread
From: James Bottomley @ 2009-03-16 13:22 UTC (permalink / raw)
To: Stephen Rothwell; +Cc: linux-next, Eric Moore, peter Bogdanovic, linux-scsi
On Mon, 2009-03-16 at 21:39 +1100, Stephen Rothwell wrote:
> Hi James,
>
> Today's linux-next build (powerpc allyesconfig) failed like this:
>
> drivers/message/built-in.o:(.data.rel+0x88): multiple definition of `ioc_list'
> drivers/scsi/built-in.o:(.data.rel+0x2760): first defined here
>
> Caused by commit 635374e7eb110e80d9918b8611198edd56a32975 ("[SCSI]
> mpt2sas v00.100.11.15"). ioc_list is not a good choice for a global
> name. The other definition is in drivers/message/fusion/mptbase.c.
>
> I have reverted that commit for today.
Eric,
Vould we get this one sorted out before the merge window opens? ... the
simple fix seems to be to rename ioc_list in mpt2sas ... and track down
any more global symbols with unfortunate names.
James
^ permalink raw reply [flat|nested] 37+ messages in thread
* linux-next: scsi tree build failure
@ 2009-03-16 10:39 Stephen Rothwell
2009-03-16 13:22 ` James Bottomley
0 siblings, 1 reply; 37+ messages in thread
From: Stephen Rothwell @ 2009-03-16 10:39 UTC (permalink / raw)
To: James Bottomley; +Cc: linux-next, Eric Moore, peter Bogdanovic
[-- Attachment #1: Type: text/plain, Size: 584 bytes --]
Hi James,
Today's linux-next build (powerpc allyesconfig) failed like this:
drivers/message/built-in.o:(.data.rel+0x88): multiple definition of `ioc_list'
drivers/scsi/built-in.o:(.data.rel+0x2760): first defined here
Caused by commit 635374e7eb110e80d9918b8611198edd56a32975 ("[SCSI]
mpt2sas v00.100.11.15"). ioc_list is not a good choice for a global
name. The other definition is in drivers/message/fusion/mptbase.c.
I have reverted that commit for today.
--
Cheers,
Stephen Rothwell sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/
[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]
^ permalink raw reply [flat|nested] 37+ messages in thread
* Re: linux-next: scsi tree build failure
2008-12-30 16:16 Stephen Rothwell
@ 2008-12-30 16:30 ` James Bottomley
0 siblings, 0 replies; 37+ messages in thread
From: James Bottomley @ 2008-12-30 16:30 UTC (permalink / raw)
To: Stephen Rothwell; +Cc: linux-next, David S. Miller, Jaswinder Singh, linux-scsi
On Wed, 2008-12-31 at 03:16 +1100, Stephen Rothwell wrote:
> Hi James,
>
> Today's linux-next build (sparc defconfig) failed like this:
>
> drivers/scsi/qlogicpti.c: In function 'qlogicpti_load_firmware':
> drivers/scsi/qlogicpti.c:480: error: 'struct qlogicpti' has no member named 'sdev'
>
> An interaction between commit 68175399c8b5ee90960e78c1bb8e3353e71fbf33
> ("[SCSI] qlogicpti: use request_firmware") from the scsi tree and commit
> 5dc2536bb8d8adb5fdfbe76cd6fdcdf9de3f40f8 ("qlogicpti: Convert to pure OF
> driver") which is in Linus' tree.
>
> The version of the firmware commit that has been in the firmware tree
> since Nov 19 has this conflict corrected.
>
> I have left it for today (as it is getting way to late). Please fix it
> up for tomorrow.
I'm slightly annoyed this hasn't been building. I accepted it into the
SCSI tree because I was under the impression it had been well tested in
linux-next.
I can't actually build this driver, but I think this is the fix, if
someone can verify, I'll fold it into the patch.
Looking at the firmware patches, they all seem to have a kconfig
deficiency in that the modified drivers should now be selecting
FW_LOADER ... it this the case?
James
---
diff --git a/drivers/scsi/qlogicpti.c b/drivers/scsi/qlogicpti.c
index e17a4d2..a342426 100644
--- a/drivers/scsi/qlogicpti.c
+++ b/drivers/scsi/qlogicpti.c
@@ -477,7 +477,7 @@ static int __devinit qlogicpti_load_firmware(struct qlogicpti *qpti)
unsigned long flags;
int i, timeout;
- err = request_firmware(&fw, FIRMWARE_1000, &qpti->sdev->ofdev.dev);
+ err = request_firmware(&fw, FIRMWARE_1000, &qpti->op->dev);
if (err) {
printk(KERN_ERR "Failed to load image \"%s\" err %d\n",
FIRMWARE_1000, err);
^ permalink raw reply related [flat|nested] 37+ messages in thread
* linux-next: scsi tree build failure
@ 2008-12-30 16:16 Stephen Rothwell
2008-12-30 16:30 ` James Bottomley
0 siblings, 1 reply; 37+ messages in thread
From: Stephen Rothwell @ 2008-12-30 16:16 UTC (permalink / raw)
To: James Bottomley; +Cc: linux-next, David S. Miller, Jaswinder Singh
[-- Attachment #1: Type: text/plain, Size: 800 bytes --]
Hi James,
Today's linux-next build (sparc defconfig) failed like this:
drivers/scsi/qlogicpti.c: In function 'qlogicpti_load_firmware':
drivers/scsi/qlogicpti.c:480: error: 'struct qlogicpti' has no member named 'sdev'
An interaction between commit 68175399c8b5ee90960e78c1bb8e3353e71fbf33
("[SCSI] qlogicpti: use request_firmware") from the scsi tree and commit
5dc2536bb8d8adb5fdfbe76cd6fdcdf9de3f40f8 ("qlogicpti: Convert to pure OF
driver") which is in Linus' tree.
The version of the firmware commit that has been in the firmware tree
since Nov 19 has this conflict corrected.
I have left it for today (as it is getting way to late). Please fix it
up for tomorrow.
--
Cheers,
Stephen Rothwell sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/
[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]
^ permalink raw reply [flat|nested] 37+ messages in thread
* Re: linux-next: scsi tree build failure
2008-10-15 6:49 Stephen Rothwell
@ 2008-10-15 12:27 ` James Bottomley
0 siblings, 0 replies; 37+ messages in thread
From: James Bottomley @ 2008-10-15 12:27 UTC (permalink / raw)
To: Stephen Rothwell; +Cc: linux-next, Martin K. Petersen
On Wed, 2008-10-15 at 17:49 +1100, Stephen Rothwell wrote:
> [Resent to get past vger's filters]
>
> Hi James,
>
> Today's linux-next build (powerpc ppc64_defconfig) failed like this:
>
> drivers/scsi/sd.c:579:27: error: macro "sd_dif_op" passed 4 arguments, but takes just 3
> drivers/scsi/sd.c: In function 'sd_prep_fn':
> drivers/scsi/sd.c:578: error: 'sd_dif_op' undeclared (first use in this function)
>
> Caused by commit 9e06688e7d60149cc9ef78ff29515c20186bb418 ("[SCSI] sd:
> Correctly handle all combinations of DIF and DIX").
>
> I added the following patch.
Thanks.
Actually this is the patch I was planning to apply:
http://marc.info/?l=linux-scsi&m=122392563103390
(inlines are better than macros because of the side effect issue) once
I'd verified we didn't have any more hidden config problems.
James
^ permalink raw reply [flat|nested] 37+ messages in thread
* linux-next: scsi tree build failure
@ 2008-10-15 6:49 Stephen Rothwell
2008-10-15 12:27 ` James Bottomley
0 siblings, 1 reply; 37+ messages in thread
From: Stephen Rothwell @ 2008-10-15 6:49 UTC (permalink / raw)
To: James Bottomley; +Cc: linux-next, Martin K. Petersen
[Resent to get past vger's filters]
Hi James,
Today's linux-next build (powerpc ppc64_defconfig) failed like this:
drivers/scsi/sd.c:579:27: error: macro "sd_dif_op" passed 4 arguments, but takes just 3
drivers/scsi/sd.c: In function 'sd_prep_fn':
drivers/scsi/sd.c:578: error: 'sd_dif_op' undeclared (first use in this function)
Caused by commit 9e06688e7d60149cc9ef78ff29515c20186bb418 ("[SCSI] sd:
Correctly handle all combinations of DIF and DIX").
I added the following patch.
--
Cheers,
Stephen Rothwell sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/
From: Stephen Rothwell <sfr@canb.auug.org.au>
Date: Wed, 15 Oct 2008 16:54:59 +1100
Subject: [PATCH] scsi: sd_dif_op has four parameters
Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au>
---
drivers/scsi/sd.h | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/scsi/sd.h b/drivers/scsi/sd.h
index a92b991..1347e00 100644
--- a/drivers/scsi/sd.h
+++ b/drivers/scsi/sd.h
@@ -106,7 +106,7 @@ extern void sd_dif_complete(struct scsi_cmnd *, unsigned int);
#else /* CONFIG_BLK_DEV_INTEGRITY */
-#define sd_dif_op(a, b, c) do { } while (0)
+#define sd_dif_op(a, b, c, d) do { } while (0)
#define sd_dif_config_host(a) do { } while (0)
#define sd_dif_prepare(a, b, c) (0)
#define sd_dif_complete(a, b) (0)
--
1.5.6.5
^ permalink raw reply related [flat|nested] 37+ messages in thread
* Re: linux-next: scsi tree build failure
2008-07-07 22:07 ` James Bottomley
@ 2008-07-07 22:48 ` Benjamin Herrenschmidt
-1 siblings, 0 replies; 37+ messages in thread
From: Benjamin Herrenschmidt @ 2008-07-07 22:48 UTC (permalink / raw)
To: James Bottomley
Cc: Stephen Rothwell, linux-next, Robert Jennings, Paul Mackerras,
linuxppc-dev, linux-scsi
On Mon, 2008-07-07 at 17:07 -0500, James Bottomley wrote:
> I thought it was all done and dusted (which is why I put it in
> scsi-misc). However, now Brian King seems to have some late arriving
> comments.
There are comments on the whole serie yup, I haven't put it in the
tree yet.
Cheers,
Ben.
^ permalink raw reply [flat|nested] 37+ messages in thread
* Re: linux-next: scsi tree build failure
@ 2008-07-07 22:48 ` Benjamin Herrenschmidt
0 siblings, 0 replies; 37+ messages in thread
From: Benjamin Herrenschmidt @ 2008-07-07 22:48 UTC (permalink / raw)
To: James Bottomley
Cc: Stephen Rothwell, linux-scsi, linuxppc-dev, linux-next, Paul Mackerras
On Mon, 2008-07-07 at 17:07 -0500, James Bottomley wrote:
> I thought it was all done and dusted (which is why I put it in
> scsi-misc). However, now Brian King seems to have some late arriving
> comments.
There are comments on the whole serie yup, I haven't put it in the
tree yet.
Cheers,
Ben.
^ permalink raw reply [flat|nested] 37+ messages in thread
* Re: linux-next: scsi tree build failure
2008-07-07 22:05 ` Benjamin Herrenschmidt
@ 2008-07-07 22:07 ` James Bottomley
-1 siblings, 0 replies; 37+ messages in thread
From: James Bottomley @ 2008-07-07 22:07 UTC (permalink / raw)
To: benh
Cc: Stephen Rothwell, linux-next, Robert Jennings, Paul Mackerras,
linuxppc-dev, linux-scsi
On Tue, 2008-07-08 at 08:05 +1000, Benjamin Herrenschmidt wrote:
> On Mon, 2008-07-07 at 09:39 -0500, James Bottomley wrote:
> > > Caused because commit 341b56db6804040aa9559e913865108424e3b18b
> > ("[SCSI]
> > > ibmvscsi: driver enablement for CMO"), which was 15/16 in a series,
> > has
> > > been merged before any of the other patches in the series. I have
> > > reverted that commit.
> >
> > Do I detect the fact that IBM sent a patch for a SCSI driver for which
> > the core features weren't yet enabled (and which I couldn't check, not
> > having a ppc build system)?
>
> I suspect the sender (Robert) was asking for comments/review :-) In any
> case, Robert, next time make it explicit on a patch CCed to a separate
> list from the rest of the serie that it isn't to be merged without
> dependencies.
>
> James, if you ack it, I'll put it in powerpc.git.
I thought it was all done and dusted (which is why I put it in
scsi-misc). However, now Brian King seems to have some late arriving
comments.
James
^ permalink raw reply [flat|nested] 37+ messages in thread
* Re: linux-next: scsi tree build failure
@ 2008-07-07 22:07 ` James Bottomley
0 siblings, 0 replies; 37+ messages in thread
From: James Bottomley @ 2008-07-07 22:07 UTC (permalink / raw)
To: benh
Cc: Stephen Rothwell, linux-scsi, linuxppc-dev, linux-next, Paul Mackerras
On Tue, 2008-07-08 at 08:05 +1000, Benjamin Herrenschmidt wrote:
> On Mon, 2008-07-07 at 09:39 -0500, James Bottomley wrote:
> > > Caused because commit 341b56db6804040aa9559e913865108424e3b18b
> > ("[SCSI]
> > > ibmvscsi: driver enablement for CMO"), which was 15/16 in a series,
> > has
> > > been merged before any of the other patches in the series. I have
> > > reverted that commit.
> >
> > Do I detect the fact that IBM sent a patch for a SCSI driver for which
> > the core features weren't yet enabled (and which I couldn't check, not
> > having a ppc build system)?
>
> I suspect the sender (Robert) was asking for comments/review :-) In any
> case, Robert, next time make it explicit on a patch CCed to a separate
> list from the rest of the serie that it isn't to be merged without
> dependencies.
>
> James, if you ack it, I'll put it in powerpc.git.
I thought it was all done and dusted (which is why I put it in
scsi-misc). However, now Brian King seems to have some late arriving
comments.
James
^ permalink raw reply [flat|nested] 37+ messages in thread
* Re: linux-next: scsi tree build failure
2008-07-07 14:39 ` James Bottomley
@ 2008-07-07 22:05 ` Benjamin Herrenschmidt
-1 siblings, 0 replies; 37+ messages in thread
From: Benjamin Herrenschmidt @ 2008-07-07 22:05 UTC (permalink / raw)
To: James Bottomley
Cc: Stephen Rothwell, linux-next, Robert Jennings, Paul Mackerras,
linuxppc-dev, linux-scsi
On Mon, 2008-07-07 at 09:39 -0500, James Bottomley wrote:
> > Caused because commit 341b56db6804040aa9559e913865108424e3b18b
> ("[SCSI]
> > ibmvscsi: driver enablement for CMO"), which was 15/16 in a series,
> has
> > been merged before any of the other patches in the series. I have
> > reverted that commit.
>
> Do I detect the fact that IBM sent a patch for a SCSI driver for which
> the core features weren't yet enabled (and which I couldn't check, not
> having a ppc build system)?
I suspect the sender (Robert) was asking for comments/review :-) In any
case, Robert, next time make it explicit on a patch CCed to a separate
list from the rest of the serie that it isn't to be merged without
dependencies.
James, if you ack it, I'll put it in powerpc.git.
Cheers,
Ben.
^ permalink raw reply [flat|nested] 37+ messages in thread
* Re: linux-next: scsi tree build failure
@ 2008-07-07 22:05 ` Benjamin Herrenschmidt
0 siblings, 0 replies; 37+ messages in thread
From: Benjamin Herrenschmidt @ 2008-07-07 22:05 UTC (permalink / raw)
To: James Bottomley
Cc: Stephen Rothwell, linux-scsi, linuxppc-dev, linux-next, Paul Mackerras
On Mon, 2008-07-07 at 09:39 -0500, James Bottomley wrote:
> > Caused because commit 341b56db6804040aa9559e913865108424e3b18b
> ("[SCSI]
> > ibmvscsi: driver enablement for CMO"), which was 15/16 in a series,
> has
> > been merged before any of the other patches in the series. I have
> > reverted that commit.
>
> Do I detect the fact that IBM sent a patch for a SCSI driver for which
> the core features weren't yet enabled (and which I couldn't check, not
> having a ppc build system)?
I suspect the sender (Robert) was asking for comments/review :-) In any
case, Robert, next time make it explicit on a patch CCed to a separate
list from the rest of the serie that it isn't to be merged without
dependencies.
James, if you ack it, I'll put it in powerpc.git.
Cheers,
Ben.
^ permalink raw reply [flat|nested] 37+ messages in thread
* Re: linux-next: scsi tree build failure
2008-07-07 12:25 ` Stephen Rothwell
@ 2008-07-07 22:00 ` Benjamin Herrenschmidt
-1 siblings, 0 replies; 37+ messages in thread
From: Benjamin Herrenschmidt @ 2008-07-07 22:00 UTC (permalink / raw)
To: Stephen Rothwell
Cc: James Bottomley, linux-next, Robert Jennings, Paul Mackerras,
linuxppc-dev
On Mon, 2008-07-07 at 22:25 +1000, Stephen Rothwell wrote:
> Hi James,
>
> Today's linux-next build (powerpc ppc64_defconfig) failed like this:
James, if you agree, I'll carry this commit in the powerpc tree along
with its dependencies (and your ack).
Cheers,
Ben.
> drivers/scsi/ibmvscsi/ibmvscsi.c: In function 'map_sg_data':
> drivers/scsi/ibmvscsi/ibmvscsi.c:431: error: 'FW_FEATURE_CMO' undeclared (first use in this function)
> drivers/scsi/ibmvscsi/ibmvscsi.c:431: error: (Each undeclared identifier is reported only once
> drivers/scsi/ibmvscsi/ibmvscsi.c:431: error: for each function it appears in.)
> drivers/scsi/ibmvscsi/ibmvscsi.c: In function 'ibmvscsi_queuecommand':
> drivers/scsi/ibmvscsi/ibmvscsi.c:750: error: 'FW_FEATURE_CMO' undeclared (first use in this function)
> drivers/scsi/ibmvscsi/ibmvscsi.c: In function 'send_mad_adapter_info':
> drivers/scsi/ibmvscsi/ibmvscsi.c:864: error: 'FW_FEATURE_CMO' undeclared (first use in this function)
> drivers/scsi/ibmvscsi/ibmvscsi.c: In function 'ibmvscsi_do_host_config':
> drivers/scsi/ibmvscsi/ibmvscsi.c:1412: error: 'FW_FEATURE_CMO' undeclared (first use in this function)
> drivers/scsi/ibmvscsi/ibmvscsi.c: At top level:
> drivers/scsi/ibmvscsi/ibmvscsi.c:1769: error: unknown field 'get_io_entitlement' specified in initializer
> drivers/scsi/ibmvscsi/ibmvscsi.c:1769: warning: missing braces around initializer
> drivers/scsi/ibmvscsi/ibmvscsi.c:1769: warning: (near initialization for 'ibmvscsi_driver.driver')
> drivers/scsi/ibmvscsi/ibmvscsi.c:1769: warning: initialization from incompatible pointer type
>
> Caused because commit 341b56db6804040aa9559e913865108424e3b18b ("[SCSI]
> ibmvscsi: driver enablement for CMO"), which was 15/16 in a series, has
> been merged before any of the other patches in the series. I have
> reverted that commit.
>
^ permalink raw reply [flat|nested] 37+ messages in thread
* Re: linux-next: scsi tree build failure
@ 2008-07-07 22:00 ` Benjamin Herrenschmidt
0 siblings, 0 replies; 37+ messages in thread
From: Benjamin Herrenschmidt @ 2008-07-07 22:00 UTC (permalink / raw)
To: Stephen Rothwell
Cc: James Bottomley, linuxppc-dev, linux-next, Paul Mackerras
On Mon, 2008-07-07 at 22:25 +1000, Stephen Rothwell wrote:
> Hi James,
>
> Today's linux-next build (powerpc ppc64_defconfig) failed like this:
James, if you agree, I'll carry this commit in the powerpc tree along
with its dependencies (and your ack).
Cheers,
Ben.
> drivers/scsi/ibmvscsi/ibmvscsi.c: In function 'map_sg_data':
> drivers/scsi/ibmvscsi/ibmvscsi.c:431: error: 'FW_FEATURE_CMO' undeclared (first use in this function)
> drivers/scsi/ibmvscsi/ibmvscsi.c:431: error: (Each undeclared identifier is reported only once
> drivers/scsi/ibmvscsi/ibmvscsi.c:431: error: for each function it appears in.)
> drivers/scsi/ibmvscsi/ibmvscsi.c: In function 'ibmvscsi_queuecommand':
> drivers/scsi/ibmvscsi/ibmvscsi.c:750: error: 'FW_FEATURE_CMO' undeclared (first use in this function)
> drivers/scsi/ibmvscsi/ibmvscsi.c: In function 'send_mad_adapter_info':
> drivers/scsi/ibmvscsi/ibmvscsi.c:864: error: 'FW_FEATURE_CMO' undeclared (first use in this function)
> drivers/scsi/ibmvscsi/ibmvscsi.c: In function 'ibmvscsi_do_host_config':
> drivers/scsi/ibmvscsi/ibmvscsi.c:1412: error: 'FW_FEATURE_CMO' undeclared (first use in this function)
> drivers/scsi/ibmvscsi/ibmvscsi.c: At top level:
> drivers/scsi/ibmvscsi/ibmvscsi.c:1769: error: unknown field 'get_io_entitlement' specified in initializer
> drivers/scsi/ibmvscsi/ibmvscsi.c:1769: warning: missing braces around initializer
> drivers/scsi/ibmvscsi/ibmvscsi.c:1769: warning: (near initialization for 'ibmvscsi_driver.driver')
> drivers/scsi/ibmvscsi/ibmvscsi.c:1769: warning: initialization from incompatible pointer type
>
> Caused because commit 341b56db6804040aa9559e913865108424e3b18b ("[SCSI]
> ibmvscsi: driver enablement for CMO"), which was 15/16 in a series, has
> been merged before any of the other patches in the series. I have
> reverted that commit.
>
^ permalink raw reply [flat|nested] 37+ messages in thread
* Re: linux-next: scsi tree build failure
2008-07-07 14:39 ` James Bottomley
@ 2008-07-07 14:53 ` Robert Jennings
-1 siblings, 0 replies; 37+ messages in thread
From: Robert Jennings @ 2008-07-07 14:53 UTC (permalink / raw)
To: James Bottomley
Cc: Stephen Rothwell, linux-next, Paul Mackerras, linuxppc-dev, benh,
linux-scsi
* James Bottomley (James.Bottomley@HansenPartnership.com) wrote:
> Correct cc's added
>
> On Mon, 2008-07-07 at 22:25 +1000, Stephen Rothwell wrote:
> > Hi James,
> >
> > Today's linux-next build (powerpc ppc64_defconfig) failed like this:
> >
> > drivers/scsi/ibmvscsi/ibmvscsi.c: In function 'map_sg_data':
> > drivers/scsi/ibmvscsi/ibmvscsi.c:431: error: 'FW_FEATURE_CMO' undeclared (first use in this function)
> > drivers/scsi/ibmvscsi/ibmvscsi.c:431: error: (Each undeclared identifier is reported only once
> > drivers/scsi/ibmvscsi/ibmvscsi.c:431: error: for each function it appears in.)
> > drivers/scsi/ibmvscsi/ibmvscsi.c: In function 'ibmvscsi_queuecommand':
> > drivers/scsi/ibmvscsi/ibmvscsi.c:750: error: 'FW_FEATURE_CMO' undeclared (first use in this function)
> > drivers/scsi/ibmvscsi/ibmvscsi.c: In function 'send_mad_adapter_info':
> > drivers/scsi/ibmvscsi/ibmvscsi.c:864: error: 'FW_FEATURE_CMO' undeclared (first use in this function)
> > drivers/scsi/ibmvscsi/ibmvscsi.c: In function 'ibmvscsi_do_host_config':
> > drivers/scsi/ibmvscsi/ibmvscsi.c:1412: error: 'FW_FEATURE_CMO' undeclared (first use in this function)
> > drivers/scsi/ibmvscsi/ibmvscsi.c: At top level:
> > drivers/scsi/ibmvscsi/ibmvscsi.c:1769: error: unknown field 'get_io_entitlement' specified in initializer
> > drivers/scsi/ibmvscsi/ibmvscsi.c:1769: warning: missing braces around initializer
> > drivers/scsi/ibmvscsi/ibmvscsi.c:1769: warning: (near initialization for 'ibmvscsi_driver.driver')
> > drivers/scsi/ibmvscsi/ibmvscsi.c:1769: warning: initialization from incompatible pointer type
> >
> > Caused because commit 341b56db6804040aa9559e913865108424e3b18b ("[SCSI]
> > ibmvscsi: driver enablement for CMO"), which was 15/16 in a series, has
> > been merged before any of the other patches in the series. I have
> > reverted that commit.
>
> Do I detect the fact that IBM sent a patch for a SCSI driver for which
> the core features weren't yet enabled (and which I couldn't check, not
> having a ppc build system)?
>
> James
I didn't not properly note in the patch description my intent to have
this merged in the linuxppc-dev tree. The patch is part of a larger set
enabling the feature. Linux-scsi was CC'ed for this driver for review.
Robert Jennings
^ permalink raw reply [flat|nested] 37+ messages in thread
* Re: linux-next: scsi tree build failure
@ 2008-07-07 14:53 ` Robert Jennings
0 siblings, 0 replies; 37+ messages in thread
From: Robert Jennings @ 2008-07-07 14:53 UTC (permalink / raw)
To: James Bottomley
Cc: Stephen Rothwell, linux-scsi, linuxppc-dev, linux-next, Paul Mackerras
* James Bottomley (James.Bottomley@HansenPartnership.com) wrote:
> Correct cc's added
>
> On Mon, 2008-07-07 at 22:25 +1000, Stephen Rothwell wrote:
> > Hi James,
> >
> > Today's linux-next build (powerpc ppc64_defconfig) failed like this:
> >
> > drivers/scsi/ibmvscsi/ibmvscsi.c: In function 'map_sg_data':
> > drivers/scsi/ibmvscsi/ibmvscsi.c:431: error: 'FW_FEATURE_CMO' undeclared (first use in this function)
> > drivers/scsi/ibmvscsi/ibmvscsi.c:431: error: (Each undeclared identifier is reported only once
> > drivers/scsi/ibmvscsi/ibmvscsi.c:431: error: for each function it appears in.)
> > drivers/scsi/ibmvscsi/ibmvscsi.c: In function 'ibmvscsi_queuecommand':
> > drivers/scsi/ibmvscsi/ibmvscsi.c:750: error: 'FW_FEATURE_CMO' undeclared (first use in this function)
> > drivers/scsi/ibmvscsi/ibmvscsi.c: In function 'send_mad_adapter_info':
> > drivers/scsi/ibmvscsi/ibmvscsi.c:864: error: 'FW_FEATURE_CMO' undeclared (first use in this function)
> > drivers/scsi/ibmvscsi/ibmvscsi.c: In function 'ibmvscsi_do_host_config':
> > drivers/scsi/ibmvscsi/ibmvscsi.c:1412: error: 'FW_FEATURE_CMO' undeclared (first use in this function)
> > drivers/scsi/ibmvscsi/ibmvscsi.c: At top level:
> > drivers/scsi/ibmvscsi/ibmvscsi.c:1769: error: unknown field 'get_io_entitlement' specified in initializer
> > drivers/scsi/ibmvscsi/ibmvscsi.c:1769: warning: missing braces around initializer
> > drivers/scsi/ibmvscsi/ibmvscsi.c:1769: warning: (near initialization for 'ibmvscsi_driver.driver')
> > drivers/scsi/ibmvscsi/ibmvscsi.c:1769: warning: initialization from incompatible pointer type
> >
> > Caused because commit 341b56db6804040aa9559e913865108424e3b18b ("[SCSI]
> > ibmvscsi: driver enablement for CMO"), which was 15/16 in a series, has
> > been merged before any of the other patches in the series. I have
> > reverted that commit.
>
> Do I detect the fact that IBM sent a patch for a SCSI driver for which
> the core features weren't yet enabled (and which I couldn't check, not
> having a ppc build system)?
>
> James
I didn't not properly note in the patch description my intent to have
this merged in the linuxppc-dev tree. The patch is part of a larger set
enabling the feature. Linux-scsi was CC'ed for this driver for review.
Robert Jennings
^ permalink raw reply [flat|nested] 37+ messages in thread
* Re: linux-next: scsi tree build failure
2008-07-07 12:25 ` Stephen Rothwell
@ 2008-07-07 14:39 ` James Bottomley
-1 siblings, 0 replies; 37+ messages in thread
From: James Bottomley @ 2008-07-07 14:39 UTC (permalink / raw)
To: Stephen Rothwell
Cc: linux-next, Robert Jennings, Paul Mackerras, linuxppc-dev, benh,
linux-scsi
Correct cc's added
On Mon, 2008-07-07 at 22:25 +1000, Stephen Rothwell wrote:
> Hi James,
>
> Today's linux-next build (powerpc ppc64_defconfig) failed like this:
>
> drivers/scsi/ibmvscsi/ibmvscsi.c: In function 'map_sg_data':
> drivers/scsi/ibmvscsi/ibmvscsi.c:431: error: 'FW_FEATURE_CMO' undeclared (first use in this function)
> drivers/scsi/ibmvscsi/ibmvscsi.c:431: error: (Each undeclared identifier is reported only once
> drivers/scsi/ibmvscsi/ibmvscsi.c:431: error: for each function it appears in.)
> drivers/scsi/ibmvscsi/ibmvscsi.c: In function 'ibmvscsi_queuecommand':
> drivers/scsi/ibmvscsi/ibmvscsi.c:750: error: 'FW_FEATURE_CMO' undeclared (first use in this function)
> drivers/scsi/ibmvscsi/ibmvscsi.c: In function 'send_mad_adapter_info':
> drivers/scsi/ibmvscsi/ibmvscsi.c:864: error: 'FW_FEATURE_CMO' undeclared (first use in this function)
> drivers/scsi/ibmvscsi/ibmvscsi.c: In function 'ibmvscsi_do_host_config':
> drivers/scsi/ibmvscsi/ibmvscsi.c:1412: error: 'FW_FEATURE_CMO' undeclared (first use in this function)
> drivers/scsi/ibmvscsi/ibmvscsi.c: At top level:
> drivers/scsi/ibmvscsi/ibmvscsi.c:1769: error: unknown field 'get_io_entitlement' specified in initializer
> drivers/scsi/ibmvscsi/ibmvscsi.c:1769: warning: missing braces around initializer
> drivers/scsi/ibmvscsi/ibmvscsi.c:1769: warning: (near initialization for 'ibmvscsi_driver.driver')
> drivers/scsi/ibmvscsi/ibmvscsi.c:1769: warning: initialization from incompatible pointer type
>
> Caused because commit 341b56db6804040aa9559e913865108424e3b18b ("[SCSI]
> ibmvscsi: driver enablement for CMO"), which was 15/16 in a series, has
> been merged before any of the other patches in the series. I have
> reverted that commit.
Do I detect the fact that IBM sent a patch for a SCSI driver for which
the core features weren't yet enabled (and which I couldn't check, not
having a ppc build system)?
James
^ permalink raw reply [flat|nested] 37+ messages in thread
* Re: linux-next: scsi tree build failure
@ 2008-07-07 14:39 ` James Bottomley
0 siblings, 0 replies; 37+ messages in thread
From: James Bottomley @ 2008-07-07 14:39 UTC (permalink / raw)
To: Stephen Rothwell; +Cc: linux-scsi, linuxppc-dev, linux-next, Paul Mackerras
Correct cc's added
On Mon, 2008-07-07 at 22:25 +1000, Stephen Rothwell wrote:
> Hi James,
>
> Today's linux-next build (powerpc ppc64_defconfig) failed like this:
>
> drivers/scsi/ibmvscsi/ibmvscsi.c: In function 'map_sg_data':
> drivers/scsi/ibmvscsi/ibmvscsi.c:431: error: 'FW_FEATURE_CMO' undeclared (first use in this function)
> drivers/scsi/ibmvscsi/ibmvscsi.c:431: error: (Each undeclared identifier is reported only once
> drivers/scsi/ibmvscsi/ibmvscsi.c:431: error: for each function it appears in.)
> drivers/scsi/ibmvscsi/ibmvscsi.c: In function 'ibmvscsi_queuecommand':
> drivers/scsi/ibmvscsi/ibmvscsi.c:750: error: 'FW_FEATURE_CMO' undeclared (first use in this function)
> drivers/scsi/ibmvscsi/ibmvscsi.c: In function 'send_mad_adapter_info':
> drivers/scsi/ibmvscsi/ibmvscsi.c:864: error: 'FW_FEATURE_CMO' undeclared (first use in this function)
> drivers/scsi/ibmvscsi/ibmvscsi.c: In function 'ibmvscsi_do_host_config':
> drivers/scsi/ibmvscsi/ibmvscsi.c:1412: error: 'FW_FEATURE_CMO' undeclared (first use in this function)
> drivers/scsi/ibmvscsi/ibmvscsi.c: At top level:
> drivers/scsi/ibmvscsi/ibmvscsi.c:1769: error: unknown field 'get_io_entitlement' specified in initializer
> drivers/scsi/ibmvscsi/ibmvscsi.c:1769: warning: missing braces around initializer
> drivers/scsi/ibmvscsi/ibmvscsi.c:1769: warning: (near initialization for 'ibmvscsi_driver.driver')
> drivers/scsi/ibmvscsi/ibmvscsi.c:1769: warning: initialization from incompatible pointer type
>
> Caused because commit 341b56db6804040aa9559e913865108424e3b18b ("[SCSI]
> ibmvscsi: driver enablement for CMO"), which was 15/16 in a series, has
> been merged before any of the other patches in the series. I have
> reverted that commit.
Do I detect the fact that IBM sent a patch for a SCSI driver for which
the core features weren't yet enabled (and which I couldn't check, not
having a ppc build system)?
James
^ permalink raw reply [flat|nested] 37+ messages in thread
* linux-next: scsi tree build failure
@ 2008-07-07 12:25 ` Stephen Rothwell
0 siblings, 0 replies; 37+ messages in thread
From: Stephen Rothwell @ 2008-07-07 12:25 UTC (permalink / raw)
To: James Bottomley
Cc: linux-next, Robert Jennings, Paul Mackerras, linuxppc-dev, benh
[-- Attachment #1: Type: text/plain, Size: 1736 bytes --]
Hi James,
Today's linux-next build (powerpc ppc64_defconfig) failed like this:
drivers/scsi/ibmvscsi/ibmvscsi.c: In function 'map_sg_data':
drivers/scsi/ibmvscsi/ibmvscsi.c:431: error: 'FW_FEATURE_CMO' undeclared (first use in this function)
drivers/scsi/ibmvscsi/ibmvscsi.c:431: error: (Each undeclared identifier is reported only once
drivers/scsi/ibmvscsi/ibmvscsi.c:431: error: for each function it appears in.)
drivers/scsi/ibmvscsi/ibmvscsi.c: In function 'ibmvscsi_queuecommand':
drivers/scsi/ibmvscsi/ibmvscsi.c:750: error: 'FW_FEATURE_CMO' undeclared (first use in this function)
drivers/scsi/ibmvscsi/ibmvscsi.c: In function 'send_mad_adapter_info':
drivers/scsi/ibmvscsi/ibmvscsi.c:864: error: 'FW_FEATURE_CMO' undeclared (first use in this function)
drivers/scsi/ibmvscsi/ibmvscsi.c: In function 'ibmvscsi_do_host_config':
drivers/scsi/ibmvscsi/ibmvscsi.c:1412: error: 'FW_FEATURE_CMO' undeclared (first use in this function)
drivers/scsi/ibmvscsi/ibmvscsi.c: At top level:
drivers/scsi/ibmvscsi/ibmvscsi.c:1769: error: unknown field 'get_io_entitlement' specified in initializer
drivers/scsi/ibmvscsi/ibmvscsi.c:1769: warning: missing braces around initializer
drivers/scsi/ibmvscsi/ibmvscsi.c:1769: warning: (near initialization for 'ibmvscsi_driver.driver')
drivers/scsi/ibmvscsi/ibmvscsi.c:1769: warning: initialization from incompatible pointer type
Caused because commit 341b56db6804040aa9559e913865108424e3b18b ("[SCSI]
ibmvscsi: driver enablement for CMO"), which was 15/16 in a series, has
been merged before any of the other patches in the series. I have
reverted that commit.
--
Cheers,
Stephen Rothwell sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/
[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]
^ permalink raw reply [flat|nested] 37+ messages in thread
* linux-next: scsi tree build failure
@ 2008-07-07 12:25 ` Stephen Rothwell
0 siblings, 0 replies; 37+ messages in thread
From: Stephen Rothwell @ 2008-07-07 12:25 UTC (permalink / raw)
To: James Bottomley; +Cc: linuxppc-dev, linux-next, Paul Mackerras
[-- Attachment #1: Type: text/plain, Size: 1736 bytes --]
Hi James,
Today's linux-next build (powerpc ppc64_defconfig) failed like this:
drivers/scsi/ibmvscsi/ibmvscsi.c: In function 'map_sg_data':
drivers/scsi/ibmvscsi/ibmvscsi.c:431: error: 'FW_FEATURE_CMO' undeclared (first use in this function)
drivers/scsi/ibmvscsi/ibmvscsi.c:431: error: (Each undeclared identifier is reported only once
drivers/scsi/ibmvscsi/ibmvscsi.c:431: error: for each function it appears in.)
drivers/scsi/ibmvscsi/ibmvscsi.c: In function 'ibmvscsi_queuecommand':
drivers/scsi/ibmvscsi/ibmvscsi.c:750: error: 'FW_FEATURE_CMO' undeclared (first use in this function)
drivers/scsi/ibmvscsi/ibmvscsi.c: In function 'send_mad_adapter_info':
drivers/scsi/ibmvscsi/ibmvscsi.c:864: error: 'FW_FEATURE_CMO' undeclared (first use in this function)
drivers/scsi/ibmvscsi/ibmvscsi.c: In function 'ibmvscsi_do_host_config':
drivers/scsi/ibmvscsi/ibmvscsi.c:1412: error: 'FW_FEATURE_CMO' undeclared (first use in this function)
drivers/scsi/ibmvscsi/ibmvscsi.c: At top level:
drivers/scsi/ibmvscsi/ibmvscsi.c:1769: error: unknown field 'get_io_entitlement' specified in initializer
drivers/scsi/ibmvscsi/ibmvscsi.c:1769: warning: missing braces around initializer
drivers/scsi/ibmvscsi/ibmvscsi.c:1769: warning: (near initialization for 'ibmvscsi_driver.driver')
drivers/scsi/ibmvscsi/ibmvscsi.c:1769: warning: initialization from incompatible pointer type
Caused because commit 341b56db6804040aa9559e913865108424e3b18b ("[SCSI]
ibmvscsi: driver enablement for CMO"), which was 15/16 in a series, has
been merged before any of the other patches in the series. I have
reverted that commit.
--
Cheers,
Stephen Rothwell sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/
[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]
^ permalink raw reply [flat|nested] 37+ messages in thread
end of thread, other threads:[~2009-06-16 2:26 UTC | newest]
Thread overview: 37+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-12-15 14:37 linux-next: scsi tree build failure Stephen Rothwell
2008-12-15 14:52 ` James Bottomley
2008-12-17 12:20 ` Stephen Rothwell
2008-12-21 3:30 ` Stephen Rothwell
2008-12-22 1:59 ` James Bottomley
2008-12-22 2:38 ` Stephen Rothwell
-- strict thread matches above, loose matches on Subject: below --
2009-06-15 2:17 Stephen Rothwell
2009-06-15 15:11 ` James Bottomley
2009-06-15 21:40 ` Benjamin Herrenschmidt
2009-06-16 2:24 ` James Bottomley
2009-04-02 5:49 Stephen Rothwell
2009-04-02 14:22 ` James Bottomley
2009-04-02 17:19 ` Robert Love
2009-04-02 18:23 ` James Bottomley
2009-04-03 0:26 ` Stephen Rothwell
2009-04-03 1:16 ` James Bottomley
2009-04-03 2:37 ` Stephen Rothwell
2009-03-16 10:39 Stephen Rothwell
2009-03-16 13:22 ` James Bottomley
2008-12-30 16:16 Stephen Rothwell
2008-12-30 16:30 ` James Bottomley
2008-10-15 6:49 Stephen Rothwell
2008-10-15 12:27 ` James Bottomley
2008-07-07 12:25 Stephen Rothwell
2008-07-07 12:25 ` Stephen Rothwell
2008-07-07 14:39 ` James Bottomley
2008-07-07 14:39 ` James Bottomley
2008-07-07 14:53 ` Robert Jennings
2008-07-07 14:53 ` Robert Jennings
2008-07-07 22:05 ` Benjamin Herrenschmidt
2008-07-07 22:05 ` Benjamin Herrenschmidt
2008-07-07 22:07 ` James Bottomley
2008-07-07 22:07 ` James Bottomley
2008-07-07 22:48 ` Benjamin Herrenschmidt
2008-07-07 22:48 ` Benjamin Herrenschmidt
2008-07-07 22:00 ` Benjamin Herrenschmidt
2008-07-07 22:00 ` Benjamin Herrenschmidt
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.