Roland Dreier wrote:
I'll roll this into the offending patch (that is in -next). But: > - wqe->recv.sgl[i].to = cpu_to_be64(((u32) wr->sg_list[i].addr) % > - (1UL << (12 + page_size[i]))); > + wqe->recv.sgl[i].to = cpu_to_be64(((u64) wr->sg_list[i].addr) & > + ((1UL << (12 + page_size[i]))-1)); Is this required? Strength reduction optimization should do this automatically (and the code has been there for quite a while, so obviously it isn't causing problems) - R.
Note that wr->sg_list[i].addr was being cast to a u32. That was wrong. -- To unsubscribe from this list: send the line "unsubscribe linux-next" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
