On Fri, Feb 15, 2008 at 08:11:58PM -0600, Vijay Sankar wrote:
> On February 15, 2008 07:49:27 pm Marco Peereboom wrote:
> > Making sure the folks who requested it see this...
> >
> > On Fri, Feb 15, 2008 at 08:21:51PM -0500, Kenneth R Westerback wrote:
> >  The diff below makes mfi use the 'backpressure' facility in the scsi
> >  layer by allowing the retry of an i/o that can't be started. It
> >  would be interesting to see if this helps those who are seeing the
> >  'not queued' errors.
> >
> >  .... Ken
> >
> >  Index: mfi.c
> >  ===================================================================
> >  RCS file: /cvs/src/sys/dev/ic/mfi.c,v
> >  retrieving revision 1.79
> >  diff -u -p -r1.79 mfi.c
> >  --- mfi.c  11 Feb 2008 01:07:02 -0000      1.79
> >  +++ mfi.c  16 Feb 2008 01:14:12 -0000
> >  @@ -986,7 +986,7 @@ mfi_scsi_cmd(struct scsi_xfer *xs)
> >
> >     if ((ccb = mfi_get_ccb(sc)) == NULL) {
> >             DNPRINTF(MFI_D_CMD, "%s: mfi_scsi_cmd no ccb\n", DEVNAME(sc));
> >  -          return (TRY_AGAIN_LATER);
> >  +          return (NO_CCB);
> >     }
> >
> >     xs->error = XS_NOERROR;
> 
> Thank you very much, we will test this next weekend. I hope to update the 
> firmware first, run the large writes to see if we can reproduce the error and 
> then apply this patch.
> 
> -- 
> Vijay Sankar, M.Eng., P.Eng.
> President & CEO
> ForeTell Technologies Limited
> 59 Flamingo Avenue, Winnipeg, MB Canada R3J 0X6
> Phone: +1 204 885 9535, E-Mail: [EMAIL PROTECTED]

Did the change that was committed fix your issues? I never saw a
report that you had been able to test it.

.... Ken

Reply via email to