RDMA/rtrs-clt: Remove outdated comment in create_con_cq_qp
authorJack Wang <jinpu.wang@cloud.ionos.com>
Fri, 23 Oct 2020 07:43:43 +0000 (09:43 +0200)
committerJason Gunthorpe <jgg@nvidia.com>
Wed, 28 Oct 2020 16:17:39 +0000 (13:17 -0300)
As run destroy_con_cq_qp many times doesn't work, remove the comments.

Fixes: 6a98d71daea1 ("RDMA/rtrs: client: main functionality")
Link: https://lore.kernel.org/r/20201023074353.21946-3-jinpu.wang@cloud.ionos.com
Suggested-by: Gioh Kim <gi-oh.kim@cloud.ionos.com>
Signed-off-by: Jack Wang <jinpu.wang@cloud.ionos.com>
Signed-off-by: Jason Gunthorpe <jgg@nvidia.com>
drivers/infiniband/ulp/rtrs/rtrs-clt.c

index 9980bb4a6f78e8d4c1d5cd08054c5009fa7c66ad..fb840b152b37c35413ad4e5aa7853984faa9958f 100644 (file)
@@ -1520,15 +1520,6 @@ static int create_con_cq_qp(struct rtrs_clt_con *con)
        int err, cq_vector;
        struct rtrs_msg_rkey_rsp *rsp;
 
-       /*
-        * This function can fail, but still destroy_con_cq_qp() should
-        * be called, this is because create_con_cq_qp() is called on cm
-        * event path, thus caller/waiter never knows: have we failed before
-        * create_con_cq_qp() or after.  To solve this dilemma without
-        * creating any additional flags just allow destroy_con_cq_qp() be
-        * called many times.
-        */
-
        if (con->c.cid == 0) {
                /*
                 * One completion for each receive and two for each send