Skip to content

Commit d375795

Browse files
[UR][L0] Re-enable driver in-order list
Signed-off-by: Zhang, Winston <[email protected]>
1 parent d21bb39 commit d375795

File tree

3 files changed

+9
-26
lines changed

3 files changed

+9
-26
lines changed

sycl/test-e2e/Graph/RecordReplay/ext_oneapi_enqueue_functions.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ int main() {
5252

5353
for (size_t i = 0; i < Size; i++) {
5454
T Ref = Pattern * i;
55-
assert(Output[i] == Ref);
55+
assert(check_value(i, Ref, Output[i], "Output"));
5656
}
5757

5858
return 0;

sycl/test-e2e/Graph/RecordReplay/usm_copy_in_order.cpp

+7-6
Original file line numberDiff line numberDiff line change
@@ -45,9 +45,9 @@ int main() {
4545
// Shouldn't be captured in graph as a dependency
4646
Queue.submit([&](handler &CGH) {
4747
CGH.parallel_for(N, [=](id<1> it) {
48-
X[it] += 0.5f;
49-
Y[it] += 0.5f;
50-
Z[it] += 0.5f;
48+
X[it] += 1.0f;
49+
Y[it] += 1.0f;
50+
Z[it] += 1.0f;
5151
});
5252
});
5353

@@ -63,14 +63,15 @@ int main() {
6363
// memcpy from 2.0 Y values to Z
6464
Queue.submit([&](handler &CGH) { CGH.memcpy(Z, Y, N * sizeof(int)); });
6565

66+
std::vector<int> Output(N);
67+
Queue.memcpy(Output.data(), Z, N * sizeof(int));
68+
6669
Graph.end_recording();
6770

6871
auto ExecGraph = Graph.finalize();
6972

7073
Queue.submit([&](handler &CGH) { CGH.ext_oneapi_graph(ExecGraph); });
71-
72-
std::vector<int> Output(N);
73-
Queue.memcpy(Output.data(), Z, N * sizeof(int)).wait();
74+
Queue.wait_and_throw();
7475

7576
const int Expected = 2;
7677
for (size_t i = 0; i < N; i++) {

unified-runtime/source/adapters/level_zero/platform.cpp

+1-19
Original file line numberDiff line numberDiff line change
@@ -547,25 +547,7 @@ ur_result_t ur_platform_handle_t_::initialize() {
547547
}
548548

549549
bool ur_platform_handle_t_::allowDriverInOrderLists(bool OnlyIfRequested) {
550-
// Use in-order lists implementation from L0 driver instead
551-
// of adapter's implementation.
552-
553-
// The following driver version is known to be passing and only this or newer
554-
// drivers should be allowed by default for in order lists.
555-
#define L0_DRIVER_INORDER_MINOR_VERSION 6
556-
#define L0_DRIVER_INORDER_PATCH_VERSION 32149
557-
558-
static const bool UseEnvVarDriverInOrderLists = [&] {
559-
const char *UrRet = std::getenv("UR_L0_USE_DRIVER_INORDER_LISTS");
560-
return UrRet ? std::atoi(UrRet) != 0 : false;
561-
}();
562-
static const bool UseDriverInOrderLists = [this] {
563-
bool CompatibleDriver = this->isDriverVersionNewerOrSimilar(
564-
1, L0_DRIVER_INORDER_MINOR_VERSION, L0_DRIVER_INORDER_PATCH_VERSION);
565-
return CompatibleDriver || UseEnvVarDriverInOrderLists;
566-
}();
567-
568-
return OnlyIfRequested ? UseEnvVarDriverInOrderLists : UseDriverInOrderLists;
550+
return true;
569551
}
570552

571553
/// Checks the version of the level-zero driver.

0 commit comments

Comments
 (0)