[Mesa-stable] [PATCH] i965/bxt: Fix conservative wm thread counts.
Mark Janes
mark.a.janes at intel.com
Mon Jan 25 23:26:23 CET 2016
Tested-by: Mark Janes <mark.a.janes at intel.com>
Ben Widawsky <benjamin.widawsky at intel.com> writes:
> When setting the conservative thread counts, I halved everything. That isn't
> correct for the wm, which has nothing to do with actual thread counts. I suck.
>
> BXT only has 1 slice, and there is some ambiguity about subslices, so just
> reserve the max possible for now. It looks like this might fix:
> piglit.spec.glsl-1_50.execution.variable-indexing.gs-output-array-vec4-index-wr.bxtm64.
> I kind of question why that is, but it is what Jenkins says.
>
> Mark is current running some of the other blacklisted tests on this patch. (it
> effects anything requiring scratch space).
>
> Cc: mesa-stable <mesa-stable at lists.freedesktop.org>
> Cc: Kenneth Graunke <kenneth at whitecape.org>
> Cc: Neil Roberts <neil at linux.intel.com>
> Cc: Mark Janes <mark.a.janes at intel.com>
> Signed-off-by: Ben Widawsky <benjamin.widawsky at intel.com>
> ---
> src/mesa/drivers/dri/i965/brw_device_info.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_device_info.c b/src/mesa/drivers/dri/i965/brw_device_info.c
> index b8148bf..7ab70fe 100644
> --- a/src/mesa/drivers/dri/i965/brw_device_info.c
> +++ b/src/mesa/drivers/dri/i965/brw_device_info.c
> @@ -390,7 +390,7 @@ static const struct brw_device_info brw_device_info_bxt = {
> .max_hs_threads = 56,
> .max_ds_threads = 56,
> .max_gs_threads = 56,
> - .max_wm_threads = 32,
> + .max_wm_threads = 64 * 3,
> .max_cs_threads = 28,
> .urb = {
> .size = 64,
> --
> 2.7.0
More information about the mesa-stable
mailing list