[vsipl++] [patch] Avoid invalid DMA sizes for vmul

Jules Bergmann jules at codesourcery.com
Sat Mar 17 02:46:10 UTC 2007


Don McCoy wrote:
> Jules Bergmann wrote:
>> This patch fixes the cleanup code to avoid DMA sizes that aren't a 
>> multiple of 16.  This fixes test failures for coverage_binary.  It 
>> also adds a new regression test that sweeps through vmul sizes from 1 
>> to 128.
>>
>> Don, is this ok to commit?  Is there a better place than bindings.hpp 
>> to put the GRANULARITY macro and is_dma_size_ok() function?
>>
> I think this is the right place.  But there is also a bit of code in the 
> vmul kernels that deals with any leftover values in cases where the 
> length is not a multiple of four floats (16 bytes).  We could probably 
> get rid of that now.
> 

Patch applied.  As before, but with removal of cleanup code for float 
vmul kernel.

-- 
Jules Bergmann
CodeSourcery
jules at codesourcery.com
(650) 331-3385 x705
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: dmasize.diff
URL: <http://sourcerytools.com/pipermail/vsipl++/attachments/20070316/d244c720/attachment.ksh>


More information about the vsipl++ mailing list