Skip to content

[XPU][LoRA] Support LoRA for DeepSeek V4 on XPU - #53689

Merged
jikunshang merged 1 commit into
vllm-project:mainfrom
chaojun-zhang:lora/dsv4-xpu
Sep 7, 2026
Merged

jikunshang merged 1 commit into
vllm-project:mainfrom
chaojun-zhang:lora/dsv4-xpu

Conversation

@chaojun-zhang

@chaojun-zhang chaojun-zhang commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Follow-up to #53361, which added LoRA support for DeepSeek V4 but only touched vllm/models/deepseek_v4/nvidia/model.py. This ports the same changes to vllm/models/deepseek_v4/xpu/model.py

Test plan: #53361 added no repo tests (validated via external verl E2E), and there's no existing DeepSeek V4 + LoRA coverage in tests/lora/ or tests/models/deepseek_v4/ to extend here either. If upstream later adds repo tests for DSV4+LoRA, they should be extended to run on XPU too.

Port the NVIDIA DSV4 LoRA support (vllm-project#53361) to the XPU model: add
SupportsLoRA, packed_modules_mapping, and lora_skip_prefixes to
DeepseekV4ForCausalLM; reconcile LoRA-wrapped (.base_layer.) parameter
names in load_weights; and add the matching .base_layer. scale regex
variants to the weights mapper.

Signed-off-by: Chaojun Zhang <chaojun.zhang@intel.com>
@chaojun-zhang
chaojun-zhang requested a review from zyongye as a code owner August 25, 2026 06:34

@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 deepseek Related to DeepSeek models DSv4 intel-gpu Related to Intel GPU labels Aug 25, 2026
@yma11

yma11 commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

@chaojun-zhang can you provide test command for this support?

@chaojun-zhang

Copy link
Copy Markdown
Contributor Author

@chaojun-zhang can you provide test command for this support?

The original #53361 PR was validated through an external verl end-to-end test, not via an in-repo script. Also, there is no publicly available LoRA adapter for testing at the moment. We can set this aside for now.

@jikunshang

Copy link
Copy Markdown
Member

cc @majian4work @xwu-intel

@majian4work majian4work left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@chaojun-zhang

Copy link
Copy Markdown
Contributor Author

/ci run

@github-actions

github-actions Bot commented Sep 3, 2026

Copy link
Copy Markdown

✅ Triggered Buildkite CI #87045 for commit 62c8a8d7319c.

@chaojun-zhang

Copy link
Copy Markdown
Contributor Author

/ci retry

@github-actions

github-actions Bot commented Sep 7, 2026

Copy link
Copy Markdown

✅ Queued 6 failed job(s) for retry in Buildkite CI #87045.

@jikunshang
jikunshang merged commit 8648446 into vllm-project:main Sep 7, 2026
71 checks passed
ItsRoy69 pushed a commit to ItsRoy69/vllm that referenced this pull request Sep 10, 2026
Signed-off-by: Chaojun Zhang <chaojun.zhang@intel.com>
Signed-off-by: Jyotirmoy Roy <jyotirmoyroy649@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

deepseek Related to DeepSeek models DSv4 intel-gpu Related to Intel GPU

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants