Skip to content

Conversation

@beicause
Copy link

@beicause beicause commented Dec 5, 2025

Description
Noted this when I was checking #8162

The offset argument of draw_indexed_indirect in hal/vulkan seems incorrect. It should + i * stride.
Also multi draw indirect should respect limits.maxDrawIndirectCount.

Actually, vkCmdDrawIndirectCount vkCmdDrawIndexedIndirectCount vkCmdDrawMeshTasksIndirectCountEXT also require that the count in the countBuffer must be <= maxDrawIndirectCount, altough on many devices it's u32::MAX(vulkan gpuinfo)

Testing
I assume the code is correct, I don't have a device to test this.

Squash or Rebase?
Squash

Checklist

  • Run cargo fmt.
  • Run taplo format.
  • Run cargo clippy --tests. If applicable, add:
    • --target wasm32-unknown-unknown
  • Run cargo xtask test to run tests.
  • If this contains user-facing changes, add a CHANGELOG.md entry.

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.

1 participant