Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WebGPU] Release assert trigged in PipelineLayout::offsetVectorForBindGroup #28716

Conversation

mwyrzykowski
Copy link
Contributor

@mwyrzykowski mwyrzykowski commented May 17, 2024

cd8059f

[WebGPU] Release assert trigged in PipelineLayout::offsetVectorForBindGroup
https://bugs.webkit.org/show_bug.cgi?id=274317
<radar://128065029>

Reviewed by Dan Glastonbury.

Invalid bind group should return nullptr as it already does in other
places in this member function instead of crashing the GPU process.

* LayoutTests/TestExpectations:
* LayoutTests/fast/webgpu/fuzz-274317-expected.txt: Added.
* LayoutTests/fast/webgpu/fuzz-274317.html: Added.
Add regression test.

* Source/WebGPU/WebGPU/PipelineLayout.mm:
(WebGPU::PipelineLayout::offsetVectorForBindGroup):

Canonical link: https://commits.webkit.org/279013@main

852547e

Misc iOS, tvOS & watchOS macOS Linux Windows
βœ… πŸ§ͺ style βœ… πŸ›  ios βœ… πŸ›  mac βœ… πŸ›  wpe   πŸ›  wincairo
❌ πŸ§ͺ bindings   πŸ›  ios-sim βœ… πŸ›  mac-AS-debug   πŸ§ͺ wpe-wk2   πŸ§ͺ wincairo-tests
βœ… πŸ§ͺ webkitperl   πŸ§ͺ ios-wk2   πŸ§ͺ api-mac   πŸ§ͺ api-wpe
❌ πŸ§ͺ webkitpy   πŸ§ͺ ios-wk2-wpt   πŸ§ͺ mac-wk1 βœ… πŸ›  wpe-cairo
  πŸ§ͺ api-ios   πŸ§ͺ mac-wk2 βœ… πŸ›  gtk
  πŸ›  tv   πŸ§ͺ mac-AS-debug-wk2   πŸ§ͺ gtk-wk2
  πŸ›  tv-sim   πŸ§ͺ mac-wk2-stress   πŸ§ͺ api-gtk
  πŸ›  watch
βœ… πŸ›  πŸ§ͺ unsafe-merge   πŸ›  watch-sim

@mwyrzykowski mwyrzykowski self-assigned this May 17, 2024
@mwyrzykowski mwyrzykowski added the WebGPU For bugs in WebGPU label May 17, 2024
@mwyrzykowski mwyrzykowski requested a review from djg May 17, 2024 23:20
Copy link
Contributor

@djg djg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@mwyrzykowski mwyrzykowski force-pushed the eng/WebGPU-Release-assert-trigged-in-PipelineLayoutoffsetVectorForBindGroup branch from b1c84de to 852547e Compare May 20, 2024 21:21
@mwyrzykowski mwyrzykowski added the unsafe-merge-queue Applied to send a pull request to merge-queue, but skip building and testing label May 20, 2024
…dGroup

https://bugs.webkit.org/show_bug.cgi?id=274317
<radar://128065029>

Reviewed by Dan Glastonbury.

Invalid bind group should return nullptr as it already does in other
places in this member function instead of crashing the GPU process.

* LayoutTests/TestExpectations:
* LayoutTests/fast/webgpu/fuzz-274317-expected.txt: Added.
* LayoutTests/fast/webgpu/fuzz-274317.html: Added.
Add regression test.

* Source/WebGPU/WebGPU/PipelineLayout.mm:
(WebGPU::PipelineLayout::offsetVectorForBindGroup):

Canonical link: https://commits.webkit.org/279013@main
@webkit-commit-queue webkit-commit-queue force-pushed the eng/WebGPU-Release-assert-trigged-in-PipelineLayoutoffsetVectorForBindGroup branch from 852547e to cd8059f Compare May 20, 2024 21:25
@webkit-commit-queue
Copy link
Collaborator

Committed 279013@main (cd8059f): https://commits.webkit.org/279013@main

Reviewed commits have been landed. Closing PR #28716 and removing active labels.

@webkit-commit-queue webkit-commit-queue merged commit cd8059f into WebKit:main May 20, 2024
@webkit-commit-queue webkit-commit-queue removed the unsafe-merge-queue Applied to send a pull request to merge-queue, but skip building and testing label May 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
WebGPU For bugs in WebGPU
Projects
None yet
4 participants