diff options
| author |   Or Gerlitz <ogerlitz@mellanox.com> | 2013-05-01 13:25:26 +0000 |
|---|---|---|
| committer |   Roland Dreier <roland@purestorage.com> | 2013-05-01 17:34:13 -0700 |
| commit | 450d1e40d597dd71843457c73d372c4b4f2a83eb (patch) | |
| tree | 75a5edd71a2b8e315481490db2fc0e4b5e610a1c /drivers/infiniband | |
| parent | IB/iser: Move informational messages from error to info level (diff) | |
| download | linux-dev-450d1e40d597dd71843457c73d372c4b4f2a83eb.tar.xz linux-dev-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>
Diffstat (limited to 'drivers/infiniband')
| -rw-r--r-- | drivers/infiniband/ulp/iser/iser_memory.c | 3 |
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; } |