执行摘要

PR #44413 修复了 LoRA 模块包装过程中，当别名路径在原始模块被替换后才解析时，同一包装器仍可能被重复注册的 bug。仅在 vllm/lora/model_manager.py 中新增一行代码，将包装器 id 也记录到去重字典中，确保后替换的别名路径能正确复用现有包装器。

功能与动机

PR #42757 已修复了同一原始模块通过多个属性路径可达时的 LoRA 去重问题，但仅追踪原始模块的 id。在 Gemma4 等模型中，self_decoder.decoder_layers 是 layers 的别名，当规范路径 layers.* 的模块先被包装后，别名路径遍历到的已是 BaseLayerWithLoRA 包装器对象，而非原始模块。由于 wrapped_by_id 中缺乏包装器的 id，去重逻辑失效，导致同一包装器被再次注册到 self.modules。适配器激活时，规范名称下的路径可以正确加载 LoRA 权重，但别名路径由于名称不匹配无法加载，会触发 reset_lora 清除同一包装器的权重，造成 LoRA 权重丢失。

实现拆解

1. 修改文件：vllm/lora/model_manager.py。
2. 变更位置：_create_modules 方法中，在已有 wrapped_by_id[id(module)] = new_module 之后新增一行。
3. 新增代码：wrapped_by_id[id(new_module)] = new_module。
4. 作用：将新创建的 BaseLayerWithLoRA 包装器的 id 也加入字典，使得后替换的别名路径（此时 module 参数已是包装器）能命中 wrapped_by_id，从而复用现有包装器，而不是注册第二份。

vllm/lora/model_manager.py

核心 LoRA 管理逻辑，包含模块包装与去重逻辑。关键变更位于 _create_modules 方法中的 wrapped_by_id 更新。

            """在模块包装循环中，当创建新的 `BaseLayerWithLoRA` 后，
            同时记录原始模块 id 和包装器 id，
            使得后替换的别名路径（已指向包装器）也能正确复用。"""
            if isinstance(new_module, BaseLayerWithLoRA):
                wrapped_by_id[id(module)] = new_module # 已有：记录原始模块 id
                wrapped_by_id[id(new_module)] = new_module # 新增：记录包装器 id

评论区精华

审核人 @jeejeelee 直接批准了 PR，无额外讨论，表明修复方案直观且正确。

风险与影响

• 风险：极低。新增一行赋值语句，位于类型检查块内，无副作用。
• 影响：仅影响具有模块别名的模型（如 Gemma4），修复后 LoRA 权重在激活时不会再被意外清除。无别名模型的用户无感知。

关联脉络

• 本 PR 是 PR #42757（[LoRA] Fix dedup for shared-alias LoRA wrapper registration）的补充修复，解决了同一问题的边界情况。
• 属于 LoRA 模块包装机制的持续改进，提升模型兼容性。