Skip to content

fix(memory): bound editor and upload arena ownership - #836

Merged
JeanPhilippeKernel merged 1 commit into
developfrom
fix/834-linux-editor-arena
Sep 24, 2026
Merged

JeanPhilippeKernel merged 1 commit into
developfrom
fix/834-linux-editor-arena

Conversation

@JeanPhilippeKernel

Copy link
Copy Markdown
Owner

Summary

  • reserve a bounded EditorContext owner before editor initialization and move editor-owned CPU allocations into it
  • materialize the VulkanDevice CPU budget and move renderer decode slabs into the bounded import/upload owner
  • validate built-in profile capacities and align allocator and memory-budget documentation with actual ownership

Validation

  • cmake --build --preset Darwin_arm64_Debug --target ZEngineTests Obelisk
  • ZEngineTests --gtest_filter=AllocatorTest.:MemoryBudgetConfigTest.:TLSFSlabTest.:WorkerThreadCountTest. (49 passed)
  • launched the Debug editor with SampleProject/project.json for 30 seconds; it remained alive past the former allocation failure
  • git diff --check and repository pre-push clang-format checks

Scope

This resolves the missing-owner startup path in #834. The separate POSIX reserve/commit redesign, complete MainArena ownership audit, and measured upload-concurrency policy are tracked in #835.

Fixes #834

@JeanPhilippeKernel JeanPhilippeKernel self-assigned this Sep 20, 2026
@JeanPhilippeKernel JeanPhilippeKernel added area-rendering area-linux Work on Linux system area-window Work on Window system area-macOS Work on macOS system labels Sep 20, 2026
@JeanPhilippeKernel
JeanPhilippeKernel merged commit 5b1bef6 into develop Sep 24, 2026
17 checks passed
@JeanPhilippeKernel
JeanPhilippeKernel deleted the fix/834-linux-editor-arena branch September 24, 2026 05:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-linux Work on Linux system area-macOS Work on macOS system area-window Work on Window system

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Crash on Linux

1 participant