From: Eric Anholt Date: Thu, 9 May 2013 16:00:11 +0000 (-0700) Subject: i965/fs: Fix segfault in instruction scheduling with LINTERP using last GRF. X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=9a31c4f9ace9ed115e54df8bc769ea9748f25d39;p=mesa.git i965/fs: Fix segfault in instruction scheduling with LINTERP using last GRF. The scheduler didn't know about uniform-type accesses, and if a uniform access was last in a 16-wide, we'd walk off the end of the array. This never happened, because we'd never coalesce out all the GRFs, due to a bug to be fixed in the next commit. Reviewed-by: Kenneth Graunke --- diff --git a/src/mesa/drivers/dri/i965/brw_schedule_instructions.cpp b/src/mesa/drivers/dri/i965/brw_schedule_instructions.cpp index 6a527544553..ccedee3ba58 100644 --- a/src/mesa/drivers/dri/i965/brw_schedule_instructions.cpp +++ b/src/mesa/drivers/dri/i965/brw_schedule_instructions.cpp @@ -579,7 +579,10 @@ fs_instruction_scheduler::calculate_deps() (inst->src[i].fixed_hw_reg.file == BRW_GENERAL_REGISTER_FILE)) { if (post_reg_alloc) { - for (int r = 0; r < reg_width; r++) + int size = reg_width; + if (inst->src[i].fixed_hw_reg.vstride == BRW_VERTICAL_STRIDE_0) + size = 1; + for (int r = 0; r < size; r++) add_dep(last_grf_write[inst->src[i].fixed_hw_reg.nr + r], n); } else { add_dep(last_fixed_grf_write, n); @@ -684,7 +687,10 @@ fs_instruction_scheduler::calculate_deps() (inst->src[i].fixed_hw_reg.file == BRW_GENERAL_REGISTER_FILE)) { if (post_reg_alloc) { - for (int r = 0; r < reg_width; r++) + int size = reg_width; + if (inst->src[i].fixed_hw_reg.vstride == BRW_VERTICAL_STRIDE_0) + size = 1; + for (int r = 0; r < size; r++) add_dep(n, last_grf_write[inst->src[i].fixed_hw_reg.nr + r]); } else { add_dep(n, last_fixed_grf_write);