diff options
author | James Bottomley <James.Bottomley@steeleye.com> | 2007-07-16 14:41:04 -0400 |
---|---|---|
committer | James Bottomley <jejb@mulgrave.localdomain> | 2007-07-18 12:16:14 -0400 |
commit | 0f05df8b3b41bc258bdf520b72e8cf7c524048b7 (patch) | |
tree | e6fd95f097da4785a1e0fb195044a745de8e89aa /drivers/scsi/aic94xx | |
parent | 3a2755af37b317d47fdc3dd15178adaf5d47263e (diff) |
[SCSI] libsas, aic94xx: fix dma mapping cockups with ATA
This one was noticed by Gilbert Wu of Adaptec:
The libata core actually does the DMA mapping for you, so there has to
be an exception in the device drivers that *don't* do dma mapping for
ATA commands. However, since we've already done this, libsas must now
dma map any ATA commands that it wishes to issue ... and yes, this is a
horrible mess.
Additionally, the test in aic94xx for ATA protocols isn't quite right.
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
Diffstat (limited to 'drivers/scsi/aic94xx')
-rw-r--r-- | drivers/scsi/aic94xx/aic94xx_task.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/scsi/aic94xx/aic94xx_task.c b/drivers/scsi/aic94xx/aic94xx_task.c index 6c12c0f19f39..d5d8caba3560 100644 --- a/drivers/scsi/aic94xx/aic94xx_task.c +++ b/drivers/scsi/aic94xx/aic94xx_task.c | |||
@@ -76,7 +76,7 @@ static inline int asd_map_scatterlist(struct sas_task *task, | |||
76 | 76 | ||
77 | /* STP tasks come from libata which has already mapped | 77 | /* STP tasks come from libata which has already mapped |
78 | * the SG list */ | 78 | * the SG list */ |
79 | if (task->task_proto == SAS_PROTOCOL_STP) | 79 | if (sas_protocol_ata(task->task_proto)) |
80 | num_sg = task->num_scatter; | 80 | num_sg = task->num_scatter; |
81 | else | 81 | else |
82 | num_sg = pci_map_sg(asd_ha->pcidev, task->scatter, | 82 | num_sg = pci_map_sg(asd_ha->pcidev, task->scatter, |
@@ -125,7 +125,7 @@ static inline int asd_map_scatterlist(struct sas_task *task, | |||
125 | 125 | ||
126 | return 0; | 126 | return 0; |
127 | err_unmap: | 127 | err_unmap: |
128 | if (task->task_proto != SAS_PROTOCOL_STP) | 128 | if (sas_protocol_ata(task->task_proto)) |
129 | pci_unmap_sg(asd_ha->pcidev, task->scatter, task->num_scatter, | 129 | pci_unmap_sg(asd_ha->pcidev, task->scatter, task->num_scatter, |
130 | task->data_dir); | 130 | task->data_dir); |
131 | return res; | 131 | return res; |