<html> <head> <base href="https://bugs.freedesktop.org/" /> </head> <body> <div> <a class="bz_bug_link bz_status_NEW " title="NEW - [GEN8+] Ungine Valley fails to run due to "intel_do_flush_locked failed: Input/output error"" href="https://bugs.freedesktop.org/show_bug.cgi?id=94924#c5">Comment # 5</a> on <a class="bz_bug_link bz_status_NEW " title="NEW - [GEN8+] Ungine Valley fails to run due to "intel_do_flush_locked failed: Input/output error"" href="https://bugs.freedesktop.org/show_bug.cgi?id=94924">bug 94924</a> from <a class="email" href="mailto:mark.a.janes@intel.com" title="Mark Janes <mark.a.janes@intel.com>"> Mark Janes</a> <pre>The proper bisection is: commit 7b208a731277b4b99b86af3df98c1219099036d7 Author: Matt Turner <<a href="mailto:mattst88@gmail.com">mattst88@gmail.com</a>> Date: Mon Feb 15 10:05:33 2016 -0800 i965: Relax restriction on scheduling last instruction. I think when this code was written, basic blocks were always ended by a control flow instruction or an end-of-thread message. That's no longer the case, and removing this restriction actually helps things: instructions in affected programs: 7267 -> 7244 (-0.32%) helped: 4 total cycles in shared programs: 66559580 -> 66431900 (-0.19%) cycles in affected programs: 28310152 -> 28182472 (-0.45%) helped: 9577 HURT: 879 GAINED: 2 The addition of the is_control_flow() checks is not a functional change, since the add_insts_from_block() does not put them in the list of instructions to schedule. I plan to change this in a later patch. Reviewed-by: Francisco Jerez <<a href="mailto:currojerez@riseup.net">currojerez@riseup.net</a>></pre> </div> <hr> You are receiving this mail because: <ul> <li>You are the QA Contact for the bug.</li> </ul> </body> </html>