Skip to content

[CI] Stabilize GLM-5.2 PCP evaluation - #51015

Merged
khluu merged 1 commit into
vllm-project:mainfrom
khluu:codex/stabilize-glm52-pcp-eval
Aug 4, 2026
Merged

khluu merged 1 commit into
vllm-project:mainfrom
khluu:codex/stabilize-glm52-pcp-eval

Conversation

@khluu

@khluu khluu commented Aug 4, 2026

Copy link
Copy Markdown
Member

Summary

  • enable PyTorch expandable CUDA allocator segments for the GLM-5.2 TP1/PCP4 GSM8K evaluation;
  • keep --max-num-batched-tokens 32768, which is needed to exercise the large-batch PCP path;
  • scope the allocator change to the memory-tight TP1/PCP4 case; TP2/PCP2 is unchanged.

Root cause

The pytest-level error in Buildkite #81913 is only a 20-minute server-start timeout. The worker log shows the actual failure during the profile run:

CUDA out of memory. Tried to allocate 24.43 GiB.
15.36 GiB is free
23.97 GiB is reserved by PyTorch but unallocated

The allocation comes from FlashInfer CUTLASS MoE's FusedMoeRunner::getWorkspaceInfo. The same signature is present in Buildkite #80082, before the recent Kimi/K3 changes, so this is not a recent model-runner regression or a bad B200 node.

The TP1/PCP4 configuration became memory-tight when --max-num-batched-tokens 32768 was added in #49294. Successful runs such as Buildkite #80689 show that the workload's real peak fits: 159.66 GiB total non-KV memory on a 178.35 GiB B200. The intermittent OOM occurs because a roughly 24 GiB cached allocator segment cannot satisfy the next slightly larger 24.43 GiB FlashInfer workspace request. Expandable segments let that reserved memory grow/reuse instead of requiring a second contiguous allocation.

Duplicate-work check

No open PR was found for the GLM-5.2 PCP FlashInfer MoE startup OOM using searches for GLM-5.2 PCP OOM, lm-eval-pcp-4xb200, and expandable_segments GLM.

Testing

  • git diff --check
  • parsed the YAML and verified both the allocator setting and retained 32K batch limit with Ruby's YAML parser
  • repository pre-commit hooks run by git commit: passed
  • GitHub pre-commit workflow: passed
  • Buildkite #82253 — LM Eval PCP (4xB200): passed on dgxB200-14 with exit status 0
    • unchanged TP2/PCP2 control: GSM8K passed
    • TP1/PCP4 with PYTORCH_CUDA_ALLOC_CONF=expandable_segments:True: profile run passed with 130.49 GiB of weights and a 28.33–30.23 GiB torch peak, the server started, and all 1,319 GSM8K examples passed

The allocator-only change does not alter model inputs, kernels, or numerics. The B200 model evaluation passed on the first validation iteration.

AI assistance

AI assistance was used for Buildkite log analysis, root-cause isolation, implementation, and drafting this description.

  • The human submitter has reviewed the changed line and CI evidence before marking this PR ready.

Enable expandable CUDA allocator segments for the TP1/PCP4 GLM-5.2 eval so the 32K-token FlashInfer MoE workspace can reuse reserved memory.

Co-authored-by: OpenAI Codex <codex@openai.com>
Signed-off-by: khluu <khluu000@gmail.com>

khluu commented Aug 4, 2026

Copy link
Copy Markdown
Member Author

/runci

khluu commented Aug 4, 2026

Copy link
Copy Markdown
Member Author

/ci run

@khluu
khluu marked this pull request as ready for review August 4, 2026 20:54

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Claude Code Review

This pull request is from a fork — automated review is disabled. A repository maintainer can comment @claude review to run a one-time review.

@khluu

khluu commented Aug 4, 2026

Copy link
Copy Markdown
Member Author

/ci run

@github-actions

github-actions Bot commented Aug 4, 2026

Copy link
Copy Markdown

✅ CI is already running for this commit: https://buildkite.com/vllm/ci/builds/82253

@LucasWilkinson LucasWilkinson left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM; thank you!

@khluu
khluu merged commit a5149b2 into vllm-project:main Aug 4, 2026
21 checks passed
@khluu khluu added this to the v0.27.0 cherry picks milestone Aug 4, 2026
khluu added a commit that referenced this pull request Aug 7, 2026
Signed-off-by: khluu <khluu000@gmail.com>
Co-authored-by: OpenAI Codex <codex@openai.com>
(cherry picked from commit a5149b2)
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