Summary: | https://tankionline.com/play/ html5 engine not working: crashes. (Metal shader not working) | ||
---|---|---|---|
Product: | WebKit | Reporter: | starfleetcaptain1927 |
Component: | WebGL | Assignee: | Kyle Piddington <kpiddington> |
Status: | RESOLVED FIXED | ||
Severity: | Critical | CC: | bfulgham, dino, ews-watchlist, jonlee, kbr, kkinnunen, kondapallykalyan, kpiddington, moxeve7896, simon.fraser, starfleetcaptain1927, webkit-bug-importer, zalan |
Priority: | P2 | Keywords: | InRadar |
Version: | Safari 15 | ||
Hardware: | All | ||
OS: | iOS 14 | ||
URL: | https://tankionline.com/play | ||
See Also: | https://bugs.webkit.org/show_bug.cgi?id=237458 | ||
Attachments: |
Description
starfleetcaptain1927
2021-10-10 11:42:22 PDT
Hello! Thank you for reporting this. The issue we're having is caused by an unused output variable. The shader in question: // // #version 300 es // #ifdef GL_FRAGMENT_PRECISION_HIGH // precision highp float; // precision highp int; // #else // precision mediump float; // precision mediump int; // #endif // // #define varying in // #define texture2D texture // #define textureCube texture // // out vec4 outColor0; // #define gl_FragColor outColor0 // // // void main() { // } // Doesn't actually do anything with outColor0. However, we still generated an initialization instruction to zero it out, despite not including this output variable from the shader. Thanks for catching this, it's a real bug and we should fix it up. Created attachment 440847 [details]
Patch
Comment on attachment 440847 [details]
Patch
Looks good to me assuming all tests pass. Would you be able to put up the new test as a pull request against the WebGL conformance suite? r+
Comment on attachment 440847 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=440847&action=review > LayoutTests/webgl/pending/resources/webgl_test_files/conformance2/glsl3/empty-shader-with-output.html:27 > +out vec4 outColor0; > +#define gl_FragColor outColor0 I don't understand this. Comment on attachment 440847 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=440847&action=review >> LayoutTests/webgl/pending/resources/webgl_test_files/conformance2/glsl3/empty-shader-with-output.html:27 >> +#define gl_FragColor outColor0 > > I don't understand this. We can probably drop this, it was directly from the shader in this bug. Created attachment 440849 [details]
Patch
Glad to help! You guys are awesome :) (In reply to Kyle Piddington from comment #2) > Hello! Thank you for reporting this. > > The issue we're having is caused by an unused output variable. The shader in > question: > // > // #version 300 es > // #ifdef GL_FRAGMENT_PRECISION_HIGH > // precision highp float; > // precision highp int; > // #else > // precision mediump float; > // precision mediump int; > // #endif > // > // #define varying in > // #define texture2D texture > // #define textureCube texture > // > // out vec4 outColor0; > // #define gl_FragColor outColor0 > // > // > // void main() { > // } > // > > Doesn't actually do anything with outColor0. However, we still generated an > initialization instruction to zero it out, despite not including this output > variable from the shader. > > Thanks for catching this, it's a real bug and we should fix it up. Comment on attachment 440849 [details]
Patch
The code just changed to add a new argument to RemoveInactiveInterfaceVariables. Is there a test covering this change?
(In reply to Kenneth Russell from comment #9) > Comment on attachment 440849 [details] > Patch > > The code just changed to add a new argument to > RemoveInactiveInterfaceVariables. Is there a test covering this change? Hey Ken, This new test should still cover this change, as well as running our existing test suite. Moving RemoveInactiveInterfaceVariables to after rewritePipelines broke some existing tests, hence the new updates. Though you remind me that I need to update the changelog as well! Note that there are important steps to take when updating ANGLE. See https://trac.webkit.org/wiki/UpdatingANGLE Comment on attachment 440849 [details]
Patch
Still r+ once the commit message is updated. Glad tests caught the problem with the first patch.
Created attachment 441270 [details]
Patch for landing
Committed r284197 (243011@main): <https://commits.webkit.org/243011@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 441270 [details]. Created attachment 441306 [details]
Hi sorry new bug :) when trying to join a battle it throws this error. The ID changes every few seconds. Probably the 3D rendering resources
Hi sorry new bug :) when trying to join a battle it throws this error. The ID changes every few seconds. Probably the 3D rendering resources
Please file a new bug rather than modifying this old one. Thanks. I think those errors are cross-origin resource loads failing. *** Bug 233096 has been marked as a duplicate of this bug. *** |