Skip to content

Update DeepGEMM tag to point to latest nv-dev branch for sm120 support - #47304

Merged
youkaichao merged 1 commit into
vllm-project:mainfrom
neuralmagic:update-deepgemm-commit-sm120
Jul 2, 2026
Merged

Update DeepGEMM tag to point to latest nv-dev branch for sm120 support#47304
youkaichao merged 1 commit into
vllm-project:mainfrom
neuralmagic:update-deepgemm-commit-sm120

Conversation

@mgoin

@mgoin mgoin commented Jul 1, 2026

Copy link
Copy Markdown
Member

Purpose

This PR #43477 landed in vLLM without bumping up our commit tag for DeepGEMM for the SM120 support, which resulted in many failures for models that trigger DeepGEMM kernels i.e. #47169, #47266

For now we should bump the DeepGEMM commit for nv-dev (which includes deepseek-ai/DeepGEMM#324), but this will likely be an ongoing issue if this gets out of sync with upstream main

Test Plan

Test Result

Tested locally on DGX Spark Qwen3-0.6B-FP8. Before it crashes during startup and now it works fine.


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.

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

@Harry-Chen Harry-Chen added the ready ONLY add when PR is ready to merge/full CI is needed label Jul 1, 2026
@youkaichao
youkaichao merged commit 84b9c27 into vllm-project:main Jul 2, 2026
231 of 234 checks passed
allenh1 pushed a commit to allenh1/vllm that referenced this pull request Jul 3, 2026
Sync 2026-07-03: picks up Model Runner V2 improvements (motivated by V2
viability reopening via PR#26). Notable: MRV2 scheduler req-slot bugfix
(vllm-project#46974), MRV2 spec-decode int32 block-verify overflow fix (vllm-project#47383),
MRV2 DSA-indexer prefill warmup for GLM-5.2/DSA (vllm-project#47285), V2-default for
dense models (vllm-project#44443 — DSv4 is MoE so stays V1), DeepGEMM tag -> nv-dev
a6b593d2 for SM120 (vllm-project#47304), TLI heterogeneous-vocab spec decode (vllm-project#38174),
Delete PagedAttention (vllm-project#47361 — no impact, our sparse-MLA runner is a
flashinfer symbol), Xqa decode kernels (vllm-project#43232).

Conflict: vllm/v1/spec_decode/llm_base_proposer.py (our DSv4 MTP
spec_step_idx routing vs upstream TLI) — resolved keeping BOTH: TLI's
heterogeneous-vocab branch routed through our _compute_logits(spec_step_idx).

Our SM12x stack preserved (persistent_topk, topk.cu, PR#26 padded-Q kernel,
dspark, attention, R1 hardening all UNCHANGED); config/vllm.py DSv4->V1
default + DSpark env-control intact.
jakki-amd pushed a commit to jakki-amd/vllm that referenced this pull request Jul 6, 2026
lkk12014402 pushed a commit to lkk12014402/vllm that referenced this pull request Jul 8, 2026
mayuyuace pushed a commit to mayuyuace/vllm that referenced this pull request Jul 9, 2026
vllm-project#47304)

Signed-off-by: mgoin <mgoin64@gmail.com>
Signed-off-by: mayuyuace <qiming1.zhang@intel.com>
philippesic pushed a commit to philippesic/vllm-semantic-cache that referenced this pull request Jul 19, 2026
plasticchris pushed a commit to plasticchris/vllm that referenced this pull request Jul 20, 2026
aditi-amd pushed a commit to aditi-amd/vllm that referenced this pull request Aug 4, 2026
vllm-project#47304)

Signed-off-by: mgoin <mgoin64@gmail.com>
Signed-off-by: root <root@smci355-ccs-aus-m02-09.cs-aus.dcgpu>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

3 participants