RDMA/hns: Update the prompt message for creating and destroy qp
authorYixian Liu <liuyixian@huawei.com>
Thu, 8 Aug 2019 14:53:43 +0000 (22:53 +0800)
committerDoug Ledford <dledford@redhat.com>
Mon, 12 Aug 2019 14:45:07 +0000 (10:45 -0400)
Current prompt message is uncorrect when destroying qp, add qpn
information when creating qp.

Signed-off-by: Yixian Liu <liuyixian@huawei.com>
Signed-off-by: Lijun Ou <oulijun@huawei.com>
Link: https://lore.kernel.org/r/1565276034-97329-4-git-send-email-oulijun@huawei.com
Signed-off-by: Doug Ledford <dledford@redhat.com>
drivers/infiniband/hw/hns/hns_roce_hw_v2.c
drivers/infiniband/hw/hns/hns_roce_qp.c

index 59f88bf..7cd4b3b 100644 (file)
@@ -4571,8 +4571,7 @@ static int hns_roce_v2_destroy_qp_common(struct hns_roce_dev *hr_dev,
                ret = hns_roce_v2_modify_qp(&hr_qp->ibqp, NULL, 0,
                                            hr_qp->state, IB_QPS_RESET);
                if (ret) {
-                       dev_err(dev, "modify QP %06lx to ERR failed.\n",
-                               hr_qp->qpn);
+                       dev_err(dev, "modify QP to Reset failed.\n");
                        return ret;
                }
        }
@@ -4641,7 +4640,8 @@ static int hns_roce_v2_destroy_qp(struct ib_qp *ibqp, struct ib_udata *udata)
 
        ret = hns_roce_v2_destroy_qp_common(hr_dev, hr_qp, udata);
        if (ret) {
-               dev_err(hr_dev->dev, "Destroy qp failed(%d)\n", ret);
+               dev_err(hr_dev->dev, "Destroy qp 0x%06lx failed(%d)\n",
+                       hr_qp->qpn, ret);
                return ret;
        }
 
index f76617b..f803209 100644 (file)
@@ -1002,7 +1002,8 @@ struct ib_qp *hns_roce_create_qp(struct ib_pd *pd,
                ret = hns_roce_create_qp_common(hr_dev, pd, init_attr, udata, 0,
                                                hr_qp);
                if (ret) {
-                       dev_err(dev, "Create RC QP failed\n");
+                       dev_err(dev, "Create RC QP 0x%06lx failed(%d)\n",
+                               hr_qp->qpn, ret);
                        kfree(hr_qp);
                        return ERR_PTR(ret);
                }