diff options
author | Bart Van Assche <bart.vanassche@wdc.com> | 2018-07-18 12:25:29 -0400 |
---|---|---|
committer | Jason Gunthorpe <jgg@mellanox.com> | 2018-07-24 18:06:37 -0400 |
commit | 12c36dcfd6f48ece2fce4dc8e6ce68224d005d37 (patch) | |
tree | f85a1d0d16f8aa22d90414827fe57e9222561fba /net/smc | |
parent | f112d53b435692331a50a3780e3ad365b66b1a69 (diff) |
net/smc: Remove a WARN_ON() statement
Remove a WARN_ON() statement that verifies 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: Ursula Braun <ubraun@linux.ibm.com>
Signed-off-by: Jason Gunthorpe <jgg@mellanox.com>
Diffstat (limited to 'net/smc')
-rw-r--r-- | net/smc/smc_wr.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/net/smc/smc_wr.c b/net/smc/smc_wr.c index dbd2605d1962..8b9bdd9bc615 100644 --- a/net/smc/smc_wr.c +++ b/net/smc/smc_wr.c | |||
@@ -274,7 +274,6 @@ int smc_wr_reg_send(struct smc_link *link, struct ib_mr *mr) | |||
274 | link->wr_reg.key = mr->rkey; | 274 | link->wr_reg.key = mr->rkey; |
275 | failed_wr = &link->wr_reg.wr; | 275 | failed_wr = &link->wr_reg.wr; |
276 | rc = ib_post_send(link->roce_qp, &link->wr_reg.wr, &failed_wr); | 276 | rc = ib_post_send(link->roce_qp, &link->wr_reg.wr, &failed_wr); |
277 | WARN_ON(failed_wr != &link->wr_reg.wr); | ||
278 | if (rc) | 277 | if (rc) |
279 | return rc; | 278 | return rc; |
280 | 279 | ||