Skip to content

Release slab memory to the OS between iterations#202

Open
chourroutm wants to merge 1 commit into
HiPCTProject:mainfrom
chourroutm:fix/slab-memory-release
Open

Release slab memory to the OS between iterations#202
chourroutm wants to merge 1 commit into
HiPCTProject:mainfrom
chourroutm:fix/slab-memory-release

Conversation

@chourroutm

Copy link
Copy Markdown
Member

Copying/downsampling processed slabs with steadily growing resident memory: the large numpy copy buffers and tensorstore shard-encode buffers were freed at the Python level but retained in the glibc allocator arena rather than returned to the OS, so RSS ratcheted up slab-by-slab until a memory-capped session was exhausted.

  • Add _release_memory() (gc.collect + guarded glibc malloc_trim) and call it at the end of _copy_slab and _downsample_block after dropping the slab/shard buffers and tensorstore handles.
  • Bound the tensorstore cache pool so the downsample read path cannot grow an unbounded in-memory cache.
  • Build the joblib job list lazily so each slab's dask view is released as it completes instead of all being pinned for the whole call.
  • Document that peak memory scales with n_processes.

Copying/downsampling processed slabs with steadily growing resident
memory: the large numpy copy buffers and tensorstore shard-encode buffers
were freed at the Python level but retained in the glibc allocator arena
rather than returned to the OS, so RSS ratcheted up slab-by-slab until a
memory-capped session was exhausted.

- Add _release_memory() (gc.collect + guarded glibc malloc_trim) and call
  it at the end of _copy_slab and _downsample_block after dropping the
  slab/shard buffers and tensorstore handles.
- Bound the tensorstore cache pool so the downsample read path cannot grow
  an unbounded in-memory cache.
- Build the joblib job list lazily so each slab's dask view is released as
  it completes instead of all being pinned for the whole call.
- Document that peak memory scales with n_processes.
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