diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2012-11-27 15:27:01 +0100 |
|---|---|---|
| committer | Nicholas Bellinger <nab@linux-iscsi.org> | 2012-11-28 07:49:21 +0100 |
| commit | 6b64e1fe2ba80ac3ce12a21f1200bac594fa1a1c (patch) | |
| tree | 700d6a4076387f7e2059d70ea005f1a4209d34c7 /drivers/target | |
| parent | iscsit: use GFP_ATOMIC under spin lock (diff) | |
| download | linux-6b64e1fe2ba80ac3ce12a21f1200bac594fa1a1c.tar.xz linux-6b64e1fe2ba80ac3ce12a21f1200bac594fa1a1c.zip | |
target: update error handling for sbc_setup_write_same()
We recently changed this to return positive subsystem error codes so the
error handling needs to be updated.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Nicholas Bellinger <nab@linux-iscsi.org>
Diffstat (limited to 'drivers/target')
| -rw-r--r-- | drivers/target/target_core_sbc.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/target/target_core_sbc.c b/drivers/target/target_core_sbc.c index 39ced25b70cd..94fd004ea3fc 100644 --- a/drivers/target/target_core_sbc.c +++ b/drivers/target/target_core_sbc.c @@ -437,7 +437,7 @@ sbc_parse_cdb(struct se_cmd *cmd, struct sbc_ops *ops) cmd->t_task_lba = get_unaligned_be64(&cdb[12]); ret = sbc_setup_write_same(cmd, &cdb[10], ops); - if (ret < 0) + if (ret) return ret; break; default: @@ -510,7 +510,7 @@ sbc_parse_cdb(struct se_cmd *cmd, struct sbc_ops *ops) cmd->t_task_lba = get_unaligned_be64(&cdb[2]); ret = sbc_setup_write_same(cmd, &cdb[1], ops); - if (ret < 0) + if (ret) return ret; break; case WRITE_SAME: @@ -528,7 +528,7 @@ sbc_parse_cdb(struct se_cmd *cmd, struct sbc_ops *ops) * of byte 1 bit 3 UNMAP instead of original reserved field */ ret = sbc_setup_write_same(cmd, &cdb[1], ops); - if (ret < 0) + if (ret) return ret; break; case VERIFY: |