[vsipl++] [patch] Tunable parameters

[Stefan Seefeld]

Jules Bergmann wrote:
> This adds tunable parameters to the CBE vmul and transpose/copy evaluators.
> 
> The idea is to include the problem size (a runtime parameter) into the 
> rt_valid decision.  For Cell, this sends small problem sizes to the PPE 
> where they can be handled faster.
> 
> The threshold is a single parameter.  Tuning is done by finding the 
> cross-over point between PPU only performance with nominal 8 SPU 
> performance.
> 
> To allow tuning data to be recollected after tuning parameters have been 
> set, tunable_threshold() functions should return 0 when 
> VSIP_IMPL_TUNE_MODE is 1.
> 
> Ok to apply?

Jules,

this looks fine. Could you please add a comment (to the rt_valid() 
function, say), clarifying the meaning of 'tunable', i.e. to prevent the 
library from dispatching, thus enabling our benchmarks to measure both, 
PPU, as well as SPU, depending on --svpp-num-spes ?

Thanks,
		Stefan

-- 
Stefan Seefeld
CodeSourcery
stefan at codesourcery.com
(650) 331-3385 x718