IB/rxe: avoid calling WARN_ON_ONCE twice

In the exit branch, WARN_ON_ONCE is called to show stack. So it is
not necessary to call WARN_ON_ONCE before going to exit.

Signed-off-by: Zhu Yanjun <yanjun.zhu@oracle.com>
Reviewed-by: Leon Romanovsky <leonro@mellanox.com>
Signed-off-by: Jason Gunthorpe <jgg@mellanox.com>

diff --git a/drivers/infiniband/sw/rxe/rxe_comp.c b/drivers/infiniband/sw/rxe/rxe_comp.c
index a285978aa7fe8061378b8973a36656ba59953e1c..98d470d1f3fc2ee105e1e57f03d9ac0b7dd42b55 100644 (file)
--- a/drivers/infiniband/sw/rxe/rxe_comp.c
+++ b/drivers/infiniband/sw/rxe/rxe_comp.c
@@ -659,7 +659,6 @@ int rxe_completer(void *arg)
                            qp->qp_timeout_jiffies)
                                mod_timer(&qp->retrans_timer,
                                          jiffies + qp->qp_timeout_jiffies);
-                       WARN_ON_ONCE(skb);
                        goto exit;
 
                case COMPST_ERROR_RETRY:
@@ -673,7 +672,6 @@ int rxe_completer(void *arg)
 
                        /* there is nothing to retry in this case */
                        if (!wqe || (wqe->state == wqe_state_posted)) {
-                               WARN_ON_ONCE(skb);
                                goto exit;
                        }
 
@@ -702,7 +700,6 @@ int rxe_completer(void *arg)
                                        skb = NULL;
                                }
 
-                               WARN_ON_ONCE(skb);
                                goto exit;
 
                        } else {
@@ -746,7 +743,6 @@ int rxe_completer(void *arg)
                                skb = NULL;
                        }
 
-                       WARN_ON_ONCE(skb);
                        goto exit;
                }
        }