The branch main has been updated by hselasky:

URL: 
https://cgit.FreeBSD.org/src/commit/?id=eafc89853835147bcbd019a974ebfa9d3a8b00a7

commit eafc89853835147bcbd019a974ebfa9d3a8b00a7
Author:     Hans Petter Selasky <[email protected]>
AuthorDate: 2021-06-16 13:01:32 +0000
Commit:     Hans Petter Selasky <[email protected]>
CommitDate: 2021-07-12 12:22:29 +0000

    ipoib: Destroying a CQ should never fail.
    
    Remove not needed error handling when destroying a CQ. The function in
    question will later on be updated to return "void".
    
    MFC after:      1 week
    Reviewed by:    kib
    Sponsored by:   Mellanox Technologies // NVIDIA Networking
---
 sys/ofed/drivers/infiniband/ulp/ipoib/ipoib_verbs.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/sys/ofed/drivers/infiniband/ulp/ipoib/ipoib_verbs.c 
b/sys/ofed/drivers/infiniband/ulp/ipoib/ipoib_verbs.c
index c459b3fb55a7..35fd7875b77d 100644
--- a/sys/ofed/drivers/infiniband/ulp/ipoib/ipoib_verbs.c
+++ b/sys/ofed/drivers/infiniband/ulp/ipoib/ipoib_verbs.c
@@ -254,11 +254,9 @@ void ipoib_transport_dev_cleanup(struct ipoib_dev_priv 
*priv)
                clear_bit(IPOIB_PKEY_ASSIGNED, &priv->flags);
        }
 
-       if (ib_destroy_cq(priv->send_cq))
-               ipoib_warn(priv, "ib_cq_destroy (send) failed\n");
+       ib_destroy_cq(priv->send_cq);
 
-       if (ib_destroy_cq(priv->recv_cq))
-               ipoib_warn(priv, "ib_cq_destroy (recv) failed\n");
+       ib_destroy_cq(priv->recv_cq);
 
        ipoib_cm_dev_cleanup(priv);
 
_______________________________________________
[email protected] mailing list
https://lists.freebsd.org/mailman/listinfo/dev-commits-src-main
To unsubscribe, send any mail to "[email protected]"

Reply via email to