On 6/7/2015 9:00 AM, ira.weiny wrote:
@@ -201,9 +202,10 @@ static void qp_event_handler(struct ib_event *event, void *context) case IB_EVENT_QP_ACCESS_ERR: case IB_EVENT_DEVICE_FATAL: default: - dprintk("svcrdma: QP ERROR event %d received for QP=%p, " + dprintk("svcrdma: QP ERROR event %s (%d) received for QP=%p, " "closing transport\n",Generally it is recommended to keep strings on a single line for easier grepping of the code. "However, never break user-visible strings such as printk messages, because that breaks the ability to grep for them."
Hey Ira, Note that this patch did not cause the line split so I prefer that this nit will be addressed in a later patch (along with the rest of the file/module). Sagi. -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
