Skip to content

Commit e097511

Browse files
Fix hardcoded values in DRM unit tests
Signed-off-by: Bartosz Dunajski <[email protected]>
1 parent 0af5cc0 commit e097511

File tree

1 file changed

+10
-7
lines changed

1 file changed

+10
-7
lines changed

opencl/test/unit_test/os_interface/linux/drm_memory_manager_tests.h

Lines changed: 10 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -90,18 +90,21 @@ class DrmMemoryManagerFixture : public MemoryManagementFixture {
9090
void TearDown() override {
9191
mock->testIoctls();
9292
mock->reset();
93-
mock->ioctl_expected.contextDestroy = static_cast<int>(device->engines.size());
94-
mock->ioctl_expected.gemClose = static_cast<int>(device->engines.size());
95-
mock->ioctl_expected.gemWait = static_cast<int>(device->engines.size());
93+
94+
int enginesCount = static_cast<int>(device->getMemoryManager()->getRegisteredEnginesCount());
95+
96+
mock->ioctl_expected.contextDestroy = enginesCount;
97+
mock->ioctl_expected.gemClose = enginesCount;
98+
mock->ioctl_expected.gemWait = enginesCount;
9699

97100
auto csr = static_cast<TestedDrmCommandStreamReceiver<DEFAULT_TEST_FAMILY_NAME> *>(device->getDefaultEngine().commandStreamReceiver);
98101
if (csr->globalFenceAllocation) {
99-
mock->ioctl_expected.gemClose += static_cast<int>(device->engines.size());
100-
mock->ioctl_expected.gemWait += static_cast<int>(device->engines.size());
102+
mock->ioctl_expected.gemClose += enginesCount;
103+
mock->ioctl_expected.gemWait += enginesCount;
101104
}
102105
if (csr->getPreemptionAllocation()) {
103-
mock->ioctl_expected.gemClose += static_cast<int>(device->engines.size());
104-
mock->ioctl_expected.gemWait += static_cast<int>(device->engines.size());
106+
mock->ioctl_expected.gemClose += enginesCount;
107+
mock->ioctl_expected.gemWait += enginesCount;
105108
}
106109
mock->ioctl_expected.gemWait += additionalDestroyDeviceIoctls.gemWait.load();
107110
mock->ioctl_expected.gemClose += additionalDestroyDeviceIoctls.gemClose.load();

0 commit comments

Comments
 (0)