执行摘要

• 一句话：用 sentinel 替换 hasattr 惰性初始化
• 推荐动作：值得精读的示例性重构：展示了如何用 sentinel 消除 hasattr 的隐式依赖，使缓存语义明确。适合作为代码可维护性改进的参考。

功能与动机

PR body 指出：之前的 hasattr 模式能工作，但防御了一个不存在的威胁（属性总是由 __init__ 保证存在），且隐藏了缓存状态区分。使用 sentinel _UNSET 类似于 Rust 的 OnceCell<Option<T>>，使生命周期和缓存命中分支都变得显式。

实现拆解

1. 添加 sentinel 和类型导入：在 model_runner.py 中定义 _UNSET: Any = object()，并从 typing 导入 Any。
2. 在 __init__ 中初始化缓存：在 ModelRunner.__init__ 中设置 self._linear_attn_registry_cache: Any = _UNSET。
3. 替换惰性初始化检查：在 _get_linear_attn_registry_result 中将 if not hasattr(self, "_linear_attn_registry_cache") 改为 if self._linear_attn_registry_cache is _UNSET。
4. 移除隐式依赖：不再依赖 hasattr 在属性不存在时触发惰性初始化，而是通过显式 sentinel 检查。

关键文件：

• python/sglang/srt/model_executor/model_runner.py（模块 模型运行器；类别 source；类型 data-contract；符号 _UNSET, _linear_attn_registry_cache, _get_linear_attn_registry_result）: 核心变更文件，修改了缓存惰性初始化模式。

关键符号：_get_linear_attn_registry_result, init

关键源码片段

python/sglang/srt/model_executor/model_runner.py

核心变更文件，修改了缓存惰性初始化模式。

# 定义全局 sentinel 对象，用于区分 " 未设置 " 与 " 结果为 None"
_UNSET: Any = object()
​
class ModelRunner:
    def __init__(self, ...):
        # ... 其他初始化代码 ...
        # 显式初始化缓存字段为 _UNSET，而非依赖 hasattr 隐式创建
        self._linear_attn_registry_cache: Any = _UNSET
        # ...
​
    def _get_linear_attn_registry_result(self):
        # 使用 sentinel 区分 " 尚未计算 " 和 " 计算结果为 None"
        # 之前用 hasattr，若属性不存在则触发惰性初始化
        # 但属性实际已在 __init__ 中设置，因此 hasattr 总是 True，掩盖了缓存语义
        if self._linear_attn_registry_cache is _UNSET:
            self._linear_attn_registry_cache = get_linear_attn_config(
                self.model_config.hf_config
            )
        return self._linear_attn_registry_cache

评论区精华

只有一个自动化评论（gemini-code-assist）提示配额已满，无人工 review 讨论。

• 暂无高价值评论线程

风险与影响

• 风险：风险极低：变更仅涉及一个私有方法内部的缓存访问模式，且为机械重构。若 sentinel 名称冲突或 __init__ 中未正确初始化，可能退化为每次调用都重新计算，但通过单元测试可覆盖。
• 影响：仅影响 ModelRunner._get_linear_attn_registry_result 的内部实现，对外部行为和性能无影响。hasattr 在 Python 中开销略高于 sentinel 比较，此变更有微小的性能改善。
• 风险标记：低风险重构

关联脉络

• PR #25448 Inline the trivial _build_model_config wrapper: 同为对 model_runner.py 的机械式重构，简化代码结构。
• PR #25437 Drop dead hasattr guards (hisparse_coordinator, metrics_collector): 同样移除了 hasattr 守卫，与本次变更精神一致。