Skip to content

Commit 40d919c

Browse files
authored
Merge pull request #295 from vinser52/svinogra_fix_test
Fix sporadic issue in memspaceHostAllProviderTest.memoryPolicyOOM test
2 parents 317afb8 + 41e7199 commit 40d919c

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

test/memspaces/memspace_host_all.cpp

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -157,9 +157,14 @@ TEST_F(memspaceHostAllProviderTest, memoryPolicyOOM) {
157157
UT_ASSERTeq(umf_ret, UMF_RESULT_SUCCESS);
158158
}
159159

160+
// TODO: we want to enable this check only when tests are running under QEMU.
161+
// Otherwise it might sporadically fail on a real system where other processes
162+
// occupied all memory from a aparticular NUMA node.
163+
#if 0
160164
// Confirm that all the NUMA nodes bound to 'HOST ALL' memspace were exhausted.
161165
for (auto &id : nodeIds) {
162166
auto it = std::find(allocNodeIds.begin(), allocNodeIds.end(), id);
163167
UT_ASSERT(it != allocNodeIds.end());
164168
}
169+
#endif
165170
}

0 commit comments

Comments
 (0)