Skip to content

temp pr for test - #1771

Draft
shepark wants to merge 5 commits into
vllm-project:releases/v0.28.0from
shepark:shepark/for_ci_28
Draft

temp pr for test#1771
shepark wants to merge 5 commits into
vllm-project:releases/v0.28.0from
shepark:shepark/for_ci_28

Conversation

@shepark

@shepark shepark commented Aug 28, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

iboiko-habana and others added 4 commits August 28, 2026 13:57
…r upstream fused-kernel inlining (#50400) vllm-project#1754

Signed-off-by: Iryna Boiko <iboiko@habana.ai>
Every HPU run (offline LLM, server, or pytest) printed a segfault block at
process teardown, right after "request processing complete; starting resource
teardown", once per EngineCore:

    WARNING [interface.py] Current platform hpu does not have 'empty_cache' attribute.
    !!!!!!! Segfault encountered !!!!!!!
      ... at::accelerator::emptyHostCache()
      ... torch::accelerator::initModule(...)::{lambda()vllm-project#5}
      ... Py_RunMain / Py_BytesMain / _start

Root cause: vLLM PR #51107 swapped cleanup_dist_env_and_memory() from
torch._C._host_emptyCache() to torch.accelerator.empty_host_cache(). On
torch>=2.9 (docker has 2.12.0a0) the attribute exists, so vLLM's
`except AttributeError` guard no longer catches it; the call dispatches to
at::accelerator::emptyHostCache(), which has no HPU host-cache hook and
segfaults. patches.py already patched empty_cache (PR vllm-project#1430) and the now-dead
torch._C._host_emptyCache, but not the live torch.accelerator.empty_host_cache.

Fix: add an HPU-safe no-op _hpu_accelerator_empty_host_cache() shim (sibling of
_hpu_accelerator_empty_cache) and install it in apply(). Harmless-but-noisy:
the crash fires after all work completes, so results/exit codes were never
affected; this just removes the crash from logs/CI.

Tested (container jha_pytorch_124-482-new, torch 2.12.0a0, gaudi3):
- After apply(): torch.accelerator.empty_host_cache resolves to the no-op shim.
- Same minimal LLM.generate() repro, patched vs. stashed-unpatched control:
    unpatched: 2 "Segfault encountered / emptyHostCache()" blocks at teardown
    patched:   0 segfault lines, clean teardown, exit 0

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Signed-off-by: Jimin Ha <jimin.ha@intel.com>
…-bias overflow- vllm-project#1749

Signed-off-by: Iryna Boiko <iboiko@habana.ai>
…ash after upstream fused-kernel inlining (#50400) vllm-project#1754"

This reverts commit 9f40e6f.
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.

4 participants