On Mon, Oct 12, 2009 at 10:04:18AM -0700, Roland Dreier wrote: > > > @@ -1523,6 +1521,7 @@ int mlx4_ib_post_send(struct ib_qp *ibqp, struct > ib_send_wr *wr, > > __be32 *lso_wqe; > > __be32 uninitialized_var(lso_hdr_sz); > > int i; > > + __be32 blh = 0; > > So isn't this a bug if multiple work requests are passed in? Shouldn't > the blh initialization be inside the loop over work requests?
Yes, it's a bug. blh should be set to 0 at the start of the for loop. I'll send another patch with a fix. -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
