[Mesa-dev] [PATCH 0/5] Enable up to 24 MRF registers in gen6

Kenneth Graunke kenneth at whitecape.org
Mon Sep 21 07:49:10 PDT 2015


On Monday, September 21, 2015 09:46:24 AM Mark Janes wrote:
> This series hits an assertion on ILK and G45:
> 
> src/mesa/drivers/dri/i965/brw_eu_emit.c:150: brw_set_dest: Assertion
> `dest.nr < (devinfo->gen == 6 ? 24 : 16)' failed.
> 
> It triggers about 8k piglit assertions on those platforms.  I'm turning
> off testing for G45 and ILK until it is resolved.
> 
> https://bugs.freedesktop.org/show_bug.cgi?id=92066
> 
> -Mark

I've pushed a fix for this:

commit c1070550c289d48ef389aeb8c564d1abd1123ad1
Author: Kenneth Graunke <kenneth at whitecape.org>
Date:   Mon Sep 21 07:42:27 2015 -0700

    i965: Fix MRF register number assertions for compr4.
    
    compr4 is represented by setting the high bit on the MRF number.
    We need to mask it out before sanity checking the register number.
    
    Fixes ~8000 assert fails on Ironlake and G45.
    
    Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=92066
    Signed-off-by: Kenneth Graunke <kenneth at whitecape.org>

Easy mistake...I always forget about compr4.  Hopefully we should be
good now.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20150921/a288f2a7/attachment.sig>


More information about the mesa-dev mailing list