On Sun, Dec 09, 2018 at 09:54:58AM -0700, Jason Gunthorpe wrote:
> On Sun, Dec 09, 2018 at 12:44:40PM +0200, Leon Romanovsky wrote:
> > -static void handle_atomics(struct mlx5_ib_qp *qp, struct mlx5_cqe64 *cqe64,
> > - u16 tail, u16 head)
> > -{
> > - u16 idx;
> > -
> > - do {
> > - idx = tail & (qp->sq.wqe_cnt - 1);
> > - handle_atomic(qp, cqe64, idx);
> > - if (idx == head)
> > - break;
> > -
> > - tail = qp->sq.w_list[idx].next;
> > - } while (1);
> > - tail = qp->sq.w_list[idx].next;
> > - qp->sq.last_poll = tail;
>
> This line is the only thing that has an external impact, is it OK to
> delete it?
I hope so, no new bugs were found.
Thanks
>
> Jason
Attachment:
signature.asc
Description: PGP signature
