Skip to content

[Bugfix][Build] Patch stable string memleak fix from 2.14 for 2.13 - #51185

Merged
ywang96 merged 3 commits into
vllm-project:mainfrom
janeyx99:patch-str-mem-leak
Aug 9, 2026
Merged

ywang96 merged 3 commits into
vllm-project:mainfrom
janeyx99:patch-str-mem-leak

Conversation

@janeyx99

@janeyx99 janeyx99 commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Purpose

The stable ABI header had a memory leak for strings that was patched recently (pytorch/pytorch@7f0ec65) and will make it into our 2.14 release. This PR ports it over for the interim. This PR should be reverted once torch 2.14 is the build version for vllm. Note that this does not change versioning support for the stable ABI, we can still target 2.11 onwards.

Test Plan

CI

It did build locally and work for me, as well as for a downstream developer.

Test Result

green, hopefully


Essential Elements of an Effective PR Description Checklist
  • The purpose of the PR, such as "Fix some issue (link existing issues this PR will resolve)".
  • The test plan, such as providing test command.
  • The test results, such as pasting the results comparison before and after, or e2e results
  • (Optional) The necessary documentation update, such as updating supported_models.md and examples for a new model.

Signed-off-by: Jane Xu <janeyx@meta.com>
@mergify mergify Bot added the ci/build label Aug 5, 2026
@janeyx99
janeyx99 marked this pull request as ready for review August 7, 2026 15:42

@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.

Signed-off-by: Shengqi Chen <harry-chen@outlook.com>
@Harry-Chen Harry-Chen changed the title Patch stable string memleak fix from 2.14 for 2.13 [BugFix] Patch stable string memleak fix from 2.14 for 2.13 Aug 8, 2026
@mergify mergify Bot added the bug Something isn't working label Aug 8, 2026
@Harry-Chen Harry-Chen changed the title [BugFix] Patch stable string memleak fix from 2.14 for 2.13 [Bugfix][Build] Patch stable string memleak fix from 2.14 for 2.13 Aug 8, 2026
@Harry-Chen Harry-Chen added the ready ONLY add when PR is ready to merge/full CI is needed label Aug 8, 2026
@github-actions

github-actions Bot commented Aug 8, 2026

Copy link
Copy Markdown

@janeyx99, CI is now available for this PR.

  • /ci run starts a CI build.
  • /ci retry retries failed jobs in the CI build for the current PR head. If the current head has no CI build, it starts a new CI build for the current head containing only jobs that failed in the latest earlier CI build for this PR.

@Harry-Chen

Copy link
Copy Markdown
Member

/ci run

@github-actions

github-actions Bot commented Aug 8, 2026

Copy link
Copy Markdown

✅ Triggered Buildkite CI #82969 for commit de3d4e37b654.

@Harry-Chen
Harry-Chen enabled auto-merge (squash) August 9, 2026 04:08
@ywang96
ywang96 disabled auto-merge August 9, 2026 04:12
@ywang96
ywang96 merged commit f8d03e7 into vllm-project:main Aug 9, 2026
247 of 250 checks passed
@Harry-Chen

Copy link
Copy Markdown
Member

cc @atalman: Should you do the torch 2.14 transition later, please include a revert of this PR in your changes. Thx!

@ashgold

ashgold commented Aug 19, 2026

Copy link
Copy Markdown

This PR fix #50150

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working ci/build ready ONLY add when PR is ready to merge/full CI is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants