Mesa (master): i965/bxt: Fix conservative wm thread counts.

Ben Widawsky bwidawsk at kemper.freedesktop.org
Tue Jan 26 00:21:17 UTC 2016


Module: Mesa
Branch: master
Commit: a443b5b7320ef0d8a63faf8a7fc6136f97460cea
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=a443b5b7320ef0d8a63faf8a7fc6136f97460cea

Author: Ben Widawsky <benjamin.widawsky at intel.com>
Date:   Mon Jan 25 11:49:10 2016 -0800

i965/bxt: Fix conservative wm thread counts.

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: Neil Roberts <neil at linux.intel.com>
Signed-off-by: Ben Widawsky <benjamin.widawsky at intel.com>
Acked-by: Kenneth Graunke <kenneth at whitecape.org>
Tested-by: Mark Janes <mark.a.janes 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,




More information about the mesa-commit mailing list