Fix thinko in headroom marking order commit

Fix a thinko bug in commit c45efd89 ("Fix data corruption triggered by
wrong headroom marking order"), which leaves s/g entries being written
in forward (rather than reverse) order.

Signed-off-by: Jack Morgenstein <jackm@dev.mellanox.co.il>
Signed-off-by: Roland Dreier <rolandd@cisco.com>

diff --git a/src/qp.c b/src/qp.c
index 9981c83b49fa3551b15490b8dc4108a7dd2e4569..b13eaad0b6e822cc4a3cd3ea0291bdbfb58398ac 100644 (file)
--- a/src/qp.c
+++ b/src/qp.c
@@ -341,7 +341,7 @@ int mlx4_post_send(struct ibv_qp *ibqp, struct ibv_send_wr *wr,
                } else {
                        struct mlx4_wqe_data_seg *seg = wqe;
 
-                       for (i = 0; i < wr->num_sge; ++i)
+                       for (i = wr->num_sge - 1; i >= 0 ; --i)
                                set_data_seg(seg + i, wr->sg_list + i);
 
                        size += wr->num_sge * (sizeof *seg / 16);