aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDarrick J. Wong <djwong@us.ibm.com>2006-10-13 19:56:25 -0400
committerJames Bottomley <jejb@mulgrave.localdomain>2007-07-18 12:14:07 -0400
commitbdab4e877819cc8b682797901c8b37567fec3c5e (patch)
tree3e89b181f0f42ff1c07635d132ed175b16a1f732
parent797f49de3d95d964a360bcf0167cc20e249bb90b (diff)
[SCSI] libsas: support NCQ for SATA disks
This patch adds SATAII NCQ support to libsas. Both the use_ncq and the dma_xfer flags in ata_task must be set for NCQ to work correctly on the Adaptec SAS controller. The rest of the patch adds ATA_FLAG_NCQ to sata_port_info and sets up ap->scsi_host so that ata_setup_ncq doesn't crash. Please note that this patch is against the aic94xx-sas git tree, not scsi-misc. Thanks also to James Bottomley for providing an earlier version of this patch from which to work. I've tested this patch on a x206m with a ST380819AS SATA2 disk plugged into the Adaptec SAS controller. The drive came up with a queue depth of 31, and I successfully ran an I/O flood test to coerce libata into sending multiple commands simultaneously. A kernel probe recorded the maximum tag number that had been seen before and after the flood test; before the test it was 2 and after it was 30, as I expected. Signed-off-by: Darrick J. Wong <djwong@us.ibm.com> Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
-rw-r--r--drivers/scsi/libsas/sas_scsi_host.c11
1 files changed, 9 insertions, 2 deletions
diff --git a/drivers/scsi/libsas/sas_scsi_host.c b/drivers/scsi/libsas/sas_scsi_host.c
index 3220b3fc6b20..274e7eb47f37 100644
--- a/drivers/scsi/libsas/sas_scsi_host.c
+++ b/drivers/scsi/libsas/sas_scsi_host.c
@@ -828,8 +828,14 @@ static unsigned int sas_ata_qc_issue(struct ata_queued_cmd *qc)
828 task->ata_task.retry_count = 1; 828 task->ata_task.retry_count = 1;
829 task->task_state_flags = SAS_TASK_STATE_PENDING; 829 task->task_state_flags = SAS_TASK_STATE_PENDING;
830 830
831 if (qc->tf.protocol == ATA_PROT_DMA) 831 switch (qc->tf.protocol) {
832 case ATA_PROT_NCQ:
833 task->ata_task.use_ncq = 1;
834 /* fall through */
835 case ATA_PROT_DMA:
832 task->ata_task.dma_xfer = 1; 836 task->ata_task.dma_xfer = 1;
837 break;
838 }
833 839
834 if (sas_ha->lldd_max_execute_num < 2) 840 if (sas_ha->lldd_max_execute_num < 2)
835 res = i->dft->lldd_execute_task(task, 1, GFP_ATOMIC); 841 res = i->dft->lldd_execute_task(task, 1, GFP_ATOMIC);
@@ -962,7 +968,7 @@ static struct ata_port_operations sas_sata_ops = {
962 968
963static struct ata_port_info sata_port_info = { 969static struct ata_port_info sata_port_info = {
964 .flags = ATA_FLAG_SATA | ATA_FLAG_NO_LEGACY | ATA_FLAG_SATA_RESET | 970 .flags = ATA_FLAG_SATA | ATA_FLAG_NO_LEGACY | ATA_FLAG_SATA_RESET |
965 ATA_FLAG_MMIO | ATA_FLAG_PIO_DMA, 971 ATA_FLAG_MMIO | ATA_FLAG_PIO_DMA | ATA_FLAG_NCQ,
966 .pio_mask = 0x1f, /* PIO0-4 */ 972 .pio_mask = 0x1f, /* PIO0-4 */
967 .mwdma_mask = 0x07, /* MWDMA0-2 */ 973 .mwdma_mask = 0x07, /* MWDMA0-2 */
968 .udma_mask = ATA_UDMA6, 974 .udma_mask = ATA_UDMA6,
@@ -1031,6 +1037,7 @@ int sas_target_alloc(struct scsi_target *starget)
1031 1037
1032 ap->private_data = found_dev; 1038 ap->private_data = found_dev;
1033 ap->cbl = ATA_CBL_SATA; 1039 ap->cbl = ATA_CBL_SATA;
1040 ap->scsi_host = shost;
1034 found_dev->sata_dev.ap = ap; 1041 found_dev->sata_dev.ap = ap;
1035 } 1042 }
1036 1043