diff options
| author | Ralph Campbell <ralph.campbell@qlogic.com> | 2007-06-19 01:24:37 +0400 |
|---|---|---|
| committer | Roland Dreier <rolandd@cisco.com> | 2007-07-10 07:12:26 +0400 |
| commit | 2c9749c3b534ea0e606b7ee2c29849bbb8d5b0a9 (patch) | |
| tree | 50e4bb91927838c2184d2e90240168096bf234f1 | |
| parent | fdc7215fbd7b7652b052d1fb7893afe324dba7aa (diff) | |
| download | linux-2c9749c3b534ea0e606b7ee2c29849bbb8d5b0a9.tar.xz | |
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>
| -rw-r--r-- | drivers/infiniband/hw/ipath/ipath_uc.c | 5 |
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) |