Call for testing and review, busdma changes

Marius Strobl marius at alchemy.franken.de
Sun Dec 16 21:16:26 UTC 2012


On Sat, Dec 08, 2012 at 08:51:12AM -1000, Jeff Roberson wrote:
> Hello,
> 
> http://people.freebsd.org/~jeff/physbio.diff
> 
> I have a relative large patch that reforms the busdma API so that new 
> types may be added without modifying every architecture's 
> busdma_machdep.c.  It does this by unifying the bus_dmamap_load_buffer() 
> routines so that they may be called from MI code.  The MD busdma is then 
> given a chance to do any final processing in the complete() callback. 
> This patch also contains cam changes to unify the bus_dmamap_load* 
> handling in cam drivers.
> 
> The arm and mips implementations saw the largest changes since they have 
> to track virtual addresses for sync().  Previously this was done in a type 
> specific way.  Now it is done in a generic way by recording the list of 
> virtuals in the map.
> 
> I have verified that this patch passes make universe which includes 
> several kernel builds from each architecture.  I suspect that if I broke 
> anything your machine simply won't boot or will throw I/O errors.  There 
> is little subtlety, it is mostly refactoring.
> 
> The next step is to allow for dma loading of physical addresses.  This 
> will permit unmapped I/O.  Which is a significant performance optimization 
> targeted for 10.0.
> 
> Many thanks for your assistance.  Any review feedback is also appreciated.
> 

Survives a buildworld on sparc64; tested with mpt(4) and sym(4).

Marius



More information about the freebsd-arm mailing list