gfx-rs/wgpu

StagingBelt memory usage problems in certain scenarios

Open

#1,441 opened on May 8, 2021

View on GitHub
 (0 comments) (3 reactions) (0 assignees)Rust (9,290 stars) (703 forks)batch import
good first issuetype: bug

Description

Recently I decided to try out Iced but run into some memory problems while testing an example, as can be seen in this issue: https://github.com/hecrj/iced/issues/786 I narrowed it down to the wgpu-rs StagingBelt, and the lack of a way to cleanup unused free Chunks from the belt. The issue with the Game of Life example was that I only added more geometry to the screen, so my buffers got larger, never fitting inside the previous Chunks and so creating new Chunks while the old Chunks stayed unused forever. I managed to fix the issue by implementing a naive GC: https://github.com/eduardobraun/wgpu-rs/commit/a62dd38538b41e2890a0cecd59e692f1d208d6e6

TLDR: StagingBelt needs a way to reclaim memory from free Chunks.

Contributor guide

Tech stack
rust
Domain
performance
Issue type
performance
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
4
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-2 days
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
Rust programmingwgpu rs basicsmemory management concepts
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
30
Research direction
Examine the linked commit a62dd38538b41e2890a0cecd59e692f1d208d6e6 which implements a naive GC for StagingBelt. Understand the current memory allocation strategy in the StagingBelt module of wgpu rs. Identify where free chunks accumulate and design a cleanup mechanism to reclaim memory. Consider integrating a periodic or explicit cleanup method that can be called to release unused chunks.
StagingBelt memory usage problems in certain scenarios · gfx-rs/wgpu#1441 | Good First Issue