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.
signature.asc
Description: Digital signature
