Merged
fix(drivers): 修正数值解码并使用独立 PR 日志 #1333
fix(drivers): 修正数值解码并使用独立 PR 日志 #1333
Merged
liuxingsenAdd on 25 days ago
liuxingsenCollaborators
25 days ago

当前PR是否有AI参与:

[ ] 否

[x] 是
__1. AI Agent 平台: OpenAI Codex Desktop
__2. AI 模型: gpt-6-astra(原业务修复及本轮日志格式规范、检查和说明更新)
__3. Prompt上下文 : 用户要求精简独立日志填写负担:只必填 PR 标题及链接、组件版本、变更要点;有不兼容改动、迁移要求或影响使用的限制时补充说明,Issue 和分类前缀可选,详细验证及实现信息保留在 PR 描述。公共规范和根入口仅由 !1333 引入,!1347 只维护自己的片段;保留业务、版本、单提交、小于999行及版权/贡献元数据,验证后刷新两个既有 PR。

PR功能描述 / 为什么需要这个合入:

修正 PRBS 打包字段的未对齐及小端读取,并让 Huawei、Samsung、DERA 的 NVMe SMART 先组合完整 128 位整数再单次转换为既有 double 属性。

将本 PR 日志移至独立文件,避免多个 PR 同时向根 CHANGELOG 顶部追加内容造成冲突。

修改方案:

  • PRBS 按字段偏移复制到对齐局部值后转换小端字节序;三厂 SMART 共用 nvme_read_u128,保留未对齐非对称输入、一次舍入及共享 mock 回归。
  • 保留 Northlink unity helper、BDF 真实通知等待及 I2C/JTAG/PCA9545 失败初始化测试对象清理。
  • 主干已通过 !1445 撤销旧准备工具,本 PR 同步移除对旧工具的扩展、恢复命令和补救片段迁移,不重新引入已撤销内容。
  • 基于当前 main a98eb95dc966ead6b0b3fe03b484bbf658025a58,新增 changelog.d/pr-1333.md,根 CHANGELOG 与基线逐字节一致。
  • 组件候选版本手工维护为 1.2.579,日志记录同一版本;不引入自动升版、合入队列或合入后回写。
  • 仅本 PR 新增公共 changelog.d/README.md 和根 README 入口;模板只必填 PR 标题链接、组件版本和变更要点。兼容性影响按需说明,Issue 与分类前缀可选,详细验证留在 PR 描述;!1347 不重复添加公共文件。
  • !1333 的公共文档提供只读汇总命令,只读取固定 HEAD 提交中的日志及版本,按 PR 编号展示后附上根历史;输出到仓库外,不消费、删除或改写原日志。既有手工根日志方式可并存,同一 PR 不重复记录。

验证方案:

检查业务文件与旧提交的逐字节一致性、根历史保留、组件版本与本 PR 日志一致、版权及提交元数据。对最终提交进行真实 Git 双向顺序合并、手工日志兼容与固定提交汇总验证,并核查新 SHA 的远端门禁。

测试建议:

按新 SHA 执行社区完整构建、UT 和 ASAN;正式依赖及目标板验证原业务修复。发布前根据实际源码提交检查版本可用性;两个 PR 的候选版本相同不是允许重复发布的依据。若后续启用要求逐 PR 修改根 CHANGELOG 的版本门禁,须先适配独立日志格式。

自验证报告:

2026-11-02 当前提交 a544cd0da81fc6de24427e4f47eecc0b182e5515,相对 a98eb95d 保持一个提交;24 个文件,+467/-127,共 594 行,小于999行。

本轮相对前版 ab545599c857d493c46170c8b6937f52480d53f1 只精简日志文档,片段由25行减至9行。业务、测试及 service.json 逐字节不变,根 CHANGELOG 保持基线内容;版权、许可证、原作者、Signed-off-by、Change-Id 及 AI 署名保留。完整变更 pre-commit、commit-msg、差异检查及最小格式/版本核对通过。

最终提交的两个合入顺序均经真实 Git 验证无冲突,最终文件树相同;公共说明只由 !1333 引入。与旧式手工根日志并存及固定 HEAD 的只读汇总验证通过,工作区/未跟踪内容不会进入汇总,命令不改写工作区。

本轮未重跑本地 C++、目标板或正式 Conan 构建;新 SHA 远端结果单独核查,不把历史结果当作新通过。日志格式仍是文档规范,未部署自动格式门禁。

该PR关联的issue

fixes #758

希望检视人员了解:

PowerOnHours 仍为 double,不能精确表示全部 128 位整数。

此变更只消除独立新增日志对共享文件的争抢;业务冲突、历史记录修订及正式版本管理仍须按原流程处理。两个 PR 当前均为未发布候选 1.2.579,若先合入的源码已经占用正式版本,后续 PR 须在合入前手工调整 service.json 及自己的日志,不能发布不同源码而复用已占用身份。

目前上游 CI 配置的 Test_version_check=false,线上强制版本门禁未启用;原 openubmc-ci !306 的根日志检查不适用于本方案,不能按原样启用。此 PR 不修改 CI 配置、不声称新发布流水线已部署。原自动定版补救任务已超出当前方案,移除其关联;当前 PR 仅关联数值解码任务。

启用合并后关闭关联 Issue;本次仅更新 PR,不请求合并。

likedislike
Pull Request has been successfully merged by openUBMC-robot.
(Thank you for liuxingsen's contribution.)
LliuxingsenCollaborators
25 days ago Created pull request, commit 32911d6b
LliuxingsenCollaborators
25 days ago Associated issue: [任务]: 整合驱动内存边界和异步生命周期修复并完成回归验收,[任务]: 按字节解码 PRBS 未对齐字段,保留 NVMe PowerOnHours 的高 64 位,消除超宽移位
openUBMC-robotopenUBMC-robotCollaborators
25 days ago Added label: sig/sig-component-drivers
openUBMC-robot
openUBMC-robotCollaborators
25 days ago Comments:

欢迎来到 openUBMC 社区

Hey @xiaoxinglxs , 感谢你对社区的贡献.

机器人使用手册

有关指令的使用,可以点击 说明文档 或 论坛 查看详情。开发人员可以在每个PR或Issue下方评论特定指令来触发机器人任务。您可以自助配置此仓库的 PR 合并规则,更多详情,请参阅机器人自助配置使用手册。

联系指引

有疑问可以联系 SIG: sig-component-drivers ,
维护者是: @dongyushan, @net_swift_wangchen, @sam_h1973, @u012924942, @weixin_37569011, @xiaoxinglxs ,
优秀贡献者是: @adellonzzy, @avery-j, @changxingOpenUBMC, @chenshengfan, @dli_bmc, @huangjiapei, @leessential, @net_swift_tianwen, @xiapengc1, @yelvsub .

likedislike
openUBMC-robotopenUBMC-robotCollaborators
25 days ago Added label: ai-co-authored
325 messages are collapsed here。 See more
huangleishengCollaborators
6 days ago Comments:

/check-pr

likedislike
openUBMC-robotopenUBMC-robotCollaborators
6 days ago Removed label: unresolved-reviews
openUBMC-robotopenUBMC-robotCollaborators
6 days ago Closed associated issue
openUBMC-robotopenUBMC-robotCollaborators
6 days ago Merged pull request, merge node SHA: 70e2e189b2f54743bd734233c97e7f6456b340d3
openUBMC-robot
openUBMC-robotCollaborators
6 days ago Comments:

合并PR失败

  • reason: {"message":"Pull Request 已合并或已关闭。"}
likedislike