Skip to content

[SYCL][NFC] Enable IPC tests#21848

Draft
KornevNikita wants to merge 19 commits into
intel:syclfrom
KornevNikita:enable-ipc-test
Draft

[SYCL][NFC] Enable IPC tests#21848
KornevNikita wants to merge 19 commits into
intel:syclfrom
KornevNikita:enable-ipc-test

Conversation

@KornevNikita
Copy link
Copy Markdown
Contributor

@KornevNikita KornevNikita commented Apr 22, 2026

UMFW-348 was fixed.

Larsen, Steffen and others added 16 commits November 27, 2025 05:49
Signed-off-by: Larsen, Steffen <steffen.larsen@intel.com>
Signed-off-by: Larsen, Steffen <steffen.larsen@intel.com>
Signed-off-by: Larsen, Steffen <steffen.larsen@intel.com>
Signed-off-by: Larsen, Steffen <steffen.larsen@intel.com>
Signed-off-by: Larsen, Steffen <steffen.larsen@intel.com>
Signed-off-by: Larsen, Steffen <steffen.larsen@intel.com>
Signed-off-by: Larsen, Steffen <steffen.larsen@intel.com>
Signed-off-by: Larsen, Steffen <steffen.larsen@intel.com>
Signed-off-by: Larsen, Steffen <steffen.larsen@intel.com>
Signed-off-by: Larsen, Steffen <steffen.larsen@intel.com>
Fix
Signed-off-by: Larsen, Steffen <steffen.larsen@intel.com>
Signed-off-by: Larsen, Steffen <steffen.larsen@intel.com>
UMF issue was fixed.
@KornevNikita KornevNikita marked this pull request as ready for review April 22, 2026 16:59
@KornevNikita KornevNikita requested a review from a team as a code owner April 22, 2026 16:59
@KornevNikita
Copy link
Copy Markdown
Contributor Author

For some reason the test is unsupported on all windows runners. Anyways, let's enable this.

@KornevNikita KornevNikita requested a review from a team as a code owner April 22, 2026 17:08
@KornevNikita KornevNikita marked this pull request as draft April 22, 2026 17:08
@KornevNikita
Copy link
Copy Markdown
Contributor Author

KornevNikita commented Apr 22, 2026

Found the reason why tests are unsupported. Re-running the pre-commit, converting to draft in case they still don't pass.

@bratpiorka bratpiorka self-requested a review May 5, 2026 13:48
@bratpiorka
Copy link
Copy Markdown
Contributor

@KornevNikita I see that the Windows tests are failing - please let me look into this

@KornevNikita
Copy link
Copy Markdown
Contributor Author

@bratpiorka thanks, any updates?

@bratpiorka
Copy link
Copy Markdown
Contributor

Hi @KornevNikita,

I have a draft PR (#21987) with changes in UR that should fix the problem. I needed also add a new UR environment flag there to enable the workaround (WA) for Windows, since the WA introduces some performance overhead that we don’t want in the default path.
I discussed this solution with the UR architect, and we agreed that adding a new environment flag is not a good idea, as users shouldn’t need to worry about the WA. Instead, this should be addressed at the application level - for example, by adding allocation hints to sycl::malloc.
This new approach is tracked internally in CMPLRLLVM-75625. Until it is implemented, the current PR needs to stay in draft status. I will also reopen UMFW-348 and link it to the CMPLRLLVM JIRA and this PR.

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.

2 participants