[perf] compressedStaging reallocated and kept per compressed Build #67

New issue

Closed

opened 2026-06-16 16:51:34 +02:00 by jorijnvdgraaf · 0 comments

jorijnvdgraaf commented

2026-06-16 16:51:34 +02:00

Owner

Subsystem: Mesh / Image decompression
Location: implementations/Crafter.Graphics-Mesh.cpp:230-237 (same pattern interfaces/Crafter.Graphics-ImageVulkan.cppm:198-215)
Impact: Memory pinning · Effort: Medium · Load-time, not per-frame
Builds on: #101 / #102 (fence-keyed deferred-deletion queue — landed)

Problem

compressedStaging is reallocated and kept per compressed Build — persistent per-mesh host-visible staging pins memory for the mesh's life.

Proposed fix

Either: (a) release after the decompress submit — compressedStaging.DeferredClear() via the deletion queue (#102), which retires it once the submit's frame has cleared the fence (the decompress references compressedStaging.address, so it must outlive the submit — exactly what the queue guarantees;

**Subsystem:** Mesh / Image decompression **Location:** `implementations/Crafter.Graphics-Mesh.cpp:230-237` (same pattern `interfaces/Crafter.Graphics-ImageVulkan.cppm:198-215`) **Impact:** Memory pinning · **Effort:** Medium · Load-time, not per-frame **Builds on:** #101 / #102 (fence-keyed deferred-deletion queue — landed) ### Problem `compressedStaging` is reallocated and kept per compressed Build — persistent per-mesh host-visible staging pins memory for the mesh's life. ### Proposed fix Either: (a) **release after the decompress submit** — `compressedStaging.DeferredClear()` via the deletion queue (#102), which retires it once the submit's frame has cleared the fence (the decompress references `compressedStaging.address`, so it must outlive the submit — exactly what the queue guarantees;

jorijnvdgraaf referenced this issue

2026-06-16 17:10:54 +02:00

[perf] Mesh vertex/index/aabb are HOST_VISIBLE — place device-local (direct on ReBAR, staged otherwise) #73

jorijnvdgraaf referenced this issue

2026-06-16 18:55:02 +02:00

[perf] Runtime upload-strategy helper: direct HOST_VISIBLE|DEVICE_LOCAL vs staging (ReBAR-aware) #89

jorijnvdgraaf added the

claude:ready

label

2026-06-17 19:21:06 +02:00