Skip to content

Disable flaky Unit_NonHost_Printf hip-tests for compiler promotion (#4081)#4245

Merged
lajagapp merged 5 commits intomainfrom
users/lajagapp/disable-hip-tests
Apr 22, 2026
Merged

Disable flaky Unit_NonHost_Printf hip-tests for compiler promotion (#4081)#4245
lajagapp merged 5 commits intomainfrom
users/lajagapp/disable-hip-tests

Conversation

@lajagapp
Copy link
Copy Markdown
Contributor

Summary

Disable three flaky Unit_NonHost_Printf hip-tests that are intermittently failing (subprocess aborted) with the compiler submodule update in #4081. This unblocks the compiler promotion merge.

Failing Tests

| Unit_NonHost_Printf_loop | Subprocess aborted |
| Unit_NonHost_Printf_multiple_Threads | Subprocess aborted |
| Unit_NonHost_Printf_BufferAvailability | Subprocess aborted |

Context

These tests fail intermittently with the compiler submodule update and are not related to a specific GPU architecture. Disabling them allows the compiler promotion to proceed while the root cause is investigated separately.

Related

…4081)

Signed-off-by: Lenine Ajagappane <Lenine.Ajagappane@amd.com>
@lajagapp lajagapp marked this pull request as ready for review March 30, 2026 18:15
Copy link
Copy Markdown
Contributor

@searlmc1 searlmc1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@searlmc1 searlmc1 requested review from ScottTodd, marbre and stellaraccident and removed request for ScottTodd, marbre and stellaraccident March 30, 2026 19:01
"Unit_hipMemsetASyncMulti",
"Unit_hipHostAlloc_AllocateMoreThanAvailGPUMemory",
"Unit_hipStreamValue_Write - TestParams<uint32_t, PtrType::DevicePtrToHost>",
# TODO(#4081): Flaky with compiler submodule update — subprocess aborted.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see this linked a PR. can we create a GitHub issue that has this info and issue? does the compiler team / hip test team know of these errors?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, updated github-issue (#4244) info now.

As Ron mentioned, Compiler team is already working on these failure tests.

@ronlieb ronlieb assigned ronlieb and lamb-j and unassigned ronlieb Mar 31, 2026
@ronlieb
Copy link
Copy Markdown
Contributor

ronlieb commented Mar 31, 2026

yes we know of it, Sam and Jacob have PRs ready to start landing to resolve this. its a comgr and llvm-project fix

lajagapp added 2 commits April 2, 2026 12:32
Signed-off-by: Lenine Ajagappane <Lenine.Ajagappane@amd.com>
@lajagapp lajagapp requested a review from geomin12 April 2, 2026 12:37
Copy link
Copy Markdown
Contributor

@geomin12 geomin12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this correct? Did we add it to the correct arch? I still see the error for gfx94X-dcgpu:

https://github.com/ROCm/TheRock/actions/runs/23900615264/job/70066496331?pr=4245#step:12:8248

@lajagapp
Copy link
Copy Markdown
Contributor Author

Is this correct? Did we add it to the correct arch? I still see the error for gfx94X-dcgpu:

https://github.com/ROCm/TheRock/actions/runs/23900615264/job/70066496331?pr=4245#step:12:8248

Yes, updated PR to disable these tests on gfx94X target family.

@lajagapp lajagapp requested a review from geomin12 April 17, 2026 07:17
@lajagapp lajagapp merged commit 821660a into main Apr 22, 2026
230 of 246 checks passed
@github-project-automation github-project-automation Bot moved this from TODO to Done in TheRock Triage Apr 22, 2026
@lajagapp lajagapp deleted the users/lajagapp/disable-hip-tests branch April 22, 2026 04:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants