[Mesa-dev] [PATCH] radeonsi/compute: Add Sea Islands support
tom at stellard.net
Wed Oct 23 20:05:03 CEST 2013
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.
> Jay Cornwall
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
More information about the mesa-dev