From: Roi Dayan Date: Thu, 31 Jul 2014 10:27:51 +0000 (+0300) Subject: IB/iser: Clarify a duplicate counters check X-Git-Tag: v4.0~1350^2^4 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=8d4aca7f04a098086c6545df158e5a022cbbfcd1;p=platform%2Fkernel%2Flinux-amlogic.git IB/iser: Clarify a duplicate counters check This is to prevent someone from thinking that this code section is redundant. Signed-off-by: Ariel Nahum Signed-off-by: Roi Dayan Signed-off-by: Or Gerlitz Signed-off-by: Roland Dreier --- diff --git a/drivers/infiniband/ulp/iser/iser_verbs.c b/drivers/infiniband/ulp/iser/iser_verbs.c index 06a49b3..3ef167f 100644 --- a/drivers/infiniband/ulp/iser/iser_verbs.c +++ b/drivers/infiniband/ulp/iser/iser_verbs.c @@ -783,7 +783,10 @@ static void iser_disconnected_handler(struct rdma_cm_id *cma_id) iser_err("iscsi_iser connection isn't bound\n"); } - /* Complete the termination process if no posts are pending */ + /* Complete the termination process if no posts are pending. This code + * block also exists in iser_handle_comp_error(), but it is needed here + * for cases of no flushes at all, e.g. discovery over rdma. + */ if (ib_conn->post_recv_buf_count == 0 && (atomic_read(&ib_conn->post_send_buf_count) == 0)) { complete(&ib_conn->flush_completion);