aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorEric Moore <eric.moore@lsi.com>2007-09-29 12:17:49 -0400
committerJames Bottomley <jejb@mulgrave.localdomain>2007-10-12 14:52:23 -0400
commite1fc2b5167f9c3614099e55d3f49763a26a5a339 (patch)
treeb8ac99997b790e2d55995ba00253d199fd180c11
parentc51d0beaaed902ef48ed1f3b1834d37b7ffd55e8 (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>
-rw-r--r--drivers/message/fusion/mptlan.c2
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;