net/rds: Remove two WARN_ON() statements
authorBart Van Assche <bart.vanassche@wdc.com>
Wed, 18 Jul 2018 16:25:27 +0000 (09:25 -0700)
committerJason Gunthorpe <jgg@mellanox.com>
Tue, 24 Jul 2018 22:06:36 +0000 (16:06 -0600)
Remove two WARN_ON() statements that verify something that is guaranteed
by the RDMA API, namely that the failed_wr pointer is not touched if an
ib_post_send() call succeeds and that it points at the failed wr if an
ib_post_send() call fails.

Signed-off-by: Bart Van Assche <bart.vanassche@wdc.com>
Acked-by: Santosh Shilimkar <santosh.shilimkar@oracle.com>
Signed-off-by: Jason Gunthorpe <jgg@mellanox.com>
net/rds/ib_frmr.c

index 48332a6..0209cd8 100644 (file)
@@ -137,7 +137,6 @@ static int rds_ib_post_reg_frmr(struct rds_ib_mr *ibmr)
 
        failed_wr = &reg_wr.wr;
        ret = ib_post_send(ibmr->ic->i_cm_id->qp, &reg_wr.wr, &failed_wr);
-       WARN_ON(failed_wr != &reg_wr.wr);
        if (unlikely(ret)) {
                /* Failure here can be because of -ENOMEM as well */
                frmr->fr_state = FRMR_IS_STALE;
@@ -257,7 +256,6 @@ static int rds_ib_post_inv(struct rds_ib_mr *ibmr)
 
        failed_wr = s_wr;
        ret = ib_post_send(i_cm_id->qp, s_wr, &failed_wr);
-       WARN_ON(failed_wr != s_wr);
        if (unlikely(ret)) {
                frmr->fr_state = FRMR_IS_STALE;
                frmr->fr_inv = false;