diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2013-08-15 01:55:41 -0400 |
---|---|---|
committer | Nicholas Bellinger <nab@linux-iscsi.org> | 2013-09-09 17:29:23 -0400 |
commit | fdb4c1f399f7632c326b6f5f4283dbd7a3934dbb (patch) | |
tree | 49a4604a345e0be1a301a1bd0bf3c742d60c142b | |
parent | 988e3a85463d9b6dabc681df3f8f131b23c19953 (diff) |
target/pscsi: remove an unneeded check
blk_get_request() just returns NULL on error, it doesn't return an
ERR_PTR.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Nicholas Bellinger <nab@linux-iscsi.org>
-rw-r--r-- | drivers/target/target_core_pscsi.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/target/target_core_pscsi.c b/drivers/target/target_core_pscsi.c index e992b27aa090..9170b3660318 100644 --- a/drivers/target/target_core_pscsi.c +++ b/drivers/target/target_core_pscsi.c | |||
@@ -1050,9 +1050,8 @@ pscsi_execute_cmd(struct se_cmd *cmd) | |||
1050 | req = blk_get_request(pdv->pdv_sd->request_queue, | 1050 | req = blk_get_request(pdv->pdv_sd->request_queue, |
1051 | (data_direction == DMA_TO_DEVICE), | 1051 | (data_direction == DMA_TO_DEVICE), |
1052 | GFP_KERNEL); | 1052 | GFP_KERNEL); |
1053 | if (!req || IS_ERR(req)) { | 1053 | if (!req) { |
1054 | pr_err("PSCSI: blk_get_request() failed: %ld\n", | 1054 | pr_err("PSCSI: blk_get_request() failed\n"); |
1055 | req ? IS_ERR(req) : -ENOMEM); | ||
1056 | ret = TCM_LOGICAL_UNIT_COMMUNICATION_FAILURE; | 1055 | ret = TCM_LOGICAL_UNIT_COMMUNICATION_FAILURE; |
1057 | goto fail; | 1056 | goto fail; |
1058 | } | 1057 | } |