path: root/net/rds
diff options
authorBart Van Assche <bart.vanassche@wdc.com>2018-07-18 09:25:27 -0700
committerJason Gunthorpe <jgg@mellanox.com>2018-07-24 16:06:36 -0600
commit197c782e3a18cc2de8766dc30a56f37b9b7a9eec (patch)
treeafa1b74a179ec565a4b84befb408764d072d5e06 /net/rds
parent72bc4d375d89d0ecb0df491c1eada201a995c471 (diff)
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>
Diffstat (limited to 'net/rds')
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 = &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;