Skip to content

Commit aa857f3

Browse files
authored
Merge pull request #1627 from pbalcer/limit-concurrency
reduce number of workers for the L0 E2E job
2 parents 056d653 + ae26472 commit aa857f3

File tree

2 files changed

+11
-2
lines changed

2 files changed

+11
-2
lines changed

.github/workflows/e2e_core.yml

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,11 @@ on:
4646
description: Tests to filter out completely
4747
type: string
4848
required: false
49-
49+
extra_lit_flags:
50+
description: Additional llvm-lit flags to use
51+
type: string
52+
required: false
53+
5054
permissions:
5155
contents: read
5256
pull-requests: write
@@ -62,7 +66,8 @@ jobs:
6266
str_name: "${{inputs.str_name}}",
6367
prefix: "${{inputs.prefix}}",
6468
config: "${{inputs.config}}",
65-
unit: "${{inputs.unit}}"}
69+
unit: "${{inputs.unit}}",
70+
extra_lit_flags: "${{inputs.extra_lit_flags}}"},
6671
]
6772
build_type: [Release]
6873
compiler: [{c: clang, cxx: clang++}]
@@ -167,6 +172,7 @@ jobs:
167172
-DSYCL_TEST_E2E_TARGETS="${{matrix.adapter.prefix}}${{matrix.adapter.str_name}}:${{matrix.adapter.unit}}"
168173
-DCMAKE_CXX_COMPILER="$(which clang++)"
169174
-DLLVM_LIT="${{github.workspace}}/sycl-repo/llvm/utils/lit/lit.py"
175+
-DSYCL_E2E_TESTS_LIT_FLAGS="-sv;${{matrix.adapter.extra_lit_flags}}"
170176
171177
- name: Set test filters for L0
172178
if: matrix.adapter.name == 'L0'

.github/workflows/e2e_level_zero.yml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,3 +31,6 @@ jobs:
3131
xfail: "ESIMD/preemption.cpp;syclcompat/atomic/atomic_class.cpp;ProgramManager/uneven_kernel_split.cpp;Plugin/level_zero_ext_intel_queue_index.cpp;Plugin/level_zero_ext_intel_cslice.cpp;Matrix/joint_matrix_rowmajorA_rowmajorB.cpp;Matrix/element_wise_ops.cpp;Matrix/element_wise_all_ops.cpp;Matrix/SG32/element_wise_all_ops.cpp"
3232
# Flaky tests
3333
filter_out: "GroupAlgorithm/root_group.cpp|Basic/exceptions-SYCL-2020.cpp|Graph/UnsupportedDevice/device_query.cpp|Graph/RecordReplay/exception_inconsistent_contexts.cpp"
34+
# These runners by default spawn upwards of 260 workers. That's too much for the GPU.
35+
# We also add a time out just in case some test hangs
36+
extra_lit_flags: "-j 50;--max-time 600"

0 commit comments

Comments
 (0)