Skip to content

dsv4 A5 NPU rebased - #338

Closed
Pingzii wants to merge 7 commits into
vllm-project:mainfrom
Pingzii:AFD-A5-rebased
Closed

dsv4 A5 NPU rebased#338
Pingzii wants to merge 7 commits into
vllm-project:mainfrom
Pingzii:AFD-A5-rebased

Conversation

@Pingzii

@Pingzii Pingzii commented Sep 11, 2026

Copy link
Copy Markdown

PLEASE FILL IN THE PR DESCRIPTION AND MAKE SURE THE CHECKLIST ITEMS HAVE BEEN CONSIDERED.

Purpose

Issue

  • Related issue(s): #
  • Closing keyword, only if fully resolved: Closes #

Scope

  • In scope:
  • Out of scope:

Implementation Notes

Test Plan

Test Result

Docs Impact

  • Files updated:
  • If none, reason:

Essential PR Checklist
  • Purpose is clear and linked to public context when possible.
  • Scope is bounded.
  • Compatibility with vLLM v0.26.0 is considered.
  • No changes are made to the vLLM source checkout.
  • Plugin-owned classes or explicit dotted class paths are preferred over monkey patches.
  • Any compat shim or monkey patch is isolated, idempotent, version-guarded, documented, and tested.
  • Imports remain CPU-safe; CUDA-heavy work is delayed or GPU-gated.
  • Validation evidence is included, including skipped GPU tests when applicable.
  • Documentation impact is stated.

ksiyuan and others added 7 commits September 8, 2026 10:03
a2e/e2a custom ops and native MC2 dispatch/combine both fail on the AFD mixed group on A5 (507035 MTE out-of-range; remote HCCL windows unreachable). Replace the four a2e/e2a call sites on A5 with plain dist.send/recv over the per-ubatch 'afd' HCCL groups. Validated by a 2-rank eager 1A+1F DeepSeek-V2-Lite completion smoke on Ascend 950PR.

- camp2p.py: A5 branches in send_attn_output/recv_ffn_output/recv_attn_output/send_ffn_output; init_afd_connector skips the custom-op loader on A5 (a2e/e2a register only ascend910_93).
- camp2p_a5.py (new): is_a5(), contiguous rank mapping (attention i -> ffn i//(attn//ffn), matching _num_tokens_for_ffn_rank), per-peer DP token counts, p2p primitives; unit tests added.
- Gate stays on FFN (only hidden_states crosses the wire). 910C custom-op path unchanged.
- Known gaps: ACL graph capture, DBO, multi-rank validation, 910C regression.
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.

2 participants