On 2018/7/27 0:51, Jason Gunthorpe wrote: > On Thu, Jul 19, 2018 at 08:23:01PM +0800, Yixian Liu wrote: > >> +static void hns_roce_set_qps_to_err(struct hns_roce_dev *hr_dev, u32 qpn) >> +{ >> + struct hns_roce_qp *qp; >> + struct ib_qp_attr attr; >> + int attr_mask; >> + int ret; >> + >> + qp = __hns_roce_qp_lookup(hr_dev, qpn); >> + if (!qp) { >> + dev_warn(hr_dev->dev, "no qp can be found!\n"); >> + return; >> + } >> + >> + if (qp->ibqp.pd->uobject) { > > All the places in this patch using pd->uobject should be just > qp->uobject > There is no uobject member in the struct hns_roce_qp, should I use qp->ibqp.uobject for all the places? Thanks > > -- 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