svn commit: r265560 - stable/9/sys/dev/isci/scil

[Jim Harris]

Author: jimharris
Date: Wed May  7 16:43:10 2014
New Revision: 265560
URL: http://svnweb.freebsd.org/changeset/base/265560

Log:
  MFC r263275:
  
  isci: Ensure ATA passthrough commands with RETURN_RESPONSE bit set
  translate their response.

Modified:
  stable/9/sys/dev/isci/scil/sati_passthrough.c
Directory Properties:
  stable/9/sys/   (props changed)
  stable/9/sys/dev/   (props changed)

Modified: stable/9/sys/dev/isci/scil/sati_passthrough.c
==============================================================================
--- stable/9/sys/dev/isci/scil/sati_passthrough.c	Wed May  7 16:42:19 2014	(r265559)
+++ stable/9/sys/dev/isci/scil/sati_passthrough.c	Wed May  7 16:43:10 2014	(r265560)
@@ -441,6 +441,13 @@ SATI_STATUS sati_passthrough_16_translat
       sati_set_ata_lba_mid_exp(register_fis, sati_get_cdb_byte(cdb, 9));
       sati_set_ata_lba_high_exp(register_fis, sati_get_cdb_byte(cdb, 11));
    }
+
+   if (PASSTHROUGH_CDB_CK_COND(cdb) ||
+       PASSTHROUGH_CDB_PROTOCOL(cdb) == PASSTHROUGH_RETURN_RESPONSE)
+   {
+      sequence->is_translate_response_required = TRUE;
+   }
+
    sati_set_ata_features(register_fis, sati_get_cdb_byte(cdb, 4));
    sati_set_ata_sector_count(register_fis, sati_get_cdb_byte(cdb, 6));
    sati_set_ata_lba_low(register_fis, sati_get_cdb_byte(cdb, 8));
@@ -483,6 +490,8 @@ SATI_STATUS sati_passthrough_translate_r
       return SATI_FAILURE_CHECK_RESPONSE_DATA;
    }
 
+   sequence->state = SATI_SEQUENCE_STATE_FINAL;
+
    // If the user set the check condition bit, fill out the sense data
    if (PASSTHROUGH_CDB_CK_COND(cdb) ||
        PASSTHROUGH_CDB_PROTOCOL(cdb) == PASSTHROUGH_RETURN_RESPONSE)
@@ -496,10 +505,9 @@ SATI_STATUS sati_passthrough_translate_r
          SCSI_ASC_NO_ADDITIONAL_SENSE,
          SCSI_ASCQ_ATA_PASS_THROUGH_INFORMATION_AVAILABLE
       );
+      return SATI_FAILURE_CHECK_RESPONSE_DATA;
    }
 
-   sequence->state = SATI_SEQUENCE_STATE_FINAL;
-
    return SATI_COMPLETE;
 }