PERFORCE change 152649 for review

Hans Petter Selasky hselasky at
Sat Nov 8 04:57:05 PST 2008

On Saturday 08 November 2008, Alexander Motin wrote:
> Hans Petter Selasky wrote:
> >
> >
> > Change 152649 by hselasky at hselasky_laptop001 on 2008/11/08 09:09:50
> >
> >
> > 	Fix a serious BUSDMA bug: The initial virtual page offset is
> > 	not retained on the bounce pages, which is what the EHCI and
> > 	the OHCI expects.
> I'm sorry, but are you sure with this? If bounce page will have the same
> page offset as original data, then how bus_dma_tag_create()'s alignment
> parameter expected to work? What if data are misaligned, but hardware
> require alignment?


I'm not saying that my patch is fully correct for all cases, but at least it 
solves the USB case. USB uses an alignment of 1-byte for data.

Probably someone needs to sit down a think this through. Like it is now, it is 
very scary with regard to USB!

Maybe it is time to create a new busdma tag flag? BUS_DMA_NOREALIGN?

If this flag is set, busdma should refuse to load the memory if the memory is 
not aligned already to the given aligment in the DMA tag ?


More information about the p4-projects mailing list