遵循新确立的代码规范（#30701）：当派生值的输入在对象生命周期内不可变时，应在 init 中一次性计算并读取属性，避免在热路径中重复推导。PR body 明确强调 "only values re-derived 3+ times in hot paths are cached; single-use derivations are left alone"，体现了性能与可读性之间的审慎权衡。
值得合并。变更集小巧、验证充分、逻辑可审计。建议作为 "Extract init-static values" 系列的标准规范示例精读，学习如何安全地进行此类提取。
__init__ 中新增两个缓存属性 self.enable_dp_attention 和 self.enable_unified_memory，分别从 server_args.enable_dp_attention 和 server_args.enable_unified_memory 赋值。随后在 __init__ 后续逻辑（decode_offload_manager 的 tp_group 选择）、init_model_worker（dp_tp_group 选择）、event_loop_overlap、run_batch、_maybe_report_active_ranks、on_idle 等 6 个方法中将原 self.server_args.* 的读取替换为 self.*。__init__ 中新增 self.decode_log_interval = self.scheduler.server_args.decode_log_interval，并在 report_decode_stats（3 处）和 update_device_timer（1 处）中替换。prepare_for_extend、prepare_for_decode 和 _mamba_radix_cache_v2_req_prepare_for_extend 方法中，将分散在方法各处的 7 次 get_server_args() 调用提升为方法开头的一次局部变量赋值。这沿用了已有模式（如 maybe_evict_swa）。_fake_server_args 中添加 decode_log_interval 的默认值，确保测试中 MetricsReporter 能正常访问该该属性。| 文件 | 模块 | 状态 | 重要度 |
|---|---|---|---|
python/sglang/srt/managers/scheduler.py |
调度器 | modified | 6.58 |
python/sglang/srt/managers/scheduler_components/metrics_reporter.py |
指标报告器 | modified | 6.18 |
python/sglang/srt/managers/schedule_batch.py |
调度批次 | modified | 6.08 |
test/registered/unit/observability/test_forward_pass_metrics.py |
观测测试 | modified | 2.71 |
python/sglang/srt/managers/scheduler.py
core-logic
调度器主文件，在 __init__ 中新增 enable_dp_attention 和 enable_unified_memory 缓存属性，并替换了 6 处热路径读取。
# python/sglang/srt/managers/scheduler.py
# 在 __init__ 方法中新增两行（位于原有配置属性之后）：
self.enable_dp_attention = server_args.enable_dp_attention
self.enable_unified_memory = server_args.enable_unified_memory
# 替换示例 1：decode_offload_manager 构建时使用 self.enable_dp_attention
self.decode_offload_manager = DecodeKVCacheOffloadManager(
req_to_token_pool=self.req_to_token_pool,
token_to_kv_pool_allocator=self.token_to_kv_pool_allocator,
tp_group=(
self.attn_tp_cpu_group
if self.enable_dp_attention # 原 self.server_args.enable_dp_attention
else self.tp_cpu_group
),
tree_cache=self.tree_cache,
server_args=self.server_args,
)
# 替换示例 2：init_model_worker 中 dp_tp_group 选择
self.dp_tp_group = (
self.attn_tp_group if self.enable_dp_attention else self.tp_group
)
# 替换示例 3：run_batch 方法中记录 forward_done 事件
if self.enable_unified_memory: # 原 self.server_args.enable_unified_memory
# Record a `forward_done` event after the forward
self.token_to_kv_pool_allocator.record_forward_done(batch)
python/sglang/srt/managers/scheduler_components/metrics_reporter.py
core-logic
指标报告器，缓存 decode_log_interval 避免每次迭代访问 server_args。
# python/sglang/srt/managers/scheduler_components/metrics_reporter.py
# 在 __init__ 的早期（约第 148 行）新增：
self.decode_log_interval = self.scheduler.server_args.decode_log_interval
# 替换示例：report_decode_stats 方法中的周期判断
# 原：if self.forward_ct_decode % self.scheduler.server_args.decode_log_interval != 0:
# 改为：
if self.forward_ct_decode % self.decode_log_interval != 0:
return
# 替换示例 2：step_time_dict 更新
self.step_time_dict[num_running_reqs].append(
gap_latency / self.decode_log_interval
)
# 替换示例 3：decode_sol_suffix 调用
msg += self._decode_sol_suffix(
batch,
gap_latency / max(1, self.decode_log_interval),
)
python/sglang/srt/managers/schedule_batch.py
core-logic
调度批次文件，将 7 次 get_server_args() 提升为局部变量，减少重复调用。
# python/sglang/srt/managers/schedule_batch.py
def prepare_for_extend(self):
self.forward_mode = ForwardMode.EXTEND
server_args = get_server_args() # 原分散在各处，现提升到方法开头
# ... 中间代码不变 ...
# 替换示例：原本的 get_server_args().enable_mamba_extra_buffer()
if server_args.enable_mamba_extra_buffer():
track_entry = self._mamba_radix_cache_v2_req_prepare_for_extend(req)
# ...
当前评论区没有形成足够清晰的争议点或结论，后续有更多讨论时会体现在这里。
风险极低。所有变更为纯属性读取替换（self.server_args.X → self.X），逻辑完全等价。作者通过 AST 静态等价验证工具确认了每个替换的正确性。测试配套仅补充了默认字段值，无新测试用例，且 CI 已通过。唯一潜在风险是：未来若有人在 __init__ 之后修改 server_args 中的对应字段，缓存值会与最新配置不一致。但该类的设计假设 server_args 在构造后不可变，因此风险可接受。
影响范围：正面的微性能优化，减少了调度器热路径上的 10 余次属性访问和 7 次函数调用（get_server_args）。对用户无功能影响，对系统性能有轻微正向提升（每次调度迭代减少若干属性查找和函数调用开销）。对团队的主要影响是确立了新的编码规范。
当前没有检测到明确关联的 Issue 链接，后续同步到相关引用后会出现在这里。
参与讨论