Skip to content

Conversation

CISC
Copy link
Collaborator

@CISC CISC commented Aug 17, 2025

The hang is probably because some sub-process is hanging, so the 10 minute timeout might not be necessary (WaitForExit only waits for main process, unlike -Wait).

@github-actions github-actions bot added the devops improvements to build systems and github actions label Aug 17, 2025
@CISC CISC requested a review from slaren August 17, 2025 09:31
@CISC CISC changed the title ci : fix hang in windows-latest-cmake-hip ci : fix hang in windows-hip build/release Aug 17, 2025
@CISC CISC merged commit b143fbc into master Aug 17, 2025
43 checks passed
@CISC CISC deleted the cisc/ci-fix-windows-latest-cmake-hip-hang branch August 17, 2025 11:30
@CISC
Copy link
Collaborator Author

CISC commented Aug 17, 2025

Interesting, we got our first timeout (notice the False in Install log):
https://github.com/ggml-org/llama.cpp/actions/runs/17022364852/job/48253189702?pr=15375

Fails at build due to missing amdhip64.dll:
https://github.com/ggml-org/llama.cpp/actions/runs/17022364852/job/48253189702?pr=15375#step:8:62

So, either the installation suddenly took more than twice as long as normal and was aborted, or we encountered the previous hang issue.

danbev added a commit that referenced this pull request Sep 10, 2025
This commit adds caching of the ROCm installation for the windows-latest-cmake-hip job. 

The motivation for this is that the installation can sometimes hang and/or not complete properly leaving an invalid installation which later fails the build. By caching the installation hopefully we can keep a good installation available in the cache and avoid the installation step.

Refs: #15365
njsyw1997 pushed a commit to aizip/llama.cpp that referenced this pull request Sep 10, 2025
)

This commit adds caching of the ROCm installation for the windows-latest-cmake-hip job. 

The motivation for this is that the installation can sometimes hang and/or not complete properly leaving an invalid installation which later fails the build. By caching the installation hopefully we can keep a good installation available in the cache and avoid the installation step.

Refs: ggml-org#15365
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

devops improvements to build systems and github actions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants