Skip to content

Relax C910V qemu utest timeout#5900

Draft
mpwaser wants to merge 1 commit into
OpenMathLib:developfrom
mpwaser:fix/c910v-qemu-utest-timeout
Draft

Relax C910V qemu utest timeout#5900
mpwaser wants to merge 1 commit into
OpenMathLib:developfrom
mpwaser:fix/c910v-qemu-utest-timeout

Conversation

@mpwaser

@mpwaser mpwaser commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Summary

This relaxes the C910V QEMU timeout for openblas_utest.

The timeout showed up while checking #5899, but #5899 only changed CMake/CTest registration. The failing C910V job uses the make build and runs openblas_utest directly under QEMU.

Details

The failing job repeatedly reaches:

TEST 126/128 fork:safety

and then restarts the whole unit-test binary with short timeout windows.

This keeps the existing retry defaults, but lets callers override them. Only the observed slow binary gets a larger budget:

run_with_retry "$QEMU_BIN ./utest/openblas_utest" 120 2 30
run_with_retry "$QEMU_BIN ./utest/openblas_utest_ext"

openblas_utest_ext keeps the default retry behavior because the observed timeout happens before it runs.

Scope

Limited to .github/workflows/c910v.yml.

Validation

Checked the relevant Actions history:

  • PR Fix CTest command for openblas_utest_ext #5899 run 29003486034, job 86079876651: timed out at openblas_utest fork:safety.
  • Post-merge develop run 29007407697, job 86082261750: same timeout pattern.
  • Prior passing develop run 28967557753, job 85954497464: same test reached timeout retries before eventually passing.

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.

1 participant