PR 21691 Review Report
标题：cgroup/dmem: reuse SUBSYS for dmem and devices to preserve KABI
审查结论：Request Changes
详细审查结果
补丁格式 pass
- hulk inclusion 正确
- bugzilla 正确
- Fixes 标签正确
- 详细描述了四种运行模式
KABI/KAPI 兼容性 pass
- 通过共享 SUBSYS slot 避免 CGROUP_SUBSYS_COUNT 增加,正确保护 KABI
- BUILD_BUG_ON 从 17 改为 16
安全性 pass
- 无直接安全问题
逻辑正确性 warning
- dmem_devices_compat_css_alloc 中版本不匹配返回 -EPERM
- static_branch 控制动态行为
- #define devices_cgrp_id dmem_cgrp_id 宏替换
Backport 验证 N/A
- 非 backport
发现问题汇总
| 严重级别 | 类别 | 问题描述 | 修复建议 |
|---|---|---|---|
| high | architecture | dmem 和 devices 共享同一个 cgroup subsys slot 是一个相当 hacky 的设计。虽然当前它们互斥(v1 vs v2),但未来如果需求变化,这种耦合会严重限制扩展性 | 建议评估是否有更干净的方式(如使用未使用的 slot 或扩展 subsys 数量),至少应在代码中添加大量注释说明维护约束 |
| medium | compatibility | 注释中提到 'cpuacct_cgrp_subsys and dmem_cgrp_subsys share the same set of cgroup_subsys hook functions' 但代码中实际共享的是 devices 和 dmem | 修正注释中的笔误 |
修复建议
- 修正注释笔误
- 确认维护者接受这种 slot 共享方案
- 添加更醒目的维护警告注释
⚠️ 免责声明
本网站内容由 AI 工具辅助生成，仅供参考和学习使用。作者不对内容的准确性、完整性和适用性做任何保证。使用本站信息所造成的任何损失，作者不承担任何责任。请在生产环境使用前进行充分测试和验证。