Skip to content

Commit

Permalink
[SYCL][Graph] Disable Windows test with L0 leak
Browse files Browse the repository at this point in the history
After PR intel#13685 merged to re-enabling
running graphs tests on Windows with immediate command lists a
non-deterministic leak has appeared in post-commit CI for a different
PR https://github.com/intel/llvm/actions/runs/9097140304/job/25008463548

Don't run the test on Windows with immediate command lists until this
can be investigated and fixed.

```
env SYCL_PI_LEVEL_ZERO_USE_IMMEDIATE_COMMANDLISTS=1 env UR_L0_LEAKS_DEBUG=1 SYCL_ENABLE_DEFAULT_CONTEXTS=0 env ONEAPI_DEVICE_SELECTOR=level_zero:gpu  D:\github\actions-runner\_work\llvm\llvm\build-e2e\Graph\Explicit\Output\host_task_single.cpp.tmp.out 2>&1 | d:\github\actions-runner\_work\llvm\llvm\install\bin\filecheck.exe D:\github\actions-runner\_work\llvm\llvm\llvm\sycl\test-e2e\Graph\Explicit\host_task_single.cpp --implicit-check-not=LEAK
```
  • Loading branch information
EwanC committed May 16, 2024
1 parent 761a5f2 commit 6a9c83d
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 2 deletions.
5 changes: 4 additions & 1 deletion sycl/test-e2e/Graph/Explicit/host_task_single.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,10 @@
// Extra run to check for leaks in Level Zero using UR_L0_LEAKS_DEBUG
// RUN: %if level_zero %{env SYCL_PI_LEVEL_ZERO_USE_IMMEDIATE_COMMANDLISTS=0 %{l0_leak_check} %{run} %t.out 2>&1 | FileCheck %s --implicit-check-not=LEAK %}
// Extra run to check for immediate-command-list in Level Zero
// RUN: %if level_zero %{env SYCL_PI_LEVEL_ZERO_USE_IMMEDIATE_COMMANDLISTS=1 %{l0_leak_check} %{run} %t.out 2>&1 | FileCheck %s --implicit-check-not=LEAK %}
// Immediate command-list testing is disabled on Windows due to a
// non-deterministic leak of the Level Zero context, and is intended
// to be re-enabled once this can be investigated and fixed.
// RUN: %if level_zero && linux %{env SYCL_PI_LEVEL_ZERO_USE_IMMEDIATE_COMMANDLISTS=1 %{l0_leak_check} %{run} %t.out 2>&1 | FileCheck %s --implicit-check-not=LEAK %}

// REQUIRES: aspect-usm_host_allocations

Expand Down
5 changes: 4 additions & 1 deletion sycl/test-e2e/Graph/RecordReplay/host_task_single.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,10 @@
// Extra run to check for leaks in Level Zero using UR_L0_LEAKS_DEBUG
// RUN: %if level_zero %{env SYCL_PI_LEVEL_ZERO_USE_IMMEDIATE_COMMANDLISTS=0 %{l0_leak_check} %{run} %t.out 2>&1 | FileCheck %s --implicit-check-not=LEAK %}
// Extra run to check for immediate-command-list in Level Zero
// RUN: %if level_zero %{env SYCL_PI_LEVEL_ZERO_USE_IMMEDIATE_COMMANDLISTS=1 %{l0_leak_check} %{run} %t.out 2>&1 | FileCheck %s --implicit-check-not=LEAK %}
// Immediate command-list testing is disabled on Windows due to a
// non-deterministic leak of the Level Zero context, and is intended
// to be re-enabled once this can be investigated and fixed.
// RUN: %if level_zero && linux %{env SYCL_PI_LEVEL_ZERO_USE_IMMEDIATE_COMMANDLISTS=1 %{l0_leak_check} %{run} %t.out 2>&1 | FileCheck %s --implicit-check-not=LEAK %}

// REQUIRES: aspect-usm_host_allocations

Expand Down

0 comments on commit 6a9c83d

Please sign in to comment.