Prhub

#48549 [Misc] Clean up "swap_space"

原始 PR 作者 wangxiyuan 合并时间 2026-07-14 12:43 文件变更 1 提交数 1 评论 0 代码增减 +0 / -11

执行摘要

清除废弃参数 swap_space 的兼容代码

PR 描述指出 swap_space 自 v0.18.0 起已废弃,现在可以安全地清理该警告,以消除遗留的兼容代码。

该 PR 属于常规维护清理,值得快速合并。无需特别审查或测试。

讨论亮点

无讨论。

实现拆解

仅涉及一个文件 vllm/entrypoints/llm.py,删除了 LLM.__init__ 方法中检查 kwargs 是否包含 swap_space 并弹出它、同时发出 DeprecationWarning 的代码块(共11行)。本次变更为纯删除,无新增代码。

文件 模块 状态 重要度
vllm/entrypoints/llm.py 入口 modified 5.01

关键源码片段

vllm/entrypoints/llm.py dependency-wiring

唯一变更文件,移除了 swap_space 参数的废弃兼容逻辑。

# vllm/entrypoints/llm.py ( 变更后 )
def __init__(
    self,
    ...
    **kwargs: Any,
) -> None:
    """LLM constructor."""
​
    # 注意:swap_space 参数兼容代码已删除(自 v0.18.0 起废弃)
​
    if "disable_log_stats" not in kwargs:
        kwargs["disable_log_stats"] = True
​
    if "worker_cls" in kwargs:
        worker_cls = kwargs["worker_cls"]
        if isinstance(worker_cls, type):
            kwargs["worker_cls"] = cloudpickle.dumps(worker_cls)
​
    if "kv_transfer_config" in kwargs and isinstance(
        kwargs["kv_transfer_config"], dict
    ):
        from vllm.config.kv_transfer import KVTransferConfig
        raw_config_dict = kwargs["kv_transfer_config"]
        try:
            kwargs["kv_transfer_config"] = KVTransferConfig(**raw_config_dict)
        except ValidationError as e:
            logger.error(...)
            raise ValueError(...)
​
    if hf_overrides is None:
        hf_overrides = {}

评论区精华

没有提炼出高价值讨论线程

当前评论区没有形成足够清晰的争议点或结论,后续有更多讨论时会体现在这里。

风险与影响

风险极低。删除的是已长期废弃参数的兼容处理,即使仍有用户传入 swap_space,该参数会被 **kwargs 接收并传入底层配置,但底层已不再使用,因此不会引发错误或行为改变。

影响范围极小,仅涉及 Python 同步调用接口的构造函数,不涉及异步接口或服务端。用户若仍使用该参数,将不再收到弃用警告,但参数本身仍会被静默忽略。

关联 Issue

未识别关联 Issue

当前没有检测到明确关联的 Issue 链接,后续同步到相关引用后会出现在这里。

完整报告

参与讨论