Skip to content

[UnifiedRadixTree] Reclaim L2 host pool slots on flush_cache#23216

Merged
hzh0425 merged 1 commit intosgl-project:hybrid_tree/hicache_integratefrom
icepoint666:diemchai/fix-unified-hicache-flush-leak
Apr 20, 2026
Merged

[UnifiedRadixTree] Reclaim L2 host pool slots on flush_cache#23216
hzh0425 merged 1 commit intosgl-project:hybrid_tree/hicache_integratefrom
icepoint666:diemchai/fix-unified-hicache-flush-leak

Conversation

@icepoint666
Copy link
Copy Markdown
Contributor

Motivation

UnifiedRadixCache._reset_full() doesn't reclaim host KV slots held by destroyed tree nodes, so after one POST /flush_cache the host pool stays full forever and all subsequent write_backup calls silently no-op, L2 is disabled until process restart.

Modifications

Call cache_controller.reset() + cache_controller.mem_pool_host.clear() at the end of _reset_full()

Accuracy Tests

Speed Tests and Profiling

Checklist

Review and Merge Process

  1. Ping Merge Oncalls to start the process. See the PR Merge Process.
  2. Get approvals from CODEOWNERS and other reviewers.
  3. Trigger CI tests with comments or contact authorized users to do so.
    • Common commands include /tag-and-rerun-ci, /tag-run-ci-label, /rerun-failed-ci
  4. After green CI and required approvals, ask Merge Oncalls or people with Write permission to merge the PR.

@gemini-code-assist
Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@hzh0425 hzh0425 merged commit 3511f1b into sgl-project:hybrid_tree/hicache_integrate Apr 20, 2026
1 check passed
hzh0425 pushed a commit that referenced this pull request Apr 21, 2026
Co-authored-by: diemchai <diemchai@tencent.com>
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