target: Report bad sector in sense data for DIF errors
authorSagi Grimberg <sagig@mellanox.com>
Thu, 23 Jan 2014 17:29:38 +0000 (19:29 +0200)
committerNicholas Bellinger <nab@linux-iscsi.org>
Sat, 25 Jan 2014 06:58:53 +0000 (06:58 +0000)
SPC-4 states that data-integrity errors shall also report
the failed sector in CHECK_CONDITION response sense data
information field.

Signed-off-by: Sagi Grimberg <sagig@mellanox.com>
Signed-off-by: Nicholas Bellinger <nab@linux-iscsi.org>
drivers/target/target_core_sbc.c
drivers/target/target_core_transport.c
include/target/target_core_base.h

index 75364c7f6219b065548c64ef9f2caa86e9fd074d..fa3cae393e13e64056da79e9a8daf43bfe8dc721 100644 (file)
@@ -1131,6 +1131,7 @@ sbc_dif_verify_write(struct se_cmd *cmd, sector_t start, unsigned int sectors,
                        if (rc) {
                                kunmap_atomic(paddr);
                                kunmap_atomic(daddr);
+                               cmd->bad_sector = sector;
                                return rc;
                        }
 
@@ -1191,6 +1192,7 @@ sbc_dif_verify_read(struct se_cmd *cmd, sector_t start, unsigned int sectors,
                        if (rc) {
                                kunmap_atomic(paddr);
                                kunmap_atomic(daddr);
+                               cmd->bad_sector = sector;
                                return rc;
                        }
 
index aebe0bb3f23b82f8d76d803f12159ae14394ef86..51a9736be7269bb1d7f63a2ff20f42aec7eb9bd9 100644 (file)
@@ -2493,6 +2493,19 @@ static int transport_get_sense_codes(
        return 0;
 }
 
+static
+void transport_err_sector_info(unsigned char *buffer, sector_t bad_sector)
+{
+       /* Place failed LBA in sense data information descriptor 0. */
+       buffer[SPC_ADD_SENSE_LEN_OFFSET] = 0xc;
+       buffer[SPC_DESC_TYPE_OFFSET] = 0; /* Information */
+       buffer[SPC_ADDITIONAL_DESC_LEN_OFFSET] = 0xa;
+       buffer[SPC_VALIDITY_OFFSET] = 0x80;
+
+       /* Descriptor Information: failing sector */
+       put_unaligned_be64(bad_sector, &buffer[12]);
+}
+
 int
 transport_send_check_condition_and_sense(struct se_cmd *cmd,
                sense_reason_t reason, int from_transport)
@@ -2695,6 +2708,7 @@ transport_send_check_condition_and_sense(struct se_cmd *cmd,
                /* LOGICAL BLOCK GUARD CHECK FAILED */
                buffer[SPC_ASC_KEY_OFFSET] = 0x10;
                buffer[SPC_ASCQ_KEY_OFFSET] = 0x01;
+               transport_err_sector_info(buffer, cmd->bad_sector);
                break;
        case TCM_LOGICAL_BLOCK_APP_TAG_CHECK_FAILED:
                /* CURRENT ERROR */
@@ -2705,6 +2719,7 @@ transport_send_check_condition_and_sense(struct se_cmd *cmd,
                /* LOGICAL BLOCK APPLICATION TAG CHECK FAILED */
                buffer[SPC_ASC_KEY_OFFSET] = 0x10;
                buffer[SPC_ASCQ_KEY_OFFSET] = 0x02;
+               transport_err_sector_info(buffer, cmd->bad_sector);
                break;
        case TCM_LOGICAL_BLOCK_REF_TAG_CHECK_FAILED:
                /* CURRENT ERROR */
@@ -2715,6 +2730,7 @@ transport_send_check_condition_and_sense(struct se_cmd *cmd,
                /* LOGICAL BLOCK REFERENCE TAG CHECK FAILED */
                buffer[SPC_ASC_KEY_OFFSET] = 0x10;
                buffer[SPC_ASCQ_KEY_OFFSET] = 0x03;
+               transport_err_sector_info(buffer, cmd->bad_sector);
                break;
        case TCM_LOGICAL_UNIT_COMMUNICATION_FAILURE:
        default:
index 0336d70d0d9fbc3bec36322ed0bcd017a2e2ef96..d28418645b000719790c5a4da0989bfd415baffb 100644 (file)
@@ -37,6 +37,9 @@
 /* Used by transport_send_check_condition_and_sense() */
 #define SPC_SENSE_KEY_OFFSET                   2
 #define SPC_ADD_SENSE_LEN_OFFSET               7
+#define SPC_DESC_TYPE_OFFSET                   8
+#define SPC_ADDITIONAL_DESC_LEN_OFFSET         9
+#define SPC_VALIDITY_OFFSET                    10
 #define SPC_ASC_KEY_OFFSET                     12
 #define SPC_ASCQ_KEY_OFFSET                    13
 #define TRANSPORT_IQN_LEN                      224
@@ -560,7 +563,7 @@ struct se_cmd {
        unsigned int            t_prot_nents;
        enum target_prot_ho     prot_handover;
        sense_reason_t          pi_err;
-       u32                     block_num;
+       sector_t                bad_sector;
 };
 
 struct se_ua {