All of lore.kernel.org
 help / color / mirror / Atom feed
From: KELEMEN Peter <Peter.Kelemen@cern.ch>
To: linux-scsi@vger.kernel.org
Subject: Re: STK9840/SYM53c875 ODD transfer in DATA OUT phase
Date: Fri, 26 Sep 2003 18:39:21 +0200	[thread overview]
Message-ID: <20030926163921.GD32502@chihiro.cern.ch> (raw)
In-Reply-To: <Pine.LNX.4.58.0309260858210.1793@filesrv1.baby-dragons.com>

* Mr. James W. Laferriere (babydr@baby-dragons.com) [20030926 09:03]:

Jes, James,

> On Fri, 26 Sep 2003, Jes Sorensen wrote:
> > Have you tried using a different SCSI controller to see if it
> > is driver related? Any chance you could try 2.4.21?

> If memory serves this is driver related . And IIRC someone
> has made changes or at least put forth a patch to attempt to
> aleviate the odd byte count problem . It is in (Again IIRC) both
> of the Symbois drivers (ie: 53c8xx.c & sym53c8xx_2) . Do try a
> newer kernel than 2.4.21 tho 2.4.22 is what I am running . Hth ,
> JimL

Well, I tried 2.4.22, the change is really small:

--- linux-2.4.20-18.7.cern/drivers/scsi/sym53c8xx_2/sym_hipd.c	2003-07-04 12:22:05.000000000 +0200
+++ linux-2.4.22/drivers/scsi/sym53c8xx_2/sym_hipd.c	2003-08-25 13:44:42.000000000 +0200
@@ -221,7 +221,7 @@
  */
 static void sym_soft_reset (hcb_p np)
 {
-	u_char istat=0;
+	u_char istat;
 	int i;
 
 	if (!(np->features & FE_ISTAT1) || !(INB (nc_istat1) & SCRUN))
@@ -4641,7 +4641,10 @@
 		case M_IGN_RESIDUE:
 			if (DEBUG_FLAGS & DEBUG_POINTER)
 				sym_print_msg(cp,"ign wide residue", np->msgin);
-			sym_modify_dp(np, tp, cp, -1);
+			if (cp->host_flags & HF_SENSE)
+				OUTL_DSP (SCRIPTA_BA (np, clrack));
+			else
+				sym_modify_dp(np, tp, cp, -1);
 			return;
 		case M_REJECT:
 			if (INB (HS_PRT) == HS_NEGOTIATE)

...to no avail, the problem persists.  However, there is a funny
thing I observed: the driver still logs the *same* error message¹
to the ringbuffer when I do even byte transfers, however for some
weird reason the driver reports success, and lo, I can read the
file back and it is intact.  Odd byte transfers trigger the same
condition and the driver reports no sense.  Very strange...

Peter

¹
Sep 26 18:29:06 x kernel: sym0:0:0:extraneous data discarded.
Sep 26 18:29:06 x kernel: sym0:0:0:ODD transfer in DATA OUT phase.
Sep 26 18:29:06 x kernel: sym0:0:0:COMMAND FAILED (87 0 9).

-- 
    .+'''+.         .+'''+.         .+'''+.         .+'''+.         .+''
 Kelemen Péter     /       \       /       \     Peter.Kelemen@cern.ch
.+'         `+...+'         `+...+'         `+...+'         `+...+'
-
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

  reply	other threads:[~2003-09-26 16:39 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-09-26 10:20 STK9840/SYM53c875 ODD transfer in DATA OUT phase KELEMEN Peter
2003-09-26 12:38 ` Jes Sorensen
2003-09-26 13:03   ` Mr. James W. Laferriere
2003-09-26 16:39     ` KELEMEN Peter [this message]
2003-09-26 17:27     ` Tony Battersby
2003-10-10  8:56       ` KELEMEN Peter

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20030926163921.GD32502@chihiro.cern.ch \
    --to=peter.kelemen@cern.ch \
    --cc=linux-scsi@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.