Re: [RESEND PATCH v7 17/21] IB/hns: Add QP operations support

2016-05-22 Thread Wei Hu (Xavier)
Hi, Doug Ledford In hns_roce_cmd_wait and hns_roce_cmd_poll, there are down seminal operations, So, there are not deadlock and conflict, right? static int hns_roce_cmd_poll(struct hns_roce_dev *hr_dev, u64 in_param, u64 *out_param, unsigned long in_modifier,

Re: [RESEND PATCH v7 17/21] IB/hns: Add QP operations support

2016-05-13 Thread Doug Ledford
On 05/09/2016 11:04 PM, Lijun Ou wrote: > +static void hns_roce_v1_cq_clean(struct hns_roce_cq *hr_cq, u32 qpn, > + struct hns_roce_srq *srq) > +{ > + spin_lock_irq(_cq->lock); > + hns_roce_v1_clean_cq(hr_cq, qpn, srq); > + spin_unlock_irq(_cq->lock); > +}

Re: [RESEND PATCH v7 17/21] IB/hns: Add QP operations support

2016-05-13 Thread Doug Ledford
On 05/09/2016 11:04 PM, Lijun Ou wrote: > +int __hns_roce_cmd(struct hns_roce_dev *hr_dev, u64 in_param, u64 *out_param, > +unsigned long in_modifier, u8 op_modifier, u16 op, > +unsigned long timeout); > + > +/* Invoke a command with no output parameter */ > +static

[RESEND PATCH v7 17/21] IB/hns: Add QP operations support

2016-05-09 Thread Lijun Ou
This patch was implementing for queue pair operations. QP Consists of a Send Work Queue and a Receive Work Queue. Send and receive queues are always created as a pair and remain that way throughout their lifetime. A Queue Pair is identified by its Queue Pair Number. QP operations as follows: