[Bug 60879] [radeonsi] X11 can't start with acceleration enabled

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Tue Sep 9 19:30:50 PDT 2014


https://bugs.freedesktop.org/show_bug.cgi?id=60879

--- Comment #89 from Michel Dänzer <michel at daenzer.net> ---
(In reply to comment #88)
> Is this patch supposed to apply cleanly against mesa 10.1.5?

No, looks like it's for Git master, should probably apply against the 10.3
branch at least though.


(In reply to comment #87)
> Fix v3
[...]
> +		for (i = 0; i < num_se; i++) {
> +			si_pm4_set_reg(pm4, GRBM_GFX_INDEX,
> +				SE_INDEX(i) |
> +				SH_BROADCAST_WRITES |
> +				INSTANCE_BROADCAST_WRITES);
> +			si_pm4_set_reg(pm4, R_028350_PA_SC_RASTER_CONFIG, raster_config);
> +		}

Since this uses the same raster_config value for all SEs, couldn't it just use
a single write with SE_BROADCAST_WRITES enabled in GRBM_GFX_INDEX?

If not:

> +		unsigned sh_per_se = MAX2(sctx->screen->b.info.max_sh_per_se, 1);
> +		unsigned num_se = MAX2(sctx->screen->b.info.max_sh_per_se, 1);

sh_per_se and num_se have the same value. Should one of them be calculated
differently, or does a single variable suffice?

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20140910/67e08cb1/attachment.html>


More information about the dri-devel mailing list