RDMA/rtrs-clt: Use %pe to print errors
authorSupriti Singh <supriti.singh@ionos.com>
Mon, 20 Nov 2023 15:41:45 +0000 (16:41 +0100)
committerLeon Romanovsky <leon@kernel.org>
Wed, 22 Nov 2023 11:42:59 +0000 (13:42 +0200)
While printing error, replace %ld by %pe. %pe prints a string
whereas %ld would print an error code.

Signed-off-by: Supriti Singh <supriti.singh@ionos.com>
Signed-off-by: Jack Wang <jinpu.wang@ionos.com>
Signed-off-by: Grzegorz Prajsner <grzegorz.prajsner@ionos.com>
Signed-off-by: Md Haris Iqbal <haris.iqbal@ionos.com>
Link: https://lore.kernel.org/r/20231120154146.920486-9-haris.iqbal@ionos.com
Signed-off-by: Leon Romanovsky <leon@kernel.org>
drivers/infiniband/ulp/rtrs/rtrs-clt.c

index 984a4a1db3c8ed13fa064ecfd8b01b0c8e117e82..f7a5ec39655434d244b5e44eab0a9e1d4f2002b1 100644 (file)
@@ -1391,9 +1391,9 @@ static int alloc_path_reqs(struct rtrs_clt_path *clt_path)
                                      clt_path->max_pages_per_mr);
                if (IS_ERR(req->mr)) {
                        err = PTR_ERR(req->mr);
+                       pr_err("Failed to alloc clt_path->max_pages_per_mr %d: %pe\n",
+                              clt_path->max_pages_per_mr, req->mr);
                        req->mr = NULL;
-                       pr_err("Failed to alloc clt_path->max_pages_per_mr %d\n",
-                              clt_path->max_pages_per_mr);
                        goto out;
                }
 
@@ -2060,10 +2060,8 @@ static int create_cm(struct rtrs_clt_con *con)
                               clt_path->s.dst_addr.ss_family == AF_IB ?
                               RDMA_PS_IB : RDMA_PS_TCP, IB_QPT_RC);
        if (IS_ERR(cm_id)) {
-               err = PTR_ERR(cm_id);
-               rtrs_err(s, "Failed to create CM ID, err: %d\n", err);
-
-               return err;
+               rtrs_err(s, "Failed to create CM ID, err: %pe\n", cm_id);
+               return PTR_ERR(cm_id);
        }
        con->c.cm_id = cm_id;
        con->cm_err = 0;