diff options
| author | Roi Dayan <roid@mellanox.com> | 2014-07-31 13:27:51 +0300 |
|---|---|---|
| committer | Roland Dreier <roland@purestorage.com> | 2014-08-01 15:10:05 -0700 |
| commit | 8d4aca7f04a098086c6545df158e5a022cbbfcd1 (patch) | |
| tree | 1ea89e757b40b1cd3092ad41588489072c023184 | |
| parent | 9a6d3234a192d4a3a51df1042c13af13f996242a (diff) | |
| download | linux-8d4aca7f04a098086c6545df158e5a022cbbfcd1.tar.gz linux-8d4aca7f04a098086c6545df158e5a022cbbfcd1.tar.bz2 linux-8d4aca7f04a098086c6545df158e5a022cbbfcd1.zip | |
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 <arieln@mellanox.com>
Signed-off-by: Roi Dayan <roid@mellanox.com>
Signed-off-by: Or Gerlitz <ogerlitz@mellanox.com>
Signed-off-by: Roland Dreier <roland@purestorage.com>
| -rw-r--r-- | drivers/infiniband/ulp/iser/iser_verbs.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/drivers/infiniband/ulp/iser/iser_verbs.c b/drivers/infiniband/ulp/iser/iser_verbs.c index 06a49b3df3fd..3ef167f97d6f 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); |