James Bottomley wrote: > This is unnecessary unless you alter it before host alloc (which is > where it takes the shost values from the template). > > Also, I think if you look at the rest of the driver, it's careful to > account for the need for required scbs in its internal queueing > algorithms, so the ASD_FREE_SCBS should be unnecessary. > >> + shost->can_queue = aic94xx_sht.can_queue; Ok then, I think it collapses to this short patch: --D Signed-off-by: Darrick J. Wong <djwong@xxxxxxxxxx> diff --git a/drivers/scsi/aic94xx/aic94xx_init.c b/drivers/scsi/aic94xx/aic94xx_init.c index 3e25e31..861d67b 100644 --- a/drivers/scsi/aic94xx/aic94xx_init.c +++ b/drivers/scsi/aic94xx/aic94xx_init.c @@ -623,6 +623,8 @@ static int __devinit asd_pci_probe(struc asd_ha->hw_prof.bios.present ? "build " : "not present", asd_ha->hw_prof.bios.bld); + shost->can_queue = asd_ha->hw_prof.max_scbs; + if (use_msi) pci_enable_msi(asd_ha->pcidev); - To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html