| Summary: | [Angle] Uniform buffer reuse causes flush, creates invalid state | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Kyle Piddington <kpiddington> | ||||||
| Component: | New Bugs | Assignee: | Kyle Piddington <kpiddington> | ||||||
| Status: | RESOLVED FIXED | ||||||||
| Severity: | Normal | CC: | dino, ews-watchlist, gman, kbr, kkinnunen, kondapallykalyan, mmaxfield, webkit-bug-importer | ||||||
| Priority: | P2 | Keywords: | InRadar | ||||||
| Version: | WebKit Nightly Build | ||||||||
| Hardware: | Unspecified | ||||||||
| OS: | Unspecified | ||||||||
| Bug Depends on: | |||||||||
| Bug Blocks: | 240948, 241894 | ||||||||
| Attachments: |
|
||||||||
|
Description
Kyle Piddington
2022-05-24 18:44:53 PDT
Created attachment 459740 [details]
Patch
Note that there are important steps to take when updating ANGLE. See https://trac.webkit.org/wiki/UpdatingANGLE Comment on attachment 459740 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=459740&action=review > Source/ThirdParty/ANGLE/src/libANGLE/renderer/metal/ContextMtl.mm:2224 > { this would need some context in the commit message. If you've not updated your checkout, maybe you need to do pull. The commit messages come now from the commit message instead of ChangeLog files. Do we have any idea in which specific case this happens? In other words, can we write a test for it? Comment on attachment 459740 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=459740&action=review > Source/ThirdParty/ANGLE/src/libANGLE/renderer/metal/ContextMtl.mm:2232 > + ASSERT(mRenderEncoder.valid()); Is this really right? ASSERT()s are supposed to only be for things that are provably true. Created attachment 459782 [details]
For landing
At the time, happened on earth.google.com. Most likely started happening once we increased the max uniforms? Could be tested in WebGL test suite, bug 240948. Committed r294877 (251007@main): <https://commits.webkit.org/251007@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 459782 [details]. |