Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1433667 > unrolled thread

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

Started byLeon Romanovsky <leon@kernel.org>
First post2016-06-29 12:50 +0200
Last post2016-06-29 12:50 +0200
Articles 1 — 1 participant

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [PATCH v10 07/22] IB/hns: Add event queue support Leon Romanovsky <leon@kernel.org> - 2016-06-29 12:50 +0200

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

FromLeon Romanovsky <leon@kernel.org>
Date2016-06-29 12:50 +0200
SubjectRe: [PATCH v10 07/22] IB/hns: Add event queue support
Message-ID<rPlea-36J-25@gated-at.bofh.it>

[Multipart message — attachments visible in raw view] — view raw

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.

[toc] | [standalone]


Back to top | Article view | linux.kernel


csiph-web