执行摘要

• 一句话：删除 vLLM rollout 中未使用的 get_device_uuid
• 推荐动作：该 PR 简单直接，无需精读。可作为代码清理的范例参考。

功能与动机

get_device_uuid 写入的 self.device_uuid 在 ServerAdapter.__init__ 后未被任何代码读取，属于死代码。PR body 简洁说明 'Delete the useless code'，目的是清理未使用的函数和属性。

实现拆解

1. 删除工具函数 get_device_uuid：在 verl/workers/rollout/vllm_rollout/utils.py 中移除了整个 get_device_uuid 函数（约 16 行）。同时删除了该函数依赖的 from verl.plugin.platform import get_platform 导入。
2. 清理导入和调用：在 verl/workers/rollout/vllm_rollout/vllm_rollout.py 中移除了 from verl.utils.device import get_device_id 和 from verl.workers.rollout.vllm_rollout.utils import get_device_uuid 导入，并删除了 __init__ 中的 self.device_uuid = get_device_uuid(get_device_id()) 语句。
3. 影响范围：仅涉及两个文件，无测试变更，无配置或部署配套改动。

关键文件：

• verl/workers/rollout/vllm_rollout/utils.py（模块 Rollout；类别 source；类型 dependency-wiring；符号 get_device_uuid）: 删除了一整个未使用的函数 get_device_uuid 及其相关导入，是本次 PR 的核心变更。
• verl/workers/rollout/vllm_rollout/vllm_rollout.py（模块 Rollout；类别 source；类型 dependency-wiring）: 移除了对已删除函数 get_device_uuid 的导入和调用，以及 get_device_id 的导入，完成了死代码清理的收尾。

关键符号：get_device_uuid

关键源码片段

verl/workers/rollout/vllm_rollout/utils.py

删除了一整个未使用的函数 get_device_uuid 及其相关导入，是本次 PR 的核心变更。

def set_death_signal():
    """Kill the current process when the parent process exits."""
    if platform.system() != "Linux":
        return
    libc = ctypes.CDLL("libc.so.6")
    libc.prctl(1, signal.SIGKILL)
    if os.getppid() == 1:
        os.kill(os.getpid(), signal.SIGKILL)
​
​
# 以下 get_device_uuid 函数已被删除，因为其调用者已被清理。
# def get_device_uuid(device_id: int) -> str:
# from vllm.platforms import current_platform
# if is_npu_available:
# if os.getenv("ASCEND_RT_VISIBLE_DEVICES") is not None:
# npu_visible_devices = os.environ["ASCEND_RT_VISIBLE_DEVICES"].split(",")
# assert device_id < len(npu_visible_devices)
# return "NPU-" + npu_visible_devices[device_id]
# else:
# return f"NPU-{device_id}"
# else:
# try:
# return current_platform.get_device_uuid(device_id)
# except Exception:
# return get_platform().get_device_uuid(device_id=device_id)
​
​
def get_vllm_max_lora_rank(lora_rank: int):
    """
    For vLLM, automatically adjusts the `max_lora_rank` to the nearest allowed value.
    """
    assert lora_rank > 0, f"lora_rank must be greater than 0, get {lora_rank}"
    ...

verl/workers/rollout/vllm_rollout/vllm_rollout.py

移除了对已删除函数 get_device_uuid 的导入和调用，以及 get_device_id 的导入，完成了死代码清理的收尾。

from verl.utils.device import is_support_ipc # 移除了 get_device_id 的导入
from verl.workers.config import HFModelConfig, RolloutConfig
from verl.workers.rollout.base import BaseRollout
from verl.workers.rollout.vllm_rollout.bucketed_weight_transfer import BucketedWeightSender
# 已删除 : from verl.workers.rollout.vllm_rollout.utils import get_device_uuid
​
# 在 __init__ 方法中，以下行已被删除：
# self.device_uuid = get_device_uuid(get_device_id())
# CUDA IPC handle 的构造现在直接使用 replica_rank 和本地 rank，不再依赖 device_uuid。
​
# 注释保留了原来解释为何不使用 GPU UUID 的理由：
# CheckpointEngineWorker and vLLM worker may see different GPU UUIDs
# when CUDA_VISIBLE_DEVICES differs between processes (common on ROCm/AMD).

评论区精华

只有一条来自 wuxibin89 的 Approved 审核，无其他讨论。

• 暂无高价值评论线程

风险与影响

• 风险：风险极低。变更仅移除未被引用的代码和导入，不会影响运行时行为。但若未来有外部代码（如插件或扩展）依赖 get_device_uuid 的导出，可能会遇到导入错误，不过该函数在模块私有范围内，且未被 __all__ 导出。
• 影响：
  • 用户影响：无，用户无需任何操作。
  • 系统影响：减少约 22 行死代码，降低维护成本。
  • 团队影响：简化了 vLLM rollout 模块的启动流程，明确了 IPC handle 的构造逻辑。
  • 风险标记：无风险

关联脉络

• PR #7139 [sglang] fix: use _base guard in _compact_for_bucket to prevent NCCL buffer race: 与 PR #7139 类似，都是对 rollout 模块中权重传输逻辑的清理/修复，但 #7139 是 bugfix 而本 PR 是死代码清理，反映了团队在持续优化 vLLM rollout 的 IPC 和权重同步路径。
• PR #7173 Revert "[megatron] fix: Qwen3.5 LoRA & MTP support (with Megatron-Bridge) (#5599)": PR #7173 回滚了部分 vLLM rollout 的变更，本 PR 进一步清理未使用的代码，两者共同指向对 vLLM rollout 模块的维护和简化。