Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753039AbYBKJtS (ORCPT ); Mon, 11 Feb 2008 04:49:18 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751430AbYBKJs6 (ORCPT ); Mon, 11 Feb 2008 04:48:58 -0500 Received: from bzq-219-195-70.pop.bezeqint.net ([62.219.195.70]:42805 "EHLO bh-buildlin2.bhalevy.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751356AbYBKJs4 (ORCPT ); Mon, 11 Feb 2008 04:48:56 -0500 Message-ID: <47B019CD.3060502@panasas.com> Date: Mon, 11 Feb 2008 11:47:57 +0200 From: Boaz Harrosh User-Agent: Thunderbird 2.0.0.9 (X11/20071031) MIME-Version: 1.0 To: James Bottomley , Andrew Morton CC: Russell King , Adrian Bunk , FUJITA Tomonori , linux-kernel@vger.kernel.org, linux-scsi@vger.kernel.org Subject: Re: scsi/arm/fas216.c compile error References: <20080209000413.GE6505@cs181133002.pp.htv.fi> <47AEF6FD.8080801@panasas.com> <20080210135851.GB25703@flint.arm.linux.org.uk> <1202653224.3136.20.camel@localhost.localdomain> <20080210220200.GA26654@flint.arm.linux.org.uk> <1202683456.3136.77.camel@localhost.localdomain> In-Reply-To: <1202683456.3136.77.camel@localhost.localdomain> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 5664 Lines: 139 On Mon, Feb 11 2008 at 0:44 +0200, James Bottomley wrote: > On Sun, 2008-02-10 at 22:02 +0000, Russell King wrote: >> On Sun, Feb 10, 2008 at 08:20:24AM -0600, James Bottomley wrote: >>> On Sun, 2008-02-10 at 13:58 +0000, Russell King wrote: >>>> On Sun, Feb 10, 2008 at 03:07:09PM +0200, Boaz Harrosh wrote: >>>>> It's in mainline 84ac86ca8c6787f9efff28bc04b1b65fe0a5c310 >>>>> [SCSI] arm: convert to accessors and !use_sg cleanup >>>>> >>>>> Thanks for checking. This patch was in scsi-pending tree since forever, And we were unable >>>>> to get a responsive maintainer to ACK on them. until the breakage cause went into mainline >>>>> we finally managed a Tested-by:. >>>>> >>>>> I guess sometimes people are so busy, you need a bulldozer to shove 20 minutes into they're >>>>> schedule. >>>> Oh, I was ill for most of December, particularly at the time that you >>>> sent the patch, and by the time I recovered, it was buried in my mailbox. >>>> >>>> Suggest you have some consideration for others who might not be able to >>>> do your beg and call at the immediate moment that you want it, and >>>> consider that their email management skills may not be as l33t as yours. >>> OK, sorry about this, it's a bit of a cockup all around. The patch that >>> fixes this problem is still in SCSI pending largely because it's patch >>> description: >>> >>> [SCSI] fas216: Use scsi_eh API for REQUEST_SENSE invocation >>> >>> Doesn't lead one to think it might be build critical, so I concentrated >>> on getting the other arm patch out. >>> >>> Russell, could you give it a quick test, and I'll put it in with a >>> tested-by tag? >> It's not looking good: >> >> CC drivers/scsi/arm/fas216.o >> drivers/scsi/arm/fas216.c: In function `fas216_rq_sns_done': >> drivers/scsi/arm/fas216.c:2021: warning: passing arg 2 of `scsi_eh_restore_cmnd' from incompatible pointer type >> drivers/scsi/arm/fas216.c: In function `fas216_std_done': >> drivers/scsi/arm/fas216.c:2107: warning: passing arg 2 of `scsi_eh_prep_cmnd' from incompatible pointer type >> >> Since the second argument of scsi_eh_prep_cmnd is 'struct scsi_eh_save *ses' >> this patch is most definitely bad. Not even booted it. > > Yes, there looks to be a fatal screw up in the definition in > FAS216_Info. Could you try this ... I think I've corrected it. > > James > > --- > >>From 35be7297dc581b42c05ea7751ee595b0ce78f669 Mon Sep 17 00:00:00 2001 > From: Boaz Harrosh > Date: Mon, 10 Sep 2007 22:39:11 +0300 > Subject: [SCSI] fas216: Use scsi_eh API for REQUEST_SENSE invocation > > - Use new scsi_eh_prep/restor_cmnd() for synchronous > REQUEST_SENSE invocation. > > Signed-off-by: Boaz Harrosh > Cc: Russell King > Signed-off-by: James Bottomley > --- > drivers/scsi/arm/fas216.c | 16 +++------------- > drivers/scsi/arm/fas216.h | 3 +++ > 2 files changed, 6 insertions(+), 13 deletions(-) > > diff --git a/drivers/scsi/arm/fas216.c b/drivers/scsi/arm/fas216.c > index fb5f202..a715632 100644 > --- a/drivers/scsi/arm/fas216.c > +++ b/drivers/scsi/arm/fas216.c > @@ -2018,6 +2018,7 @@ static void fas216_rq_sns_done(FAS216_Info *info, struct scsi_cmnd *SCpnt, > * the upper layers to process. This would have been set > * correctly by fas216_std_done. > */ > + scsi_eh_restore_cmnd(SCpnt, &info->ses); > SCpnt->scsi_done(SCpnt); > } > > @@ -2103,23 +2104,12 @@ request_sense: > if (SCpnt->cmnd[0] == REQUEST_SENSE) > goto done; > > + scsi_eh_prep_cmnd(SCpnt, &info->ses, NULL, 0, ~0); > fas216_log_target(info, LOG_CONNECT, SCpnt->device->id, > "requesting sense"); > - memset(SCpnt->cmnd, 0, sizeof (SCpnt->cmnd)); > - SCpnt->cmnd[0] = REQUEST_SENSE; > - SCpnt->cmnd[1] = SCpnt->device->lun << 5; > - SCpnt->cmnd[4] = sizeof(SCpnt->sense_buffer); > - SCpnt->cmd_len = COMMAND_SIZE(SCpnt->cmnd[0]); > - SCpnt->SCp.buffer = NULL; > - SCpnt->SCp.buffers_residual = 0; > - SCpnt->SCp.ptr = (char *)SCpnt->sense_buffer; > - SCpnt->SCp.this_residual = sizeof(SCpnt->sense_buffer); > - SCpnt->SCp.phase = sizeof(SCpnt->sense_buffer); > + init_SCp(SCpnt); > SCpnt->SCp.Message = 0; > SCpnt->SCp.Status = 0; > - SCpnt->request_bufflen = sizeof(SCpnt->sense_buffer); > - SCpnt->sc_data_direction = DMA_FROM_DEVICE; > - SCpnt->use_sg = 0; > SCpnt->tag = 0; > SCpnt->host_scribble = (void *)fas216_rq_sns_done; > > diff --git a/drivers/scsi/arm/fas216.h b/drivers/scsi/arm/fas216.h > index 00e5f05..b65f4cf 100644 > --- a/drivers/scsi/arm/fas216.h > +++ b/drivers/scsi/arm/fas216.h > @@ -16,6 +16,8 @@ > #define NO_IRQ 255 > #endif > > +#include > + > #include "queue.h" > #include "msgqueue.h" > > @@ -311,6 +313,7 @@ typedef struct { > > /* miscellaneous */ > int internal_done; /* flag to indicate request done */ > + struct scsi_eh_save ses; /* holds request sense restore info */ > unsigned long magic_end; > } FAS216_Info; > Yes the pointer thing, Thanks James. Andrew this patch was in -mm for two month or so. I was under the impression that you have an arm cross compiler that tries to build every -mm kernel. Is it possible that for some reason this portion did not get compiled? Is there a place that one can inspect the output of -mm compilations, Specially for cross compiled ARCHs? Boaz -- 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/