[PR #8] [MERGED] Aggregate allocations between polling intervals to catch OOMs #57

New issue

Closed

opened 2026-10-26 22:24:11 +00:00 by bsakaguchi · 0 comments

bsakaguchi commented

2026-10-26 22:24:11 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/Comfy-Org/comfy-aimdo/pull/8
Author: @rattus128
Created: 3/1/2026
Status: ✅ Merged
Merged: 3/1/2026
Merged by: @rattus128

Base: master ← Head: dev/vram-math-projection

📝 Commits (1)

89b377a Aggregate allocations between polling intervals to catch OOMs

📊 Changes

7 files changed (+91 additions, -62 deletions)

View changed files

📝 src-win/shmem-detect.c (+18 -13)
➕ src/aimdo-time.h (+14 -0)
📝 src/control.c (+20 -19)
📝 src/model-vbar.c (+7 -1)
📝 src/plat.h (+30 -5)
📝 src/pyt-cu-plug-alloc-async.c (+1 -23)
📝 src/vrambuf.c (+1 -1)

📄 Description

Budget deficit detection now polls platform state every 2s and tracks allocation deltas between polls, catching OOMs that were previously missed between sync points.

Contribution Agreement

[ X ] I agree that my contributions are licensed under the GPLv3.
[ X ] I grant Comfy Org the rights to relicense these contributions as outlined in CONTRIBUTING.md.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/Comfy-Org/comfy-aimdo/pull/8 **Author:** [@rattus128](https://github.com/rattus128) **Created:** 3/1/2026 **Status:** ✅ Merged **Merged:** 3/1/2026 **Merged by:** [@rattus128](https://github.com/rattus128) **Base:** `master` ← **Head:** `dev/vram-math-projection` --- ### 📝 Commits (1) - [`89b377a`](https://github.com/Comfy-Org/comfy-aimdo/commit/89b377aebd3d4c4bf2fc68cb38b2d4101064c52d) Aggregate allocations between polling intervals to catch OOMs ### 📊 Changes **7 files changed** (+91 additions, -62 deletions) <details> <summary>View changed files</summary> 📝 `src-win/shmem-detect.c` (+18 -13) ➕ `src/aimdo-time.h` (+14 -0) 📝 `src/control.c` (+20 -19) 📝 `src/model-vbar.c` (+7 -1) 📝 `src/plat.h` (+30 -5) 📝 `src/pyt-cu-plug-alloc-async.c` (+1 -23) 📝 `src/vrambuf.c` (+1 -1) </details> ### 📄 Description Budget deficit detection now polls platform state every 2s and tracks allocation deltas between polls, catching OOMs that were previously missed between sync points. ### Contribution Agreement - [ X ] I agree that my contributions are licensed under the GPLv3. - [ X ] I grant **Comfy Org** the rights to relicense these contributions as outlined in [CONTRIBUTING.md](./CONTRIBUTING.md). --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

bsakaguchi

2026-10-26 22:24:11 +00:00

closed this issue
added the
pull-request
label

No labels

pull-request

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

Comfy-Org/comfy-aimdo#57

No description provided.

Rows
Columns