Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1433667 > unrolled thread
| Started by | Leon Romanovsky <leon@kernel.org> |
|---|---|
| First post | 2016-06-29 12:50 +0200 |
| Last post | 2016-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.
Re: [PATCH v10 07/22] IB/hns: Add event queue support Leon Romanovsky <leon@kernel.org> - 2016-06-29 12:50 +0200
| From | Leon Romanovsky <leon@kernel.org> |
|---|---|
| Date | 2016-06-29 12:50 +0200 |
| Subject | Re: [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.
Back to top | Article view | linux.kernel
csiph-web