diff options
author | Eric Moore <eric.moore@lsi.com> | 2007-09-29 12:17:49 -0400 |
---|---|---|
committer | James Bottomley <jejb@mulgrave.localdomain> | 2007-10-12 14:52:23 -0400 |
commit | e1fc2b5167f9c3614099e55d3f49763a26a5a339 (patch) | |
tree | b8ac99997b790e2d55995ba00253d199fd180c11 /drivers/message | |
parent | c51d0beaaed902ef48ed1f3b1834d37b7ffd55e8 (diff) |
[SCSI] mptlan: bug fix, only half the message frame is dma'd resulting in corruption
NB = number of blocks. This represents the number of blocks to
transfer. The block size is based on the message frame size provided
in the ioc_facts. A value of zero indicates the entire message frame
should be copied. This is two bit value. So by setting this to
non-zero vaule, you increase performance by reducing amount of data
needing to be dma'd. The value that is stored in ioc->ReqeustNB is
sometimes a non-zero vaule, which creates a bug in mptlan, where not
the entire message frame is getting transfer to firware, resulting in
corruption. This fix sets the default to zero, thus entire message
frame is copied.
Signed-off-by: Eric Moore <Eric.Moore@lsi.com>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
Diffstat (limited to 'drivers/message')
-rw-r--r-- | drivers/message/fusion/mptlan.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/message/fusion/mptlan.c b/drivers/message/fusion/mptlan.c index b550471e3042..855832207369 100644 --- a/drivers/message/fusion/mptlan.c +++ b/drivers/message/fusion/mptlan.c | |||
@@ -1224,6 +1224,8 @@ mpt_lan_post_receive_buckets(struct mpt_lan_priv *priv) | |||
1224 | } | 1224 | } |
1225 | pRecvReq = (LANReceivePostRequest_t *) mf; | 1225 | pRecvReq = (LANReceivePostRequest_t *) mf; |
1226 | 1226 | ||
1227 | i = le16_to_cpu(mf->u.frame.hwhdr.msgctxu.fld.req_idx); | ||
1228 | mpt_dev->RequestNB[i] = 0; | ||
1227 | count = buckets; | 1229 | count = buckets; |
1228 | if (count > max) | 1230 | if (count > max) |
1229 | count = max; | 1231 | count = max; |