| Message ID | 1473109698-31408-23-git-send-email-jgunthorpe@obsidianresearch.com (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
diff --git a/libnes/src/nes_uverbs.c b/libnes/src/nes_uverbs.c index 80891d6243c7..6458c51c3f55 100644 --- a/libnes/src/nes_uverbs.c +++ b/libnes/src/nes_uverbs.c @@ -518,8 +518,8 @@ int nes_ima_upoll_cq(struct ibv_cq *cq, int num_entries, struct ibv_wc *entry) /* Working on a RQ Completion*/ if (++nesuqp->rq_tail >= nesuqp->rq_size) nesuqp->rq_tail = 0; - if (entry->status == NES_CQ_BUF_OV_ERR) - entry->status = IBV_WC_LOC_LEN_ERR; + if (entry->status == NES_CQ_BUF_OV_ERR) + entry->status = IBV_WC_LOC_LEN_ERR; } if (++head >= cq_size)
../providers/nes/nes_uverbs.c:489:5: warning: this 'if' clause does not guard... [-Wmisleading-indentation] if (++nesuqp->rq_tail >= nesuqp->rq_size) ^~ ../providers/nes/nes_uverbs.c:491:6: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the 'if' if (entry->status == NES_CQ_BUF_OV_ERR) ^~ Presumably this has been tested as is, so I've opted to preserve the behaviour, but I can't tell if that is right or not. Signed-off-by: Jason Gunthorpe <jgunthorpe@obsidianresearch.com> --- libnes/src/nes_uverbs.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)