diff options
| author | Bart Van Assche <bart.vanassche@wdc.com> | 2018-07-18 09:25:27 -0700 |
|---|---|---|
| committer | Jason Gunthorpe <jgg@mellanox.com> | 2018-07-24 16:06:36 -0600 |
| commit | 197c782e3a18cc2de8766dc30a56f37b9b7a9eec (patch) | |
| tree | afa1b74a179ec565a4b84befb408764d072d5e06 | |
| parent | 72bc4d375d89d0ecb0df491c1eada201a995c471 (diff) | |
| download | linux-197c782e3a18cc2de8766dc30a56f37b9b7a9eec.tar.bz2 | |
net/rds: Remove two WARN_ON() statements
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>
| -rw-r--r-- | net/rds/ib_frmr.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/net/rds/ib_frmr.c b/net/rds/ib_frmr.c index 48332a6ed738..0209cd81546c 100644 --- a/net/rds/ib_frmr.c +++ b/net/rds/ib_frmr.c @@ -137,7 +137,6 @@ static int rds_ib_post_reg_frmr(struct rds_ib_mr *ibmr) failed_wr = ®_wr.wr; ret = ib_post_send(ibmr->ic->i_cm_id->qp, ®_wr.wr, &failed_wr); - WARN_ON(failed_wr != ®_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; |