执行摘要

• 一句话：修复 diffusion LLM 禁用 PP 条件取反
• 推荐动作：该 PR 变更简单且关键，值得快速合并。建议阅读 _handle_dllm_inference() 方法以理解 diffusion LLM 推理下的参数覆盖模式，并检查其他类似 if not 模式是否存在类似反转问题。

功能与动机

_handle_dllm_inference() 旨在为 diffusion LLM 推理禁用 pipeline parallelism（不支持 pp_size > 1），但条件 not self.pp_size > 1 导致：当 pp_size > 1 时 PP 未被禁用，产生错误配置；当 pp_size == 1 时发出无用警告。PR body 明确指出了这个反转和带来的误导。

实现拆解

1. 定位问题行：在 python/sglang/srt/server_args.py 的 _handle_dllm_inference 方法中，第 4561 行条件 if not self.pp_size > 1 需要修正。
2. 修正条件：去除 not，改为 if self.pp_size > 1，使得仅在 pipeline parallelism 实际启用（pp_size > 1）时执行日志警告并将 self.pp_size 置为 1。
3. 保持其他逻辑不变：该方法中其余禁用逻辑（如 overlap schedule、radix cache、LoRA 等）未受影响。
4. 未新增测试：由于变更极小且为逻辑修复，未引入额外测试，依赖 CI 覆盖。

关键文件：

• python/sglang/srt/server_args.py（模块 启动参数；类别 source；类型 core-logic；符号 _handle_dllm_inference）: 包含 _handle_dllm_inference() 方法，其中 PP 禁用条件被修正，是本次变更的唯一文件。

关键符号：_handle_dllm_inference

关键源码片段

python/sglang/srt/server_args.py

包含 _handle_dllm_inference() 方法，其中 PP 禁用条件被修正，是本次变更的唯一文件。

def _handle_dllm_inference(self):
    # ... 其他参数覆盖逻辑 ...
​
    # 修正前 : if not self.pp_size > 1: # 条件取反，错误行为
    # 修正后 : if self.pp_size > 1: # 仅在 PP 启用时报错并禁用
    if self.pp_size > 1:
        logger.warning(
            "Pipeline parallelism is disabled because of using diffusion LLM inference"
        )
        self.pp_size = 1
​
    if self.enable_lora:
        # ...

评论区精华

Review 过程简短：btw616 和 ClawSeven 均给予 LGTM。CI 中存在部分 flaky 失败，但确认与 PR 无关。ClawSeven 和 Ronnie-Rui 多次 rerun failed CI 以尝试通过全部检查。

• CI 失败是否为 flaky (other): 确认 CI 失败与 PR 无关，但需所有检查通过，等待 rerun 后成功合并。

风险与影响

• 风险：该 PR 仅修改一行条件，将 not self.pp_size > 1 改为 self.pp_size > 1，逻辑纠正后风险极低。唯一潜在风险是若其他代码路径依赖于错误的 PP 禁用行为（pp_size == 1 时日志触发），但经评估不存在此类依赖。
• 影响：影响范围仅限于 diffusion LLM 推理流程。修复后，当用户配置 pp_size > 1 并启用 diffusion LLM 时，pipeline parallelism 会被正确禁用并给出提示警告，避免运行时错误。对于 pp_size == 1 的用户，不再出现误导性日志。功能影响程度低但正确性意义高。
• 风险标记：极低风险

关联脉络

• PR #22659 Add sleep/wake support for diffusion engine: 同为 diffusion 引擎相关功能，且涉及 scheduler 和参数配置，本 PR 的 _handle_dllm_inference 是 diffusion 推理的配置入口。