Skip to content

Commit

Permalink
[CI] Disable E2E tests failing on PVC (#14720)
Browse files Browse the repository at this point in the history
This PR disable tests failing on PVC.
GH issue to track disabled tests:
#14826
  • Loading branch information
uditagarwal97 authored Sep 5, 2024
1 parent 243f237 commit 4cdce55
Show file tree
Hide file tree
Showing 29 changed files with 80 additions and 4 deletions.
3 changes: 3 additions & 0 deletions sycl/test-e2e/Basic/accessor/accessor.cpp
Original file line number Diff line number Diff line change
@@ -1,6 +1,9 @@
// RUN: %{build} -DSYCL2020_DISABLE_DEPRECATION_WARNINGS -o %t.out
// RUN: %{run} %t.out

// https://github.com/intel/llvm/issues/14826
// XFAIL: arch-intel_gpu_pvc

//==----------------accessor.cpp - SYCL accessor basic test ----------------==//
//
// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
Expand Down
4 changes: 4 additions & 0 deletions sycl/test-e2e/Basic/accessor/host_task_accessor_deduction.cpp
Original file line number Diff line number Diff line change
@@ -1,2 +1,6 @@
// RUN: %{build} -Daccessor_new_api_test %S/Inputs/host_task_accessor.cpp -o %t.out
// RUN: %{run} %t.out

// Disabled on PVC without igc-dev due to timeout.
// https://github.com/intel/llvm/issues/14826
// UNSUPPORTED: arch-intel_gpu_pvc && !igc-dev
3 changes: 3 additions & 0 deletions sycl/test-e2e/DeprecatedFeatures/set_arg_interop.cpp
Original file line number Diff line number Diff line change
@@ -1,5 +1,8 @@
// REQUIRES: opencl, opencl_icd

// https://github.com/intel/llvm/issues/14826
// XFAIL: arch-intel_gpu_pvc

// RUN: %{build} -D__SYCL_INTERNAL_API -o %t.out %opencl_lib -O3
// RUN: %{run} %t.out

Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,8 @@
// REQUIRES: arch-intel_gpu_pvc, ocloc

// https://github.com/intel/llvm/issues/14826
// XFAIL: arch-intel_gpu_pvc

// RUN: %clangxx -fsycl -fsycl-targets=intel_gpu_pvc %s -o %t.out
// RUN: %{run} %t.out

Expand Down
3 changes: 3 additions & 0 deletions sycl/test-e2e/DeviceCodeSplit/grf.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,9 @@
// compiler option

// REQUIRES: arch-intel_gpu_pvc
// https://github.com/intel/llvm/issues/14826
// XFAIL: *

// RUN: %{build} -o %t.out
// Don't use SYCL_UR_TRACE as the output from the L0 adapter logging interferes
// with the regular UR traces we are checking.
Expand Down
6 changes: 5 additions & 1 deletion sycl/test-e2e/ESIMD/lit.local.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,11 @@ if 'gpu-intel-gen9' in config.available_features and platform.system() == 'Windo
# The current DG2 postcommit job only runs L0 anyway,
# so there's no difference in coverage.
# We should investigate why OCL fails separately.
if 'gpu-intel-dg2' in config.available_features:

# Check if any device has arch-intel_gpu_pvc
has_arch_gpu_intel_pvc = any('arch-intel_gpu_pvc' in T for T in config.sycl_dev_features.values())

if 'gpu-intel-dg2' in config.available_features or has_arch_gpu_intel_pvc:
config.required_features += ['level_zero']

# TODO: Remove this once the warnings are resolved
Expand Down
3 changes: 3 additions & 0 deletions sycl/test-e2e/ESIMD/mask_expand_load.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,9 @@
// RUN: %{build} -fsycl-device-code-split=per_kernel -o %t.out
// RUN: %{run} %t.out

// https://github.com/intel/llvm/issues/14826
// XFAIL: arch-intel_gpu_pvc

// This is a basic test to validate the expanded load API.

#include "esimd_test_utils.hpp"
Expand Down
6 changes: 5 additions & 1 deletion sycl/test-e2e/ESIMD/named_barriers/loop.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,11 @@
// REQUIRES: arch-intel_gpu_pvc
// RUN: %{build} -o %t.out
// RUN: %{run} %t.out
//

// Disabled on PVC without igc-dev due to flaky failures.
// https://github.com/intel/llvm/issues/14826
// UNSUPPORTED: arch-intel_gpu_pvc && !igc-dev

// Test checks support of named barrier in a loop in ESIMD kernel.
// SLM and surface size is 32 bytes, 16 bytes per iteration.
// Each iteration has 1 barrier and 1 producer. Producer stores data to SLM,
Expand Down
5 changes: 4 additions & 1 deletion sycl/test-e2e/ESIMD/named_barriers/loop_extended.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,10 @@
// RUN: %{build} -o %t.out
// RUN: %{run} %t.out

//
// Disabled on PVC due to flaky failures.
// https://github.com/intel/llvm/issues/14826
// UNSUPPORTED: arch-intel_gpu_pvc

// Test checks support of named barrier in a loop in ESIMD kernel.
// First iteration has 1 barrier and 1 producer, second - 2 barriers and 2
// producers. Producer stores data to SLM, then all threads read SLM and store
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,9 @@
// 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 %S/../Inputs/Kernels/saxpy.spv 2>&1 | FileCheck %s --implicit-check-not=LEAK %}

// https://github.com/intel/llvm/issues/14826
// XFAIL: arch-intel_gpu_pvc

#define GRAPH_E2E_EXPLICIT

#include "../Inputs/interop-level-zero-launch-kernel.cpp"
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,9 @@
// 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 %S/../Inputs/Kernels/saxpy.spv 2>&1 | FileCheck %s --implicit-check-not=LEAK %}

// https://github.com/intel/llvm/issues/14826
// XFAIL: arch-intel_gpu_pvc

#define GRAPH_E2E_RECORD_REPLAY

#include "../Inputs/interop-level-zero-launch-kernel.cpp"
4 changes: 4 additions & 0 deletions sycl/test-e2e/GroupAlgorithm/reduce_sycl2020.cpp
Original file line number Diff line number Diff line change
@@ -1,6 +1,10 @@
// RUN: %{build} -fsycl-device-code-split=per_kernel -I . -o %t.out
// RUN: %{run} %t.out

// Disabled on PVC without igc-dev due to timeout.
// https://github.com/intel/llvm/issues/14826
// UNSUPPORTED: arch-intel_gpu_pvc && !igc-dev

#include "support.h"
#include <algorithm>
#include <cassert>
Expand Down
3 changes: 3 additions & 0 deletions sycl/test-e2e/KernelAndProgram/target_register_alloc_mode.cpp
Original file line number Diff line number Diff line change
@@ -1,5 +1,8 @@
// REQUIRES: arch-intel_gpu_pvc

// https://github.com/intel/llvm/issues/14826
// XFAIL: arch-intel_gpu_pvc

// RUN: %{build} -ftarget-register-alloc-mode=pvc:auto -o %t_with.out
// RUN: %{build} -o %t_without.out
// RUN: %{build} -ftarget-register-alloc-mode=pvc:default -o %t_default.out
Expand Down
3 changes: 3 additions & 0 deletions sycl/test-e2e/KernelCompiler/kernel_compiler_sycl.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,9 @@

// REQUIRES: (opencl || level_zero)

// https://github.com/intel/llvm/issues/14826
// XFAIL: arch-intel_gpu_pvc && igc-dev

// RUN: %{build} -o %t.out
// RUN: %{run} %t.out

Expand Down
2 changes: 1 addition & 1 deletion sycl/test-e2e/Matrix/SG32/get_coord_int8_matB.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
// SG size = 32 is not currently supported for SYCL Joint Matrix by IGC on DG2
// UNSUPPORTED: gpu-intel-dg2
// REQUIRES: aspect-ext_intel_matrix
// REQUIRES-INTEL-DRIVER: lin: 27501, win: 101.4943
// REQUIRES-INTEL-DRIVER: lin: 30049, win: 101.4943

// RUN: %{build} -o %t.out
// RUN: %{run} %t.out
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,9 @@
// REQUIRES: aspect-ext_intel_matrix, gpu
// UNSUPPORTED: gpu-intel-dg2

// https://github.com/intel/llvm/issues/14826
// XFAIL: arch-intel_gpu_pvc && igc-dev

// RUN: %{build} -D__SPIRV_USE_COOPERATIVE_MATRIX -o %t_gpu_vnni.out -ffp-model=precise -DOOB -DVNNI
// RUN: %{run} %t_gpu_vnni.out

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,8 @@
// RUN: %{run} %t.out

// XFAIL: cpu
// https://github.com/intel/llvm/issues/14826
// XFAIL: arch-intel_gpu_pvc && !dev-igc

#include "../common.hpp"

Expand Down
1 change: 1 addition & 0 deletions sycl/test-e2e/Matrix/element_wise_all_ops_1d.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@
//
//===----------------------------------------------------------------------===//
// REQUIRES: aspect-ext_intel_matrix, gpu
// REQUIRES-INTEL-DRIVER: lin: 30049

// RUN: %{build} -o %t.out
// RUN: env IGC_JointMatrixLoadStoreOpt=1 %{run} %t.out
Expand Down
1 change: 1 addition & 0 deletions sycl/test-e2e/Matrix/element_wise_all_ops_1d_cont.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@
//
//===----------------------------------------------------------------------===//
// REQUIRES: aspect-ext_intel_matrix, gpu
// REQUIRES-INTEL-DRIVER: lin: 30049

// RUN: %{build} -o %t.out
// RUN: env IGC_JointMatrixLoadStoreOpt=2 %{run} %t.out
Expand Down
1 change: 1 addition & 0 deletions sycl/test-e2e/Matrix/element_wise_all_ops_scalar.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@
//
//===----------------------------------------------------------------------===//
// REQUIRES: aspect-ext_intel_matrix, gpu
// REQUIRES-INTEL-DRIVER: lin: 30049

// RUN: %{build} -o %t.out
// RUN: env IGC_JointMatrixLoadStoreOpt=0 %{run} %t.out
Expand Down
1 change: 1 addition & 0 deletions sycl/test-e2e/Matrix/get_coord_int8_matB.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@

// RUN: %{build} -o %t.out
// RUN: %{run} %t.out
// REQUIRES-INTEL-DRIVER: lin: 30049

#include "common.hpp"
#include "get_coord_int8_matB_impl.hpp"
3 changes: 3 additions & 0 deletions sycl/test-e2e/Matrix/joint_matrix_bf16_fill_k_cache_OOB.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,9 @@
// REQUIRES: aspect-ext_intel_matrix, gpu
// UNSUPPORTED: gpu-intel-dg2

// https://github.com/intel/llvm/issues/14826
// XFAIL: arch-intel_gpu_pvc && igc-dev

// RUN: %{build} -o %t_gpu_vnni.out -ffp-model=precise -DOOB -DVNNI
// RUN: %{run} %t_gpu_vnni.out

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,5 +19,8 @@
// This test will be adjusted to run on the CPU
// XFAIL:cpu

// https://github.com/intel/llvm/issues/14826
// XFAIL: arch-intel_gpu_pvc && !igc-dev

#include "common.hpp"
#include "joint_matrix_bf16_fill_k_cache_impl.hpp"
3 changes: 3 additions & 0 deletions sycl/test-e2e/Matrix/joint_matrix_prefetch.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,9 @@
// RUN: %{build} -o %t.out
// RUN: %{run} %t.out

// https://github.com/intel/llvm/issues/14826
// XFAIL: arch-intel_gpu_pvc && !igc-dev

#include "common.hpp"

constexpr size_t TN = 16;
Expand Down
1 change: 1 addition & 0 deletions sycl/test-e2e/Matrix/joint_matrix_rowmajorA_rowmajorB.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@
// REQUIRES: aspect-ext_intel_matrix
// VNNI transform is not supported yet by IGC on DG2
// UNSUPPORTED: gpu-intel-dg2
// REQUIRES-INTEL-DRIVER: lin: 30049

// RUN: %{build} -o %t.out
// RUN: %{run} %t.out
Expand Down
2 changes: 2 additions & 0 deletions sycl/test-e2e/Plugin/level_zero_ext_intel_cslice.cpp
Original file line number Diff line number Diff line change
@@ -1,6 +1,8 @@
// REQUIRES: level_zero
// REQUIRES: aspect-ext_intel_device_id
// UNSUPPORTED: gpu-intel-pvc-1T
// https://github.com/intel/llvm/issues/14826
// XFAIL: arch-intel_gpu_pvc

// RUN: %{build} -Wno-error=deprecated-declarations -o %t.out

Expand Down
3 changes: 3 additions & 0 deletions sycl/test-e2e/Plugin/level_zero_ext_intel_queue_index.cpp
Original file line number Diff line number Diff line change
@@ -1,6 +1,9 @@
// REQUIRES: aspect-ext_intel_device_id
// REQUIRES: level_zero
// UNSUPPORTED: gpu-intel-pvc-1T

// https://github.com/intel/llvm/issues/14826
// XFAIL: arch-intel_gpu_pvc
// RUN: %{build} -o %t.out

// TODO: at this time PVC 1T systems are not correctly supporting CSLICE
Expand Down
2 changes: 2 additions & 0 deletions sycl/test-e2e/Plugin/level_zero_sub_sub_device.cpp
Original file line number Diff line number Diff line change
@@ -1,5 +1,7 @@
// REQUIRES: arch-intel_gpu_pvc, level_zero

// https://github.com/intel/llvm/issues/14826
// XFAIL: arch-intel_gpu_pvc
// UNSUPPORTED: gpu-intel-pvc-1T

// RUN: %{build} %level_zero_options -o %t.out
Expand Down
3 changes: 3 additions & 0 deletions sycl/test-e2e/syclcompat/launch/launch_policy_lmem.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,9 @@
* launch<F> with policy & use local memory tests
**************************************************************************/

// https://github.com/intel/llvm/issues/14826
// XFAIL: arch-intel_gpu_pvc

// RUN: %{build} -fsycl-device-code-split=per_kernel -o %t.out
// RUN: %{run} %t.out

Expand Down

0 comments on commit 4cdce55

Please sign in to comment.