From mboxrd@z Thu Jan 1 00:00:00 1970 From: bugzilla-daemon@freedesktop.org Subject: [Bug 111241] Shadertoy shader causing hang Date: Mon, 02 Sep 2019 23:47:12 +0000 Message-ID: References: Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============1391691096==" Return-path: Received: from culpepper.freedesktop.org (culpepper.freedesktop.org [131.252.210.165]) by gabe.freedesktop.org (Postfix) with ESMTP id 15A91898BC for ; Mon, 2 Sep 2019 23:47:12 +0000 (UTC) In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org --===============1391691096== Content-Type: multipart/alternative; boundary="15674680310.E5af.22215" Content-Transfer-Encoding: 7bit --15674680310.E5af.22215 Date: Mon, 2 Sep 2019 23:47:11 +0000 MIME-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Bugzilla-URL: http://bugs.freedesktop.org/ Auto-Submitted: auto-generated https://bugs.freedesktop.org/show_bug.cgi?id=3D111241 Timothy Arceri changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution|--- |FIXED --- Comment #13 from Timothy Arceri --- (In reply to Dieter N=C3=BCtzel from comment #12) > (In reply to Pierre-Eric Pelloux-Prayer from comment #11) > > (In reply to Dieter N=C3=BCtzel from comment #8) > > > BTW > > >=20 > > > Pierre-Eric can you look into this > > >=20 > > > Shadertoy shader corruption, too? > > > https://www.shadertoy.com/view/Xt3cWS > > > > >=20 > > The "Buffer A" shader doesn't write fragColor when uv.y is < 0.1 or > 0= .9. > >=20 > > So the content is undefined and may be black on some platform or random. > >=20 > > radeonsi is correct here, but we might want to replace undef values wit= h 0x0 > > to get a default value instead of random. >=20 > Cool to have you around for bug hunting...;-) >=20 > Any hints where I shoud change 'undef values with 0x0' for testing? >=20 > And sorry that I 'hijacked' this thread - should I open a new ticket? I don't think you need to open a bug for it at all. As its not a bug in Mesa its a shader bug. Closing this bug report as it should be fixed by: commit 47cc660d9c19572e5ef2dce7c8ae1766a2ac9885 glsl: replace 'x + (-x)' with constant 0 This fixes a hang in shadertoy for radeonsi where a buffer was initializ= ed with: value -=3D value with value being undefined. In this case LLVM replace the operation with an assignment to NaN. Cc: 19.1 19.2 Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=3D111241 Reviewed-by: Marek Ol=C5=A1=C3=A1k --=20 You are receiving this mail because: You are the assignee for the bug.= --15674680310.E5af.22215 Date: Mon, 2 Sep 2019 23:47:11 +0000 MIME-Version: 1.0 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Bugzilla-URL: http://bugs.freedesktop.org/ Auto-Submitted: auto-generated Timothy Arceri changed bug 11124= 1
What Removed Added
Status NEW RESOLVED
Resolution --- FIXED

Comme= nt # 13 on bug 11124= 1 from Timothy Arceri
(In reply to Dieter N=C3=BCtzel from comment #12)
> (In reply to Pierre-Eric Pelloux-Prayer from comment #11)
> > (In reply to Dieter N=C3=BCtzel from comment #8)
> > > BTW
> > >=20
> > > Pierre-Eric can you look into this
> > >=20
> > > Shadertoy shader corruption, too?
> > > https://ww=
w.shadertoy.com/view/Xt3cWS
> > >
> >=20
> > The "Buffer A" shader doesn't write fragColor when uv.y=
 is < 0.1 or > 0.9.
> >=20
> > So the content is undefined and may be black on some platform or =
random.
> >=20
> > radeonsi is correct here, but we might want to replace undef valu=
es with 0x0
> > to get a default value instead of random.
>=20
> Cool to have you around for bug hunting...;-)
>=20
> Any hints where I shoud change 'undef values with 0x0' for testing?
>=20
> And sorry that I 'hijacked' this thread - should I open a new ticket?<=
/span >

I don't think you need to open a bug for it at all. As its not a bug in Mesa
its a shader bug.

Closing this bug report as it should be fixed by:

commit  47cc660d9c19572e5ef2dce7c8ae1766a2ac9885
glsl: replace 'x + (-x)' with constant 0
   This fixes a hang in shadertoy for radeonsi where a buffer was initializ=
ed
with:

   value -=3D value

   with value being undefined.
   In this case LLVM replace the operation with an assignment to NaN.

   Cc: 19.1 19.2 <mesa-stable@lists.freedesktop.org>
   Bugzilla: https://bugs.freedesktop.org/show_bug.=
cgi?id=3D111241
   Reviewed-by: Marek Ol=C5=A1=C3=A1k <marek.olsak@amd.com>


You are receiving this mail because:
  • You are the assignee for the bug.
= --15674680310.E5af.22215-- --===============1391691096== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVs IG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlz dHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVs --===============1391691096==--