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

[GL] Unbind Vertex Buffers After Renderpass #3459

Merged

Conversation

cwfitzgerald
Copy link
Member

Checklist

  • Run cargo clippy.
  • Run RUSTFLAGS=--cfg=web_sys_unstable_apis cargo clippy --target wasm32-unknown-unknown if applicable.
  • Add change to CHANGELOG.md. See simple instructions inside file.

Connections

Closes #3457

Description

We never reset the vertex buffers on the same command buffer.

Testing

Added regression test

@cwfitzgerald cwfitzgerald force-pushed the fix-vertex-buffer-invalidation branch from 0fc91e0 to bc1ecb3 Compare February 8, 2023 20:13
@cwfitzgerald cwfitzgerald force-pushed the fix-vertex-buffer-invalidation branch from bc1ecb3 to ca08915 Compare February 8, 2023 22:42
@cwfitzgerald cwfitzgerald marked this pull request as ready for review February 9, 2023 00:19
@cwfitzgerald cwfitzgerald changed the title Unbind Vertex Buffers After Renderpass [GL] Unbind Vertex Buffers After Renderpass Feb 9, 2023
@cwfitzgerald cwfitzgerald enabled auto-merge (squash) February 9, 2023 00:22
@cwfitzgerald cwfitzgerald force-pushed the fix-vertex-buffer-invalidation branch from d57d0d9 to 7599867 Compare February 9, 2023 00:23
auto-merge was automatically disabled February 9, 2023 00:26

Merge queue setting changed

@cwfitzgerald cwfitzgerald added this pull request to the merge queue Feb 9, 2023
@cwfitzgerald cwfitzgerald removed this pull request from the merge queue due to a manual request Feb 9, 2023
@cwfitzgerald cwfitzgerald added this pull request to the merge queue Feb 9, 2023
@cwfitzgerald cwfitzgerald removed this pull request from the merge queue due to the queue being cleared Feb 9, 2023
@cwfitzgerald cwfitzgerald added this pull request to the merge queue Feb 9, 2023
@cwfitzgerald cwfitzgerald removed this pull request from the merge queue due to a manual request Feb 9, 2023
@cwfitzgerald cwfitzgerald added this pull request to the merge queue Feb 9, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Feb 9, 2023
@cwfitzgerald cwfitzgerald merged commit 49cfc88 into gfx-rs:master Feb 9, 2023
@cwfitzgerald cwfitzgerald added the PR: needs back-porting PR with a fix that needs to land on crates label Feb 9, 2023
@cwfitzgerald cwfitzgerald deleted the fix-vertex-buffer-invalidation branch February 9, 2023 17:06
cwfitzgerald added a commit to cwfitzgerald/wgpu that referenced this pull request Feb 9, 2023
cwfitzgerald added a commit that referenced this pull request Feb 9, 2023
@cwfitzgerald cwfitzgerald removed the PR: needs back-porting PR with a fix that needs to land on crates label Feb 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

GL Vertex Buffers Not Reset After Ending Renderpass
1 participant