Skip to content

Commit 864038a

Browse files
[CI] Run E2E tests on PVC on postcommit (#15308)
This PR enables running E2E tests on PVC in the post-commit. Currently, the latest PVC driver hangs when a large number of jobs are submitted to it in parallel. Due to this, E2E tests might timeout spuriously on PVC. The PVC bug is being tracked internally via GSD-10241. To prevent spurious test timeouts, we restrict the number of parallel E2E test runs on PVC and to prevent disruption in CI, we are currently enabling PVC in post-commit only. When the bug gets fixed, we can move PVC testing to pre-commit.
1 parent a62b220 commit 864038a

File tree

14 files changed

+51
-5
lines changed

14 files changed

+51
-5
lines changed

.github/workflows/sycl-linux-run-tests.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -110,6 +110,7 @@ on:
110110
- '["Linux", "gen12"]'
111111
- '["amdgpu"]'
112112
- '["Linux", "arc"]'
113+
- '["Linux", "pvc"]'
113114
- '["cts-cpu"]'
114115
- '["Linux", "build"]'
115116
image:

.github/workflows/sycl-post-commit.yml

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -60,6 +60,21 @@ jobs:
6060
image_options: -u 1001 --device=/dev/dri --device=/dev/kfd
6161
target_devices: ext_oneapi_hip:gpu
6262
reset_intel_gpu: false
63+
- name: E2E tests on Intel Ponte Vecchio GPU
64+
runner: '["Linux", "pvc"]'
65+
env: '{"LIT_FILTER_OUT":"ESIMD/unified_memory_api/"}'
66+
image: ghcr.io/intel/llvm/ubuntu2404_intel_drivers:latest
67+
image_options: -u 1001 --device=/dev/dri --privileged --cap-add SYS_ADMIN
68+
target_devices: ext_oneapi_level_zero:gpu;opencl:gpu
69+
extra_lit_opts: -j 50
70+
- name: E2E tests with dev igc on Intel Ponte Vecchio GPU
71+
runner: '["Linux", "pvc"]'
72+
env: '{"LIT_FILTER_OUT":"ESIMD/unified_memory_api/"}'
73+
image: ghcr.io/intel/llvm/ubuntu2404_intel_drivers:devigc
74+
image_options: -u 1001 --device=/dev/dri --privileged --cap-add SYS_ADMIN
75+
target_devices: ext_oneapi_level_zero:gpu;opencl:gpu
76+
use_igc_dev: true
77+
extra_lit_opts: -j 50
6378
# Performance tests below. Specifics:
6479
# - only run performance tests (use LIT_FILTER env)
6580
# - ask llvm-lit to show all the output, even for PASS (-a)

sycl/test-e2e/AddressSanitizer/out-of-bounds/DeviceGlobal/device_global.cpp

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,10 @@
66
// RUN: %{build} %device_asan_flags -O2 -g -o %t3.out
77
// RUN: %{run} not %t3.out 2>&1 | FileCheck %s
88

9+
// Flakily timesout on PVC
10+
// UNSUPPORTED: arch-intel_gpu_pvc
11+
// UNSUPPORTED-TRACKER: https://github.com/intel/llvm/issues/16401
12+
913
#include <sycl/detail/core.hpp>
1014

1115
#include <sycl/ext/oneapi/device_global/device_global.hpp>

sycl/test-e2e/Basic/accessor/accessor.cpp

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,10 @@
11
// RUN: %{build} -DSYCL2020_DISABLE_DEPRECATION_WARNINGS -o %t.out
22
// RUN: %{run} %t.out
33

4+
// Test flakily fails on PVC.
5+
// UNSUPPORTED: arch-intel_gpu_pvc
6+
// UNSUPPORTED-TRACKER: https://github.com/intel/llvm/issues/16401
7+
48
//==----------------accessor.cpp - SYCL accessor basic test ----------------==//
59
//
610
// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.

sycl/test-e2e/DeviceArchitecture/device_architecture_comparison_on_device_aot.cpp

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,8 @@
11
// REQUIRES: arch-intel_gpu_pvc, ocloc
22

3+
// XFAIL: arch-intel_gpu_pvc
4+
// XFAIL-TRACKER: https://github.com/intel/llvm/issues/16401
5+
36
// RUN: %clangxx -fsycl -fsycl-targets=intel_gpu_pvc %s -o %t.out
47
// RUN: %{run} %t.out
58

sycl/test-e2e/DeviceCodeSplit/grf.cpp

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,8 @@
1414
// compiler option
1515

1616
// REQUIRES: arch-intel_gpu_pvc
17+
// XFAIL: arch-intel_gpu_pvc
18+
// XFAIL-TRACKER: https://github.com/intel/llvm/issues/16401
1719

1820
// RUN: %{build} -Wno-error=deprecated-declarations -o %t1.out
1921
// RUN: env SYCL_UR_TRACE=2 %{run} %t1.out 2>&1 | FileCheck %s --check-prefixes=CHECK,CHECK-NO-VAR

sycl/test-e2e/KernelAndProgram/persistent-cache-multi-device.cpp

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,9 @@
55
// RUN: env NEOReadDebugKeys=1 CreateMultipleRootDevices=3 SYCL_CACHE_PERSISTENT=1 SYCL_CACHE_TRACE=1 SYCL_CACHE_DIR=%t/cache_dir env -u XDG_CACHE_HOME env -u HOME %{run} %t.out 2>&1 | FileCheck %s --check-prefixes=CHECK-BUILD
66
// RUN: env NEOReadDebugKeys=1 CreateMultipleRootDevices=3 SYCL_CACHE_PERSISTENT=1 SYCL_CACHE_TRACE=1 SYCL_CACHE_DIR=%t/cache_dir env -u XDG_CACHE_HOME env -u HOME %{run} %t.out 2>&1 | FileCheck %s --check-prefixes=CHECK-CACHE
77

8+
// XFAIL: arch-intel_gpu_pvc
9+
// XFAIL-TRACKER: https://github.com/intel/llvm/issues/16401
10+
811
// Test checks that persistent cache works correctly with multiple devices.
912

1013
#include <sycl/detail/core.hpp>

sycl/test-e2e/MemorySanitizer/lit.local.cfg

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,8 @@
11
# TRACKER: https://github.com/intel/llvm/issues/16184
2-
has_arch_gpu_intel_pvc = any('arch-intel_gpu_pvc' in T for T in config.sycl_dev_features.values())
3-
if not has_arch_gpu_intel_pvc:
4-
config.unsupported_features += ['gpu']
2+
# TRACKER for PVC: https://github.com/intel/llvm/issues/16401
3+
#has_arch_gpu_intel_pvc = any('arch-intel_gpu_pvc' in T for T in config.sycl_dev_features.values())
4+
#if not has_arch_gpu_intel_pvc:
5+
config.unsupported_features += ['gpu']
56

67
config.substitutions.append(
78
("%device_msan_flags", "-Xarch_device -fsanitize=memory")

sycl/test-e2e/OnlineCompiler/online_compiler_L0.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
// REQUIRES: level_zero, level_zero_dev_kit, cm-compiler
2-
// XFAIL: gpu
2+
// XFAIL: gpu && !(arch-intel_gpu_pvc && igc-dev)
33
// XFAIL-TRACKER: https://github.com/intel/llvm/issues/16406
44
// RUN: %{build} -Wno-error=deprecated-declarations -DRUN_KERNELS %level_zero_options -o %t.out
55
// RUN: %{run} %t.out

sycl/test-e2e/OnlineCompiler/online_compiler_OpenCL.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
// REQUIRES: opencl, opencl_icd, cm-compiler
2-
// XFAIL: gpu || cpu || accelerator
2+
// XFAIL: (gpu && !(arch-intel_gpu_pvc && igc-dev)) || cpu || accelerator
33
// XFAIL-TRACKER: https://github.com/intel/llvm/issues/16406
44
// RUN: %{build} -Wno-error=deprecated-declarations -DRUN_KERNELS %opencl_lib -o %t.out
55
// RUN: %{run} %t.out

sycl/test-e2e/ProgramManager/multi_device_bundle/build_twice.cpp

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,9 @@
66
// RUN: %{build} -o %t.out
77
// RUN: env NEOReadDebugKeys=1 CreateMultipleRootDevices=3 SYCL_UR_TRACE=2 %{run} %t.out | FileCheck %s
88

9+
// XFAIL: arch-intel_gpu_pvc
10+
// XFAIL-TRACKER: https://github.com/intel/llvm/issues/16401
11+
912
#include <sycl/detail/core.hpp>
1013
#include <sycl/kernel_bundle.hpp>
1114

sycl/test-e2e/ProgramManager/multi_device_bundle/device_libs_and_caching.cpp

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,9 @@
2929
// Check the case when in-memory caching of the programs is disabled.
3030
// RUN: env SYCL_CACHE_IN_MEM=0 NEOReadDebugKeys=1 CreateMultipleRootDevices=4 %{run} %t.out
3131

32+
// XFAIL: arch-intel_gpu_pvc
33+
// XFAIL-TRACKER: https://github.com/intel/llvm/issues/16401
34+
3235
#include <cmath>
3336
#include <complex>
3437
#include <sycl/detail/core.hpp>

sycl/test-e2e/Regression/multithread_write_accessor.cpp

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,9 @@
11
// RUN: %{build} -o %t.out %threads_lib
22
// RUN: %{run} %t.out
3+
4+
// XFAIL: arch-intel_gpu_pvc
5+
// XFAIL-TRACKER: https://github.com/intel/llvm/issues/16401
6+
37
#include <sycl/detail/core.hpp>
48

59
#include <cassert>

sycl/test-e2e/syclcompat/launch/launch_policy_lmem.cpp

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,9 @@
2626
// UNSUPPORTED: linux && opencl && (gpu-intel-gen12 || gpu-intel-dg2)
2727
// UNSUPPORTED-TRACKER: https://github.com/intel/llvm/issues/15275
2828

29+
// XFAIL: arch-intel_gpu_pvc
30+
// XFAIL-TRACKER: https://github.com/intel/llvm/issues/16401
31+
2932
#include <sycl/detail/core.hpp>
3033
#include <sycl/ext/oneapi/properties/properties.hpp>
3134
#include <sycl/group_barrier.hpp>

0 commit comments

Comments
 (0)