[Mesa-dev] [PATCH 4/4] i965/fs: Disable unlit_centroid_workaround on Haswell.

Matt Turner mattst88 at gmail.com
Sun Jun 29 23:18:17 PDT 2014


Although the HSW PRM shows it, the BSpec lists this workaround as being
for Ivybridge only.

total instructions in shared programs: 1994951 -> 1993675 (-0.06%)
instructions in affected programs:     27325 -> 26049 (-4.67%)
---
 src/mesa/drivers/dri/i965/brw_device_info.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/mesa/drivers/dri/i965/brw_device_info.c b/src/mesa/drivers/dri/i965/brw_device_info.c
index f28d754..18e4c80 100644
--- a/src/mesa/drivers/dri/i965/brw_device_info.c
+++ b/src/mesa/drivers/dri/i965/brw_device_info.c
@@ -110,11 +110,11 @@ static const struct brw_device_info brw_device_info_snb_gt2 = {
    .must_use_separate_stencil = true,               \
    .has_llc = true,                                 \
    .has_pln = true,                                 \
-   .has_surface_tile_offset = true,                 \
-   .needs_unlit_centroid_workaround = true
+   .has_surface_tile_offset = true
 
 static const struct brw_device_info brw_device_info_ivb_gt1 = {
    GEN7_FEATURES, .is_ivybridge = true, .gt = 1,
+   .needs_unlit_centroid_workaround = true,
    .max_vs_threads = 36,
    .max_gs_threads = 36,
    .max_wm_threads = 48,
@@ -128,6 +128,7 @@ static const struct brw_device_info brw_device_info_ivb_gt1 = {
 
 static const struct brw_device_info brw_device_info_ivb_gt2 = {
    GEN7_FEATURES, .is_ivybridge = true, .gt = 2,
+   .needs_unlit_centroid_workaround = true,
    .max_vs_threads = 128,
    .max_gs_threads = 128,
    .max_wm_threads = 172,
@@ -141,6 +142,7 @@ static const struct brw_device_info brw_device_info_ivb_gt2 = {
 
 static const struct brw_device_info brw_device_info_byt = {
    GEN7_FEATURES, .is_baytrail = true, .gt = 1,
+   .needs_unlit_centroid_workaround = true,
    .has_llc = false,
    .max_vs_threads = 36,
    .max_gs_threads = 36,
-- 
1.8.3.2



More information about the mesa-dev mailing list