Re: WARNING: at drivers/ata/libata-core.c:5988 ata_qc_issue()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, 2008-01-25 at 10:18 -0500, Mark Lord wrote:
> James Bottomley wrote:
> > On Thu, 2008-01-24 at 23:29 -0500, Mark Lord wrote:
> >> Tejun Heo wrote:
> >>> Mark Lord wrote:
> >>> ..
> >>>> Super!  You've done a great job with this stuff, Tejun!
> >>> Thanks but I can't really say nice things about how sata_sil24's
> >>> qc_defer() is implemented or how we generally handle command deferring.
> >>>  We really need the control at the higher level - request_queue group.
> >>> Oh well... I guess you guys will be talking about it over beer again
> >>> soon.  :-)
> >> ..
> >>
> >> You too?
> >>
> >> Another one for those beers, is a way to tell the IOMMU code about
> >> physical segment limitations -- so we can stop having to allocate
> >> PRD tables 2X as big as necessary in drivers like sata_mv.
> > 
> > If the IOMMU would observe the queue dma_boundary parameter, is that
> > enough?  (it is for the 64k PRD limit).  If so, there are already
> > patches in the works to solve this permanently.  If not, could we get
> > the requirements to see how it might be done?
> ..
> 
> That sounds like the right thing.
> 
> We just have ensure that:
> 
> 1. single SG/PRD entries never cross a 64KB address boundary.
> 
> and these two then naturally follow for free:
> 
> 2. single SG/PRD entries never exceed 64KB.
> 3. single SG/PRD entries never cross a 32-bit address boundary.
> 
> Are the patches going into 2.6.25 ?

Tomo can answer that one ... they're in his patch series

> How do we take advantage of them ?

you need to set the dma_boundary in the host template for the driver.
ATA already provides a ATA_DMA_BOUNDARY constant for the 64k case.

James


-
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Filesystems]     [Linux SCSI]     [Linux RAID]     [Git]     [Kernel Newbies]     [Linux Newbie]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Samba]     [Device Mapper]

  Powered by Linux