recent scsi changes break fc4 build

recent scsi changes break fc4 build

[David S. Miller]

Please, at the very least, try to compile the changes you send the
Linus instead of doing a cut&paste job and hoping for the best.

Thanks a lot.

drivers/fc4/fc.c: In function `fcp_scsi_done':
drivers/fc4/fc.c:768: warning: unused variable `flags'
drivers/fc4/fc.c: In function `fcp_scsi_abort':
drivers/fc4/fc.c:910: warning: unused variable `flags'
drivers/fc4/fc.c: In function `fcp_scsi_dev_reset':
drivers/fc4/fc.c:987: error: `flags' undeclared (first use in this function)
drivers/fc4/fc.c:987: error: (Each undeclared identifier is reported only once
drivers/fc4/fc.c:987: error: for each function it appears in.)
drivers/fc4/fc.c: In function `fcp_scsi_host_reset':
drivers/fc4/fc.c:1033: error: `flags' undeclared (first use in this function)

Re: recent scsi changes break fc4 build

[Christoph Hellwig]

On Sat, Jun 18, 2005 at 11:16:46PM -0700, David S. Miller wrote:
> 
> Please, at the very least, try to compile the changes you send the
> Linus instead of doing a cut&paste job and hoping for the best.
> 
> Thanks a lot.

Sorry Dave, but fc4 is broken beyon imagination and not mentioned in
the common drivers/Kconfig at all.  My suggestion to fix it is an
rm -rf.

Re: recent scsi changes break fc4 build

[Jeff Garzik]

[-- Attachment #1: Type: text/plain, Size: 1060 bytes --]

David S. Miller wrote:
> Please, at the very least, try to compile the changes you send the
> Linus instead of doing a cut&paste job and hoping for the best.
> 
> Thanks a lot.
> 
> drivers/fc4/fc.c: In function `fcp_scsi_done':
> drivers/fc4/fc.c:768: warning: unused variable `flags'
> drivers/fc4/fc.c: In function `fcp_scsi_abort':
> drivers/fc4/fc.c:910: warning: unused variable `flags'
> drivers/fc4/fc.c: In function `fcp_scsi_dev_reset':
> drivers/fc4/fc.c:987: error: `flags' undeclared (first use in this function)
> drivers/fc4/fc.c:987: error: (Each undeclared identifier is reported only once
> drivers/fc4/fc.c:987: error: for each function it appears in.)
> drivers/fc4/fc.c: In function `fcp_scsi_host_reset':
> drivers/fc4/fc.c:1033: error: `flags' undeclared (first use in this function)

Regardless of Christoph's comments, 3 out of 4 problems you list are 
probably my fault.

fc4/fc doesn't get built for me, under 'make allyesconfig', one of the 
few that fell through the cracks.  Sorry.

Does the attached patch work for you?

	Jeff



[-- Attachment #2: patch --]
[-- Type: text/plain, Size: 928 bytes --]

diff --git a/drivers/fc4/fc.c b/drivers/fc4/fc.c
--- a/drivers/fc4/fc.c
+++ b/drivers/fc4/fc.c
@@ -765,8 +765,6 @@ void fcp_release(fc_channel *fcchain, in
 
 static void fcp_scsi_done (Scsi_Cmnd *SCpnt)
 {
-	unsigned long flags;
-
 	if (FCP_CMND(SCpnt)->done)
 		FCP_CMND(SCpnt)->done(SCpnt);
 }
@@ -907,8 +905,6 @@ int fcp_scsi_abort(Scsi_Cmnd *SCpnt)
 	 */
 
 	if (++fc->abort_count < (fc->can_queue >> 1)) {
-		unsigned long flags;
-
 		SCpnt->result = DID_ABORT;
 		fcmd->done(SCpnt);
 		printk("FC: soft abort\n");
@@ -931,6 +927,7 @@ void fcp_scsi_reset_done(Scsi_Cmnd *SCpn
 
 int fcp_scsi_dev_reset(Scsi_Cmnd *SCpnt)
 {
+	unsigned long flags;
 	fcp_cmd *cmd;
 	fcp_cmnd *fcmd;
 	fc_channel *fc = FC_SCMND(SCpnt);
@@ -1028,6 +1025,7 @@ static int __fcp_scsi_host_reset(Scsi_Cm
 
 int fcp_scsi_host_reset(Scsi_Cmnd *SCpnt)
 {
+	unsigned long flags;
 	int rc;
 
 	spin_lock_irqsave(SCpnt->device->host->host_lock, flags);

Re: recent scsi changes break fc4 build

[David S. Miller]

From: Christoph Hellwig <hch@infradead.org>
Date: Sun, 19 Jun 2005 10:42:37 +0100

> Sorry Dave, but fc4 is broken beyon imagination and not mentioned in
> the common drivers/Kconfig at all.  My suggestion to fix it is an
> rm -rf.

It doesn't take much work to cross compile check the changes
you made, Al Viro has very easy descriptions of how to set
this up.

Please fix the build, thanks.

Re: recent scsi changes break fc4 build

[David S. Miller]

From: Jeff Garzik <jgarzik@pobox.com>
Date: Sun, 19 Jun 2005 11:43:31 -0400

> Does the attached patch work for you?

Yep, that fixes the build.  Please push upstream :)

Thanks a lot Jeff.