Alejandro Piñeiro <apinhe...@igalia.com> writes:

> Among other thigs, fix a gpu hang when using INTEL_DEBUG=shader_time
> for any shader.
>
> Signed-off-by: Jason Ekstrand <ja...@jlekstrand.net>
> Signed-off-by: Alejandro Piñeiro <apinhe...@igalia.com>
> ---
>
> This is the change suggested by Jason (so I added him as signed-off). I just
> run a full piglit run to check that there isn't any regression.
>
>  src/mesa/drivers/dri/i965/brw_eu_emit.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_eu_emit.c 
> b/src/mesa/drivers/dri/i965/brw_eu_emit.c
> index 2538f0d..8218f9c 100644
> --- a/src/mesa/drivers/dri/i965/brw_eu_emit.c
> +++ b/src/mesa/drivers/dri/i965/brw_eu_emit.c
> @@ -2909,7 +2909,7 @@ brw_set_dp_untyped_atomic_message(struct brw_codegen *p,
>  
>     if (devinfo->gen >= 8 || devinfo->is_haswell) {
>        if (brw_inst_access_mode(devinfo, p->current) == BRW_ALIGN_1) {
> -         if (brw_inst_exec_size(devinfo, p->current) == BRW_EXECUTE_8)
> +         if (brw_inst_exec_size(devinfo, p->current) != BRW_EXECUTE_16)
>              msg_control |= 1 << 4; /* SIMD8 mode */
>  
>           brw_inst_set_dp_msg_type(devinfo, insn,
> @@ -2922,7 +2922,7 @@ brw_set_dp_untyped_atomic_message(struct brw_codegen *p,
>        brw_inst_set_dp_msg_type(devinfo, insn,
>                                 GEN7_DATAPORT_DC_UNTYPED_ATOMIC_OP);
>  
> -      if (brw_inst_exec_size(devinfo, p->current) == BRW_EXECUTE_8)
> +      if (brw_inst_exec_size(devinfo, p->current) != BRW_EXECUTE_16)

LGTM,

Reviewed-by: Francisco Jerez <curroje...@riseup.net>

>           msg_control |= 1 << 4; /* SIMD8 mode */
>     }
>  
> -- 
> 2.7.4

Attachment: signature.asc
Description: PGP signature

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to