Revert "fix: Increase size of reported global memory available to 98%"

fhazubski-Intel · Compute-Runtime-Automation · commit 8729d75d8484 · 2023-07-26T10:55:42.000+02:00
Temporarily reverting back to previously reported value to avoid corner-case regression. Intent is reintroduce as soon as regressions are rootcaused. This only affects Linux. Related-To: GSD-5474, GSD-5412 This reverts commit e8ac22c. Signed-off-by: Filip Hazubski <filip.hazubski@intel.com> Source: 6217d97
diff --git a/shared/source/os_interface/linux/drm_memory_manager.cpp b/shared/source/os_interface/linux/drm_memory_manager.cpp
@@ -1191,7 +1191,7 @@ uint64_t DrmMemoryManager::getSystemSharedMemory(uint32_t rootDeviceIndex) {
 
 double DrmMemoryManager::getPercentOfGlobalMemoryAvailable(uint32_t rootDeviceIndex) {
     if (isLocalMemorySupported(rootDeviceIndex)) {
-        return 0.98;
+        return 0.95;
     }
     return 0.8;
 }
diff --git a/shared/test/unit_test/os_interface/linux/drm_memory_manager_tests.cpp b/shared/test/unit_test/os_interface/linux/drm_memory_manager_tests.cpp
@@ -4549,7 +4549,7 @@ TEST_F(DrmMemoryManagerTest, givenCustomAlignmentWhenWddmMemoryManagerIsCreatedT
 TEST_F(DrmMemoryManagerTest, givenDrmManagerWithLocalMemoryWhenGettingGlobalMemoryPercentThenCorrectValueIsReturned) {
     TestedDrmMemoryManager memoryManager(true, false, false, *executionEnvironment);
     uint32_t rootDeviceIndex = 0u;
-    EXPECT_EQ(memoryManager.getPercentOfGlobalMemoryAvailable(rootDeviceIndex), 0.98);
+    EXPECT_EQ(memoryManager.getPercentOfGlobalMemoryAvailable(rootDeviceIndex), 0.95);
 }
 
 TEST_F(DrmMemoryManagerTest, givenDrmManagerWithoutLocalMemoryWhenGettingGlobalMemoryPercentThenCorrectValueIsReturned) {

Original file line number	Diff line number	Diff line change
`@@ -1191,7 +1191,7 @@ uint64_t DrmMemoryManager::getSystemSharedMemory(uint32_t rootDeviceIndex) {`
`1191`	`1191`
`1192`	`1192`	`double DrmMemoryManager::getPercentOfGlobalMemoryAvailable(uint32_t rootDeviceIndex) {`
`1193`	`1193`	`if (isLocalMemorySupported(rootDeviceIndex)) {`
`1194`		`- return 0.98;`
	`1194`	`+ return 0.95;`
`1195`	`1195`	`}`
`1196`	`1196`	`return 0.8;`
`1197`	`1197`	`}`
Original file line number	Diff line number	Diff line change
`@@ -4549,7 +4549,7 @@ TEST_F(DrmMemoryManagerTest, givenCustomAlignmentWhenWddmMemoryManagerIsCreatedT`
`4549`	`4549`	`TEST_F(DrmMemoryManagerTest, givenDrmManagerWithLocalMemoryWhenGettingGlobalMemoryPercentThenCorrectValueIsReturned) {`
`4550`	`4550`	`TestedDrmMemoryManager memoryManager(true, false, false, *executionEnvironment);`
`4551`	`4551`	`uint32_t rootDeviceIndex = 0u;`
`4552`		`- EXPECT_EQ(memoryManager.getPercentOfGlobalMemoryAvailable(rootDeviceIndex), 0.98);`
	`4552`	`+ EXPECT_EQ(memoryManager.getPercentOfGlobalMemoryAvailable(rootDeviceIndex), 0.95);`
`4553`	`4553`	`}`
`4554`	`4554`
`4555`	`4555`	`TEST_F(DrmMemoryManagerTest, givenDrmManagerWithoutLocalMemoryWhenGettingGlobalMemoryPercentThenCorrectValueIsReturned) {`