IB/ipath: Fix problem with next WQE after a UC completion

This patch fixes a bug introduced when moving some code around for
readability.

Setting the wqe pointer at the end of the function is a NOP since it
isn't used.  Move it back to where it is used.

Signed-off-by: Ralph Campbell <ralph.campbell@qlogic.com>
Signed-off-by: Roland Dreier <rolandd@cisco.com>

1 files changed, 3 insertions, 2 deletions

diff --git a/drivers/infiniband/hw/ipath/ipath_uc.c b/drivers/infiniband/hw/ipath/ipath_uc.c
index 1c2b03c2ef5e..49d650cabccf 100644
--- a/drivers/infiniband/hw/ipath/ipath_uc.c
+++ b/drivers/infiniband/hw/ipath/ipath_uc.c
@@ -58,7 +58,6 @@ static void complete_last_send(struct ipath_qp *qp, struct ipath_swqe *wqe,
                 wc->port_num = 0;
                 ipath_cq_enter(to_icq(qp->ibqp.send_cq), wc, 0);
         }
-        wqe = get_swqe_ptr(qp, qp->s_last);
 }
 
 /**
@@ -97,8 +96,10 @@ int ipath_make_uc_req(struct ipath_qp *qp,
                  * Signal the completion of the last send
                  * (if there is one).
                  */
-                if (qp->s_last != qp->s_tail)
+                if (qp->s_last != qp->s_tail) {
                         complete_last_send(qp, wqe, &wc);
+                        wqe = get_swqe_ptr(qp, qp->s_last);
+                }
 
                 /* Check if send work queue is empty. */
                 if (qp->s_tail == qp->s_head)