RDS: IB: include faddr in connection log
authorSantosh Shilimkar <santosh.shilimkar@oracle.com>
Mon, 14 Mar 2016 14:43:55 +0000 (07:43 -0700)
committerSantosh Shilimkar <santosh.shilimkar@oracle.com>
Mon, 2 Jan 2017 22:02:41 +0000 (14:02 -0800)
Also use pr_* for it.

Signed-off-by: Santosh Shilimkar <santosh.shilimkar@oracle.com>
net/rds/ib_cm.c
net/rds/ib_recv.c
net/rds/ib_send.c

index 5b2ab95afa072f4970e25bbc856e916fe28187aa..b9da1e59ecc1f164ea3e95faf7789c6b30a78b1e 100644 (file)
@@ -113,19 +113,18 @@ void rds_ib_cm_connect_complete(struct rds_connection *conn, struct rdma_cm_even
        }
 
        if (conn->c_version < RDS_PROTOCOL(3, 1)) {
-               printk(KERN_NOTICE "RDS/IB: Connection to %pI4 version %u.%u failed,"
-                      " no longer supported\n",
-                      &conn->c_faddr,
-                      RDS_PROTOCOL_MAJOR(conn->c_version),
-                      RDS_PROTOCOL_MINOR(conn->c_version));
+               pr_notice("RDS/IB: Connection <%pI4,%pI4> version %u.%u no longer supported\n",
+                         &conn->c_laddr, &conn->c_faddr,
+                         RDS_PROTOCOL_MAJOR(conn->c_version),
+                         RDS_PROTOCOL_MINOR(conn->c_version));
                rds_conn_destroy(conn);
                return;
        } else {
-               printk(KERN_NOTICE "RDS/IB: connected to %pI4 version %u.%u%s\n",
-                      &conn->c_faddr,
-                      RDS_PROTOCOL_MAJOR(conn->c_version),
-                      RDS_PROTOCOL_MINOR(conn->c_version),
-                      ic->i_flowctl ? ", flow control" : "");
+               pr_notice("RDS/IB: connected <%pI4,%pI4> version %u.%u%s\n",
+                         &conn->c_laddr, &conn->c_faddr,
+                         RDS_PROTOCOL_MAJOR(conn->c_version),
+                         RDS_PROTOCOL_MINOR(conn->c_version),
+                         ic->i_flowctl ? ", flow control" : "");
        }
 
        /*
index 606a11f681d28b18879629e758b89f8a9ae8b33f..6803b75eb8bdfeeb2ea3120e1c3c7c7b68893f06 100644 (file)
@@ -980,8 +980,8 @@ void rds_ib_recv_cqe_handler(struct rds_ib_connection *ic,
        } else {
                /* We expect errors as the qp is drained during shutdown */
                if (rds_conn_up(conn) || rds_conn_connecting(conn))
-                       rds_ib_conn_error(conn, "recv completion on %pI4 had status %u (%s), disconnecting and reconnecting\n",
-                                         &conn->c_faddr,
+                       rds_ib_conn_error(conn, "recv completion on <%pI4,%pI4> had status %u (%s), disconnecting and reconnecting\n",
+                                         &conn->c_laddr, &conn->c_faddr,
                                          wc->status,
                                          ib_wc_status_msg(wc->status));
        }
index 84d90c97332f9178552f34f2e13973a351f6213a..19eca5c4c00c646777d6b4445782cb515cb2035a 100644 (file)
@@ -300,8 +300,8 @@ void rds_ib_send_cqe_handler(struct rds_ib_connection *ic, struct ib_wc *wc)
 
        /* We expect errors as the qp is drained during shutdown */
        if (wc->status != IB_WC_SUCCESS && rds_conn_up(conn)) {
-               rds_ib_conn_error(conn, "send completion on %pI4 had status %u (%s), disconnecting and reconnecting\n",
-                                 &conn->c_faddr, wc->status,
+               rds_ib_conn_error(conn, "send completion on <%pI4,%pI4> had status %u (%s), disconnecting and reconnecting\n",
+                                 &conn->c_laddr, &conn->c_faddr, wc->status,
                                  ib_wc_status_msg(wc->status));
        }
 }