2007-02-08 02:54:53

by Patro, Sumant

[permalink] [raw]
Subject: RE: [PATCH 4/5] scsi: megaraid_sas - preallocate memory for ioctlprocessing

Thanks.
I will resubmit the patch.

Regards,
Sumant

-----Original Message-----
From: James Bottomley [mailto:[email protected]]
Sent: Wednesday, February 07, 2007 2:02 PM
To: Andrew Morton
Cc: Patro, Sumant; [email protected];
[email protected]; Kolli, Neela; Yang, Bo; Patro, Sumant
Subject: Re: [PATCH 4/5] scsi: megaraid_sas - preallocate memory for
ioctlprocessing

On Wed, 2007-02-07 at 13:30 -0800, Andrew Morton wrote:
> I suspect all this horror is due to stupidity in the DMA API.
>
> pci_alloc_consistent() just goes and assumes GFP_ATOMIC, whereas the
> caller (megasas_mgmt_fw_ioctl) would have been perfectly happy to use
> GFP_KERNEL.
>
> I bet this fixes it

It does, but the DMA API was expanded to cope with this exact case, so
use dma_alloc_coherent() directly in the megaraid code instead. The dev
is just &pci_dev->dev.

James