已合并
fix(driver): 修复注册失败清理并补齐插件测试部署 #416
liuxingsen创建于 9月24日
fix(driver): 修复注册失败清理并补齐插件测试部署 #416
已合并
Pull Request已成功合入, 合并人@openUBMC-robot
(感谢 liuxingsen 的贡献)9月24日 创建了 pull request,commit 1794adf6
9月24日 添加了label:sig/sig-component-drivers
openUBMC-robot
9月24日 评论:
9月24日 评论:
欢迎来到 openUBMC 社区
Hey @xiaoxinglxs , 感谢你对社区的贡献.
机器人使用手册
有关指令的使用,可以点击 说明文档 或 论坛 查看详情。开发人员可以在每个PR或Issue下方评论特定指令来触发机器人任务。您可以自助配置此仓库的 PR 合并规则,更多详情,请参阅机器人自助配置使用手册。
联系指引
有疑问可以联系 SIG: sig-component-drivers ,
维护者是: @dongyushan, @net_swift_wangchen, @sam_h1973, @u012924942, @weixin_37569011, @xiaoxinglxs ,
优秀贡献者是: @WeliveWelove, @chenshengfan, @leessential, @zjp99 .


9月24日 添加了label:ai-co-authored
9月24日 添加了label:openUBMC-cla/yes
此处折叠了72条消息 查看更多
lishengxiao
22 天前 评论:
22 天前 评论:
/lgtm
/approve
/check-pr


22 天前 添加了label:approvedlgtm
22 天前 删除了label:unresolved-reviews
openUBMC-robot
22 天前 评论:
22 天前 评论:
22 天前 合入了pull request,合并节点 SHA:82f7d573ddb39914ad1491fd5788e0561b6a5e79
当前PR是否有AI参与:
[x ] 否
[] 是
__1. AI Agent 平台:
__2. AI 模型:
__3. Prompt上下文 :
PR功能描述 / 为什么需要这个合入:
动态库打开后注册失败时,原实现正常返回并保留句柄;重复加载还会覆盖旧句柄。修复后,注册失败清理资源并向调用方报错,串行重复加载保留原句柄并拒绝操作。
CI 的 temp/bin/devmon_test 会运行两条真实插件异常用例,但原测试插件只构建、未安装,导致缺夹具失败。本次补齐构建和安装入口,并整改同次 CI 报告的 13 个文件头。
修改方案:
验证方案:
本地构建后运行配置内完整 Meson 套件,再执行真实 meson install,取消插件目录变量,从安装布局运行驱动专项回归;移走夹具验证用例确实检出缺包,恢复后执行随机顺序重复测试。使用 CI 原版文件头检查器及仓库 pre-commit 检查。推送后单独核对当前 SHA 的远端流水线。
测试建议:
在标准 CI/Bingo 依赖环境复跑完整 UT;后续补充完整 unidev 运行、ASAN 和目标板验证。同一实例并发 load/unload 不在接口支持范围内,不能将串行守卫解释为线程安全。
自验证报告:
2026-10-13,本次实际验证:
c93b0d1a286e328ae86afad0cda9470c31b1f171,2026-10-13 23:07 完成:pre-commit、PR 描述、防投毒、开源合规、编译构建、UT、头文件、AI Review 共 8 项启用检查全部 SUCCESS。UT 日志确认两个插件已安装到 Conan 测试包 bin,实际入口仍为 temp/bin/devmon_test,并报告 UT测试通过。该PR关联的issue
fixes #144
希望检视人员了解:
在主线 9eea32c 上整理为两个提交:功能及测试修复、文件头整改。原提交已留存备份,保持同一个 PR。三个既有检视意见已在整改推送并核实后标记解决。远端 run #91 已全部通过。保留两个提交,stat/needs-squash 按社区流程由评审设置扁平化合并;unresolved-reviews 是尚未刷新的汇总标签,实际三条意见均已 resolved=true。不以旧 ai-compliance-successful 标签代替复检。未调用可能自动合入的 /check-pr。本次不执行合入。