Skip to content

ci: unify pto-isa pin default#694

Merged
zhangstevenunity merged 2 commits into
hw-native-sys:mainfrom
HecreReed:codex/unify-pto-isa-pin-sync-20260522
May 22, 2026
Merged

ci: unify pto-isa pin default#694
zhangstevenunity merged 2 commits into
hw-native-sys:mainfrom
HecreReed:codex/unify-pto-isa-pin-sync-20260522

Conversation

@HecreReed
Copy link
Copy Markdown
Collaborator

Summary

  • unify the remote board-validation fallback PTO_ISA_COMMIT with the workflow_dispatch default
  • keep the runtime default aligned with the repo-pinned docker/Dockerfile commit

Notes

  • the repo already has a weekly pto-isa pin updater in .github/workflows/update_pto_isa_pin.yml
  • that workflow currently runs only when github.repository == "zhangstevenunity/PTOAS", so it does not auto-update pins in hw-native-sys/PTOAS or forks

Verification

  • python3 .github/scripts/update_pto_isa_pin.py --commit 662d7f2a916d6bbde3109ce4a16ed5c28f5d900a --check

@gemini-code-assist
Copy link
Copy Markdown

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@reedhecre
Copy link
Copy Markdown

reedhecre commented May 22, 2026

Codex Review

该评论由 review 机器人自动更新。

  • PR: ci: unify pto-isa pin default #694 ci: unify pto-isa pin default
  • Author: HecreReed
  • Base/Head: main / codex/unify-pto-isa-pin-sync-20260522
  • Head SHA: 0855a129d140
  • Trigger: 检测到新的 open PR
  • Generated At: 2026-05-22T06:47:58Z
  • Status: completed

Summary

未检查到 PR #694 存在问题

Findings

No issues found.

@HecreReed HecreReed marked this pull request as ready for review May 22, 2026 07:57
@zhangstevenunity zhangstevenunity merged commit b0fabcf into hw-native-sys:main May 22, 2026
11 checks passed
@reedhecre
Copy link
Copy Markdown

A5 板测失败

  • 触发方式:merged
  • 源码提交:b0fabcf3e47d
  • 结果汇总:OK 20 / FAIL 1 / SKIP 0
  • 日志:/root/ptoas-board-monitor-a5/logs/20260522_181305_merged_pr694.log
  • 结果 TSV:/root/ptoas-board-monitor-a5/logs/20260522_181305_merged_pr694.tsv
  • 失败阶段:board-validation-deepseek / exit=1

失败用例

  • attention_swa_test_incore_40 (run, exit=139)

@reedhecre
Copy link
Copy Markdown

A5 板测失败详情:PR #694

attention_swa_test_incore_40

stage=run info=exit=139

./test/npu_validation/scripts/run_remote_npu_validation.sh: line 366: 1581532 Segmentation fault      (core dumped) LD_LIBRARY_PATH="${LD_LIBRARY_PATH_NPU}" ./build/${testcase}
[2026-05-22 18:21:30] ERROR: testcase failed (exit 139): attention_swa_test_incore_40

@reedhecre
Copy link
Copy Markdown

A3 板测失败

  • 触发方式:merged
  • 源码提交:b0fabcf3e47d
  • 结果汇总:OK 216 / FAIL 2 / SKIP 2
  • 日志:/home/zhongxuan/ptoas-board-monitor/runtime/logs/20260522_181305_merged_pr694.log
  • 失败阶段:board-validation / exit=1

失败用例

  • syncall_binding (run, exit=2)
  • tprefetch_async_binding (run, exit=2)

@reedhecre
Copy link
Copy Markdown

A3 板测失败详情:PR #694

syncall_binding

stage=run info=exit=2

/home/zhongxuan/ptoas-board-monitor/runtime/runs/20260522_181305_merged_pr694/npu_validation/SyncAll/syncall_binding/syncall_binding_kernel.cpp:87:11: error: use of undeclared identifier 'SyncAllMode'
  SYNCALL<SyncAllMode::Soft, SyncCoreType::AIVOnly>(v8, v9, v2);
          ^
/home/zhongxuan/ptoas-board-monitor/runtime/runs/20260522_181305_merged_pr694/npu_validation/SyncAll/syncall_binding/syncall_binding_kernel.cpp:88:11: error: use of undeclared identifier 'SyncAllMode'
  SYNCALL<SyncAllMode::Soft, SyncCoreType::Mix>(v8, v9, v10, v2);
          ^
/home/zhongxuan/ptoas-board-monitor/runtime/runs/20260522_181305_merged_pr694/npu_validation/SyncAll/syncall_binding/syncall_binding_kernel.cpp:89:11: error: use of undeclared identifier 'SyncCoreType'
  SYNCALL<SyncCoreType::Mix>();
          ^
3 errors generated.
gmake[2]: *** [CMakeFiles/syncall_binding_kernel.dir/build.make:76: CMakeFiles/syncall_binding_kernel.dir/syncall_binding_kernel.cpp.o] Error 1
gmake[2]: *** Waiting for unfinished jobs....
gmake[1]: *** [CMakeFiles/Makefile2:85: CMakeFiles/syncall_binding_kernel.dir/all] Error 2
gmake: *** [Makefile:91: all] Error 2
[2026-05-22 18:38:18] ERROR: testcase failed (exit 2): syncall_binding
tprefetch_async_binding

stage=run info=exit=2

/home/zhongxuan/ptoas-board-monitor/runtime/runs/20260522_181305_merged_pr694/npu_validation/TPrefetchAsync/tprefetch_async_binding/tprefetch_async_binding_kernel.cpp:88:8: error: no type named 'PrefetchAsyncContext' in namespace 'pto'
  pto::PrefetchAsyncContext v8 = pto::PrefetchAsyncContext((__gm__ uint8_t*) v2);
  ~~~~~^
/home/zhongxuan/ptoas-board-monitor/runtime/runs/20260522_181305_merged_pr694/npu_validation/TPrefetchAsync/tprefetch_async_binding/tprefetch_async_binding_kernel.cpp:88:39: error: no member named 'PrefetchAsyncContext' in namespace 'pto'
  pto::PrefetchAsyncContext v8 = pto::PrefetchAsyncContext((__gm__ uint8_t*) v2);
                                 ~~~~~^
2 errors generated.
gmake[2]: *** [CMakeFiles/tprefetch_async_binding_kernel.dir/build.make:76: CMakeFiles/tprefetch_async_binding_kernel.dir/tprefetch_async_binding_kernel.cpp.o] Error 1
gmake[2]: *** Waiting for unfinished jobs....
gmake[1]: *** [CMakeFiles/Makefile2:85: CMakeFiles/tprefetch_async_binding_kernel.dir/all] Error 2
gmake: *** [Makefile:91: all] Error 2
[2026-05-22 18:38:55] ERROR: testcase failed (exit 2): tprefetch_async_binding

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants