Skip to content

[SYCL][E2E][TEST] Unplug sycl_device_globals on NVIDIA RTX 3090#21867

Merged
sarnex merged 2 commits intointel:syclfrom
kweronsx:flush-sycl-device-globals-test-prints
Apr 27, 2026
Merged

[SYCL][E2E][TEST] Unplug sycl_device_globals on NVIDIA RTX 3090#21867
sarnex merged 2 commits intointel:syclfrom
kweronsx:flush-sycl-device-globals-test-prints

Conversation

@kweronsx
Copy link
Copy Markdown
Contributor

@kweronsx kweronsx commented Apr 24, 2026

Temporarily disable sycl_device_globals on NVIDIA RTX 3090

The test is expected to work on this device, but is currently experiencing a high rate of intermittent failures (#20408). To avoid blocking contributors whose changes are unrelated to this issue, execution is being turned off on RTX 3090 until the root cause is identified and fixed.

Additionally, occurrences of "\n" in print statements have been replaced with std::endl to ensure output is flushed immediately and remains visible in test logs in the future.

@kweronsx kweronsx requested a review from a team as a code owner April 24, 2026 07:13
@kweronsx kweronsx requested a review from againull April 24, 2026 07:13
@kweronsx kweronsx changed the title Force flushing of test print [SYCL][E2E][TEST] Unplug sycl_device_globals on NVIDIA RTX 3090 Apr 24, 2026
@kweronsx kweronsx force-pushed the flush-sycl-device-globals-test-prints branch 3 times, most recently from d5391be to 882eb12 Compare April 24, 2026 11:05

// UNSUPPORTED: gpu-nvidia-geforce-rtx-3090
// UNSUPPORTED-INTENDED: High rate of intermittent failures on NVIDIA RTX 3090.
// More details can be found here: https://github.com/intel/llvm/issues/20408
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.

use UNSUPPORTED-TRACKER here

@bratpiorka
Copy link
Copy Markdown
Contributor

@kweronsx please update PR description

@kweronsx kweronsx force-pushed the flush-sycl-device-globals-test-prints branch from 882eb12 to 72f8178 Compare April 27, 2026 06:45
Also, force buffer flush to make debug prints visible.
@kweronsx
Copy link
Copy Markdown
Contributor Author

@intel/llvm-gatekeepers please merge, the failures are unrelated.

@KornevNikita
Copy link
Copy Markdown
Contributor

KornevNikita commented Apr 27, 2026

I doubt that this affects BMG run, but let's re-run just in case.
UPD. oops, I see there already 19 attempts:D
UPD 2. OK, then merging as is.

@KornevNikita
Copy link
Copy Markdown
Contributor

@intel/llvm-reviewers-runtime please take a look.

@sarnex sarnex merged commit 929fd90 into intel:sycl Apr 27, 2026
306 of 326 checks passed
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.

4 participants