执行摘要

• 一句话：新增环境变量强制启用非 CUDA 平台的 WAR barrier
• 推荐动作：值得精读：这是一个小而关键的修复，展示了通过环境变量处理平台差异的良好实践。AMD 用户应关注此 flag 并评估启用后的收益。

功能与动机

overlap scheduler 使用 WAR barrier 确保当前迭代 schedule-stream 对共享 GPU 缓冲区（如 req_to_token_pool/SWA mapping）的写入等待前一迭代前向计算的读取完成。原有逻辑仅对 CUDA 平台启用该 barrier，但 AMD/ROCm 上存在相同的写后读风险。PR 旨在通过环境变量让非 CUDA 平台也能按需启用 barrier，提升 AMD 等后端的正确性。

实现拆解

1. 在 python/sglang/srt/environ.py 中新增环境变量定义：SGLANG_ENABLE_WAR_BARRIER = EnvBool(False)，默认关闭。
2. 在 python/sglang/srt/managers/scheduler.py 的 run_event_loop 方法中修改 barrier 启用逻辑：将 self._war_barrier_enabled = is_cuda() and not self.spec_algorithm.is_dflash() 改为 (is_cuda() or envs.SGLANG_ENABLE_WAR_BARRIER.get()) and not self.spec_algorithm.is_dflash()。当 flag 为 True 时，在非 CUDA 平台也启用 barrier；DFLASH 的排除条件保留。
3. 未引入新的测试文件：本次改动不涉及测试，但通过 CI stage-a 验证（合并者确认非操作）。

关键文件：

• python/sglang/srt/managers/scheduler.py（模块 调度器；类别 source；类型 core-logic；符号 run_event_loop）: 核心变更点：修改 _war_barrier_enabled 的条件，将 is_cuda() 替换为 is_cuda() or envs.SGLANG_ENABLE_WAR_BARRIER.get()，使非 CUDA 平台在 flag 下也可启用 barrier。
• python/sglang/srt/environ.py（模块 配置；类别 source；类型 configuration）: 新增环境变量声明，定义 SGLANG_ENABLE_WAR_BARRIER 并默认 False，为 scheduler 提供配置入口。

关键符号：run_event_loop

评论区精华

讨论集中在权限和 CI 状态上：

• oulgen 表示没有添加标签/审查者的权限，询问是否运行 CI 或直接合并。
• hnyls2002 触发 CI 后，确认 stage-a 通过且该 PR 为非操作（non-op），直接合并。

• yinghai 表示 LGTM。

• CI 状态与合并确认 (other): CI stage-a 通过，合并。

风险与影响

• 风险：风险较低，属于 opt-in 的环境变量扩展。
• 回归风险：默认行为不变（False），现有 CUDA 用户无影响。
• 性能风险：在 AMD 上启用 barrier 可能引入额外同步开销，但这是为了正确性，用户可根据测试决定是否开启。
• 兼容性：未修改现有 API 或数据结构，仅新增环境变量。
• 影响：直接影响 AMD/ROCm 用户，允许他们通过设置 SGLANG_ENABLE_WAR_BARRIER=1 来启用 WAR barrier，避免潜在的写后读数据竞争。CUDA 用户无感知。对 DFLASH 推测解码路径无影响。
• 风险标记：新增环境变量, 默认关闭无影响, AMD 平台 opt-in

关联脉络

• PR #27919 Revert "[AMD] Fix DeepSeek V4 Pro c128 state tensor dtype mismatch error and c4_sparse_raw_indices attribute error in cuda graph phase": 同为 AMD 相关修复，涉及 cuda graph 和 kernel 调整
• PR #27850 [AMD] Fix DSA device-to-host direct test on rocm720 (page_size%16 assert): 同为 AMD 平台测试修复，体现团队对 AMD 后端的持续关注