-
Notifications
You must be signed in to change notification settings - Fork 227
[AMD][AgentX] DeepSeek-V4 MI355X agentic disaggregated benchmarking #2170
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
ichbinblau
wants to merge
40
commits into
main
Choose a base branch
from
amd/agentx-v1.0-yanfei
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
40 commits
Select commit
Hold shift + click to select a range
c2f8a5f
[AMD][AgentX] Add DeepSeek V4 MI355X agentic disagg
cquil11 5f2cc06
run-sweep: fix broken conc-list interpolation for multi-node agentic
ichbinblau 7bfdf38
bump image to rocm/sgl-dev:sglang-0.5.14-rocm720-mi35x-mori-0706
ichbinblau 1a7b88c
run-sweep: fix sequence-typed conc input for multi-node agentic
ichbinblau 5fa76de
agentic: run one task per concurrency for multi-node sweeps
ichbinblau a5763f7
Bump image to latest upstream image with 4 PR fixes included.
ichbinblau ea54838
update con-list
ichbinblau 183a987
setup_deps: remove disabled decode_tp_queue_agree patch
ichbinblau 711baf8
setup_deps: broaden GLM transformers gate; disable prefill bootstrap-…
ichbinblau 5d671a3
agentic: node-0 sibling benchmark-client container for DSv4 sweeps (#…
AMD-yanfeiwang 8278751
enable log info and extend timeout
ichbinblau 0225236
fix: address PR 2170 review comments
Duyi-Wang cd4c3f9
setup_deps: remove prefill bootstrap-desync patch entirely
ichbinblau a361ad8
fix perf changelog
billishyahao 775c22c
consolidate sglang agentic envs
ichbinblau 36613c7
select models.yaml recipe by IS_AGENTIC (AgentX/DI variants)
ichbinblau 9ccd032
DPTP enabling.
ichbinblau 0df082a
tune hicache ratio, router barrier timeout, and agentic conc sweep
ichbinblau f588eda
raise hicache ratio to 16
ichbinblau c9e7a32
fix: restore qwen3.5 changelog entry removed during rebase
ichbinblau 3cc2625
fix: validate agentic hicache master config for changelog gate
ichbinblau 444e44b
feat: wire KV_P2P_TRANSFER env into SGLang launch flags
ichbinblau cadad83
chore: drop max_total_tokens from DSv4 AgentX prefill no_dp recipe
ichbinblau c68375d
tune: raise router barrier timeout to 3000s for worker nodes
ichbinblau 6344aaa
test: fix sweep config fixtures for kv-p2p-transfer validation
ichbinblau 90af0b6
refactor: centralize sync barrier timeouts for DSV4 startup
ichbinblau 6b48abe
fix: forward KV offload backend metadata to agentic client
ichbinblau df41192
Merge branch 'main' into amd/agentx-v1.0-yanfei
ichbinblau dfcc261
revert: drop metadata-derived KV_OFFLOAD_BACKEND from DSv4 recipe
ichbinblau aa95a53
fix: quote KV_OFFLOAD_BACKEND_METADATA in docker run env forwarding
ichbinblau fbde2b3
Merge remote-tracking branch 'origin/amd/agentx-v1.0-yanfei'
ichbinblau 8116833
Fix client.env generation to flatten multi-line env values
ichbinblau 315760c
Re-serialize KV_OFFLOAD_BACKEND_METADATA as compact JSON instead of s…
ichbinblau ceee754
Fix regression: skip JSON round-trip for empty KV_OFFLOAD_BACKEND_MET…
ichbinblau c7aa128
Merge remote-tracking branch 'origin/main' into agent/batch-null-2170
cquil11 5fdef46
fix(changelog): append PR entry after main
cquil11 d8cbb70
fix(changelog): omit null optional metadata
cquil11 e625173
fix(agentx): set DeepSeek reasoning effort high
cquil11 47018bd
Merge remote-tracking branch 'origin/main' into amd/agentx-v1.0-yanfei
ichbinblau b656337
Remove stale patches/README.md documenting an already-retired mori_co…
ichbinblau File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
172 changes: 172 additions & 0 deletions
172
benchmarks/multi_node/agentic/dsv4_fp4_mi355x_sglang-disagg.sh
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,172 @@ | ||
| #!/usr/bin/env bash | ||
|
|
||
| # Agentic trace-replay recipe for a disaggregated SGLang server on MI355X | ||
| # (DeepSeek-V4-Pro FP4, 1P1D TP8). | ||
| # | ||
| # CI-style sibling of dsr1_fp4_mi355x_sglang-disagg.sh: driven entirely by | ||
| # environment variables and submits a SLURM job via submit.sh. The agentic / | ||
| # HiCache-offload configuration mirrors the DSR1 recipe but uses DSV4-Pro | ||
| # specific flags (dsv4 attention backend, page-size 256, SWA settings). | ||
|
|
||
| SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)" | ||
| source "$SCRIPT_DIR/../../benchmark_lib.sh" | ||
|
|
||
| check_env_vars \ | ||
| CONC_LIST \ | ||
| ISL \ | ||
| OSL \ | ||
| IMAGE \ | ||
| SPEC_DECODING \ | ||
| MODEL_PATH \ | ||
| PREFILL_NUM_WORKERS \ | ||
| PREFILL_TP \ | ||
| PREFILL_EP \ | ||
| PREFILL_DP_ATTN \ | ||
| DECODE_NUM_WORKERS \ | ||
| DECODE_TP \ | ||
| DECODE_EP \ | ||
| DECODE_DP_ATTN \ | ||
| PREFILL_NODES \ | ||
| DECODE_NODES \ | ||
| RANDOM_RANGE_RATIO \ | ||
| DURATION \ | ||
| KV_OFFLOADING \ | ||
| IS_AGENTIC \ | ||
| FRAMEWORK | ||
|
|
||
| if [[ -n "$SLURM_JOB_ID" ]]; then | ||
| echo "JOB $SLURM_JOB_ID running on $SLURMD_NODENAME" | ||
| fi | ||
|
|
||
| set -x | ||
|
|
||
| # Use upstreamed multi_node scripts (no external clone needed) | ||
| cd "$GITHUB_WORKSPACE/benchmarks/multi_node/amd_utils" || exit 1 | ||
|
|
||
| # Set up SGL launch script-specific environment variables | ||
| export TIME_LIMIT="${TIME_LIMIT:-08:00:00}" | ||
| export MODEL_PATH=$MODEL_PATH | ||
| export MODEL_NAME=$MODEL_NAME | ||
| export CONTAINER_IMAGE=$IMAGE | ||
|
|
||
| # ── Identity / result naming ── | ||
| export MODEL_PREFIX="${MODEL_PREFIX:-dsv4}" | ||
| export PRECISION="${PRECISION:-fp4}" | ||
| export RESULT_FILENAME="${RESULT_FILENAME:-${RUNNER_NAME:-dsv4-fp4-agentic}}" | ||
|
|
||
| # ── Agentic benchmark params ── | ||
| export DURATION="${DURATION:-1800}" | ||
| # DSV4-Pro max model len for agentic traces (matches single-node recipe). | ||
| export MAX_MODEL_LEN="${MAX_MODEL_LEN:-1000000}" | ||
|
|
||
| # ── In-tree sglang patches ── | ||
| # mori_conn.py targets hybrid-state bugs (GLM-5, Qwen3.5). DSV4-Pro uses a | ||
| # pure MoE/DSA architecture without hybrid state; skip to avoid interference. | ||
| export MORI_CONN_PATCH="${MORI_CONN_PATCH:-skip}" | ||
|
|
||
| # ── Aiter fault mitigation ── | ||
| # --disable-custom-all-reduce avoids a known aiter fault on MI355X. | ||
| export DISABLE_CUSTOM_ALL_REDUCE="${DISABLE_CUSTOM_ALL_REDUCE:-0}" | ||
|
|
||
| # ── KV cache offloading (HiCache) ── | ||
| # KV_OFFLOADING=none | dram (passed from YAML; default none for disagg). | ||
| # KV_OFFLOAD_BACKEND selects the backend when offloading is on; this recipe | ||
| # only implements HiCache, so "hicache" is the only supported value. | ||
| # HICACHE_TIER: L2 -> GPU + CPU-DRAM host pool. L3 -> + Mooncake store. | ||
| export KV_OFFLOADING="${KV_OFFLOADING:-none}" | ||
| if [[ "$KV_OFFLOADING" != "none" ]]; then | ||
| export KV_OFFLOAD_BACKEND="${KV_OFFLOAD_BACKEND:-hicache}" | ||
| fi | ||
| # HiCache/Mooncake tunables only matter when KV offloading is enabled. | ||
| if [[ "$KV_OFFLOADING" != "none" && "${KV_OFFLOAD_BACKEND:-}" == "hicache" ]]; then | ||
| export HICACHE_TIER="${HICACHE_TIER:-L2}" | ||
| export HICACHE_HOST_POOL_COUNT="${HICACHE_HOST_POOL_COUNT:-1}" | ||
| # DSV4 uses page-size 256 (set in models.yaml); HiCache must match. | ||
| export HICACHE_PAGE_SIZE="${HICACHE_PAGE_SIZE:-256}" | ||
| # HiCache ratio (host pool = ratio * GPU KV pool). Default derived in server_sglang.sh. | ||
| export HICACHE_RATIO="${HICACHE_RATIO:-}" | ||
|
|
||
| # ── HiCache layout/backend by tier ── | ||
| # L3 (Mooncake): page_first + direct + write_through + storage=mooncake | ||
| # L2 (CPU DRAM): layer_first + direct + write_through_selective + storage=none | ||
| # NOTE: write_through_selective evicts only under GPU memory pressure, avoiding | ||
| # the mori RDMA race that causes GPU memory access faults with write_through. | ||
| if [[ "${HICACHE_TIER^^}" == "L3" ]]; then | ||
| export HICACHE_MEM_LAYOUT="${HICACHE_MEM_LAYOUT:-page_first}" | ||
| export HICACHE_IO_BACKEND="${HICACHE_IO_BACKEND:-direct}" | ||
| export HICACHE_WRITE_POLICY="${HICACHE_WRITE_POLICY:-write_through}" | ||
| export HICACHE_STORAGE_BACKEND="${HICACHE_STORAGE_BACKEND:-mooncake}" | ||
| else | ||
| export HICACHE_MEM_LAYOUT="${HICACHE_MEM_LAYOUT:-page_first}" | ||
| export HICACHE_IO_BACKEND="${HICACHE_IO_BACKEND:-direct}" | ||
| export HICACHE_WRITE_POLICY="${HICACHE_WRITE_POLICY:-write_through}" | ||
| export HICACHE_STORAGE_BACKEND="${HICACHE_STORAGE_BACKEND:-}" | ||
| fi | ||
| export HICACHE_PREFETCH_POLICY="${HICACHE_PREFETCH_POLICY:-best_effort}" | ||
| # Shared nodes: use non-default Mooncake ports to avoid collisions. | ||
| export MC_MASTER_PORT="${MC_MASTER_PORT:-58137}" | ||
| export MC_METADATA_PORT="${MC_METADATA_PORT:-8080}" | ||
| export MC_METRICS_PORT="${MC_METRICS_PORT:-19003}" | ||
| export MC_MASTER_THREADS="${MC_MASTER_THREADS:-64}" | ||
| export MC_EVICTION_HIGH_WATERMARK="${MC_EVICTION_HIGH_WATERMARK:-0.95}" | ||
| export MC_PATCH_HOSTPOOL="${MC_PATCH_HOSTPOOL:-1}" | ||
| export MC_PROTOCOL="${MC_PROTOCOL:-tcp}" | ||
| export MC_GLOBAL_SEG="${MC_GLOBAL_SEG:-64gb}" | ||
| export MC_DEVICE="${MC_DEVICE:-}" | ||
| export MC_MASTER_ADDR="${MC_MASTER_ADDR:-}" | ||
| export MC_METADATA_SERVER="${MC_METADATA_SERVER:-}" | ||
| fi | ||
|
|
||
| # ── MoRIIO RDMA Send Queue tuning ── | ||
| export MORI_IO_SQ_BACKOFF_TIMEOUT_US="${MORI_IO_SQ_BACKOFF_TIMEOUT_US:-500000}" | ||
| export MORI_IO_QP_MAX_SEND_WR="${MORI_IO_QP_MAX_SEND_WR:-32768}" | ||
|
|
||
| # ── SGLang PD router policy + server metrics ── | ||
| export PREFILL_ROUTER_POLICY="${PREFILL_ROUTER_POLICY:-cache_aware}" | ||
| export ENABLE_METRICS="${ENABLE_METRICS:-1}" | ||
|
|
||
| # ── MTP ── | ||
| export DECODE_MTP_SIZE="${DECODE_MTP_SIZE:-0}" | ||
|
|
||
| # Derive EP/DP enable flags from the topology inputs. | ||
| if [[ "${PREFILL_EP:-1}" -eq 1 ]]; then | ||
| export PREFILL_ENABLE_EP=false | ||
| else | ||
| export PREFILL_ENABLE_EP=true | ||
| fi | ||
|
|
||
| if [[ "$PREFILL_DP_ATTN" == "true" ]]; then | ||
| export PREFILL_ENABLE_DP=true | ||
| else | ||
| export PREFILL_ENABLE_DP=false | ||
| fi | ||
|
|
||
| if [[ "${DECODE_EP:-1}" -eq 1 ]]; then | ||
| export DECODE_ENABLE_EP=false | ||
| else | ||
| export DECODE_ENABLE_EP=true | ||
| fi | ||
|
|
||
| if [[ "$DECODE_DP_ATTN" == "true" ]]; then | ||
| export DECODE_ENABLE_DP=true | ||
| else | ||
| export DECODE_ENABLE_DP=false | ||
| fi | ||
|
|
||
| # Launch the job. CONC_LIST is space-delimited in YAML; submit.sh wants 'x'. | ||
| JOB_ID=$(bash ./submit.sh $PREFILL_NODES \ | ||
| $PREFILL_NUM_WORKERS \ | ||
| $DECODE_NODES \ | ||
| $DECODE_NUM_WORKERS \ | ||
| $ISL $OSL "${CONC_LIST// /x}" inf \ | ||
| ${PREFILL_ENABLE_EP} ${PREFILL_ENABLE_DP} \ | ||
| ${DECODE_ENABLE_EP} ${DECODE_ENABLE_DP} \ | ||
| ${PREFILL_TP} ${DECODE_TP} \ | ||
| ${RANDOM_RANGE_RATIO}) | ||
|
|
||
| if [[ $? -ne 0 ]]; then | ||
| echo "Failed to submit job" >&2 | ||
| exit 1 | ||
| fi | ||
|
|
||
| echo "$JOB_ID" | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@cquil11 plz check how this var works and if it follows the rule of ALLOWED_CPU_MEM = amount_of_cpu_mem * (# of gpus used per node / total GPUs per node)