Skip to content

Commit

Permalink
fix: Removed the condition variable check from Sysman Memory CTS
Browse files Browse the repository at this point in the history
During the call to fetch the memory and the RAS states, the condition
variable check has been removed.

Related-To: VLCLJ-2343

Signed-off-by: Pratik Bari <pratik.bari@intel.com>
  • Loading branch information
pratikbariintel committed Jan 3, 2025
1 parent e3bfc16 commit 3a1f1fa
Showing 1 changed file with 1 addition and 7 deletions.
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
/*
*
* Copyright (C) 2020-2023 Intel Corporation
* Copyright (C) 2020-2025 Intel Corporation
*
* SPDX-License-Identifier: MIT
*
Expand All @@ -18,8 +18,6 @@ namespace lzt = level_zero_tests;

namespace {

std::mutex mem_mutex;

#ifdef USE_ZESINIT
class MemoryModuleZesTest : public lzt::ZesSysmanCtsClass {};
#define MEMORY_TEST MemoryModuleZesTest
Expand Down Expand Up @@ -401,12 +399,10 @@ void getMemoryState(ze_device_handle_t device) {
FAIL() << "No handles found: "
<< _ze_result_t(ZE_RESULT_ERROR_UNSUPPORTED_FEATURE);
}
std::unique_lock<std::mutex> lock(mem_mutex);
for (auto mem_handle : mem_handles) {
ASSERT_NE(nullptr, mem_handle);
lzt::get_mem_state(mem_handle);
}
lock.unlock();
}

void getRasState(ze_device_handle_t device) {
Expand All @@ -417,13 +413,11 @@ void getRasState(ze_device_handle_t device) {
FAIL() << "No handles found: "
<< _ze_result_t(ZE_RESULT_ERROR_UNSUPPORTED_FEATURE);
}
std::unique_lock<std::mutex> lock(mem_mutex);
for (auto ras_handle : ras_handles) {
ASSERT_NE(nullptr, ras_handle);
ze_bool_t clear = 0;
lzt::get_ras_state(ras_handle, clear);
}
lock.unlock();
}

TEST_F(
Expand Down

0 comments on commit 3a1f1fa

Please sign in to comment.