Bug 92066

Summary: [ILK,G45,regression] New assertion on BRW_MAX_MRF breaks ilk and g45
Product: Mesa Reporter: Mark Janes <mark.a.janes>
Component: Drivers/DRI/i965Assignee: Iago Toral <itoral>
Status: RESOLVED FIXED QA Contact: Intel 3D Bugs Mailing List <intel-3d-bugs>
Severity: blocker    
Priority: medium CC: kenneth, mark.a.janes
Version: git   
Hardware: x86-64 (AMD64)   
OS: Linux (All)   
Whiteboard:
i915 platform: i915 features:

Description Mark Janes 2015-09-21 13:43:37 UTC 
the series d48ac93066190077510d635e71631b6574261d08 - 5d23ce2f15bda866990750b49d7860144dff2e68 generates about 8k assertions in piglit on g45 and ilk:

src/mesa/drivers/dri/i965/brw_eu_emit.c:150: brw_set_dest: Assertion `dest.nr < (devinfo->gen == 6 ? 24 : 16)' failed.

I'm disabling testing on g45 and ilk until this bug is resolved.
Comment 1 Kenneth Graunke 2015-09-21 14:47:28 UTC 
Fixed on master with:

commit c1070550c289d48ef389aeb8c564d1abd1123ad1
Author: Kenneth Graunke <kenneth@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@whitecape.org>

I decided just to push the patch because it was a simple mistake with an obvious fix, and I'd rather not leave every GL program on G45/ILK broken longer than I have to :)

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.