All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mario Giammarco <mgiammarco-ml@4warding.com>
To: "Shirron, Stephen" <Stephen.Shirron@lsi.com>
Cc: Mario Giammarco <mgiammarco@gmail.com>,
	"Hickerson, Roger" <Roger.Hickerson@lsi.com>,
	linux-scsi@vger.kernel.org
Subject: Re: LSI Logic 40919o fibre channel: scsi works ip not
Date: Thu, 20 Sep 2007 17:34:12 +0200	[thread overview]
Message-ID: <200709201734.13350.mgiammarco-ml@4warding.com> (raw)
In-Reply-To: <F6F6225F9738D94FB5B5876E3707A919011B4D9C@NAMAIL2.ad.lsil.com>

Il Monday 05 March 2007 20:46:10 Shirron, Stephen ha scritto:
> The code should look like this, in mpt_lan_post_receive_buckets():
>
> 			goto out;
> 		}
> 		pRecvReq = (LANReceivePostRequest_t *) mf;
>
> 		i = le16_to_cpu(mf->u.frame.hwhdr.msgctxu.fld.req_idx);
> 		mpt_dev->RequestNB[i] = 0;
>
> 		count = buckets;
> 		if (count > max)
> 			count = max;
>
> If you send me mptlan.c, I will edit it for you.  Then there cannot
> be a mistake.
>
> stephen
>
> -----Original Message-----
> From: Mario Giammarco [mailto:mgiammarco@gmail.com]
> Sent: Monday, March 05, 2007 2:39 PM
> To: Shirron, Stephen
> Cc: Mario Giammarco; Hickerson, Roger
> Subject: Re: LSI Logic 40919o fibre channel: scsi works ip not
>
> Sorry for delay replying. I have seen kernel 2.6.18 and 2.6.20 source
> and these lines are missinig:
>
>    i = le16_to_cpu(mf->u.frame.hwhdr.msgctxu.fld.req_idx);
>                 mpt_dev->RequestNB[i] = 0;
>
> I do not understand where to put them. Perhaps inside the for loop (i
> is the loop variable...) but I do not understand if I can choose a
> random point or I have to put it in a specific place.
>
> Thanks again for interest!
>
>
> 2007/2/16, Shirron, Stephen <Stephen.Shirron@lsi.com>:
> Mario,
>
> There appears to be a bug in the driver.
>
> Go to mpt_lan_post_receive_buckets(), in mptlan.c.  Find:
>
>                 mf = mpt_get_msg_frame(LanCtx, mpt_dev);
>
> Later find:
>
>                 pRecvReq = (LANReceivePostRequest_t *) mf;
>
> Below that, should be:
>
>                 i = le16_to_cpu(mf->u.frame.hwhdr.msgctxu.fld.req_idx);
>                 mpt_dev->RequestNB[i] = 0;
>
> The above two lines of code are present in some versions of mptctl.c
> (the 2.06.xx versions and 3.02.xx versions) but are missing from some
> others (3.03.xx, 3.04.xx, and 4.00.xx).
>
> If you can add those two lines of code, rebuild mptctl.ko, and retest,
> I believe that should take care of your reported problems.
>
> Let me know, please.
>
> stephen


Hello,
month ago I replied to you saying that this patch solved my bug.

Today I am trying kernel 2.6.22 and I see that it does not contain yet the 
patch.

Can you please put the patch in official driver?

Thank you very much!

Mario Giammarco

  parent reply	other threads:[~2007-09-20 16:21 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-02-15 22:06 LSI Logic 40919o fibre channel: scsi works ip not Mario Giammarco
2007-02-17 16:04 ` Dan Aloni
     [not found] ` <F6F6225F9738D94FB5B5876E3707A9190112A445@NAMAIL2.ad.lsil.com>
2007-02-18 11:15   ` Mario Giammarco
2007-02-27 13:02     ` Shirron, Stephen
     [not found] ` <eaa59be00703051138r1a2382cfmec9d8c3258e557ae@mail.gmail.com>
     [not found]   ` <F6F6225F9738D94FB5B5876E3707A919011B4D9C@NAMAIL2.ad.lsil.com>
2007-09-20 15:34     ` Mario Giammarco [this message]
2007-09-20 17:48       ` Shirron, Stephen
2007-02-20 18:01 Moore, Eric
2007-02-20 18:01 ` Moore, Eric

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=200709201734.13350.mgiammarco-ml@4warding.com \
    --to=mgiammarco-ml@4warding.com \
    --cc=Roger.Hickerson@lsi.com \
    --cc=Stephen.Shirron@lsi.com \
    --cc=linux-scsi@vger.kernel.org \
    --cc=mgiammarco@gmail.com \
    /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.