[Mesa-dev] [PATCH] radeonsi/compute: Add Sea Islands support

[Tom Stellard]

On Wed, Oct 23, 2013 at 09:47:30AM -0500, Jay Cornwall wrote:
> On 2013-10-22 21:05, Tom Stellard wrote:
> 
> > ---
> >  src/gallium/drivers/radeonsi/radeonsi_compute.c | 15 ++++++++++++---
> >  1 file changed, 12 insertions(+), 3 deletions(-)
> > 
> [..]
> > +		si_pm4_set_reg(pm4, R_00B82C_COMPUTE_MAX_WAVE_ID,
> > +						0x190 /* Default value */);
> 
> Is there any visible effect on performance if this is set very low?
> 

I haven't tested with any other values.

> If there is, perhaps we should compute this value for SI. I think all 
> the necessary parameters are provided by DRM (MAX_SE * MAX_SH_PER_SE * 
> MAX_PIPES * 40 wavefronts per CU, off the top of my head).

The formula from the comment is:

(number of compute units) * 4 * (waves per simd) - 1

I think this may be a simplified version of your formula.

-Tom
> 
> -- 
> Jay Cornwall
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev