> -----Original Message----- > From: Leon Romanovsky [mailto:leon@xxxxxxxxxx] > Sent: Monday, September 12, 2016 1:33 PM > To: Salil Mehta > Cc: dledford@xxxxxxxxxx; Huwei (Xavier); oulijun; Zhuangyuzeng (Yisen); > xuwei (O); mehta.salil.lnk@xxxxxxxxx; linux-rdma@xxxxxxxxxxxxxxx; > linux-kernel@xxxxxxxxxxxxxxx; Linuxarm > Subject: Re: [PATCH for-next 09/20] IB/hns: Optimize code of aeq and > ceq interrupt handle and fix the bug of qpn > > On Fri, Sep 09, 2016 at 06:30:40PM +0800, Salil Mehta wrote: > > .... > > > + HNS_ROCE_AEQE_EVENT_QP_EVENT_QP_QPN_M, > > + HNS_ROCE_AEQE_EVENT_QP_EVENT_QP_QPN_S); > > + phy_port = roce_get_field(aeqe->event.qp_event.qp, > > + HNS_ROCE_AEQE_EVENT_QP_EVENT_PORT_NUM_M, > > + HNS_ROCE_AEQE_EVENT_QP_EVENT_PORT_NUM_S); > > + if (qpn <= 1) > > + qpn = HNS_ROCE_MAX_PORTS * qpn + phy_port; > > + > > + switch (event_type) { > > + case HNS_ROCE_EVENT_TYPE_INV_REQ_LOCAL_WQ_ERROR: > > + dev_warn(dev, "Invalid Req Local Work Queue Error.\n"); > > + dev_warn(dev, "QP %d, phy_port %d.\n", qpn, phy_port); > > dev_warn like all other printk flavours supports multi-line messages. Yes, agreed. This can be done using single dev_warn as well, like below: dev_warn(dev, "Invalid Req Local Work Queue Error.\n" "QP %d, phy_port %d.\n", qpn, phy_port); Will change and refloat the patch. Thanks Salil > > > + break; -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html