Re: [PATCH 1/3] scsi : megaraid_{mm,mbox}: 64-bit DMA capability checker

[Vasily Averin <vvs@sw.ru>]

Ju, Seokmann wrote:
> Hi,
> With patch, the driver access PCIconfiguration space with dedicated 
> offset to read a signature. If the signature read, it means that the 
> controller has capability to handle 64-bit DMA. 
> Without this patch, the driver blindly claimed 64-bit DMA capability
> without checking with controller.
> The issue has been reported by Vasily Averin. 

> +	if (((magic64 == HBA_SIGNATURE_64_BIT) &&
> +		(adapter->pdev->subsystem_device !=
> +		PCI_SUBSYS_ID_MEGARAID_SATA_150_6)) ||

I would note that I've reported about issue on SATA_150_4 device. I can also
accept that similar patch fixes this issue.

> +		(adapter->pdev->vendor == PCI_VENDOR_ID_LSI_LOGIC && 
> +		adapter->pdev->device == PCI_DEVICE_ID_VERDE) || 
> +		(adapter->pdev->vendor == PCI_VENDOR_ID_LSI_LOGIC && 
> +		adapter->pdev->device == PCI_DEVICE_ID_DOBSON) || 
> +		(adapter->pdev->vendor == PCI_VENDOR_ID_LSI_LOGIC && 
> +		adapter->pdev->device == PCI_DEVICE_ID_LINDSAY) || 
> +		(adapter->pdev->vendor == PCI_VENDOR_ID_DELL && 
> +		adapter->pdev->device == PCI_DEVICE_ID_PERC4_DI_EVERGLADES) || 
> +		(adapter->pdev->vendor == PCI_VENDOR_ID_DELL && 
> +		adapter->pdev->device == PCI_DEVICE_ID_PERC4E_DI_KOBUK)) {

Thank you,
	Vasily Averin

SWsoft Virtuozzo/OpenVZ Linux kernel team