Skip to content

[Bugfix]Reject invalid FlashInfer MNNVL workspaces - #49043

Merged
mgoin merged 2 commits into
vllm-project:mainfrom
lengrongfu:fix/mnnvl-mc_ptr
Jul 27, 2026
Merged

mgoin merged 2 commits into
vllm-project:mainfrom
lengrongfu:fix/mnnvl-mc_ptr

Conversation

@lengrongfu

@lengrongfu lengrongfu commented Jul 18, 2026

Copy link
Copy Markdown
Contributor

Purpose

Fixs: #49041

On topologies without symmetric-memory multicast support, FlashInfer can construct an MNNVL allreduce workspace whose mc_ptr is null. vLLM treated the workspace as initialized and passed it into fused allreduce kernels, which dereferenced the null multicast mapping and failed with a CUDA illegal memory access.

Validate mc_ptr immediately after workspace creation. Destroy invalid MNNVL workspaces and return None so the existing backend selection can fall back to TRTLLM on supported single-node configurations, or disable the fusion safely when no fallback is available.

https://docs.nvidia.com/cuda/cuda-driver-api/group__CUDA__MULTICAST.html

Test Plan

Rerun using the previous command.

Test Result


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: lengrongfu <lenronfu@gmail.com>
@lengrongfu
lengrongfu marked this pull request as ready for review July 18, 2026 15:39

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

@mergify mergify Bot added nvidia bug Something isn't working labels Jul 18, 2026
@lengrongfu

Copy link
Copy Markdown
Contributor Author

@mmangkad @mgoin Hi, this pr is fix about this pr #48064, in _resolve_fi_ar_backend auto select mnnvl. please take look.

@ZJY0516

ZJY0516 commented Jul 25, 2026

Copy link
Copy Markdown
Member

According to #48075, TRTLLM is ~10× slower than no-fuse

@github-project-automation github-project-automation Bot moved this to Ready in NVIDIA Jul 25, 2026
@mgoin mgoin added performance Performance-related issues ready ONLY add when PR is ready to merge/full CI is needed labels Jul 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working nvidia performance Performance-related issues ready ONLY add when PR is ready to merge/full CI is needed

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants