On Fri, 18 May 2007 10:34:35 -0400
Jeff Garzik <[email protected]> wrote:
> Alan Cox wrote:
> >> That shouldn't be a problem, libata default DMA mask is 32 bits (which
> >> isn't overridden with this controller) and so the block layer will
> >> bounce any data being read/written above that point with IOMMU or
> >> swiotlb. The comment is a bit unnecessarily scary.
> >
> > Adding a BUG_ON for this would be wise. Its trivial to check and a BUG
> > rather than corruption if this assumption ever changes would be far
> > preferable
>
> The default DMA mask -everywhere- is 32 bits.
>
> A lot of code will break if this assumption ever changes, not just libata.
Little lesson from history..
Over ten years ago someone (Eric Youngdale I guess) stuck a panic check
for DMA over 16MBytes in the AHA 1542 ISA SCSI driver. Last year it
triggered. The panic probably saved someone from corruption and meant the
bug could be fixed.
Alan
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
