i965/bxt: Fix conservative wm thread counts.
authorBen Widawsky <benjamin.widawsky@intel.com>
Mon, 25 Jan 2016 19:49:10 +0000 (11:49 -0800)
committerBen Widawsky <benjamin.widawsky@intel.com>
Mon, 25 Jan 2016 23:51:17 +0000 (15:51 -0800)
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@lists.freedesktop.org>
Cc: Neil Roberts <neil@linux.intel.com>
Signed-off-by: Ben Widawsky <benjamin.widawsky@intel.com>
Acked-by: Kenneth Graunke <kenneth@whitecape.org>
Tested-by: Mark Janes <mark.a.janes@intel.com>
src/mesa/drivers/dri/i965/brw_device_info.c

index b8148bf701e80c5092db6ded3437b1c096ab2b7f..7ab70fef1e69661155bc19bd46eca52d4f1f6921 100644 (file)
@@ -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,