aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDarrick J. Wong <djwong@us.ibm.com>2006-08-30 17:18:33 -0400
committerJames Bottomley <jejb@mulgrave.il.steeleye.com>2006-08-30 17:30:06 -0400
commitf19eaa7f53736449a6eac89c3863eca2c64d5913 (patch)
treedd60f697af06bf3b6b1ece3fa7a29485ff2b4e92
parentbc229b3663dcd7d8f266cb13b0839efdee6d95b5 (diff)
[SCSI] aic94xx: Increase can_queue for better performance
This patch sets can_queue in the aic94xx driver's scsi_host to better performing values than what's there currently. It seems that asd_ha->seq.can_queue reflects the number of requests that can be queued per controller; so long as there's one scsi_host per controller, it seems logical that the scsi_host ought to have the same can_queue value. To the best of my (still limited) knowledge, this method provides the correct value. The effect of leaving this value set to 1 is terrible performance in the case of either (a) certain Maxtor SAS drives flying solo or (b) flooding several disks with I/O simultaneously (md-raid). There may be more scenarios where we see similar problems that I haven't uncovered. Signed-off-by: Darrick J. Wong <djwong@us.ibm.com> Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
-rw-r--r--drivers/scsi/aic94xx/aic94xx_init.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/scsi/aic94xx/aic94xx_init.c b/drivers/scsi/aic94xx/aic94xx_init.c
index 3ec2e46f80c6..69aa70887530 100644
--- a/drivers/scsi/aic94xx/aic94xx_init.c
+++ b/drivers/scsi/aic94xx/aic94xx_init.c
@@ -620,6 +620,8 @@ static int __devinit asd_pci_probe(struct pci_dev *dev,
620 asd_ha->hw_prof.bios.present ? "build " : "not present", 620 asd_ha->hw_prof.bios.present ? "build " : "not present",
621 asd_ha->hw_prof.bios.bld); 621 asd_ha->hw_prof.bios.bld);
622 622
623 shost->can_queue = asd_ha->seq.can_queue;
624
623 if (use_msi) 625 if (use_msi)
624 pci_enable_msi(asd_ha->pcidev); 626 pci_enable_msi(asd_ha->pcidev);
625 627