diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2012-11-27 09:27:01 -0500 |
---|---|---|
committer | Nicholas Bellinger <nab@linux-iscsi.org> | 2012-11-28 01:49:21 -0500 |
commit | 6b64e1fe2ba80ac3ce12a21f1200bac594fa1a1c (patch) | |
tree | 700d6a4076387f7e2059d70ea005f1a4209d34c7 | |
parent | 3c989d7603872bf878840f7ce3ea49b73bea4c6c (diff) |
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>
-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) | |||
437 | cmd->t_task_lba = get_unaligned_be64(&cdb[12]); | 437 | cmd->t_task_lba = get_unaligned_be64(&cdb[12]); |
438 | 438 | ||
439 | ret = sbc_setup_write_same(cmd, &cdb[10], ops); | 439 | ret = sbc_setup_write_same(cmd, &cdb[10], ops); |
440 | if (ret < 0) | 440 | if (ret) |
441 | return ret; | 441 | return ret; |
442 | break; | 442 | break; |
443 | default: | 443 | default: |
@@ -510,7 +510,7 @@ sbc_parse_cdb(struct se_cmd *cmd, struct sbc_ops *ops) | |||
510 | cmd->t_task_lba = get_unaligned_be64(&cdb[2]); | 510 | cmd->t_task_lba = get_unaligned_be64(&cdb[2]); |
511 | 511 | ||
512 | ret = sbc_setup_write_same(cmd, &cdb[1], ops); | 512 | ret = sbc_setup_write_same(cmd, &cdb[1], ops); |
513 | if (ret < 0) | 513 | if (ret) |
514 | return ret; | 514 | return ret; |
515 | break; | 515 | break; |
516 | case WRITE_SAME: | 516 | case WRITE_SAME: |
@@ -528,7 +528,7 @@ sbc_parse_cdb(struct se_cmd *cmd, struct sbc_ops *ops) | |||
528 | * of byte 1 bit 3 UNMAP instead of original reserved field | 528 | * of byte 1 bit 3 UNMAP instead of original reserved field |
529 | */ | 529 | */ |
530 | ret = sbc_setup_write_same(cmd, &cdb[1], ops); | 530 | ret = sbc_setup_write_same(cmd, &cdb[1], ops); |
531 | if (ret < 0) | 531 | if (ret) |
532 | return ret; | 532 | return ret; |
533 | break; | 533 | break; |
534 | case VERIFY: | 534 | case VERIFY: |