summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRafael Antognolli <rafael.antognolli@intel.com>2018-01-24 16:33:58 -0800
committerRafael Antognolli <rafael.antognolli@intel.com>2018-01-26 10:07:44 -0800
commitbcfd78e4489f538e34138269650fc6cbe8c9d75f (patch)
tree756539e969a8d70e1bc9594263fc101fcb77a212
parent78c125af3904c539ea69bec2dd9fdf7a5162854f (diff)
i965/gen10: Re-enable push constants.
The GPU hang caused by push constants is apparently fixed, so let's enable them again. Signed-off-by: Rafael Antognolli <rafael.antognolli@intel.com> Cc: "18.0" <mesa-stable@lists.freedesktop.org> Reviewed-by: Kenneth Graunke <kenneth@whitecape.org> Reviewed-by: Jason Ekstrand <jason@jlekstrand.net>
-rw-r--r--src/intel/compiler/brw_fs.cpp9
1 files changed, 0 insertions, 9 deletions
diff --git a/src/intel/compiler/brw_fs.cpp b/src/intel/compiler/brw_fs.cpp
index 09adcbc2df1..6fb46e7374c 100644
--- a/src/intel/compiler/brw_fs.cpp
+++ b/src/intel/compiler/brw_fs.cpp
@@ -2096,15 +2096,6 @@ fs_visitor::assign_constant_locations()
if (subgroup_id_index >= 0)
max_push_components--; /* Save a slot for the thread ID */
- /* FIXME: We currently have some GPU hangs that happen apparently when using
- * push constants. Since we have no solution for such hangs yet, just
- * go ahead and use pull constants for now.
- */
- if (devinfo->gen == 10 && compiler->supports_pull_constants) {
- compiler->shader_perf_log(log_data, "Disabling push constants.");
- max_push_components = 0;
- }
-
/* We push small arrays, but no bigger than 16 floats. This is big enough
* for a vec4 but hopefully not large enough to push out other stuff. We
* should probably use a better heuristic at some point.