[Beignet] [PATCH] use r112 as source of EOT message

Zhigang Gong zhigang.gong at linux.intel.com
Thu Aug 15 23:51:20 PDT 2013


Nice catch! We have been sufferred from this random hang for such a long time.
Thanks.

On Fri, Aug 16, 2013 at 07:56:08AM +0800, Zou Nan hai wrote:
> Fix random hang cases.
> use r112 as source of EOT message.
> Bspec requires r112-r127 as EOT message source.
> 
> Signed-off-by: Zou Nanhai <nanhai.zou at intel.com>
> ---
>  backend/src/backend/gen_context.cpp |    3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/backend/src/backend/gen_context.cpp b/backend/src/backend/gen_context.cpp
> index 364ed1d..ff18c46 100644
> --- a/backend/src/backend/gen_context.cpp
> +++ b/backend/src/backend/gen_context.cpp
> @@ -822,11 +822,12 @@ namespace gbe
>    }
>  
>    void GenContext::emitEotInstruction(const SelectionInstruction &insn) {
> +    p->MOV(GenRegister::ud8grf(112, 0), GenRegister::ud8grf(0, 0));
>      p->push();
>        p->curr.predicate = GEN_PREDICATE_NONE;
>        p->curr.execWidth = 8;
>        p->curr.noMask = 1;
> -      p->EOT(0);
> +      p->EOT(112);
>      p->pop();
>    }
>  
> -- 
> 1.7.9.5
> 
> _______________________________________________
> Beignet mailing list
> Beignet at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/beignet


More information about the Beignet mailing list