Set page table cloning on for event device allocs

aravindksg · Compute-Runtime-Automation · commit e4a93f987044 · 2022-02-24T22:35:03.000+01:00
Related-To: NEO-5968

Signed-off-by: Aravind Gopalakrishnan &lt;aravind.gopalakrishnan@intel.com&gt;
diff --git a/opencl/test/unit_test/memory_manager/storage_info_tests.cpp b/opencl/test/unit_test/memory_manager/storage_info_tests.cpp
@@ -439,7 +439,7 @@ TEST_F(MultiDeviceStorageInfoTest, givenGpuTimestampAllocationWhenUsingSingleTil
                                     singleTileMask};
     auto storageInfo = memoryManager->createStorageInfoFromProperties(properties);
     EXPECT_EQ(singleTileMask, storageInfo.memoryBanks);
-    EXPECT_FALSE(storageInfo.cloningOfPageTables);
+    EXPECT_TRUE(storageInfo.cloningOfPageTables);
     EXPECT_FALSE(storageInfo.tileInstanced);
     EXPECT_EQ(singleTileMask, storageInfo.pageTablesVisibility);
 }
diff --git a/shared/source/memory_manager/definitions/storage_info.cpp b/shared/source/memory_manager/definitions/storage_info.cpp
@@ -102,11 +102,9 @@ StorageInfo MemoryManager::createStorageInfoFromProperties(const AllocationPrope
         }
         break;
     case AllocationType::GPU_TIMESTAMP_DEVICE_BUFFER:
-        if (properties.flags.multiOsContextCapable) {
-            storageInfo.cloningOfPageTables = true;
-        } else {
+        storageInfo.cloningOfPageTables = true;
+        if (!properties.flags.multiOsContextCapable) {
             storageInfo.pageTablesVisibility = preferredTile;
-            storageInfo.cloningOfPageTables = false;
         }
         break;
     case AllocationType::BUFFER:

Original file line number	Diff line number	Diff line change
`@@ -439,7 +439,7 @@ TEST_F(MultiDeviceStorageInfoTest, givenGpuTimestampAllocationWhenUsingSingleTil`
`439`	`439`	`singleTileMask};`
`440`	`440`	`auto storageInfo = memoryManager->createStorageInfoFromProperties(properties);`
`441`	`441`	`EXPECT_EQ(singleTileMask, storageInfo.memoryBanks);`
`442`		`- EXPECT_FALSE(storageInfo.cloningOfPageTables);`
	`442`	`+ EXPECT_TRUE(storageInfo.cloningOfPageTables);`
`443`	`443`	`EXPECT_FALSE(storageInfo.tileInstanced);`
`444`	`444`	`EXPECT_EQ(singleTileMask, storageInfo.pageTablesVisibility);`
`445`	`445`	`}`
Original file line number	Diff line number	Diff line change
`@@ -102,11 +102,9 @@ StorageInfo MemoryManager::createStorageInfoFromProperties(const AllocationPrope`
`102`	`102`	`}`
`103`	`103`	`break;`
`104`	`104`	`case AllocationType::GPU_TIMESTAMP_DEVICE_BUFFER:`
`105`		`- if (properties.flags.multiOsContextCapable) {`
`106`		`- storageInfo.cloningOfPageTables = true;`
`107`		`- } else {`
	`105`	`+ storageInfo.cloningOfPageTables = true;`
	`106`	`+ if (!properties.flags.multiOsContextCapable) {`
`108`	`107`	`storageInfo.pageTablesVisibility = preferredTile;`
`109`		`- storageInfo.cloningOfPageTables = false;`
`110`	`108`	`}`
`111`	`109`	`break;`
`112`	`110`	`case AllocationType::BUFFER:`