On 2021/2/10 1:13, Jason Gunthorpe wrote: > On Tue, Feb 09, 2021 at 04:38:43PM +0800, Weihang Li wrote: >> @@ -545,7 +545,10 @@ static int set_rc_opcode(struct hns_roce_v2_rc_send_wqe *rc_sq_wqe, >> rc_sq_wqe->va = cpu_to_le64(atomic_wr(wr)->remote_addr); >> break; >> case IB_WR_REG_MR: >> - set_frmr_seg(rc_sq_wqe, reg_wr(wr)); >> + if (hr_dev->pci_dev->revision >= PCI_REVISION_ID_HIP09) >> + set_frmr_seg(rc_sq_wqe, reg_wr(wr)); >> + else >> + ret = -EOPNOTSUPP; > > The driver must also clear IB_DEVICE_MEM_MGT_EXTENSIONS if it won't > support IB_WR_REG_MR > > Jason > Thanks for the reminder, I will fix it. Weihang