summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorOr Gerlitz <ogerlitz@mellanox.com>2013-05-01 13:25:26 +0000
committerRoland Dreier <roland@purestorage.com>2013-05-01 17:34:13 -0700
commit450d1e40d597dd71843457c73d372c4b4f2a83eb (patch)
tree75a5edd71a2b8e315481490db2fc0e4b5e610a1c
parent4f3638826124b722b2c0dde5097bca182bf74125 (diff)
downloadlinux-stable-450d1e40d597dd71843457c73d372c4b4f2a83eb.tar.gz
linux-stable-450d1e40d597dd71843457c73d372c4b4f2a83eb.tar.bz2
linux-stable-450d1e40d597dd71843457c73d372c4b4f2a83eb.zip
IB/iser: Return error to upper layers on EAGAIN registration failures
Commit 819a087316a6 ("IB/iser: Avoid error prints on EAGAIN registration failures") not only eliminated the error print on that case, but rather also modified the code such that it doesn't return any error to upper layers. As a result a wrong mapping was used. Fix this to correctly return the error in that case. Signed-off-by: Or Gerlitz <ogerlitz@mellanox.com> Signed-off-by: Roland Dreier <roland@purestorage.com>
-rw-r--r--drivers/infiniband/ulp/iser/iser_memory.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/infiniband/ulp/iser/iser_memory.c b/drivers/infiniband/ulp/iser/iser_memory.c
index be1edb04b085..68ebb7fe072a 100644
--- a/drivers/infiniband/ulp/iser/iser_memory.c
+++ b/drivers/infiniband/ulp/iser/iser_memory.c
@@ -416,8 +416,9 @@ int iser_reg_rdma_mem(struct iscsi_iser_task *iser_task,
for (i=0 ; i<ib_conn->page_vec->length ; i++)
iser_err("page_vec[%d] = 0x%llx\n", i,
(unsigned long long) ib_conn->page_vec->pages[i]);
- return err;
}
+ if (err)
+ return err;
}
return 0;
}