Re: [PATCH v10 07/22] IB/hns: Add event queue support

From: Leon Romanovsky
Date: Wed Jun 29 2016 - 06:41:58 EST


On Wed, Jun 29, 2016 at 04:53:39PM +0800, oulijun wrote:

> >> +
> >> + for (i = 0; i < npages; ++i)
> >> + if (eq->buf_list[i].buf)
> >
> > Is it possible situation to have eq->buf_list[i].buf == NULL at the
> > middle of iteration?
> >
> We have analysized it according to your reivews. We think that the eq->buf_list[i].buf
> is not NULL at the middle of iteration because only the eq->buf_list[i].buf allocated
> when the hns_roce_free_eq be called. As result, the if (eq->buf_list[i].buf) condition
> should be deleted, right?

Yes, please.

Attachment: signature.asc
Description: Digital signature