[Bug 101834] WebGL shaders cause system freeze

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Wed Apr 11 04:34:32 UTC 2018


https://bugs.freedesktop.org/show_bug.cgi?id=101834

Timothy Arceri <t_arceri at yahoo.com.au> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|NEW                         |RESOLVED

--- Comment #1 from Timothy Arceri <t_arceri at yahoo.com.au> ---
There has been a series of changes to loop unrolling since this was reported
but this in now working and I believe the final fix was likely the following
patch as this was still crashing when I tested it before working on this fix.
It entirely possible something else fixed it but its working now so closing.

commit 56b867395dee1a48594b27987d3bf68a4e745dda
Author: Timothy Arceri <tarceri at itsqueeze.com>
Date:   Mon Mar 26 10:31:26 2018 +1100

    glsl: fix infinite loop caused by bug in loop unrolling pass

    Just checking for 2 jumps is not enough to be sure we can do a
    complex loop unroll. We need to make sure we also have also found
    2 loop terminators.

    Without this we were attempting to unroll a loop where the second
    jump was nested inside multiple ifs which loop analysis is unable
    to detect as a terminator. We ended up splicing out the first
    terminator but failed to actually unroll the loop, this resulted
    in the creation of a possible infinite loop.

    Fixes: 646621c66da9 "glsl: make loop unrolling more like the nir unrolling
path"

    Tested-by: Gert Wollny <gw.fossdev at gmail.com>
    Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>
    Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=105670

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20180411/cdc8412c/attachment.html>


More information about the dri-devel mailing list