Skip to content

Fix Vulkan texture publication ordering#263

Open
cwervo wants to merge 2 commits into
mainfrom
ac/vulkan-anim-fix
Open

Fix Vulkan texture publication ordering#263
cwervo wants to merge 2 commits into
mainfrom
ac/vulkan-anim-fix

Conversation

@cwervo
Copy link
Copy Markdown
Collaborator

@cwervo cwervo commented May 14, 2026

Summary

  • keep Vulkan texture uploads asynchronous while avoiding sampling unpublished descriptor slots
  • drain pending texture descriptor work before canvas and display command recording
  • retire freed textures safely by redirecting descriptor slots to debugIm before delayed destruction

Validation

@cwervo cwervo requested a review from osnr May 14, 2026 02:46
@cwervo
Copy link
Copy Markdown
Collaborator Author

cwervo commented May 14, 2026

Works on folk-hex, folk-convivial, and on @ppkn's system. Will merge tomorrow unless there are any objections

@cwervo cwervo requested a review from ppkn May 14, 2026 15:36
@osnr
Copy link
Copy Markdown
Collaborator

osnr commented May 14, 2026

I still want to read through it more closely and understand the texture pipeline which has gotten pretty weird, but it works on gadget-blue and folk-omar as well so no objection rn

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.

2 participants